On 11/20/24 06:18, Bart Van Assche wrote: > On 11/18/24 11:53 PM, Damien Le Moal wrote: >> On 11/19/24 09:28, Bart Van Assche wrote: >>> Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx> >> >> The patch title seems incorrect. This is not documenting anything but adding >> lockdep checks. > > Hmm ... what this patch does is to document what the locking assumptions > are for the modified functions. So I'm not sure why the patch title is > considered incorrect? Seeing the word "Document" in the patch title, I expected text/comments. These are lockdep checks so may be just state that, e.g. "Add lockdep checks..." > > Thanks, > > Bart. > -- Damien Le Moal Western Digital Research