[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7ee5d705c12d770bf7566bce7d664bf733b25206.camel@intel.com>
Date: Tue, 25 Jun 2019 20:47:26 +0000
From: "Derrick, Jonathan" <jonathan.derrick@...el.com>
To: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"zub@...ux.fjfi.cvut.cz" <zub@...ux.fjfi.cvut.cz>,
"linux-block@...r.kernel.org" <linux-block@...r.kernel.org>,
"sbauer@...donthack.me" <sbauer@...donthack.me>,
"axboe@...nel.dk" <axboe@...nel.dk>
CC: "jonas.rabenstein@...dium.uni-erlangen.de"
<jonas.rabenstein@...dium.uni-erlangen.de>
Subject: Re: [PATCH v2 0/3] block: sed-opal: add support for shadow MBR done
flag and write
These are still good with me and we'll likely have a similar future use
for passing data through the ioctl.
Could we get this staged for 5.3?
On Tue, 2019-05-21 at 22:46 +0200, David Kozub wrote:
> This patch series extends SED Opal support: it adds IOCTL for setting the shadow
> MBR done flag which can be useful for unlocking an Opal disk on boot and it adds
> IOCTL for writing to the shadow MBR.
>
> This applies on current master.
>
> I successfully tested toggling the MBR done flag and writing the shadow MBR
> using some tools I hacked together[1] with a Samsung SSD 850 EVO drive.
>
> Changes from v1:
> * PATCH 2/3: remove check with access_ok, just rely on copy_from_user as
> suggested in [2] (I tested passing data == 0 and I got the expected EFAULT)
>
> [1] https://gitlab.com/zub2/opalctl
> [2] https://lore.kernel.org/lkml/20190501134833.GB24132@infradead.org/
>
> Jonas Rabenstein (3):
> block: sed-opal: add ioctl for done-mark of shadow mbr
> block: sed-opal: ioctl for writing to shadow mbr
> block: sed-opal: check size of shadow mbr
>
> block/opal_proto.h | 16 ++++
> block/sed-opal.c | 157 +++++++++++++++++++++++++++++++++-
> include/linux/sed-opal.h | 2 +
> include/uapi/linux/sed-opal.h | 20 +++++
> 4 files changed, 193 insertions(+), 2 deletions(-)
>
Download attachment "smime.p7s" of type "application/x-pkcs7-signature" (3278 bytes)
Powered by blists - more mailing lists