From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AB000CA9ED4 for ; Tue, 5 Nov 2019 00:57:33 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8D65320717 for ; Tue, 5 Nov 2019 00:57:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730203AbfKEA5d (ORCPT ); Mon, 4 Nov 2019 19:57:33 -0500 Received: from verein.lst.de ([213.95.11.211]:42272 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730066AbfKEA5c (ORCPT ); Mon, 4 Nov 2019 19:57:32 -0500 Received: by verein.lst.de (Postfix, from userid 2407) id A9B7268BE1; Tue, 5 Nov 2019 01:57:29 +0100 (CET) Date: Tue, 5 Nov 2019 01:57:29 +0100 From: Christoph Hellwig To: Bart Van Assche Cc: Avri Altman , "James E . J . Bottomley" , "Martin K . Petersen" , linux-scsi@vger.kernel.org, Christoph Hellwig , Yaniv Gardi , Subhash Jadavani , Stanley Chu , Tomas Winkler Subject: Re: [PATCH RFC v2 2/5] ufs: Use reserved tags for TMFs Message-ID: <20191105005729.GA29695@lst.de> References: <20191105004226.232635-1-bvanassche@acm.org> <20191105004226.232635-3-bvanassche@acm.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191105004226.232635-3-bvanassche@acm.org> User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-scsi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org On Mon, Nov 04, 2019 at 04:42:23PM -0800, Bart Van Assche wrote: > Reserved tags are numerically lower than non-reserved tags. Compensate the > change caused by reserving tags by subtracting the number of reserved tags > from the tag number assigned by the block layer. Why would you do that? Do we really care about the exact tag number? If so would it make sense to reverse in the block layer how we allocate private vs normal tags? Also this change should probably merged into the patch that actually starts using the private tags by actually allocating requests using them.