Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1399419 > unrolled thread

Patch: V4L stable versions 4.5.3 and 4.5.4

Started byDavid R <david@unsolicited.net>
First post2016-05-11 20:50 +0200
Last post2016-05-11 23:10 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel


Contents

  Patch: V4L stable versions 4.5.3 and 4.5.4 David R <david@unsolicited.net> - 2016-05-11 20:50 +0200
    Re: Patch: V4L stable versions 4.5.3 and 4.5.4 David R <david@unsolicited.net> - 2016-05-11 21:10 +0200
      Re: Patch: V4L stable versions 4.5.3 and 4.5.4 Mauro Carvalho Chehab <mchehab@osg.samsung.com> - 2016-05-11 22:40 +0200
        Re: Patch: V4L stable versions 4.5.3 and 4.5.4 Sakari Ailus <sakari.ailus@iki.fi> - 2016-05-11 23:10 +0200

#1399419 — Patch: V4L stable versions 4.5.3 and 4.5.4

FromDavid R <david@unsolicited.net>
Date2016-05-11 20:50 +0200
SubjectPatch: V4L stable versions 4.5.3 and 4.5.4
Message-ID<rxHmO-5Az-21@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

Hi

Please consider applying the attached patch (or something like it) to
V4L2, and whatever is appropriate to the mainstream kernel. Without this
my media server crashes and burns at boot.

See https://lkml.org/lkml/2016/5/7/88 for more details

Thanks
David

[toc] | [next] | [standalone]


#1399437

FromDavid R <david@unsolicited.net>
Date2016-05-11 21:10 +0200
Message-ID<rxHGb-6be-27@gated-at.bofh.it>
In reply to#1399419
On 11/05/16 19:47, David R wrote:
> Hi
> 
> Please consider applying the attached patch (or something like it) to
> V4L2, and whatever is appropriate to the mainstream kernel. Without this
> my media server crashes and burns at boot.
> 
> See https://lkml.org/lkml/2016/5/7/88 for more details
> 
> Thanks
> David
> 
I see the offending patch was reverted earlier today. My box is fine
with my (more simple) alternative, but your call.

David

[toc] | [prev] | [next] | [standalone]


#1399501

FromMauro Carvalho Chehab <mchehab@osg.samsung.com>
Date2016-05-11 22:40 +0200
Message-ID<rxJ5g-7qx-27@gated-at.bofh.it>
In reply to#1399437
Hi David,

Em Wed, 11 May 2016 20:05:22 +0100
David R <david@unsolicited.net> escreveu:

> On 11/05/16 19:47, David R wrote:
> > Hi
> > 
> > Please consider applying the attached patch (or something like it) to
> > V4L2, and whatever is appropriate to the mainstream kernel. Without this
> > my media server crashes and burns at boot.
> > 
> > See https://lkml.org/lkml/2016/5/7/88 for more details
> > 
> > Thanks
> > David
> >   
> I see the offending patch was reverted earlier today. My box is fine
> with my (more simple) alternative, but your call.

Yes, I noticed the bug earlier today, while testing a DVB device.
As this affects 2 stable releases plus the upcoming Kernel 4.6,
I decided to just revert it for now, while we don't solve the
issue.

Your patch looks good. So, eventually it will be merged on a new
version of this patch, after we test it properly.

Thanks!
Mauro

[toc] | [prev] | [next] | [standalone]


#1399508

FromSakari Ailus <sakari.ailus@iki.fi>
Date2016-05-11 23:10 +0200
Message-ID<rxJyi-84d-11@gated-at.bofh.it>
In reply to#1399501
Hi Mauro and David,

On Wed, May 11, 2016 at 05:38:23PM -0300, Mauro Carvalho Chehab wrote:
> Hi David,
> 
> Em Wed, 11 May 2016 20:05:22 +0100
> David R <david@unsolicited.net> escreveu:
> 
> > On 11/05/16 19:47, David R wrote:
> > > Hi
> > > 
> > > Please consider applying the attached patch (or something like it) to
> > > V4L2, and whatever is appropriate to the mainstream kernel. Without this
> > > my media server crashes and burns at boot.
> > > 
> > > See https://lkml.org/lkml/2016/5/7/88 for more details
> > > 
> > > Thanks
> > > David
> > >   
> > I see the offending patch was reverted earlier today. My box is fine
> > with my (more simple) alternative, but your call.
> 
> Yes, I noticed the bug earlier today, while testing a DVB device.
> As this affects 2 stable releases plus the upcoming Kernel 4.6,
> I decided to just revert it for now, while we don't solve the
> issue.
> 
> Your patch looks good. So, eventually it will be merged on a new
> version of this patch, after we test it properly.

Indeed vb2_core_dqbuf() is called with pb == NULL in file I/O, which was
unfortunately missed until now.

The proposed fix looks good to me.

-- 
Kind regards,

Sakari Ailus
e-mail: sakari.ailus@iki.fi	XMPP: sailus@retiisi.org.uk

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web