linux-kernel - Re: [PATCH 3/5] cciss: Use kernel provided PCI state save and restore functions

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: <4CC1EACC.9040500@kernel.dk>
Date:	Fri, 22 Oct 2010 21:49:32 +0200
From:	Jens Axboe <axboe@...nel.dk>
To:	Stephen Cameron <smcameron@...oo.com>
CC:	"Stephen M. Cameron" <scameron@...rdog.cce.hp.com>,
	akpm@...ux-foundation.org, linux-kernel@...r.kernel.org,
	mike.miller@...com, thenzl@...hat.com
Subject: Re: [PATCH 3/5] cciss: Use kernel provided PCI state save and restore
 functions

On 2010-10-22 21:36, Stephen Cameron wrote:
> --- On Fri, 10/22/10, Stephen M. Cameron <scameron@...rdog.cce.hp.com> wrote:
> 
>> From: Stephen M. Cameron <scameron@...rdog.cce.hp.com>
>> Subject: [PATCH 3/5] cciss: Use kernel provided PCI state save and restore functions
>> To: axboe@...nel.dk
>> Cc: akpm@...ux-foundation.org, smcameron@...oo.com, linux-kernel@...r.kernel.org, mike.miller@...com, thenzl@...hat.com
>> Date: Friday, October 22, 2010, 2:21 PM
>> From: Stephen M. Cameron <scameron@...rdog.cce.hp.com>
>>
>> and use the doorbell reset method if available (which doesn't
>> lock up the controller if you properly save and restore
>> all the PCI registers that you're supposed to.)
> 
> 
> This patch, and all these cciss patches in this series, are vs. the jens/for-2.6.37/drivers branch.
> 
> One difference I noticed between linux-2.6.36 and jens/for-2.6.37/drivers is that linux-2.6.36 contains this in drivers/block/cciss.c
> in the function cciss_kdump_hard_reset_controller:
> 
>         /* The doorbell reset seems to cause lockups on some Smart
>          * Arrays (e.g. P410, P410i, maybe others).  Until this is
>          * fixed or at least isolated, avoid the doorbell reset.
>          */
>         use_doorbell = 0;
> 
> This is not present in jens/for-2.6.37/drivers.
> 
> That was added to linus tree by me, because I'd noticed we were
> getting controller lockups and subsequent data corruption when
> attempting kdumps using the doorbell reset when (unbeknownst to me at
> the time) the pci state saving and restoring code wasn't quite right.
> So these patches fix the pci state saving and restoring code, so the
> above "use_doorbell = 0" needs to come out when these patches go into
> Linus's tree.  But such a patch cannot be made vs. Jens's tree,
> because that code isn't in Jens's tree.

I sent a pull request to Linus earlier today and I know the drivers
branch merges cleanly into -git, so we should be fine.

I'll get this series queued up as well, but it'll be for a later round
early next week. JFYI.

-- 
Jens Axboe

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/