Re: [GIT PULL] USB driver patches for 3.19-rc1

[Stephen Rothwell <sfr@canb.auug.org.au>]

[-- Attachment #1.1: Type: text/plain, Size: 892 bytes --]

Hi Linus,

On Sun, 14 Dec 2014 15:06:07 -0800 Linus Torvalds <torvalds@linux-foundation.org> wrote:
>
> On Sun, Dec 14, 2014 at 2:35 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> >
> > Hans de Goede (1):
> >       uas: Make uas work with blk-mq
> 
> So I got some fairly trivial conflicts on this one (conflicting with
> the scsi cleanups mainly by Christoph Hellwig.
> 
> I resolved the conflict easily, and it all *looks* fine, but quite
> frankly, I'd be a lot happier about it if somebody who has the
> hardware were to actually test the end result.   The whole interaction
> with ".use_blk_tags" etc should be verified by somebody who knows the
> code.
> 
> Hans? Christoph?

Attached is the message I sent about this on Nov 24 to which I received
no response and so assumed it was ok ...
-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #1.2: Type: message/rfc822, Size: 3407 bytes --]

[-- Attachment #1.2.1.1: Type: text/plain, Size: 1269 bytes --]

Hi James,

Today's linux-next merge of the scsi tree got a conflict in
drivers/usb/storage/uas.c between commit e28e2f2f7c42 ("uas: Make uas
work with blk-mq") from the usb tree and commits 125c99bc8b6b ("scsi:
add new scsi-command flag for tagged commands"), abd0c533e377
("scsi: remove ordered_tag host template field") and 2ecb204d07ac
("scsi: always assign block layer tags if enabled") from the scsi tree.

I fixed it up (maybe, please check - see below) and can carry the fix
as necessary (no action is required).

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

diff --cc drivers/usb/storage/uas.c
index 004ebc12bc21,33f211b56a42..000000000000
--- a/drivers/usb/storage/uas.c
+++ b/drivers/usb/storage/uas.c
@@@ -806,7 -816,14 +805,7 @@@ static struct scsi_host_template uas_ho
  	.sg_tablesize = SG_NONE,
  	.cmd_per_lun = 1,	/* until we override it */
  	.skip_settle_delay = 1,
- 	.ordered_tag = 1,
 -
 -	/*
 -	 * The uas drivers expects tags not to be bigger than the maximum
 -	 * per-device queue depth, which is not true with the blk-mq tag
 -	 * allocator.
 -	 */
 -	.disable_blk_mq = true,
+ 	.use_blk_tags = 1,
  };
  
  #define UNUSUAL_DEV(id_vendor, id_product, bcdDeviceMin, bcdDeviceMax, \

[-- Attachment #1.2.1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]