Re: [PATCH 03/13 V2] blk: make the bioset rescue_workqueue optional.

From	Ming Lei <tom.leiming@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 03/13 V2] blk: make the bioset rescue_workqueue optional.
Date	2017-05-03 05:30 +0200
Message-ID	<tCT9f-7qB-9@gated-at.bofh.it> (permalink)
References	<tCwZ3-ZU-3@gated-at.bofh.it> <tCwZ3-ZU-1@gated-at.bofh.it> <tCOCC-4jl-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Wed, May 3, 2017 at 6:34 AM, NeilBrown <neilb@suse.com> wrote:
> From 09017acf74ec4df674b78ca66f0924187f10d8a4 Mon Sep 17 00:00:00 2001
> From: NeilBrown <neilb@suse.com>
> Date: Fri, 10 Mar 2017 13:59:50 +1100
> Subject: [PATCH] blk: make the bioset rescue_workqueue optional.
>
> This patch converts bioset_create() to not create a workqueue by
> default, so alloctions will never trigger punt_bios_to_rescuer().  It
> also introduces a new flag BIOSET_NEED_RESCUER() which tells
> bioset_create() to preserve the old behavior.
>
> All callers of bioset_create() that are inside block device drivers,
> are given the BIOSET_NEED_RESCUER().
>
> biosets used by filesystems or other top-level users do not
> need rescuing as the bio can never be queued behind other
> bios.  This includes fs_bio_set, blkdev_dio_pool,
> btrfs_bioset, xfs_ioend_bioset, and one allocated by
> target_core_iblock.c.
>
> biosets used by md/raid do not need rescuing as
> their usage was recently audited and revised to never
> risk deadlock.
>
> It is hoped that most, if not all, of the remaining biosets
> can end up being the non-rescued version.
>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
> Credit-to: Ming Lei <ming.lei@redhat.com> (minor fixes)
> Signed-off-by: NeilBrown <neilb@suse.com>

Reviewed-by: Ming Lei <ming.lei@redhat.com>


Thanks,
Ming

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 03/13] blk: make the bioset rescue_workqueue optional. NeilBrown <neilb@suse.com> - 2017-05-02 05:50 +0200
  Re: [PATCH 03/13] blk: make the bioset rescue_workqueue optional. Christoph Hellwig <hch@infradead.org> - 2017-05-02 10:20 +0200
  Re: [PATCH 03/13] blk: make the bioset rescue_workqueue optional. Ming Lei <ming.lei@redhat.com> - 2017-05-02 13:10 +0200
    Re: [PATCH 03/13] blk: make the bioset rescue_workqueue optional. NeilBrown <neilb@suse.com> - 2017-05-03 00:20 +0200
  [PATCH 03/13 V2] blk: make the bioset rescue_workqueue optional. NeilBrown <neilb@suse.com> - 2017-05-03 00:40 +0200
    Re: [PATCH 03/13 V2] blk: make the bioset rescue_workqueue optional. Ming Lei <tom.leiming@gmail.com> - 2017-05-03 05:30 +0200

csiph-web