Re: 2.6.25-rc regression: bttv: oops on radio access (bisected)

[Marcin Slusarz <marcin.slusarz@gmail.com>]

On Tue, Apr 01, 2008 at 05:10:51PM -0300, Mauro Carvalho Chehab wrote:
> Hi Marcin
> 
> On Sun, 30 Mar 2008 18:20:49 +0200
> Marcin Slusarz <marcin.slusarz@gmail.com> wrote:
> 
> > Hi
> > 2.6.25-rc7 kernel oopses on exec of "radio -c /dev/radio0".
> > I bisected it down to:
> > 
> > 402aa76aa5e57801b4db5ccf8c7beea9f580bb1b is first bad commit
> > commit 402aa76aa5e57801b4db5ccf8c7beea9f580bb1b
> > Author: Douglas Schilling Landgraf <dougsland@gmail.com>
> > Date:   Thu Dec 27 22:20:58 2007 -0300
> > 
> >     V4L/DVB (6911): Converted bttv to use video_ioctl2
> > 
> >     Signed-off-by: Douglas Schilling Landgraf <dougsland@gmail.com>
> >     Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
> > 
> 
> There are three patches that are meant to fix the bugs with radio. On the tests
> I did here, they worked, but the Bongani still points that the fixes didn't
> solve for him.
> 
> On the tests I did here, the three patches seemed to work [1]. Maybe you could
> test with those patches and post us some results.
> 
> There are three patches meant to fix several issues caused by the conversion: 
> 
> http://git.kernel.org/?p=linux/kernel/git/mchehab/v4l-dvb.git;a=commitdiff;h=bdd38d9b5c6365ea004df6d8a183dd1344b4801f
> http://git.kernel.org/?p=linux/kernel/git/mchehab/v4l-dvb.git;a=commitdiff;h=055a6282cabd311cf010a5a83f0494558504f7d0
> http://git.kernel.org/?p=linux/kernel/git/mchehab/v4l-dvb.git;a=commitdiff;h=6e07ff78274752fe812a1e8bddb6013a278e62e8
> 
> Could you test please and give us some feedback?

These patches fix my regression. Thanks!
(no oops, radio app does not crash/hang, no messages about invalid ioctls, i can hear the radio)
 
Marcin