Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751135AbdHXAvI (ORCPT ); Wed, 23 Aug 2017 20:51:08 -0400 Received: from esa3.hgst.iphmx.com ([216.71.153.141]:18213 "EHLO esa3.hgst.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751003AbdHXAvH (ORCPT ); Wed, 23 Aug 2017 20:51:07 -0400 X-IronPort-AV: E=Sophos;i="5.41,419,1498492800"; d="scan'208";a="44703984" Subject: Re: [GIT PULL] SCSI fixes for 4.13-rc6 To: "Martin K. Petersen" , Bart Van Assche References: <1503470559.2729.4.camel@HansenPartnership.com> <1503500560.2484.1.camel@wdc.com> Cc: "torvalds@linux-foundation.org" , "James.Bottomley@HansenPartnership.com" , "akpm@linux-foundation.org" , "linux-scsi@vger.kernel.org" , "linux-kernel@vger.kernel.org" From: Damien Le Moal Message-ID: <90e92658-eb4b-8f32-6dd3-b5fd0c229fdc@wdc.com> Date: Thu, 24 Aug 2017 09:49:55 +0900 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1013 Lines: 29 On 8/24/17 00:27, Martin K. Petersen wrote: > > Hi Bart, > >> Had you noticed that Damien had asked not to send the "sd_zbc: Write >> unlock zone from sd_uninit_cmnd()" patch to Linus without my "scsi-mq: >> Always unprepare before requeuing a request" patch? > > He did change his mind later in that thread, though. > > However, what's more important is that we still need a good version of > your patch for 4.13. I took Brian's workaround for ipr but I still think > Christoph's concerns need to be addressed for me to put your change back > in. Yes, I did ask for the patch to be kept since it does not make things worse for the SMR/mq case (still have a potential dispatch deadlock) but does cleanup the code in the legacy/sq case. Since it looks like the jiffies_at_alloc and retry counter initialization fixes are progressing, we hopefully should be able to get Bart's pacth back in soon (this week ?) and remove the mq dispatch deadlock. Best regards. -- Damien Le Moal, Western Digital Research