Re: [PATCH 06/12] block: factor out flag_{store,show} helper for integrity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Christoph Hellwig <hch@xxxxxx>
Subject: Re: [PATCH 06/12] block: factor out flag_{store,show} helper for integrity
From: Keith Busch <kbusch@xxxxxxxxxx>
Date: Wed, 5 Jun 2024 10:00:35 -0600
Cc: Jens Axboe <axboe@xxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, Mike Snitzer <snitzer@xxxxxxxxxx>, Mikulas Patocka <mpatocka@xxxxxxxxxx>, Song Liu <song@xxxxxxxxxx>, Yu Kuai <yukuai3@xxxxxxxxxx>, Dan Williams <dan.j.williams@xxxxxxxxx>, Vishal Verma <vishal.l.verma@xxxxxxxxx>, Dave Jiang <dave.jiang@xxxxxxxxx>, Ira Weiny <ira.weiny@xxxxxxxxx>, Sagi Grimberg <sagi@xxxxxxxxxxx>, Chaitanya Kulkarni <kch@xxxxxxxxxx>, linux-block@xxxxxxxxxxxxxxx, dm-devel@xxxxxxxxxxxxxxx, linux-raid@xxxxxxxxxxxxxxx, nvdimm@xxxxxxxxxxxxxxx, linux-nvme@xxxxxxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx
In-reply-to: <20240605063031.3286655-7-hch@lst.de>
References: <20240605063031.3286655-1-hch@lst.de> <20240605063031.3286655-7-hch@lst.de>

On Wed, Jun 05, 2024 at 08:28:35AM +0200, Christoph Hellwig wrote:
> Factor the duplicate code for the generate and verify attributes into
> common helpers.

Looks good.

Reviewed-by: Keith Busch <kbusch@xxxxxxxxxx>

References:
- move integrity settings to queue_limits
  - From: Christoph Hellwig
- [PATCH 06/12] block: factor out flag_{store,show} helper for integrity
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 01/12] dm-integrity: use the nop integrity profile
Next by Date: Re: [PATCH 07/12] block: use kstrtoul in flag_store
Previous by thread: [PATCH 06/12] block: factor out flag_{store,show} helper for integrity
Next by thread: Re: [PATCH 06/12] block: factor out flag_{store,show} helper for integrity
Index(es):
- Date
- Thread

[Index of Archives] [Linux RAID] [Linux SCSI] [Linux ATA RAID] [IDE] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Device Mapper]