[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]
Re: [dm-devel] [PATCH 42/41 v2] dm: convey that all flushes are processed as empty
- From: Tejun Heo <tj kernel org>
- To: Mike Snitzer <snitzer redhat com>
- Cc: k-ueda ct jp nec com, jack suse cz, linux-scsi vger kernel org, mst redhat com, jaxboe fusionio com, rusty rustcorp com au, linux-kernel vger kernel org, swhiteho redhat com, linux-raid vger kernel org, linux-ide vger kernel org, device-mapper development <dm-devel redhat com>, James Bottomley suse de, mpatocka redhat com, tytso mit edu, konishi ryusuke lab ntt co jp, linux-fsdevel vger kernel org, vst vlnb net, rwheeler redhat com, Christoph Hellwig <hch lst de>, chris mason oracle com, Milan Broz <mbroz redhat com>
- Subject: Re: [dm-devel] [PATCH 42/41 v2] dm: convey that all flushes are processed as empty
- Date: Wed, 08 Sep 2010 18:09:43 +0200
On 09/08/2010 04:04 AM, Mike Snitzer wrote:
> Rename __clone_and_map_flush to __clone_and_map_empty_flush for added
> clarity.
>
> Simplify logic associated with REQ_FLUSH conditionals.
>
> Introduce a BUG_ON() and add a few more helpful comments to the code
> so that it is clear that all flushes are empty.
>
> Cleanup __split_and_process_bio() so that an empty flush isn't processed
> by a 'sector_count' focused while loop.
>
> Signed-off-by: Mike Snitzer <snitzer redhat com>
> ---
> drivers/md/dm.c | 34 +++++++++++++++-------------------
> 1 files changed, 15 insertions(+), 19 deletions(-)
>
> Tejun, please feel free to fold this patch into (or insert after)
> 0025-dm-relax-ordering-of-bio-based-flush-implementation.patch
I added it after 26 and updated flush-fua branch accordingly.
Thank you.
--
tejun
[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]