linux-kernel - Re: [PATCH] virt: acrn: Use EPOLLIN instead of POLLIN

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: <20210310080845.GA28452@shuo-intel.sh.intel.com>
Date:   Wed, 10 Mar 2021 16:08:45 +0800
From:   Shuo A Liu <shuo.a.liu@...el.com>
To:     Yejune Deng <yejune.deng@...il.com>
Cc:     gregkh@...uxfoundation.org, linux-kernel@...r.kernel.org,
        kernel test robot <lkp@...el.com>
Subject: Re: [PATCH] virt: acrn: Use EPOLLIN instead of POLLIN

On Wed 10.Mar'21 at 15:49:01 +0800, Yejune Deng wrote:
>This fixes the following sparse warning:
>"sparse warnings: (new ones prefixed by >>)"
>>> drivers/virt/acrn/irqfd.c:163:13: sparse: sparse: restricted __poll_t
> degrades to integer
>
>Fixes: 803c1aadecdb("virt: acrn: Use vfs_poll() instead of f_op->poll()")
>Reported-by: kernel test robot <lkp@...el.com>
>Signed-off-by: Yejune Deng <yejune.deng@...il.com>

Thanks for the patch.
Acked-by: Shuo Liu <shuo.a.liu@...el.com>

>---
> drivers/virt/acrn/irqfd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/virt/acrn/irqfd.c b/drivers/virt/acrn/irqfd.c
>index 98d6e9b18f9e..df5184979b28 100644
>--- a/drivers/virt/acrn/irqfd.c
>+++ b/drivers/virt/acrn/irqfd.c
>@@ -160,7 +160,7 @@ static int acrn_irqfd_assign(struct acrn_vm *vm, struct acrn_irqfd *args)
> 	/* Check the pending event in this stage */
> 	events = vfs_poll(f.file, &irqfd->pt);
>
>-	if (events & POLLIN)
>+	if (events & EPOLLIN)
> 		acrn_irqfd_inject(irqfd);
>
> 	fdput(f);
>-- 
>2.29.0
>