From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Snitzer Subject: Re: blk-mq DM changes for 3.20 [was: Re: blk-mq request allocation stalls]X Date: Thu, 29 Jan 2015 18:09:24 -0500 Message-ID: <20150129230924.GA24657@redhat.com> References: <54B40E8A.6010005@kernel.dk> <54B532EE.20700@kernel.dk> <20150127184258.GB2078@redhat.com> <54C91172.10002@kernel.dk> <20150128174442.GA11951@redhat.com> <54C92127.4030408@kernel.dk> Reply-To: device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Keith Busch Cc: Jens Axboe , Bart Van Assche , device-mapper development , Jun'ichi Nomura , Christoph Hellwig List-Id: dm-devel.ids On Thu, Jan 29 2015 at 5:43pm -0500, Keith Busch wrote: > On Wed, 28 Jan 2015, Jens Axboe wrote: > >On 01/28/2015 10:44 AM, Mike Snitzer wrote: > >>On Wed, Jan 28 2015 at 11:42am -0500, > >>Jens Axboe wrote: > >>>I'd prefer to take these prep patches through the block tree. > >> > >>Great, should I send the patches or can you cherry-pick? > > > >I already cherry picked them, they are in the for-3.20/core branch. > > I might be getting ahead of myself for trying this right now (sorry if > that's the case) but 'for-3.20/core' is missing necessary parts of the > series and hits a BUG_ON at blk-core.c:2333. The original request is > initialized when setting up the clone, and in this branch, that happens > in prep_fn before the request was dequeued so it's not in the queuelist > when it started. > > One of my commits relocated the initialization, but I didn't realize it > had a hard dependency on the follow-on commit. Should we reorder that > part of the series? Which follow on commit are you referring to? Please be specific about which commits you think are out of order. Also, what are you testing... are you using the linux-dm.git tree's 'dm-for-3.20' branch which builds on Jens' 'for-3.20/core'? If not please test that and see if you still have problems (without the associated DM changes all the block preparation changes that Jens cherry-picked shouldn't cause any problems at all).