From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oliver Neukum Subject: Re: [PATCH] sky2: detect and prevent kernel panic from a possible faulty hardware Date: Wed, 31 Jul 2013 10:18:13 +0200 Message-ID: <1375258693.17175.4.camel@linux-fkkt.site> References: <1374957354-10142-1-git-send-email-sam8641@gmail.com> <20130730.161906.1898764862564749288.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: sam8641@gmail.com, mlindner@marvell.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: David Miller Return-path: Received: from cantor2.suse.de ([195.135.220.15]:44165 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759231Ab3GaISP (ORCPT ); Wed, 31 Jul 2013 04:18:15 -0400 In-Reply-To: <20130730.161906.1898764862564749288.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, 2013-07-30 at 16:19 -0700, David Miller wrote: > From: Samuel Williams > Date: Sat, 27 Jul 2013 15:35:54 -0500 > > > A possible faulty hardware might interrupt with a status of 0xffffffff which > > may kernel panic if sky2 driver tries to handle it. Detecting this problem > > may avoid kernel panic. > > > > Signed-off-by: Samuel Williams > > Sure, but then the device is basically just going to stop. > > I think this change needs to be more comprehensive, for example it > needs to fully reset the device and try to get it operational again. But sky2 is used in some hotremovable cards. You can hit the window for this race. It is unlikely but still this simple check fixes a bug. Regards Oliver