Re: [PATCH v2 1/2] media: atomisp: fix memory leak in atomisp_pci_probe()

From: Andy Shevchenko

Date: Mon Jun 15 2026 - 09:49:40 EST


On Mon, Jun 15, 2026 at 02:39:42PM +0300, Dan Carpenter wrote:
> On Mon, Jun 15, 2026 at 02:11:12PM +0300, Andy Shevchenko wrote:

...

> > > The bug was first flagged by an experimental analysis tool we are
> > > developing for kernel memory-management bugs while analyzing
> > > v6.13-rc1. The tool is still under development and is not yet publicly
> > > available. Manual inspection confirms that the bug is still present in
> > > v7.1-rc7.
> > >
> > > An x86_64 allyesconfig build showed no new warnings. As we do not have
> > > an Intel Atom ISP platform with the required camera sensor hardware to
> > > test with, no runtime testing was able to be performed.
> >
> > These last two paragraphs do not suit the commit message. Please, drop them
> > here and better to describe all this in the cover letter (if not yet).
>
> This is how the documentation says to write commit messages.
>
> https://lore.kernel.org/all/ahgaOigklcDCYvRp@stanley.mountain/

Isn't it enough to have in the cover letter?

Thanks for commenting there, but I would insist to move these two paragraphs
from the commit message here. Maybe Kees is okay with that, I'm thinking that
this is too much (since we have lore archives).

--
With Best Regards,
Andy Shevchenko