2023-10-06 01:23:32

by Max Chou

[permalink] [raw]
Subject: RE: Regression: devcoredump patch broke Realtek usb bluetooth adapter

Hi!
Realtek will send a patch to fix this issue.

BRs,
Max


> -----Original Message-----
> From: Kirill A. Shutemov <[email protected]>
> Sent: Wednesday, October 4, 2023 7:46 PM
> To: Max Chou <[email protected]>
> Cc: Bagas Sanjaya <[email protected]>; Hilda Wu
> <[email protected]>; [email protected]; Luiz Augusto von Dentz
> <[email protected]>; Marcel Holtmann <[email protected]>;
> Johan Hedberg <[email protected]>; Juerg Haefliger
> <[email protected]>; Linux Bluetooth
> <[email protected]>; Thorsten Leemhuis <[email protected]>;
> Linux Regressions <[email protected]>; Linux Kernel Mailing List
> <[email protected]>
> Subject: Re: Regression: devcoredump patch broke Realtek usb bluetooth
> adapter
>
>
> External mail.
>
>
>
> On Wed, Oct 04, 2023 at 07:28:45AM +0000, Max Chou wrote:
> > Hi! Kirill,
> > I guess the root cause is as below.
> > ===
> > static int btrtl_register_devcoredump_support(struct hci_dev *hdev) {
> > int err;
> >
> > err = hci_devcd_register(hdev, btrtl_coredump, btrtl_dmp_hdr,
> > NULL);
> >
> > return err;
> > }
> > ===
> > If CONFIG_DEV_COREDUMP is not enabled, it would return -EOPNOTSUPP
> for hci_devcd_register().
> > Unfortunately, btrtl_register_devcoredump_support() will return it.
> > Finally, -EOPNOSTUPP will be returned for btrtl_setup_realtek().
> >
> > Could you have the following workaround for the root cause checked?
> > Please share dmesg as well. Thanks,
>
> Yes. It works and I see
>
> [ 3.640539] Bluetooth: hci0: RTL: btrtl_register_devcoredump_support():
> err = -95
>
> in dmesg. -EOPNOTSUPP indeed.
>
> --
> Kiryl Shutsemau / Kirill A. Shutemov