On 10/5/18 4:37 PM, Omar Sandoval wrote: > On Fri, Oct 05, 2018 at 08:18:00AM -0600, Jens Axboe wrote: >> On 10/4/18 11:35 AM, Bart Van Assche wrote: >>> When debugging e.g. the SCSI timeout handler it is important that >>> requests that have not yet been started or that already have >>> completed are also reported through debugfs. >> >> Thanks, I like this better - applied. BTW, what's up with the >> reverse ordering on this: >> >>> Signed-off-by: Bart Van Assche <[email protected]> >>> Cc: Christoph Hellwig <[email protected]> >>> Cc: Ming Lei <[email protected]> >>> Cc: Hannes Reinecke <[email protected]> >>> Cc: Johannes Thumshirn <[email protected]> >>> Cc: Martin K. Petersen <[email protected]> >> >> For some reason that really annoys me, and I see it in various >> patches these days. IMHO the SOB should be last, with whatever >> acks, reviews, CC, before that. > > I could've sworn that this guideline was even documented somewhere, but > I can't find it now ¯\_(ツ)_/¯
My guess is that it's some newer git thing - but if it is, it's really annoying and should be reverted. I end up fixing these up by hand. -- Jens Axboe
