RE: [PATCH net] hvsock: fix epollout hang from race condition
From: Sunil Muthuswamy <hidden>
Date: 2019-06-17 19:28:29
Also in:
linux-hyperv, lkml
-----Original Message----- From: David Miller <davem@davemloft.net> Sent: Monday, June 17, 2019 11:56 AM To: Sunil Muthuswamy <redacted> Cc: Dexuan Cui <decui@microsoft.com>; KY Srinivasan <kys@microsoft.com>; Haiyang Zhang <haiyangz@microsoft.com>; Stephen Hemminger [off-list ref]; sashal@kernel.org; Michael Kelley [off-list ref]; netdev@vger.kernel.org; linux-hyperv@vger.kernel.org; linux-kernel@vger.kernel.org Subject: Re: [PATCH net] hvsock: fix epollout hang from race condition From: Sunil Muthuswamy <redacted> Date: Mon, 17 Jun 2019 18:47:08 +0000quoted
quoted
-----Original Message----- From: linux-hyperv-owner@vger.kernel.org <redacted> On Behalf Of David Miller Sent: Sunday, June 16, 2019 1:55 PM To: Dexuan Cui <decui@microsoft.com> Cc: Sunil Muthuswamy <redacted>; KY Srinivasan <kys@microsoft.com>; Haiyang Zhang[off-list ref];quoted
quoted
Stephen Hemminger [off-list ref]; sashal@kernel.org; Michael Kelley [off-list ref]; netdev@vger.kernel.org; linux-hyperv@vger.kernel.org; linux-kernel@vger.kernel.org Subject: Re: [PATCH net] hvsock: fix epollout hang from race condition From: Dexuan Cui <decui@microsoft.com> Date: Sat, 15 Jun 2019 03:22:32 +0000quoted
These warnings are not introduced by this patch from Sunil. I'm not sure why I didn't notice these warnings before. Probably my gcc version is not new eought? Actually these warnings are bogus, as I checked the related functions, which may confuse the compiler's static analysis. I'm going to make a patch to initialize the pointers to NULL to suppress the warnings. My patch will be based on the latest's net.git + this patch from Sunil.Sunil should then resubmit his patch against something that has the warning suppression patch applied.David, Dexuan's patch to suppress the warnings seems to be applied now to the 'net' branch. Can we please get this patch applied as well?I don't know how else to say "Suni should then resubmit his patch" Please just resubmit it!
The patch does not change at all. So, I was hoping we could reapply it. But, I have resubmitted the patch. Thanks.