linux-kernel - Re: linux-next: build warning after merge of the hid tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20230405114807.5a57bf46@canb.auug.org.au>
Date:   Wed, 5 Apr 2023 11:48:07 +1000
From:   Stephen Rothwell <sfr@...b.auug.org.au>
To:     Bastien Nocera <hadess@...ess.net>
Cc:     Jiri Kosina <jikos@...nel.org>,
        Benjamin Tissoires <benjamin.tissoires@...hat.com>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux Next Mailing List <linux-next@...r.kernel.org>
Subject: Re: linux-next: build warning after merge of the hid tree

Hi Bastien,

On Tue, 04 Apr 2023 11:18:35 +0200 Bastien Nocera <hadess@...ess.net> wrote:
>
> Before I send patches, does this look correct to you?
> 
> diff --git a/drivers/usb/core/message.c b/drivers/usb/core/message.c
> index 7930dca84616..7c7f88ca4f62 100644
> --- a/drivers/usb/core/message.c
> +++ b/drivers/usb/core/message.c
> @@ -1926,7 +1926,7 @@ static void __usb_wireless_status_intf(struct work_struct *ws)
> 1926⋮1926│ 
> 1927⋮1927│ /**
> 1928⋮1928│  * usb_set_wireless_status - sets the wireless_status struct member
> 1929⋮    │- * @dev: the device to modify
>     ⋮1929│+ * @iface: the interface to modify
> 1930⋮1930│  * @status: the new wireless status
> 1931⋮1931│  *
> 1932⋮1932│  * Set the wireless_status struct member to the new value, and emit

That looks good.

> diff --git a/include/linux/usb.h b/include/linux/usb.h
> index a48eeec62a66..3d523ebf0c09 100644
> --- a/include/linux/usb.h
> +++ b/include/linux/usb.h
> @@ -261,7 +261,8 @@ struct usb_interface {
>  261⋮ 261│      unsigned needs_binding:1;       /* needs delayed unbind/rebind */
>  262⋮ 262│      unsigned resetting_device:1;    /* true: bandwidth alloc after reset */
>  263⋮ 263│      unsigned authorized:1;          /* used for interface authorization */
>  264⋮    │-     enum usb_wireless_status wireless_status;
>     ⋮ 264│+     enum usb_wireless_status wireless_status; /* whether the emitter is
>     ⋮ 265│+                                                * connected to this receiver */
>  265⋮ 266│      struct work_struct wireless_status_work;
>  266⋮ 267│ 
>  267⋮ 268│      struct device dev;              /* interface specific device info */
> 

This is not right, though.  The warning was about
"wireless_status_work" not "wireless_status", so I think you need a
description of wireless_status_work in the block comment above the
struct definition.

-- 
Cheers,
Stephen Rothwell

Content of type "application/pgp-signature" skipped