Messages in this thread | | | Subject | Re: [PATCH v2] exfat: add missing brelse() calls on error paths | From | Markus Elfring <> | Date | Wed, 10 Jun 2020 20:56:26 +0200 |
| |
>>> If the second exfat_get_dentry() call fails then we need to release >>> "old_bh" before returning. There is a similar bug in exfat_move_file(). >> >> Would you like to convert any information from this change description >> into an imperative wording? >> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?id=5b14671be58d0084e7e2d1cc9c2c36a94467f6e0#n151 > > I really feel like imperative doesn't add anything. I understand that > some people feel really strongly about it, but I don't know why. It > doesn't make commit messages more understandable.
Do you insist to deviate from the given guideline?
> The important thing is that the problem is clear, the fix is clear and > the runtime impact is clear.
I have got further ideas to improve also this commit message. I am curious if other contributors would like to add another bit of patch review.
Regards, Markus
| |