linux-kernel - Re: Bug fix for pvrusb2 driver [was: Re: Kernel 2.6.26.5 -> 2.6.27.2 [USB REGRESSION] (USB -> D

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <37219a840810201101g6382179ft8e3b03416aeeb957@mail.gmail.com>
Date:	Mon, 20 Oct 2008 14:01:42 -0400
From:	"Michael Krufky" <mkrufky@...uxtv.org>
To:	"Greg KH" <greg@...ah.com>
Cc:	"Mike Isely" <isely@...ox.com>,
	"Alan Stern" <stern@...land.harvard.edu>,
	"Justin Piszcz" <jpiszcz@...idpixels.com>,
	"Oliver Neukum" <oliver@...kum.org>,
	"Kernel development list" <linux-kernel@...r.kernel.org>,
	"USB list" <linux-usb@...r.kernel.org>
Subject: Re: Bug fix for pvrusb2 driver [was: Re: Kernel 2.6.26.5 -> 2.6.27.2 [USB REGRESSION] (USB -> D_STATE)]

On Mon, Oct 20, 2008 at 1:29 PM, Greg KH <greg@...ah.com> wrote:
> On Mon, Oct 20, 2008 at 11:37:41AM -0500, Mike Isely wrote:
>>
>> I was incomplete in my previous response.  See further below for nack
>> and another patch...
>>
>>
>> On Mon, 20 Oct 2008, Alan Stern wrote:
>>
>>    [...]
>>
>> >
>> > Index: usb-2.6/drivers/media/video/pvrusb2/pvrusb2-main.c
>> > ===================================================================
>> > --- usb-2.6.orig/drivers/media/video/pvrusb2/pvrusb2-main.c
>> > +++ usb-2.6/drivers/media/video/pvrusb2/pvrusb2-main.c
>> > @@ -68,6 +68,16 @@ static void pvr_setup_attach(struct pvr2
>> >  #endif /* CONFIG_VIDEO_PVRUSB2_SYSFS */
>> >  }
>> >
>> > +static int pvr_pre_reset(struct usb_interface *intf)
>> > +{
>> > +   return 0;
>> > +}
>> > +
>> > +static int pvr_post_reset(struct usb_interface *intf)
>> > +{
>> > +   return 0;
>> > +}
>> > +
>> >  static int pvr_probe(struct usb_interface *intf,
>> >                  const struct usb_device_id *devid)
>> >  {
>> > @@ -109,7 +119,9 @@ static struct usb_driver pvr_driver = {
>> >     .name =         "pvrusb2",
>> >     .id_table =     pvr2_device_table,
>> >     .probe =        pvr_probe,
>> > -   .disconnect =   pvr_disconnect
>> > +   .disconnect =   pvr_disconnect,
>> > +   .pre_reset =    pvr_pre_reset,
>> > +   .post_reset =   pvr_post_reset,
>> >  };
>> >
>> >  /*
>> >
>> > --
>> > To unsubscribe from this list: send the line "unsubscribe linux-usb" in
>> > the body of a message to majordomo@...r.kernel.org
>> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
>> >
>>
>> Nacked-by: Mike Isely <isely@...ox.com>
>>
>> There is already another patch ready to go which eliminates the reset
>> entirely.  It can be found here:
>>
>> http://linuxtv.org/hg/~mcisely/pvrusb2/rev/0bb411d8d2e4
>
> Will this patch be sent to the -stable group, to fix this regression in
> 2.6.27?  Or should they take Alan's fix instead?

Greg,

Please queue the patch that Mike Isely sent for 2.6.27.y -- It's not
in Mauro's tree yet, but it will get there eventually.  (it might take
him some before he does his next sync to git)

Reviewed-by: Michael Krufky <mkrufky@...uxtv.org>

Regards,

Mike Krufky
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/