From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751157AbdJCHV7 (ORCPT ); Tue, 3 Oct 2017 03:21:59 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36094 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750841AbdJCHV5 (ORCPT ); Tue, 3 Oct 2017 03:21:57 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com C76FA4A6F7 Authentication-Results: ext-mx09.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx09.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=ming.lei@redhat.com Date: Tue, 3 Oct 2017 15:21:45 +0800 From: Ming Lei To: Christoph Hellwig Cc: Jens Axboe , linux-block@vger.kernel.org, linux-scsi@vger.kernel.org, "Martin K . Petersen" , "James E . J . Bottomley" , Bart Van Assche , Oleksandr Natalenko , Johannes Thumshirn , Cathy Avery , Martin Steigerwald , linux-kernel@vger.kernel.org, Hannes Reinecke , Bart Van Assche Subject: Re: [PATCH V7 3/6] block: pass flags to blk_queue_enter() Message-ID: <20171003072144.GA23235@ming.t460p> References: <20170930061214.10622-1-ming.lei@redhat.com> <20170930061214.10622-4-ming.lei@redhat.com> <20171002134032.GC19119@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171002134032.GC19119@infradead.org> User-Agent: Mutt/1.8.3 (2017-05-23) X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.38]); Tue, 03 Oct 2017 07:21:57 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Oct 02, 2017 at 06:40:32AM -0700, Christoph Hellwig wrote: > On Sat, Sep 30, 2017 at 02:12:11PM +0800, Ming Lei wrote: > > We need to pass PREEMPT flags to blk_queue_enter() > > for allocating request with RQF_PREEMPT in the > > following patch. > > I don't like having another name space for flags. It seems like > we should simply pass ops on, and then map the nowait allocations > to RQF_PREEMPT to REQ_PREEMPT as done in the series from Bart. OK, no problem, will integrate Bart's patches in this patchset. -- Ming