[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <34848777-b370-4a63-8b08-c3246d214167@suse.com>
Date: Wed, 3 Dec 2025 09:42:38 +0100
From: Hannes Reinecke <hare@...e.com>
To: Christoph Hellwig <hch@...radead.org>, Po-Wen Kao <powenkao@...gle.com>
Cc: "James E.J. Bottomley" <James.Bottomley@...senpartnership.com>,
"Martin K. Petersen" <martin.petersen@...cle.com>,
"open list:SCSI SUBSYSTEM" <linux-scsi@...r.kernel.org>,
open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 1/1] scsi: core: Fix error handler encryption support
On 12/3/25 08:38, Christoph Hellwig wrote:
> On Wed, Dec 03, 2025 at 07:33:08AM +0000, Po-Wen Kao wrote:
>> From: Brian Kao <powenkao@...gle.com>
>>
>> Some low-level drivers (LLD) access block layer crypto fields, such as
>> rq->crypt_keyslot and rq->crypt_ctx within `struct request`, to
>> configure hardware for inline encryption.
>
> So don't do that except for commands that can actually be encrypted,
> i.e. those that have non-zero payload size. I think you really want
> to fix this in the driver.
>
> And we really need to stop passing scsi_cmnds to the error handler.
>
> Hannes, any chance you could send another batch of your decades old
> series?
>
There had been an intersection with the reserved command stuff, but
now that Bart has dusted things off there I guess I should give it
another go.
Cheers,
Hannes
--
Dr. Hannes Reinecke Kernel Storage Architect
hare@...e.com +49 911 74053 688
SUSE Software Solutions GmbH, Frankenstr. 146, 90461 Nürnberg
HRB 36809 (AG Nürnberg), GF: I. Totev, A. McDonald, W. Knoblich
Powered by blists - more mailing lists