Bug 1904734 Comment 6 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to C.M.Chang[:chunmin] from comment #5)

Commenting this [piece of code](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3877-3899) doesn't solve the issue, since the [`output_dev_desc.mChannelsPerFrame`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3918) reports `2` while the [`device_layout`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3919) is set to `[BackLeft, BackRight, FrontCenter, LowFrequency, FrontLeft, FrontRight]` (via Audio MIDI Setup). In this case, `Mixer` forces the channel layout to [`[FrontLeft, FrontRight]`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#207-212).  Additionally, forcing the channel to `6` doesn't solve the issue, and it would lead to break the [`output_buffer_size >= size_needed` assertion](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#128) in [`audiounit_output_callback`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#950). 

More investigation is required to know why the assertion is reached when mutating the channel from `2` to `6`.
Revision 1 by
on 
(In reply to C.M.Chang[:chunmin] from comment #5)

Commenting this [piece of code](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3877-3899) doesn't solve the issue, since the [`output_dev_desc.mChannelsPerFrame`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3918) (on my [Cubilux Surround Sound Card (CA7)](https://www.cubilux.com/products/7-1-5-1surround-sound-crad)) reports `2` while the [`device_layout`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3919) is set to `[BackLeft, BackRight, FrontCenter, LowFrequency, FrontLeft, FrontRight]` (via Audio MIDI Setup). In this case, `Mixer` forces the channel layout to [`[FrontLeft, FrontRight]`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#207-212).  Additionally, forcing the channel to `6` doesn't solve the issue, and it would lead to break the [`output_buffer_size >= size_needed` assertion](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#128) in [`audiounit_output_callback`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#950). 

More investigation is required to know why the assertion is reached when mutating the channel from `2` to `6`.
Revision 2 by
on 
(In reply to C.M.Chang[:chunmin] from comment #5)

Commenting this [piece of code](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3877-3899) doesn't solve the issue, since the [`output_dev_desc.mChannelsPerFrame`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3918) (on my [Cubilux Surround Sound Card (CA7)](https://www.cubilux.com/products/7-1-5-1surround-sound-crad)) reports `2` while the [`device_layout`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3919) reports that the hardware layout is set to `[BackLeft, BackRight, FrontCenter, LowFrequency, FrontLeft, FrontRight]` (via Audio MIDI Setup). In this case, `Mixer` forces the channel layout to [`[FrontLeft, FrontRight]`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#207-212).  Additionally, forcing the channel to `6` doesn't solve the issue, and it would lead to break the [`output_buffer_size >= size_needed` assertion](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#128) in [`audiounit_output_callback`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#950). 

More investigation is required to know why the assertion is reached when mutating the channel from `2` to `6`.
Revision 3 by
on 
(In reply to C.M.Chang[:chunmin] from comment #5)

Commenting this [piece of code](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3877-3899) doesn't solve the issue, since the [`output_dev_desc.mChannelsPerFrame`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3918) (on my [Cubilux Surround Sound Card (CA7)](https://www.cubilux.com/products/7-1-5-1surround-sound-crad)) reports `2` while the [`device_layout`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3919) reports that the hardware layout is set to `[BackLeft, BackRight, FrontCenter, LowFrequency, FrontLeft, FrontRight]` (via Audio MIDI Setup). These two pieces of hardware information contradict each other.

In this absurd case, `Mixer` forces the channel layout to [`[FrontLeft, FrontRight]`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#207-212).  Additionally, forcing the channel to `6` doesn't solve the issue, and it would lead to break the [`output_buffer_size >= size_needed` assertion](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#128) in [`audiounit_output_callback`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#950). 

More investigation is required to know why the assertion is reached when mutating the channel from `2` to `6`.
Revision 4 by
on 
(In reply to C.M.Chang[:chunmin] from comment #5)

Commenting this [piece of code](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3877-3899) doesn't solve the issue, since the [`output_dev_desc.mChannelsPerFrame`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3918) (on my [Cubilux Surround Sound Card (CA7)](https://www.cubilux.com/products/7-1-5-1surround-sound-crad)) reports `2` while the [`device_layout`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3919) reports that the hardware layout is set to `[BackLeft, BackRight, FrontCenter, LowFrequency, FrontLeft, FrontRight]` (via Audio MIDI Setup). These two pieces of hardware information contradict each other.

In this absurd case, `Mixer` forces the channel layout to [`[FrontLeft, FrontRight]`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#207-212).  However, forcing the channel to `6` doesn't solve the issue, and it would lead to break the [`output_buffer_size >= size_needed` assertion](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#128) in [`audiounit_output_callback`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#950). 

More investigation is required to know why the assertion is reached when mutating the channel from `2` to `6`.
Revision 5 by
on 
(In reply to C.M.Chang[:chunmin] from comment #5)

Commenting this [piece of code](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3877-3899) doesn't solve the issue, since the [`output_dev_desc.mChannelsPerFrame`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3918) (and [`output_hw_desc.mChannelsPerFrame`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3802)), on my [Cubilux Surround Sound Card (CA7)](https://www.cubilux.com/products/7-1-5-1surround-sound-crad),  reports `2` while the [`device_layout`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#3919) reports that the hardware layout is set to `[BackLeft, BackRight, FrontCenter, LowFrequency, FrontLeft, FrontRight]` (via Audio MIDI Setup) ([`get_channel_count`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#1618) reports `6` in this case). These two pieces of hardware information contradict each other.

In this absurd case, `Mixer` forces the channel layout to [`[FrontLeft, FrontRight]`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#207-212).  However, forcing the channel to `6` doesn't solve the issue, and it would lead to break the [`output_buffer_size >= size_needed` assertion](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mixer.rs#128) in [`audiounit_output_callback`](https://searchfox.org/mozilla-central/rev/b60bff00c0fe136f35e4d6f3316911c090a39f23/third_party/rust/cubeb-coreaudio/src/backend/mod.rs#950). 

More investigation is required to know why the assertion is reached when mutating the channel from `2` to `6`.

Back to Bug 1904734 Comment 6