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
| ||
|
Message-ID: <CAKgT0UeMvwrjyj9VvJu63JjYSWKWvDNg9YLerachV8wezQES_g@mail.gmail.com> Date: Tue, 26 Jul 2016 09:59:08 -0700 From: Alexander Duyck <alexander.duyck@...il.com> To: Tom Walsh <tomwalsh73@...il.com> Cc: Netdev <netdev@...r.kernel.org> Subject: Re: patch for PCH_GBE rotten packet (Intel EG20T Platform Controller). On Tue, Jul 26, 2016 at 9:00 AM, Tom Walsh <tomwalsh73@...il.com> wrote: > Not sure how this goes, but, we found a race condition in > drivers/net/ethernet/oki-semi/pch_gbe/ > > The attached patch corrects for this. > > TomW So a few minor changes are probably needed. First we don't recommend the use of camel case for variable names. So "resetStatus" should probably be renamed to "reset_status". Second is the fact that you seemed to have removed a tab that didn't need to be removed before the read of the INT_EN register. Third you need to add a "Signed-off-by" containing your name and email address. Finally we usually require the patches to be emailed inline to avoid having the attachments stripped. You might also wan to go through the steps called out in https://www.kernel.org/doc/Documentation/SubmittingPatches Hope this helps. - Alex
Powered by blists - more mailing lists