Messages in this thread | | | Date | Fri, 1 May 2015 19:11:26 +0200 | From | Christoph Hellwig <> | Subject | Re: [PATCH 1/5] blk: clean up plug |
| |
On Thu, Apr 30, 2015 at 10:45:14AM -0700, Shaohua Li wrote: > Current code looks like inner plug gets flushed with a > blk_finish_plug(). Actually it's a nop. All requests/callbacks are added > to current->plug, while only outmost plug is assigned to current->plug. > So inner plug always has empty request/callback list, which makes > blk_flush_plug_list() a nop. This tries to make the code more clear.
Looks good,
Reviewed-by: Christoph Hellwig <hch@lst.de>
| |