mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* Subject: [PATCH 0/1] Initial NVME storage erase operation
@ 2025-01-16 10:52 Renaud Barbier
  2025-01-16 11:15 ` Ahmad Fatoum
  0 siblings, 1 reply; 2+ messages in thread
From: Renaud Barbier @ 2025-01-16 10:52 UTC (permalink / raw)
  To: Barebox List

This patch introduces the erase entry point for the NVME storage.
It uses the nvme sanitize command 2 for a low level block erase
of the entire device as start and end blocks cannot be specified.

Note that in the current state, the user should first remove existing
partitions with the command 'parted rmpart' as they will stay dangling
in barebox device nodes and mount points.

Renaud Barbier (1):
  nvme: add erase operation through sanitize command

 drivers/nvme/host/core.c | 82 ++++++++++++++++++++++++++++++++++++++++
 drivers/nvme/host/pci.c  |  2 +
 include/nvme/types.h     | 38 +++++++++++++++++++
 3 files changed, 122 insertions(+)
 create mode 100644 include/nvme/types.h

-- 
2.34.1



^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Subject: [PATCH 0/1] Initial NVME storage erase operation
  2025-01-16 10:52 Subject: [PATCH 0/1] Initial NVME storage erase operation Renaud Barbier
@ 2025-01-16 11:15 ` Ahmad Fatoum
  0 siblings, 0 replies; 2+ messages in thread
From: Ahmad Fatoum @ 2025-01-16 11:15 UTC (permalink / raw)
  To: Renaud Barbier, Barebox List

Hello Renaud,

On 16.01.25 11:52, Renaud Barbier wrote:
> This patch introduces the erase entry point for the NVME storage.
> It uses the nvme sanitize command 2 for a low level block erase
> of the entire device as start and end blocks cannot be specified.
> 
> Note that in the current state, the user should first remove existing
> partitions with the command 'parted rmpart' as they will stay dangling
> in barebox device nodes and mount points.

Thanks for your patch. Your mailer seems to have mangled the series
a bit:
   - The patch is not a reply to the cover letter
   - "Subject: [PATCH 0/1]" ends up in the commit message title

I think Sascha can fix it up on apply, but jfyi, you can use b4 with barebox
now, including the kernel.org web relay:

https://b4.docs.kernel.org/en/latest/contributor/send.html#authenticating-with-the-web-submission-endpoint

This takes care to send patches unmangled by corporate IT.

Cheers,
Ahmad
  

> 
> Renaud Barbier (1):
>   nvme: add erase operation through sanitize command
> 
>  drivers/nvme/host/core.c | 82 ++++++++++++++++++++++++++++++++++++++++
>  drivers/nvme/host/pci.c  |  2 +
>  include/nvme/types.h     | 38 +++++++++++++++++++
>  3 files changed, 122 insertions(+)
>  create mode 100644 include/nvme/types.h
> 


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-01-16 11:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-01-16 10:52 Subject: [PATCH 0/1] Initial NVME storage erase operation Renaud Barbier
2025-01-16 11:15 ` Ahmad Fatoum

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox