Re: [PATCH] usbnet: rx_submit() should return an error code.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Oliver Neukum <oneukum@xxxxxxx>
Date: Tue, 10 Aug 2010 09:42:05 +0200

> Am Dienstag, 10. August 2010, 09:18:55 schrieb David Miller:
>> From: Elly Jones <ellyjones@xxxxxxxxxx>
>> Date: Mon, 9 Aug 2010 18:37:50 -0400
>> 
>> > This patch makes rx_submit() return an error code, and makes some call sites
>> > that care check the return value. This is important because it lets us properly
>> > handle cases where the device isn't ready to handle URB submissions (e.g., when
>> > it is autosuspended under some drivers); previously, we would attempt and fail
>> > to submit URBs and reschedule ourselves to try and fail again. This patch is
>> > against Linus's 2.6 repo commit 45d7f32c7a43cbb9592886d38190e379e2eb2226.
>> > 
>> > Signed-Off-By: Elizabeth Jones <ellyjones@xxxxxxxxxx>
>> 
>> What gets kevent() running again to run the rx_submit() calls when the
>> device comes back from being suspended?
>> 
> 
> usbnet_resume() schedules usbnet_bh() which will take care of that.

That makes sense, I've applied Elizabeth's patch.

But Elizabeth, your patch was severely corrupted by your email client.
Long lines were split up into two, etc.  This corrupts the patch and
makes it unusable for us.  Please make sure that you have fixed your
email client setup for the next patch you submit.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux