public inbox for linux-block@vger.kernel.org
 help / color / mirror / Atom feed
From: Milan Broz <gmazyland@gmail.com>
To: Ondrej Kozina <okozina@redhat.com>, linux-block@vger.kernel.org
Cc: gjoyce@ibm.com, jonathan.derrick@linux.dev
Subject: Re: [PATCH 0/6] Extend Single User Mode (SUM) support in sed-opal.
Date: Thu, 5 Feb 2026 12:03:19 +0100	[thread overview]
Message-ID: <7ac95fb5-b206-4bb1-ad75-20a5000cd5f7@gmail.com> (raw)
In-Reply-To: <20260130162527.570255-1-okozina@redhat.com>

On 1/30/26 5:25 PM, Ondrej Kozina wrote:
> This patch series extends support for TCG Storage Opal SSC Single User
> Mode (SUM) in the sed-opal kernel interface. SUM addresses few
> shortcomins described in the chapter 2 of the OPAL2 sepecification
> extension (see below). It better isolates Admin authority from User
> stored data by taking the Lock/Unlock/Enable/Disable privileges from
> Admin when managing Locking Ranges configured for SUM.
> 
> This permission split necessitates separate ioctl calls for operations
> that previously could be performed atomically under a single authority.
> The series refactors existing code and introduces new ioctls to support
> this model:
> 
> - IOC_OPAL_REACTIVATE_LSP: Switch an active OPAL2 device to/from SUM
> - IOC_OPAL_LR_SET_START_LEN: Set locking range boundaries only
> - IOC_OPAL_ENABLE_DISABLE_LR: Set lock enable states only
> - IOC_OPAL_GET_SUM_STATUS: Query SUM configuration status
> 
> References:
> - TCG Storage Opal SSC Feature Set: Single User Mode specification
> - cryptsetup code using the extended sed-opal interface:
>    https://gitlab.com/cryptsetup/cryptsetup/-/merge_requests/832

I have tested this + cryptsetup userspace on my collection of SED devices
(some SATA, but mostly NVMe).

There are several broken Opal2 implementations, as vendors
usually refuses to fix it as it is out of support, we need detect
"supportable" SUM.

Anyway, kernel ioctls should remain simple here, in general, LGTM.

Please check comments on for separate patches.
Once fixed, I'll add my review/tested signature.

