[dm-devel] [PATCH 1/5] block: remove the unused block_sleeprq tracepoint

2020-11-30 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig --- include/trace/events/block.h | 18 -- kernel/trace/blktrace.c | 22 -- 2 files changed, 40 deletions(-) diff --git a/include/trace/events/block.h b/include/trace/events/block.h index 34d64ca306b1c7..76459cf750e14d 1006

[dm-devel] [PATCH 1/5] block: remove the unused block_sleeprq tracepoint

2020-12-03 Thread Christoph Hellwig
The block_sleeprq tracepoint was only used by the legacy request code. Remove it now that the legacy request code is gone. Signed-off-by: Christoph Hellwig Reviewed-by: Damien Le Moal Reviewed-by: Hannes Reinecke --- include/trace/events/block.h | 18 -- kernel/trace/blktrace.c

Re: [dm-devel] [PATCH 1/5] block: remove the unused block_sleeprq tracepoint

2020-12-03 Thread Damien Le Moal
On 2020/12/01 3:03, Christoph Hellwig wrote: > Signed-off-by: Christoph Hellwig Even if it is just repeating the patch title, a commit message would be nice. You could also mention when the last use of this tracepoint was removed. > --- > include/trace/events/block.h | 18 -- >

Re: [dm-devel] [PATCH 1/5] block: remove the unused block_sleeprq tracepoint

2020-12-03 Thread Hannes Reinecke
On 11/30/20 6:58 PM, Christoph Hellwig wrote: Signed-off-by: Christoph Hellwig --- include/trace/events/block.h | 18 -- kernel/trace/blktrace.c | 22 -- 2 files changed, 40 deletions(-) Reviewed-by: Hannes Reinecke https://www.redhat.com/mailman/li

Re: [dm-devel] [PATCH 1/5] block: remove the unused block_sleeprq tracepoint

2020-12-03 Thread Tejun Heo
On Thu, Dec 03, 2020 at 05:21:35PM +0100, Christoph Hellwig wrote: > The block_sleeprq tracepoint was only used by the legacy request code. > Remove it now that the legacy request code is gone. > > Signed-off-by: Christoph Hellwig > Reviewed-by: Damien Le Moal > Reviewed-by: Hannes Reinecke Ac

Re: [dm-devel] [PATCH 1/5] block: remove the unused block_sleeprq tracepoint

2020-12-03 Thread Chaitanya Kulkarni
On 12/3/20 08:24, Christoph Hellwig wrote: > The block_sleeprq tracepoint was only used by the legacy request code. > Remove it now that the legacy request code is gone. > > Signed-off-by: Christoph Hellwig > Reviewed-by: Damien Le Moal > Reviewed-by: Hannes Reinecke Looks good. Reviewed-by: C