Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp4575217pxj; Tue, 25 May 2021 11:03:28 -0700 (PDT) X-Google-Smtp-Source: ABdhPJykS8btBos2/VPybbSVVrCHKEG7h3sgGzEKkHKTrKTU/N4yCfLhs/up+OEmOBd4V3TveSrw X-Received: by 2002:a5e:930d:: with SMTP id k13mr22201428iom.61.1621965808701; Tue, 25 May 2021 11:03:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1621965808; cv=none; d=google.com; s=arc-20160816; b=zjPDdFfy0sefDORz/1tiyTVM3F+OkOqLRk9Mu19NpbpNrTnHdADWGJZBR7fTbZ+y9h r8iCnVa9gneJ0iPpdTV8unXtapTP8po5jxjVqg7HaQ+ToAO1xnmWXSjYnT8X+2HV5A5W rAE07lWRubmVk7SnhC/MfkeDRlxBQgm+VM1CS0nKwlo6W++nPF6N9N838THc4GXttEIQ X0XFiF/l08RI+LcW71gOR+5OGbFXSOocjk0B1aM2/Vma1ymRnWtXtBlTxXEvESZpVrCk UJ06s2kMKrKTa+b+uOP3YUhA+rF789szNc0I4vwMz2VjrqDgn7/+6chHNQQjy6QKrP7m FEcA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject; bh=6kZjbGQduH1O16Py6H5dph2xaDYJhSr3BHluI3PjlOA=; b=PLKbzdvIv1auRMsl3QUHm1zf+OSxSgM8mkg0gIn0fTcGOcrIDeWP1KzZ1aPhlTtt3G P8XaNoDphrS7I7LOfOCK+EoHsT74gVlMliWX6C5sgyrQF3bFzPFIlK3uPgcVa/irZxcO u4d7XcnckwfWn4IHL+6ZZxh/PxgI7543BWlVqDDjNv4McCpmRJHaQK2hRgdxyE8AVPym E5vY9xfj6kdB/3afPHTj5G++KRtVeiWxW0S2Z8BMdYX9M8FrmxE5xkqXoBBJOTGYWAVC Aqfxry98zX7JKo69dzgpObL26HlyDY6QKLyLKzcIwVxJFf9h/Sp8JvIA2HvE3DdNzLc4 oRjQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id i6si9609030ilj.74.2021.05.25.11.03.15; Tue, 25 May 2021 11:03:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233521AbhEYQmJ (ORCPT + 99 others); Tue, 25 May 2021 12:42:09 -0400 Received: from mail-pf1-f178.google.com ([209.85.210.178]:37882 "EHLO mail-pf1-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230377AbhEYQmI (ORCPT ); Tue, 25 May 2021 12:42:08 -0400 Received: by mail-pf1-f178.google.com with SMTP id q67so6991408pfb.4; Tue, 25 May 2021 09:40:37 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=6kZjbGQduH1O16Py6H5dph2xaDYJhSr3BHluI3PjlOA=; b=QHcUQCDgZ+6bYQJlLahxOJHBGDJCoNAtj612tuFwLx4CWota6Srq2SLb1UZ24vpIM1 hEI2vEuh6LUG/Dg8ge4TtUwaVRtsvpvwDueKC5tXmbGVnPHr2JP7D3OXTTZaBxHVxjl4 jKmI1InCtK0oTb8gI4jJujzXawej6qSOT3s1TFaF/rMApG+YGYO6zWx7PKmWUuTyFVeL QnpxKLbsROJWcd8Qc+PxUmEovnkTCQ1hMVYuY6raXGoPuqK5dXnKZhj7wm97oFCiSS4s iFX3me0gRAiHMd19tVGBZm6ByL24/1Ggq84BtST85uJyN1WXOLDThOJFytCzDd7W7cY2 7oLQ== X-Gm-Message-State: AOAM532yoW0XVUxrmEIK+RD6EWBDrItF5kdXLldTpbmGnmmT5BgCMndJ 6DOO8HWNCHWmxWaJHVDwUUs= X-Received: by 2002:a65:6156:: with SMTP id o22mr19679020pgv.71.1621960837224; Tue, 25 May 2021 09:40:37 -0700 (PDT) Received: from [192.168.3.217] (c-73-241-217-19.hsd1.ca.comcast.net. [73.241.217.19]) by smtp.gmail.com with ESMTPSA id i16sm12897203pji.30.2021.05.25.09.40.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 25 May 2021 09:40:36 -0700 (PDT) Subject: Re: [PATCH v1 2/3] scsi: ufs: Optimize host lock on transfer requests send/compl paths To: Can Guo Cc: asutoshd@codeaurora.org, nguyenb@codeaurora.org, hongwus@codeaurora.org, linux-scsi@vger.kernel.org, kernel-team@android.com, Stanley Chu , Alim Akhtar , Avri Altman , "James E.J. Bottomley" , "Martin K. Petersen" , Matthias Brugger , Bean Huo , Jaegeuk Kim , Adrian Hunter , Kiwoong Kim , Satya Tangirala , open list , "moderated list:ARM/Mediatek SoC support" , "moderated list:ARM/Mediatek SoC support" References: <1621845419-14194-1-git-send-email-cang@codeaurora.org> <1621845419-14194-3-git-send-email-cang@codeaurora.org> <41a08b3e-122d-4f1a-abbd-4b5730f880b2@acm.org> <0cfbf580e340073ff972be493a59dbe7@codeaurora.org> From: Bart Van Assche Message-ID: Date: Tue, 25 May 2021 09:40:34 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.2 MIME-Version: 1.0 In-Reply-To: <0cfbf580e340073ff972be493a59dbe7@codeaurora.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 5/24/21 6:40 PM, Can Guo wrote: > Agree with all above, and what you ask is right what we are doing in > the 3rd change - get rid of host lock on dispatch and completion > paths. > > I agree with using dedicated spin locks for dedicated purposes in > UFS driver, e.g., clk gating has its own gating_lock and clk scaling > has its own scaling_lock. But this specific series is only for > improving performance. We will take your comments into consideration > and address it in future. Thanks! Bart.