linux-kernel - Re: [PATCH] vDPA: add missing MODULE

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 for Android: free password hash cracker in your pocket

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

Message-ID: <b1868e2d-7181-4713-8db9-b4d8a29e3fe6@quicinc.com>
Date: Wed, 26 Jun 2024 09:45:42 -0700
From: Jeff Johnson <quic_jjohnson@...cinc.com>
To: Zhu Lingshan <lingshan.zhu@...el.com>,
        "Michael S. Tsirkin"
	<mst@...hat.com>,
        Jason Wang <jasowang@...hat.com>,
        Xuan Zhuo
	<xuanzhuo@...ux.alibaba.com>,
        Eugenio Pérez
	<eperezma@...hat.com>
CC: <virtualization@...ts.linux.dev>, <linux-kernel@...r.kernel.org>,
        <kernel-janitors@...r.kernel.org>
Subject: Re: [PATCH] vDPA: add missing MODULE_DESCRIPTION() macros

On 6/11/2024 12:22 PM, Jeff Johnson wrote:
> With ARCH=x86, make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/vdpa/vdpa.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/vdpa/ifcvf/ifcvf.o
> 
> Add the missing invocations of the MODULE_DESCRIPTION() macro.
> 
> Signed-off-by: Jeff Johnson <quic_jjohnson@...cinc.com>
> ---
>  drivers/vdpa/ifcvf/ifcvf_main.c | 1 +
>  drivers/vdpa/vdpa.c             | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/drivers/vdpa/ifcvf/ifcvf_main.c b/drivers/vdpa/ifcvf/ifcvf_main.c
> index 80d0a0460885..ccf64d7bbfaa 100644
> --- a/drivers/vdpa/ifcvf/ifcvf_main.c
> +++ b/drivers/vdpa/ifcvf/ifcvf_main.c
> @@ -894,4 +894,5 @@ static struct pci_driver ifcvf_driver = {
>  
>  module_pci_driver(ifcvf_driver);
>  
> +MODULE_DESCRIPTION("Intel IFC VF NIC driver for virtio dataplane offloading");
>  MODULE_LICENSE("GPL v2");
> diff --git a/drivers/vdpa/vdpa.c b/drivers/vdpa/vdpa.c
> index 8d391947eb8d..1ca445e31acb 100644
> --- a/drivers/vdpa/vdpa.c
> +++ b/drivers/vdpa/vdpa.c
> @@ -1538,4 +1538,5 @@ core_initcall(vdpa_init);
>  module_exit(vdpa_exit);
>  
>  MODULE_AUTHOR("Jason Wang <jasowang@...hat.com>");
> +MODULE_DESCRIPTION("vDPA bus");
>  MODULE_LICENSE("GPL v2");
> 
> ---
> base-commit: 83a7eefedc9b56fe7bfeff13b6c7356688ffa670
> change-id: 20240611-md-drivers-vdpa-391206d17ec3
> 
Following up to see if anything else is needed from me. Hoping to see this in
linux-next so I can remove it from my tracking spreadsheet :)

/jeff