RE: [PATCH] blkcg: add plugging support for punt bio

2020-09-08 Thread Tianxianting
...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: Re: [PATCH] blkcg: add plugging support for punt bio On Sat, Sep 05, 2020 at 11:25:03AM +, Tianxianting wrote: > Hi jens,tj > Could you share a couple of minutes to comment it? > I really appreciate it The result looks f

Re: [PATCH] blkcg: add plugging support for punt bio

2020-09-08 Thread Tejun Heo
On Sat, Sep 05, 2020 at 11:25:03AM +, Tianxianting wrote: > Hi jens,tj > Could you share a couple of minutes to comment it? > I really appreciate it The result looks fine to me but can you please summarize that in the commit message of the patch? Thanks. -- tejun

Re: [PATCH] blkcg: add plugging support for punt bio

2020-08-06 Thread Jens Axboe
On 8/6/20 7:54 AM, Xianting Tian wrote: > Try to merge continuous bio to current task's plug fisrt. The patch looks fine to me, but I'd really like to see a bit more changelog here. The commit message should explain why the change is made, rather it's a very brief explanation of what it does. If

[PATCH] blkcg: add plugging support for punt bio

2020-08-06 Thread Xianting Tian
Try to merge continuous bio to current task's plug fisrt. Signed-off-by: Xianting Tian --- block/blk-cgroup.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/block/blk-cgroup.c b/block/blk-cgroup.c index 0ecc897..fe5d361 100644 --- a/block/blk-cgroup.c +++ b/block/blk-cgroup.c @@ -122,6

Re: [PATCH] blkcg: add plugging support for punt bio

2020-08-06 Thread Tejun Heo
On Thu, Aug 06, 2020 at 09:54:42AM -0400, Xianting Tian wrote: > Try to merge continuous bio to current task's plug fisrt. > > Signed-off-by: Xianting Tian Acked-by: Tejun Heo Thanks! -- tejun