Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp653507ybz; Wed, 15 Apr 2020 16:01:46 -0700 (PDT) X-Google-Smtp-Source: APiQypKzizN/zijOu6PfWIG8SbkXlR8TFZO51QQwsOgepsmwX882zbbVEtayijaJRFeXCormGOV5 X-Received: by 2002:a17:907:4420:: with SMTP id om24mr6719981ejb.99.1586991706522; Wed, 15 Apr 2020 16:01:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1586991706; cv=none; d=google.com; s=arc-20160816; b=FCSUgRN0Er1aGKBw39R4X47O52VQuR4RU90vwEyuqMomCYS4vVQAZxD151Cn4whEqI m+XEIE+8h70rhBP6TC2AL02EWMAzeL+ZpIZW7Zeth1hFI4UttKkyWwRVzU+z7zaJ+DyM cQV9jGvdVhgqiGlXGzk3Kmb93e+0zRQ97z8J9byHDLhzX7wUHkeb7tDV9YBeyXpzB/sM nJY+R1+eS3e9VWrcT4ks6peMIlS/erCgI4bDYJvDPDf3v0mLxwD0JLWEbtpMq+VqzsGY W5oj9gO1CHV+Y3VkltETmf5yrb+ALpmGZjnJ6cW5oGxiklFK08O3O7NfDOkY12j5kItn AZJw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:dkim-signature:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=eXKsl5AwjUrhoGQdcNjptVpYoo1UjOgc5wmqJgrcC5Q=; b=AtLDAx8OsjOprWXgbox2XVAHqizfohCFkh2gehhFi1inz2xl+X9k/4h7CrvjdPTI97 pj+uR/g7wzPpsl3cUmxCrEfwqRwrorRQCsc1WgZaxvLfJGg60e6V/jAyrqKbSum93PbA PrITt/8iG9raCSjil+XI6nHZ+kmbWJwipjsaLzQPk4VRN3dLVPKVfS3ZdBIjQiZ4dz+V aLgczXnRWoEfBnFmy8P8ItnxLRfphxyxLcuUYlXIVcSaHuwVs/mlvsjS/ucLtvqO/JP6 ROJwvSSh3gU/yUJEGiE+bQunn1T9RKmeaSRg2CcMLnPtolxKnqm3SFubVX0U7ClODIPj ZQ3Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nvidia.com header.s=n1 header.b=BufMhBhR; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nvidia.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id dn12si8203546ejc.52.2020.04.15.16.01.23; Wed, 15 Apr 2020 16:01:46 -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; dkim=pass header.i=@nvidia.com header.s=n1 header.b=BufMhBhR; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nvidia.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2896958AbgDOLZr (ORCPT + 99 others); Wed, 15 Apr 2020 07:25:47 -0400 Received: from hqnvemgate26.nvidia.com ([216.228.121.65]:11983 "EHLO hqnvemgate26.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2896948AbgDOLZa (ORCPT ); Wed, 15 Apr 2020 07:25:30 -0400 Received: from hqpgpgate102.nvidia.com (Not Verified[216.228.121.13]) by hqnvemgate26.nvidia.com (using TLS: TLSv1.2, DES-CBC3-SHA) id ; Wed, 15 Apr 2020 04:25:16 -0700 Received: from hqmail.nvidia.com ([172.20.161.6]) by hqpgpgate102.nvidia.com (PGP Universal service); Wed, 15 Apr 2020 04:25:29 -0700 X-PGP-Universal: processed; by hqpgpgate102.nvidia.com on Wed, 15 Apr 2020 04:25:29 -0700 Received: from DRHQMAIL107.nvidia.com (10.27.9.16) by HQMAIL109.nvidia.com (172.20.187.15) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Wed, 15 Apr 2020 11:25:28 +0000 Received: from [10.24.37.103] (10.124.1.5) by DRHQMAIL107.nvidia.com (10.27.9.16) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Wed, 15 Apr 2020 11:25:17 +0000 Subject: Re: [TEGRA194_CPUFREQ Patch 2/3] cpufreq: Add Tegra194 cpufreq driver To: Viresh Kumar CC: , , , , , , , , , , , References: <20200326115023.xy3n5bl7uetuw7mx@vireshk-i7> <20200406025549.qfwzlk3745y3r274@vireshk-i7> <3ab4136c-8cca-c2f9-d286-b82dac23e720@nvidia.com> <20200408055301.jhvu5bc2luu3b5qr@vireshk-i7> <08307e54-0e14-14a3-7d6a-d59e1e04a683@nvidia.com> <20200409074415.twpzu2n4frqlde7b@vireshk-i7> <00390070-38a1-19aa-ca59-42c4658bee7e@nvidia.com> <20200413062141.a6hmwipexhv3sctq@vireshk-i7> <64b609f1-efb1-425f-a91a-27a492bd3ec4@nvidia.com> <20200414054504.e3qn2cnxqur4sclw@vireshk-i7> From: Sumit Gupta Message-ID: Date: Wed, 15 Apr 2020 16:55:53 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: <20200414054504.e3qn2cnxqur4sclw@vireshk-i7> X-Originating-IP: [10.124.1.5] X-ClientProxiedBy: HQMAIL105.nvidia.com (172.20.187.12) To DRHQMAIL107.nvidia.com (10.27.9.16) Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nvidia.com; s=n1; t=1586949916; bh=eXKsl5AwjUrhoGQdcNjptVpYoo1UjOgc5wmqJgrcC5Q=; h=X-PGP-Universal:Subject:To:CC:References:From:Message-ID:Date: User-Agent:MIME-Version:In-Reply-To:X-Originating-IP: X-ClientProxiedBy:Content-Type:Content-Language: Content-Transfer-Encoding; b=BufMhBhRaHC5kGcUpG2IkNn/Bfg4VQWvgSaQNbxoHn0VvAvRLQsl7xjWkgBtNJ7Kh t2ipKJv2gpkrMXWf0a8+Ok30Adzu+bjmHntEskmxM78ojT6yDs3Va1eAN41kin2EQB W+EPOcwvBy5bflyNRl0ZUZerXfolh7aB+8mr7Vx+u3MtYcKh9Es5gkcD8UvlpU2LkP 5m2FgGLBadbB6U4ySZHwUGPeFcU3Dfd6MhmIDTxqDa51lUqqbxBSRxwc1cFSXF+Ool VPAofppZofnYUY//2D9agcpOmHfefVFjPxzlCoajeCWLbuFHF8tPKE3VTuS9GST+Lw 3w3Q2+kkG0IeA== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 14/04/20 11:15 AM, Viresh Kumar wrote: > External email: Use caution opening links or attachments > > > On 13-04-20, 17:50, Sumit Gupta wrote: >> This was done considering long delay value as explained previously. >> Do you think that smp_call_function_single() would be better than work queue >> here? > > Don't work with assumptions, you should test both and see which one > works better. Workqueue should never be faster than > smp_call_function_single() with my understanding. Checked the time taken and its almost same in both cases. Earlier we used smp_call_function_single(), but delay time period was small in that SOC. In T194, the time period was more. So, this is an optimization done because using work queue has advantage as interrupts will not be disabled for that period. If you think work queue is not required, then can remove it. The functionality works fine in both cases. > > -- > viresh >