Re: [PATCH 2/2] block: check that there is a plug in blk_flush_plug

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

To: Christoph Hellwig <hch@xxxxxx>, "axboe@xxxxxxxxx" <axboe@xxxxxxxxx>
Subject: Re: [PATCH 2/2] block: check that there is a plug in blk_flush_plug
From: Chaitanya Kulkarni <chaitanyak@xxxxxxxxxx>
Date: Thu, 27 Jan 2022 07:47:52 +0000
Accept-language: en-US
Cc: "linux-block@xxxxxxxxxxxxxxx" <linux-block@xxxxxxxxxxxxxxx>
In-reply-to: <20220127070549.1377856-2-hch@lst.de>
References: <20220127070549.1377856-1-hch@lst.de> <20220127070549.1377856-2-hch@lst.de>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0

On 1/26/22 11:05 PM, Christoph Hellwig wrote:
> Rename blk_flush_plug to __blk_flush_plug and add a wrapper that includes
> the NULL check instead of open coding that check everywhere.
> 
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>


Looks good.

Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx>

References:
- [PATCH 1/2] block: remove blk_needs_flush_plug
  - From: Christoph Hellwig
- [PATCH 2/2] block: check that there is a plug in blk_flush_plug
  - From: Christoph Hellwig

Prev by Date: [GIT PULL] nvme fixes for Linux 5.17
Next by Date: Re: [v5 PATCH] block: introduce block_rq_error tracepoint
Previous by thread: [PATCH 2/2] block: check that there is a plug in blk_flush_plug
Next by thread: Re: [PATCH 1/2] block: remove blk_needs_flush_plug
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]

Powered by Linux