linux-kernel - Re: linux-next: manual merge of the device-mapper tree with the block tree

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: <20250718185642.0f2454c3@canb.auug.org.au>
Date: Fri, 18 Jul 2025 18:56:42 +1000
From: Stephen Rothwell <sfr@...b.auug.org.au>
To: John Garry <john.g.garry@...cle.com>
Cc: Alasdair G Kergon <agk@...hat.com>, Mike Snitzer <snitzer@...nel.org>,
 Jens Axboe <axboe@...nel.dk>, Linux Kernel Mailing List
 <linux-kernel@...r.kernel.org>, Linux Next Mailing List
 <linux-next@...r.kernel.org>, Mikulas Patocka <mpatocka@...hat.com>
Subject: Re: linux-next: manual merge of the device-mapper tree with the
 block tree

Hi John,

On Fri, 18 Jul 2025 09:26:57 +0100 John Garry <john.g.garry@...cle.com> wrote:
>
> I think that this is the proper merge resolution:
> 
> static void stripe_io_hints(struct dm_target *ti,
>      struct queue_limits *limits)
> {
> 	struct stripe_c *sc = ti->private;
> 	unsigned int io_min, io_opt;
> 
> 	if (!check_shl_overflow(sc->chunk_size, SECTOR_SHIFT, &io_min) && !check_mul_overflow(io_min, sc->stripes, &io_opt)) {
> 		limits->io_min = io_min;
> 		limits->io_opt = io_opt;
> 	}
> 	limits->chunk_sectors = sc->chunk_size;
> }
> 
> 
> For purpose of atomic writes, we should always set chunk_sectors.

OK, I have changed my resolution to that starting from Monday.

> BTW, I tried to apply the conflicting patches from the block tree on -next from 17 July, and I was getting strange behaviour:
> 
> # vgcreate vg00 /dev/sda /dev/sdb /dev/sdc /dev/sdd
>   WARNING: Unknown logical_block_size for device /dev/sda.
>   WARNING: Unknown logical_block_size for device /dev/sdb.
>   WARNING: Unknown logical_block_size for device /dev/sdc.
>   WARNING: Unknown logical_block_size for device /dev/sdd.
>   Physical volume "/dev/sda" successfully created.
>   Physical volume "/dev/sdb" successfully created.
>   Physical volume "/dev/sdc" successfully created.
>   Physical volume "/dev/sdd" successfully created.
>   Volume group "vg00" successfully created
> #
> 
> I had no such problem on Jens' block for-6.17 tree.

I have no idea what caused that, sorry.

-- 
Cheers,
Stephen Rothwell

Content of type "application/pgp-signature" skipped