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: <4638c185-ac5c-41e3-8c79-5c995228ec0e@wdc.com>
Date: Tue, 7 Oct 2025 13:08:00 +0000
From: Johannes Thumshirn <Johannes.Thumshirn@....com>
To: hch <hch@....de>
CC: Jens Axboe <axboe@...nel.dk>, Steven Rostedt <rostedt@...dmis.org>, Masami
 Hiramatsu <mhiramat@...nel.org>, Mathieu Desnoyers
	<mathieu.desnoyers@...icios.com>, "linux-block@...r.kernel.org"
	<linux-block@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "linux-trace-kernel@...r.kernel.org"
	<linux-trace-kernel@...r.kernel.org>, "linux-btrace@...r.kernel.org"
	<linux-btrace@...r.kernel.org>, John Garry <john.g.garry@...cle.com>, Hannes
 Reinecke <hare@...e.de>, Damien Le Moal <dlemoal@...nel.org>, Naohiro Aota
	<Naohiro.Aota@....com>, Shinichiro Kawasaki <shinichiro.kawasaki@....com>,
	Chaitanya Kulkarni <chaitanyak@...dia.com>, "Martin K . Petersen"
	<martin.petersen@...cle.com>
Subject: Re: [PATCH v2 11/15] blktrace: add block trace commands for zone
 operations

On 10/3/25 9:33 AM, Christoph Hellwig wrote:
> On Thu, Sep 25, 2025 at 05:02:27PM +0200, Johannes Thumshirn wrote:
>> Add block trace commands for zone operations. These are added as a
>> separate set of 'block trace commands' shifted by 32bit so that they do
>> not interfere with the old 16bit wide trace command field in 'struct
>> blk_io_trace' action.
> Can you explain how the commands are handled for old/new here?
>
> Because I'd still much prefer to sort things out so that they make
> sense for the new code if possible.  i.e. have a 32-bit command
> and 32 bit flags, and use sensible encoding for the new one, and
> remap the supported once to the old organically grown one.

Sure for the old commands everything is still in the lower 32bits, this 
has the nice property that we don't need to duplicate all the code for 
v1 and v2.

The commands added afterwards are intended to be in the upper 32bits, 
which are discarded if the user requests the v1 format.

At least this was the original plan. I think I badly messed up v2 as the 
new commands should re-start at 0 and be shifted up by 32bits.


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