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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D2614C4167B for ; Tue, 5 Dec 2023 05:59:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:CC:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=3LK9jcnO2LKsXvZ4d4db8H6yT9gdmGLVtbrluxK8Gv0=; b=uhf1JvQAIWBgxBQEeZMSpooIZj 8H7mW0wLUFmWBfWsiKNhIsKor70rkzVcVPpe9Y83GFUn9TYkUT6P/TFe8pSpeG4ndcBjdVYDTtTA3 3vFQsZj33VpaNfgB+yR8G17psjgc3JpEXy9WIzZgP6dfdkYu3qbB6ZnQV6NEYfgUI0cK/C2qmgOK8 VAR+5ErF72SMAJeIElyeok1o3vKrjhPoR0swR78Ajp6//dLc5/f7bp75bF1kDyEX12cuS9AuedeIb HvXNx0j5+Wfqw4dGOx7uzYmLfoK3TQ7N4l+ZtFpvkcrd8BWnluVDRYTGdTrUdlrnlcsanWOYsaVcZ Q6Ugt6Dg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rAOT7-006LyU-25; Tue, 05 Dec 2023 05:59:42 +0000 Received: from mx0a-0031df01.pphosted.com ([205.220.168.131]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rAOT3-006Lx3-1s; Tue, 05 Dec 2023 05:59:39 +0000 Received: from pps.filterd (m0279867.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 3B530TCh010074; Tue, 5 Dec 2023 05:59:26 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h=message-id : date : mime-version : subject : to : cc : references : from : in-reply-to : content-type : content-transfer-encoding; s=qcppdkim1; bh=3LK9jcnO2LKsXvZ4d4db8H6yT9gdmGLVtbrluxK8Gv0=; b=o8m8mIqi/aC0QrUWWiuWVZjs9GXCeX+xG8xlFpLj4irv22PQlGNrgUs/TZN7Hy7ycN0m RkkZxtrCGk4XibHSUdjiDc3ZwFLx7Zh+ghQXRkvB6iKND+aNNlT68UeAH7/FZvBMwM+S igz96hsNywiVf/AHgU3eEcPaEJn6JqsRuiIPtsavJxD139PVLgJN52EJnCS8NYPLH6IB gn0tBNKjJarZ8lNvL9JJCuwUFvnc0daqEVOGQHDsEmYUCfKizlU3KQ9I+SeBRzR82GG8 BIdONGQ3MZsJCSenhCcJ0o13Mn4IjpFQOTStJXW9WFUCau87kpugwGVCYjEfJg6Dc8rA Ng== Received: from nalasppmta04.qualcomm.com (Global_NAT1.qualcomm.com [129.46.96.20]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3usdf7jdek-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 05 Dec 2023 05:59:26 +0000 Received: from nalasex01b.na.qualcomm.com (nalasex01b.na.qualcomm.com [10.47.209.197]) by NALASPPMTA04.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 3B55x0Y2022211 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 5 Dec 2023 05:59:00 GMT Received: from [10.217.219.220] (10.80.80.8) by nalasex01b.na.qualcomm.com (10.47.209.197) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.40; Mon, 4 Dec 2023 21:58:54 -0800 Message-ID: <692cd503-5b14-4be6-831d-d8e9c282a95e@quicinc.com> Date: Tue, 5 Dec 2023 11:28:51 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH V2 1/3] ufs: core: Add CPU latency QoS support for ufs driver To: Bart Van Assche , "James E.J. Bottomley" , "Martin K. Petersen" , Peter Wang , Manivannan Sadhasivam , Andy Gross , Bjorn Andersson , Konrad Dybcio , "Matthias Brugger" , AngeloGioacchino Del Regno , CC: Alim Akhtar , Avri Altman , , , , , , , , Nitin Rawat References: <20231204143101.64163-1-quic_mnaresh@quicinc.com> <20231204143101.64163-2-quic_mnaresh@quicinc.com> <590ade27-b4da-49be-933b-e9959aa0cd4c@acm.org> Content-Language: en-US From: Naresh Maramaina In-Reply-To: <590ade27-b4da-49be-933b-e9959aa0cd4c@acm.org> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01b.na.qualcomm.com (10.46.141.250) To nalasex01b.na.qualcomm.com (10.47.209.197) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-ORIG-GUID: 94TVaSz2EVX3106dVDD8YYFE_UR_i5ND X-Proofpoint-GUID: 94TVaSz2EVX3106dVDD8YYFE_UR_i5ND X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.997,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2023-12-05_03,2023-12-04_01,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 suspectscore=0 lowpriorityscore=0 priorityscore=1501 bulkscore=0 spamscore=0 clxscore=1015 adultscore=0 phishscore=0 malwarescore=0 mlxscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2311060000 definitions=main-2312050048 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231204_215937_630407_836DC7C8 X-CRM114-Status: GOOD ( 20.79 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On 12/5/2023 12:30 AM, Bart Van Assche wrote: > On 12/4/23 06:30, Maramaina Naresh wrote: >> +    u32    (*config_qos_vote)(struct ufs_hba *hba); > > Please remove the above callback since this patch series does not > introduce any instances of this callback. > Sure Bart, i will take care of this comment in next patch set. If some SoC vendor have a different qos vote value then this callback can be added in future. >> + >> +    /* This capability allows the host controller driver to use the >> PM QoS >> +     * feature. >> +     */ >> +    UFSHCD_CAP_PM_QOS                = 1 << 13, >>   }; > > Why does it depend on the host driver whether or not PM QoS is > enabled? Why isn't it enabled unconditionally? For some platform vendors power KPI might be more important than random io KPI. Hence this flag is disabled by default and can be enabled based on platform requirement. > >> + * @pm_qos_req: PM QoS request handle >> + * @pm_qos_init: flag to check if pm qos init completed >>    */ > > Documentation for pm_qos_init is missing. > Sorry, i didn't get your comment, i have already added documentation for @pm_qos_init, @pm_qos_req variable as above. Do you want me to add this information some where else as well? >>   struct ufs_hba { >>       void __iomem *mmio_base; >> @@ -1076,6 +1089,9 @@ struct ufs_hba { >>       struct ufs_hw_queue *uhq; >>       struct ufs_hw_queue *dev_cmd_queue; >>       struct ufshcd_mcq_opr_info_t mcq_opr[OPR_MAX]; >> +    struct pm_qos_request pm_qos_req; >> +    bool pm_qos_init; >> +    u32 qos_vote; > > Please rename "pm_qos_init" into "pm_qos_initialized". > Sure Bart, i will take care of this comment in next patch set. > Thanks, > > Bart. > Thanks, Naresh.