Re: [GIT pull for 3.14 v3] xhci streams and uas fixes + usbfs streams support
On Mon, Nov 18, 2013 at 03:39:05PM +0100, Hans de Goede wrote: Hi Sarah et al, Hi Hans, Here is v3 of my patch-series / pull-req for you to pull into your xhci tree for 3.14. I was just about to start testing the code I merged from you, when I realized the last round of patches didn't Cc the usb-storage mailing list or Matthew Dharm. Since four of the patches interact with usb-storage, it would be good to get his ACK on those: https://git.kernel.org/cgit/linux/kernel/git/sarah/xhci.git/commit/?h=for-usb-next-queueid=07f652720e3f https://git.kernel.org/cgit/linux/kernel/git/sarah/xhci.git/commit/?h=for-usb-next-queueid=dda9e7db4e29 https://git.kernel.org/cgit/linux/kernel/git/sarah/xhci.git/commit/?h=for-usb-next-queueid=91a4c4b94a40 https://git.kernel.org/cgit/linux/kernel/git/sarah/xhci.git/commit/?h=for-usb-next-queueid=326aff723f82 Matthew, any objections to these patches? Do you want Hans to resend the entire 70 patch set to the USB storage mailing list, or do you think sending it to the USB mailing list was good enough? Sarah Sharp New this version is that the uas: Use proper packet size when submitting reponse urbs patch has been dropped. As Gerd rightfully pointed out we won't know if we get a sense or response iu on the status endpoint when using usb-2, so the urb must be large enough to receive the largest iu type, which is the sense iu. So the old code was correct. The following changes since commit 7d49f0bac41ee9b012af1efe2f725d91a87a8fe9: USB: Maintainers change for usb serial drivers (2013-10-31 08:53:52 -0700) are available in the git repository at: git://linuxtv.org/hgoede/gspca.git usb-next-for-sarah for you to fetch changes up to 8d0f187e6c19ed23e7df4b2f0b943ee784c7f1b2: uas: Remove comment about registering a uas scsi controller for each usb bus (2013-11-18 15:13:35 +0100) Gerd Hoffmann (6): xhci: fix usb3 streams uas: properly reinitialize in uas_eh_bus_reset_handler uas: make work list per-device uas: add dead request list uas: replace BUG_ON() + WARN_ON() with WARN_ON_ONCE() uas: remove BROKEN Hans de Goede (62): xhci: Free streams when they are still allocated on a set_interface call xhci: Check size rather then number of streams when allocating stream ctxs xhci: For streams the css flag most be read from the stream-ctx on ep stop xhci: Set SCT field for Set TR dequeue on streams xhci: For streams the dequeue ptr must be read from the stream ctx xhci: use usb_ss_max_streams in xhci_check_streams_endpoint usb-core: Fix usb_free_streams return value documentation usb-core: Move USB_MAXENDPOINTS definitions to usb.h usb-core: Track if an endpoint has streams usb-core: Free bulk streams on interface release usbfs: Kill urbs on interface before doing a set_interface usbfs: proc_do_submiturb use a local variable for number_of_packets usbfs: Add support for bulk stream ids usbfs: Add ep_to_host_endpoint helper function usbfs: Add support for allocating / freeing streams uas: Urbs must be anchored before submitting them uas: Properly set interface to altsetting 0 on probe failure uas: Avoid unnecessary unlock / lock calls around unlink_data_urbs uas: uas_alloc_cmd_urb: drop unused stream_id parameter uas: Fix uas not working when plugged into an ehci port uas: Fix reset locking uas: Fix reset handling for externally triggered reset uas: s/response_ui/response_iu/ uas: Fix response iu struct definition uas: Pack iu struct definitions uas: Use all available stream ids uas: Add a uas_find_uas_alt_setting helper function uas: Move uas detect code to uas-detect.h xhci: xhci_mem_cleanup: make sure cmd_ring_reserved_trbs really is 0 xhci: The trb_address_map radix tree expects 1KB segment memory aligment xhci: Handle MaxPSASize == 0 usb: Clear host_endpoint-streams when implicitly freeing streams usb: Reset USB-3 devices on USB-3 link bounce uas: Add the posibilty to blacklist uas devices from using the uas driver usb-storage: Don't bind to uas devices if the uas driver is enabled usb-storage: Modify and export adjust_quirks so that it can be used by uas uas: Honor no-uas quirk set in usb-storage's quirks module parameter uas: Add uas_find_endpoints() helper function uas: Fix bounds check in uas_find_endpoints uas: Move uas_find_endpoints to uas-detect.h uas: Drop fixed endpoint config handling uas: Verify endpoint descriptors from uas_use_uas_driver() uas: Not being able to alloc streams when connected through usb-3 is an error uas: task_mgmt: Kill the sense-urb if we fail to submit the cmd urb uas: Don't allow more then one
Re: [GIT pull for 3.14 v3] xhci streams and uas fixes + usbfs streams support
I'm okay with these patches. No need to resend to the USB Storage list, tho I would appreciate such in the future. Acked-By: Matthew Dharm mdharm-...@one-eyed-alien.net Matt On Wed, Dec 4, 2013 at 3:58 PM, Sarah Sharp sarah.a.sh...@linux.intel.com wrote: On Mon, Nov 18, 2013 at 03:39:05PM +0100, Hans de Goede wrote: Hi Sarah et al, Hi Hans, Here is v3 of my patch-series / pull-req for you to pull into your xhci tree for 3.14. I was just about to start testing the code I merged from you, when I realized the last round of patches didn't Cc the usb-storage mailing list or Matthew Dharm. Since four of the patches interact with usb-storage, it would be good to get his ACK on those: https://git.kernel.org/cgit/linux/kernel/git/sarah/xhci.git/commit/?h=for-usb-next-queueid=07f652720e3f https://git.kernel.org/cgit/linux/kernel/git/sarah/xhci.git/commit/?h=for-usb-next-queueid=dda9e7db4e29 https://git.kernel.org/cgit/linux/kernel/git/sarah/xhci.git/commit/?h=for-usb-next-queueid=91a4c4b94a40 https://git.kernel.org/cgit/linux/kernel/git/sarah/xhci.git/commit/?h=for-usb-next-queueid=326aff723f82 Matthew, any objections to these patches? Do you want Hans to resend the entire 70 patch set to the USB storage mailing list, or do you think sending it to the USB mailing list was good enough? Sarah Sharp New this version is that the uas: Use proper packet size when submitting reponse urbs patch has been dropped. As Gerd rightfully pointed out we won't know if we get a sense or response iu on the status endpoint when using usb-2, so the urb must be large enough to receive the largest iu type, which is the sense iu. So the old code was correct. The following changes since commit 7d49f0bac41ee9b012af1efe2f725d91a87a8fe9: USB: Maintainers change for usb serial drivers (2013-10-31 08:53:52 -0700) are available in the git repository at: git://linuxtv.org/hgoede/gspca.git usb-next-for-sarah for you to fetch changes up to 8d0f187e6c19ed23e7df4b2f0b943ee784c7f1b2: uas: Remove comment about registering a uas scsi controller for each usb bus (2013-11-18 15:13:35 +0100) Gerd Hoffmann (6): xhci: fix usb3 streams uas: properly reinitialize in uas_eh_bus_reset_handler uas: make work list per-device uas: add dead request list uas: replace BUG_ON() + WARN_ON() with WARN_ON_ONCE() uas: remove BROKEN Hans de Goede (62): xhci: Free streams when they are still allocated on a set_interface call xhci: Check size rather then number of streams when allocating stream ctxs xhci: For streams the css flag most be read from the stream-ctx on ep stop xhci: Set SCT field for Set TR dequeue on streams xhci: For streams the dequeue ptr must be read from the stream ctx xhci: use usb_ss_max_streams in xhci_check_streams_endpoint usb-core: Fix usb_free_streams return value documentation usb-core: Move USB_MAXENDPOINTS definitions to usb.h usb-core: Track if an endpoint has streams usb-core: Free bulk streams on interface release usbfs: Kill urbs on interface before doing a set_interface usbfs: proc_do_submiturb use a local variable for number_of_packets usbfs: Add support for bulk stream ids usbfs: Add ep_to_host_endpoint helper function usbfs: Add support for allocating / freeing streams uas: Urbs must be anchored before submitting them uas: Properly set interface to altsetting 0 on probe failure uas: Avoid unnecessary unlock / lock calls around unlink_data_urbs uas: uas_alloc_cmd_urb: drop unused stream_id parameter uas: Fix uas not working when plugged into an ehci port uas: Fix reset locking uas: Fix reset handling for externally triggered reset uas: s/response_ui/response_iu/ uas: Fix response iu struct definition uas: Pack iu struct definitions uas: Use all available stream ids uas: Add a uas_find_uas_alt_setting helper function uas: Move uas detect code to uas-detect.h xhci: xhci_mem_cleanup: make sure cmd_ring_reserved_trbs really is 0 xhci: The trb_address_map radix tree expects 1KB segment memory aligment xhci: Handle MaxPSASize == 0 usb: Clear host_endpoint-streams when implicitly freeing streams usb: Reset USB-3 devices on USB-3 link bounce uas: Add the posibilty to blacklist uas devices from using the uas driver usb-storage: Don't bind to uas devices if the uas driver is enabled usb-storage: Modify and export adjust_quirks so that it can be used by uas uas: Honor no-uas quirk set in usb-storage's quirks module parameter uas: Add uas_find_endpoints() helper function uas: Fix bounds check in uas_find_endpoints uas: Move uas_find_endpoints to uas-detect.h uas: Drop fixed endpoint config
[GIT pull for 3.14 v3] xhci streams and uas fixes + usbfs streams support
Hi Sarah et al, Here is v3 of my patch-series / pull-req for you to pull into your xhci tree for 3.14. New this version is that the uas: Use proper packet size when submitting reponse urbs patch has been dropped. As Gerd rightfully pointed out we won't know if we get a sense or response iu on the status endpoint when using usb-2, so the urb must be large enough to receive the largest iu type, which is the sense iu. So the old code was correct. The following changes since commit 7d49f0bac41ee9b012af1efe2f725d91a87a8fe9: USB: Maintainers change for usb serial drivers (2013-10-31 08:53:52 -0700) are available in the git repository at: git://linuxtv.org/hgoede/gspca.git usb-next-for-sarah for you to fetch changes up to 8d0f187e6c19ed23e7df4b2f0b943ee784c7f1b2: uas: Remove comment about registering a uas scsi controller for each usb bus (2013-11-18 15:13:35 +0100) Gerd Hoffmann (6): xhci: fix usb3 streams uas: properly reinitialize in uas_eh_bus_reset_handler uas: make work list per-device uas: add dead request list uas: replace BUG_ON() + WARN_ON() with WARN_ON_ONCE() uas: remove BROKEN Hans de Goede (62): xhci: Free streams when they are still allocated on a set_interface call xhci: Check size rather then number of streams when allocating stream ctxs xhci: For streams the css flag most be read from the stream-ctx on ep stop xhci: Set SCT field for Set TR dequeue on streams xhci: For streams the dequeue ptr must be read from the stream ctx xhci: use usb_ss_max_streams in xhci_check_streams_endpoint usb-core: Fix usb_free_streams return value documentation usb-core: Move USB_MAXENDPOINTS definitions to usb.h usb-core: Track if an endpoint has streams usb-core: Free bulk streams on interface release usbfs: Kill urbs on interface before doing a set_interface usbfs: proc_do_submiturb use a local variable for number_of_packets usbfs: Add support for bulk stream ids usbfs: Add ep_to_host_endpoint helper function usbfs: Add support for allocating / freeing streams uas: Urbs must be anchored before submitting them uas: Properly set interface to altsetting 0 on probe failure uas: Avoid unnecessary unlock / lock calls around unlink_data_urbs uas: uas_alloc_cmd_urb: drop unused stream_id parameter uas: Fix uas not working when plugged into an ehci port uas: Fix reset locking uas: Fix reset handling for externally triggered reset uas: s/response_ui/response_iu/ uas: Fix response iu struct definition uas: Pack iu struct definitions uas: Use all available stream ids uas: Add a uas_find_uas_alt_setting helper function uas: Move uas detect code to uas-detect.h xhci: xhci_mem_cleanup: make sure cmd_ring_reserved_trbs really is 0 xhci: The trb_address_map radix tree expects 1KB segment memory aligment xhci: Handle MaxPSASize == 0 usb: Clear host_endpoint-streams when implicitly freeing streams usb: Reset USB-3 devices on USB-3 link bounce uas: Add the posibilty to blacklist uas devices from using the uas driver usb-storage: Don't bind to uas devices if the uas driver is enabled usb-storage: Modify and export adjust_quirks so that it can be used by uas uas: Honor no-uas quirk set in usb-storage's quirks module parameter uas: Add uas_find_endpoints() helper function uas: Fix bounds check in uas_find_endpoints uas: Move uas_find_endpoints to uas-detect.h uas: Drop fixed endpoint config handling uas: Verify endpoint descriptors from uas_use_uas_driver() uas: Not being able to alloc streams when connected through usb-3 is an error uas: task_mgmt: Kill the sense-urb if we fail to submit the cmd urb uas: Don't allow more then one task to run at the same time uas: Use GFP_NOIO rather then GFP_ATOMIC where possible uas: Add suspend/resume support uas: Reset device on reboot uas: Fix task-management not working when connected over USB-2 uas: uas_alloc_data_urb: Remove unnecessary use_streams check uas: Properly complete inflight commands on bus-reset or disconnect uas: add uas_mark_cmd_dead helper function uas: cmdinfo: use only one list head uas: Fix command / task mgmt submission racing with disconnect uas: Fix memory management uas: Clear cmdinfo on command queue-ing uas: Use the right error codes for different kinds of errors uas: Improve error reporting uas: Add some data in/out ready iu sanity checks uas: Make sure sg elements are properly aligned uas: Add Hans de Goede as uas maintainer uas: Remove comment about registering a uas scsi controller for each usb bus Sarah Sharp (1): xhci: Remove segments from radix tree on failed insert.