From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH] scsi: fix regression that accidentally disabled block-based tcq Date: Sun, 14 Sep 2014 11:41:40 +0200 Message-ID: <541562D4.90801@redhat.com> References: <540EC52C.3000804@redhat.com> <20140909152730.GA17882@infradead.org> <540FFBF4.9090001@redhat.com> <20140910154535.GA5102@infradead.org> <541172E9.8070309@redhat.com> <20140911161331.GA24488@infradead.org> <54134E50.6040805@redhat.com> <20140912230032.GA21143@infradead.org> <54141C59.7040707@redhat.com> <20140913175047.GA28882@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140913175047.GA28882@infradead.org> Sender: linux-kernel-owner@vger.kernel.org To: Christoph Hellwig Cc: linux-usb , SCSI development list , Linux Kernel Mailing List List-Id: linux-scsi@vger.kernel.org Hi, On 09/13/2014 07:50 PM, Christoph Hellwig wrote: > On Sat, Sep 13, 2014 at 12:28:41PM +0200, Hans de Goede wrote: >> Yes this one does the trick and fixes things. Note the git tree I used for >> testing also had your previous fix to split up the blk_tcq union in 2 >> separate struct members. Let me know if you want me to re-test without that >> fix. > > I'm pretty sure that isn't needed, so double testing it indeed doesn't > would be helpful. I can confirm that things still work fine with the union left in place. > I'd like to add your Tested-by: tag after that, too. Ack: Tested-by: Hans de Goede Regards, Hans