<div dir="ltr"><div>Hi Gao and Ajay,</div><div><br></div><div>Thank you for the review and for pointing that out.</div><div><br></div><div>I apologize for the formatting; I made a mistake while using the CLI SMTP instead of my usual Gmail GUI. I will correct the subject line and commit message body and resend the patch shortly.</div><div><br></div><div>Thanks,</div><div>Shubham Vishwakarma</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, Mar 20, 2026 at 7:07 AM Gao Xiang <<a href="mailto:hsiangkao@linux.alibaba.com">hsiangkao@linux.alibaba.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
On 2026/3/20 09:16, Ajay Rajera wrote:<br>
> Hi Vi-shub,<br>
> just a review :<br>
> I think the fix looks correct and it is the right approach but the<br>
> commit message formatting needs work. The entire description is in the<br>
> subject line. Per kernel conventions, the subject should be a short<br>
> one-liner, e.g: erofs-utils: lib: fix QPL job leak on early error<br>
> paths<br>
> The detailed explanation (which error paths leak, why, and how the fix<br>
> works) should go in the commit message body, separated from the<br>
> subject.<br>
> <br>
> So you can resend with the subject/body split fixed? so It will look more clear.<br>
<br>
yes, the commit message is in a mess.<br>
<br>
Thanks,<br>
Gao Xiang<br>
<br>
> Thanks, Ajay<br>
<br>
</blockquote></div>