linux-kernel - Re: [PATCH v3] media: add virtio-media driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAAVeFuLPAwMyq7cTDE4Fp3+i883RdDWkg=GOMGDzMxZz6rgSuw@mail.gmail.com>
Date: Mon, 28 Jul 2025 20:51:43 +0900
From: Alexandre Courbot <gnurou@...il.com>
To: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
Cc: Ricardo Ribalda <ribalda@...omium.org>, "Michael S. Tsirkin" <mst@...hat.com>, 
	Mauro Carvalho Chehab <mchehab@...nel.org>, Hans Verkuil <hverkuil@...all.nl>, 
	Albert Esteve <aesteve@...hat.com>, Jason Wang <jasowang@...hat.com>, 
	Xuan Zhuo <xuanzhuo@...ux.alibaba.com>, Eugenio Pérez <eperezma@...hat.com>, 
	gurchetansingh@...gle.com, daniel.almeida@...labora.com, adelva@...gle.com, 
	changyeon@...gle.com, nicolas.dufresne@...labora.com, 
	linux-kernel@...r.kernel.org, linux-media@...r.kernel.org, 
	virtualization@...ts.linux.dev, Alexandre Courbot <acourbot@...gle.com>
Subject: Re: [PATCH v3] media: add virtio-media driver

Hi Mauro,

On Fri, Jul 25, 2025 at 2:24 AM Mauro Carvalho Chehab
<mchehab+huawei@...nel.org> wrote:
>
> Em Sun, 1 Jun 2025 12:01:22 +0200
> Ricardo Ribalda <ribalda@...omium.org> escreveu:
>
> > Hi Mauro
> >
> > On Sun, 1 Jun 2025 at 11:34, Mauro Carvalho Chehab
> > <mchehab+huawei@...nel.org> wrote:
> > >
> > > Em Wed, 28 May 2025 18:23:02 +0200
> > > Ricardo Ribalda <ribalda@...omium.org> escreveu:
> > >
> > > > > +static int scatterlist_builder_add_userptr(struct scatterlist_builder *builder,
> > > > > +                                          unsigned long userptr,
> > > > > +                                          unsigned long length)
> > > > > +{
> > > > > +       int ret;
> > > > > +       int nents;
> > > > Could you initialize nents and sg_list?
> > > > old versions of gcc are a bit picky
> > > > https://gitlab.freedesktop.org/linux-media/users/ribalda/-/jobs/77042562#L4381
> > >
> > > Please don't. In this specific case, ret is always initialized:
> > >
> > > > +       struct virtio_media_sg_entry *sg_list;
> > > > +
> > > > +       ret = __scatterlist_builder_add_userptr(builder, userptr, length,
> > > > +                                               &sg_list, &nents);
> > >
> > > nents and sg_list may or may not be initialized at the function,
> > > but initializing it is wrong, as, when they are not initialized, the
> > > ret code shall catch it (and if not, we *do* want gcc to warn).
> > >
> > > So, if our CI is warning about that due to an old version, please upgrade
> > > the version at the CI runner.
> >
> > The main version of gcc works fine. It is the minimal version (8.1) required by
> > https://www.kernel.org/doc/html/next/process/changes.html
> > that  complains.
>
> Ricardo,
>
> gcc 8.1 was released in May 2, 2018. I don't think it makes sense to
> address bogus warnings with that old gcc versions. I would just disable
> WERROR for such versions on our CI tests.
>
> ---
>
> Alexandre/Michael,
>
> I need a couple of full days to properly review virtio-media.
> I was planning to do it during this Kernel cycle, but I ended
> allocating too much time just to be able to create a crossvm
> that would allow testing it. Afterwards, I got sidetracked with other
> issues. I won't be able to review it in time for this merge window.
>
> I'm planning to do it at the beginning of the next merge cycle.

Not a worry at all and I appreciate the time you are putting aside to
review this properly!

Thanks,
Alex.