linux-kernel - Re: [axboe-block:for-next] [block] bd4a633b6f: fsmark.files_per

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <Znl4lXRmK2ukDB7r@ryzen.lan>
Date: Mon, 24 Jun 2024 15:45:57 +0200
From: Niklas Cassel <cassel@...nel.org>
To: Christoph Hellwig <hch@....de>
Cc: kernel test robot <oliver.sang@...el.com>, oe-lkp@...ts.linux.dev,
	lkp@...el.com, Jens Axboe <axboe@...nel.dk>,
	Damien Le Moal <dlemoal@...nel.org>, Hannes Reinecke <hare@...e.de>,
	linux-m68k@...ts.linux-m68k.org, linux-um@...ts.infradead.org,
	linux-kernel@...r.kernel.org, linux-block@...r.kernel.org,
	drbd-dev@...ts.linbit.com, nbd@...er.debian.org,
	linuxppc-dev@...ts.ozlabs.org, ceph-devel@...r.kernel.org,
	virtualization@...ts.linux.dev, xen-devel@...ts.xenproject.org,
	linux-bcache@...r.kernel.org, dm-devel@...ts.linux.dev,
	linux-raid@...r.kernel.org, linux-mmc@...r.kernel.org,
	linux-mtd@...ts.infradead.org, nvdimm@...ts.linux.dev,
	linux-nvme@...ts.infradead.org, linux-s390@...r.kernel.org,
	linux-scsi@...r.kernel.org, ying.huang@...el.com,
	feng.tang@...el.com, fengwei.yin@...el.com
Subject: Re: [axboe-block:for-next] [block]  bd4a633b6f: fsmark.files_per_sec
 -64.5% regression

On Mon, Jun 24, 2024 at 10:35:37AM +0200, Christoph Hellwig wrote:
> This is odd to say at least.  Any chance you can check the value
> of /sys/block/$DEVICE/queue/rotational for the relevant device before
> and after this commit?  And is this an ATA or NVMe SSD?
> 

Seems to be ATA SSD:
https://download.01.org/0day-ci/archive/20240624/202406241546.6bbd44a7-oliver.sang@intel.com/job.yaml

ssd_partitions: "/dev/disk/by-id/ata-INTEL_SSDSC2BG012T4_BTHC428201ZX1P2OGN-part1"

Most likely btrfs does something different depending on the nonrot flag
being set or not. (And like you are suggesting, most likely the value of
the nonrot flag is somehow different after commit bd4a633b6f)


Kind regards,
Niklas