Re: [PATCH 07/11] block: use kstrtoul in flag_store

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

To: Christoph Hellwig <hch@xxxxxx>
Subject: Re: [PATCH 07/11] block: use kstrtoul in flag_store
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Date: Mon, 10 Jun 2024 08:05:24 -0400
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>, Keith Busch <kbusch@xxxxxxxxxx>, 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: <20240607055912.3586772-8-hch@lst.de> (Christoph Hellwig's message of "Fri, 7 Jun 2024 07:59:01 +0200")
Organization: Oracle Corporation
References: <20240607055912.3586772-1-hch@lst.de> <20240607055912.3586772-8-hch@lst.de>

Christoph,

> Use the text to integer helper that has error handling and doesn't modify
> the input pointer.

Reviewed-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx>

-- 
Martin K. Petersen	Oracle Linux Engineering

References:
- move integrity settings to queue_limits v2
  - From: Christoph Hellwig
- [PATCH 07/11] block: use kstrtoul in flag_store
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 08/11] block: don't require stable pages for non-PI metadata
Next by Date: Re: [PATCH 03/11] block: remove the BIP_IP_CHECKSUM flag
Previous by thread: Re: [PATCH 07/11] block: use kstrtoul in flag_store
Next by thread: [PATCH 08/11] block: don't require stable pages for non-PI metadata
Index(es):
- Date
- Thread

[Index of Archives] [Linux RAID Wiki] [ATA RAID] [Linux SCSI Target Infrastructure] [Linux Block] [Linux IDE] [Linux SCSI] [Linux Hams] [Device Mapper] [Device Mapper Cryptographics] [Kernel] [Linux Admin] [Linux Net] [GFS] [RPM] [git] [Yosemite Forum]