Thanks!
Milan


  parent reply	other threads:[~2026-02-05 11:03 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-30 16:25 [PATCH 0/6] Extend Single User Mode (SUM) support in sed-opal Ondrej Kozina
2026-01-30 16:25 ` [PATCH 1/6] sed-opal: add IOC_OPAL_REACTIVATE_LSP Ondrej Kozina
2026-02-02 23:17   ` Hannes Reinecke
2026-02-05 10:53   ` Milan Broz
2026-01-30 16:25 ` [PATCH 2/6] sed-opal: refactor (split) IOC_OPAL_LR_SETUP internals Ondrej Kozina
2026-01-30 16:25 ` [PATCH 3/6] sed-opal: add IOC_OPAL_LR_SET_START_LEN ioctl Ondrej Kozina
2026-01-30 16:25 ` [PATCH 4/6] sed-opal: add IOC_OPAL_ENABLE_DISABLE_LR Ondrej Kozina
2026-01-30 16:25 ` [PATCH 5/6] sed-opal: increase column attribute type size to 64 bits Ondrej Kozina
2026-01-30 16:25 ` [PATCH 6/6] sed-opal: add IOC_OPAL_GET_SUM_STATUS ioctl Ondrej Kozina
2026-02-05 10:58   ` Milan Broz
2026-02-05 11:03 ` Milan Broz [this message]
2026-02-05 15:09 ` [PATCH v2 0/9] Extend Single User Mode (SUM) support in sed-opal Ondrej Kozina
2026-02-06 14:17   ` [PATCH v3 " Ondrej Kozina
2026-02-26 13:47     ` Ondrej Kozina
2026-02-26 15:34       ` Jens Axboe
2026-02-26 15:41     ` Jens Axboe
2026-02-06 14:17   ` [PATCH v3 1/9] sed-opal: add UID of Locking Table Ondrej Kozina
2026-02-06 15:30     ` Milan Broz
2026-02-06 14:17   ` [PATCH v3 2/9] sed-opal: add RangeStartRangeLengthPolicy parameter Ondrej Kozina
2026-02-06 15:31     ` Milan Broz
2026-02-06 14:17   ` [PATCH v3 3/9] sed-opal: add Admin1PIN parameter Ondrej Kozina
2026-02-06 15:31     ` Milan Broz
2026-02-06 14:17   ` [PATCH v3 4/9] sed-opal: add IOC_OPAL_REACTIVATE_LSP Ondrej Kozina
2026-02-06 15:32     ` Milan Broz
2026-02-06 14:17   ` [PATCH v3 5/9] sed-opal: refactor (split) IOC_OPAL_LR_SETUP internals Ondrej Kozina
2026-02-06 15:33     ` Milan Broz
2026-02-06 14:18   ` [PATCH v3 6/9] sed-opal: add IOC_OPAL_LR_SET_START_LEN ioctl Ondrej Kozina
2026-02-06 15:34     ` Milan Broz
2026-02-06 14:18   ` [PATCH v3 7/9] sed-opal: add IOC_OPAL_ENABLE_DISABLE_LR Ondrej Kozina
2026-02-06 15:35     ` Milan Broz
2026-02-06 14:18   ` [PATCH v3 8/9] sed-opal: increase column attribute type size to 64 bits Ondrej Kozina
2026-02-06 15:35     ` Milan Broz
2026-02-06 14:18   ` [PATCH v3 9/9] sed-opal: add IOC_OPAL_GET_SUM_STATUS ioctl Ondrej Kozina
2026-02-06 15:36     ` Milan Broz
2026-02-05 15:09 ` [PATCH v2 1/9] sed-opal: add UID of Locking Table Ondrej Kozina
2026-02-06  0:04   ` Hannes Reinecke
2026-02-05 15:09 ` [PATCH v2 2/9] sed-opal: add RangeStartRangeLengthPolicy parameter Ondrej Kozina
2026-02-06  0:05   ` Hannes Reinecke
2026-02-05 15:09 ` [PATCH v2 3/9] sed-opal: add Admin1PIN parameter Ondrej Kozina
2026-02-06  0:06   ` Hannes Reinecke
2026-02-05 15:09 ` [PATCH v2 4/9] sed-opal: add IOC_OPAL_REACTIVATE_LSP Ondrej Kozina
2026-02-06  0:08   ` Hannes Reinecke
2026-02-05 15:09 ` [PATCH v2 5/9] sed-opal: refactor (split) IOC_OPAL_LR_SETUP internals Ondrej Kozina
2026-02-05 15:09 ` [PATCH v2 6/9] sed-opal: add IOC_OPAL_LR_SET_START_LEN ioctl Ondrej Kozina
2026-02-05 15:09 ` [PATCH v2 7/9] sed-opal: add IOC_OPAL_ENABLE_DISABLE_LR Ondrej Kozina
2026-02-05 15:09 ` [PATCH v2 8/9] sed-opal: increase column attribute type size to 64 bits Ondrej Kozina
2026-02-05 15:09 ` [PATCH v2 9/9] sed-opal: add IOC_OPAL_GET_SUM_STATUS ioctl Ondrej Kozina
2026-02-06  9:05   ` Milan Broz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7ac95fb5-b206-4bb1-ad75-20a5000cd5f7@gmail.com \
    --to=gmazyland@gmail.com \
    --cc=gjoyce@ibm.com \
    --cc=jonathan.derrick@linux.dev \
    --cc=linux-block@vger.kernel.org \
    --cc=okozina@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox