[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080430103759.GM18935@agk.fab.redhat.com>
Date: Wed, 30 Apr 2008 11:38:00 +0100
From: Alasdair G Kergon <agk@...hat.com>
To: Jens Axboe <jens.axboe@...cle.com>
Cc: Mike Anderson <andmike@...ux.vnet.ibm.com>,
Mikulas Patocka <mpatocka@...hat.com>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Optimize lock in queue unplugging
On Wed, Apr 30, 2008 at 09:14:15AM +0200, Jens Axboe wrote:
> So it's basically dm calling into blk_unplug() all the time, which
> doesn't check if the queue is plugged. The reason why I didn't like the
> initial patch is that ->unplug_fn() really should not be called unless
> the queue IS plugged. So how about this instead:
>
> http://git.kernel.dk/?p=linux-2.6-block.git;a=commit;h=c44993018887e82abd49023e92e8d8b6000e03ed
>
> That's a lot more appropriate, imho.
Makes sense to me, ack.
Alasdair
--
agk@...hat.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists