Received: by 2002:ab2:1689:0:b0:1f7:5705:b850 with SMTP id d9csp1160188lqa; Sun, 28 Apr 2024 22:55:29 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCUTx07rcuUkYkKIU0SwYZShcG1o0qfodJndLvl8IzV8flz/YUgQB9jXttLI7SliMC+5XDzb/aSK8WusYXR+yJAmdxU7TWvXtcPpPpq/MQ== X-Google-Smtp-Source: AGHT+IGJcjJ/fFcc4IE6GYP04wOn2Uhqsk5l0KowamzeJi6Qm6gOVVP3L/Di2+3xBPC342ZndwJ1 X-Received: by 2002:a05:622a:591:b0:43b:6e0:4291 with SMTP id c17-20020a05622a059100b0043b06e04291mr374400qtb.53.1714370129639; Sun, 28 Apr 2024 22:55:29 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1714370129; cv=pass; d=google.com; s=arc-20160816; b=ZkAwS72cGmYcVyQR46gfoqjW4w2isa/U+wvAJBXsYLdARHyTyZfm++JFxIMV4nK1Fl ZDg84zYiFNdwjYeHEzlTmkFNiphwRAq1YhV4esus5E5GgRfQbegd5+aBGFIhFlHdMHf/ UpmjbKMsaxAqqum/Zb6jhHyXw4pfFndakHvWiLx/4WhoJ4rye8NdnxQAp0S7wVnOdI1p qtu5ho+qsporXwqWHi5G6SUvZEe5BVGMrM9UO0ztKlyL+hi8+P4Crlow3BX3C4+lHu4f 2JjV5r253zD/VUUA+0pgRxoGvB4vu6SQfTuRkq0mg0wMOKIOoTsNuLXkgxYYPdOfTGpF ugZw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id:dkim-signature; bh=gLzy6b8OJzfnnQC7tsrjEadostV0B9ZF2+TL8e/KtDY=; fh=PrIEA24EGWDjht/xuLxe9RY74vGG29dgBTfn193yTHc=; b=zL1FsnbWgMmgSsQkU7BlJhpN1UcUO0x61sB2rEGxMOxaHMYhikLRHNmCsT+kk8sJ1E vBIn94vY9FD7/tGrpxKXxirgL4vE8mlwG3TaK5RiA3afIgOuZaiet+wleoS7DnCJ4X6M 4XZ0c6lgRiWRomz0V5EadgR9FBFaINp0ampGWVbc2GAuqJv7olWssteD/S/cFD8nF0hg FE9LEcChJ94LvUH3jv59yzAXVsy+4XEGssu6jRrG331BO4GlspBwU0XDYZZgmidXXw/n 90q2xJQnkoKRgvpCLL+X1CbvKD5+JwbjVF4lMt0XejMPid5f9PwBFTUxhKTs8DYq3quF R9HQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=izBL91tb; arc=pass (i=1 spf=pass spfdomain=ti.com dkim=pass dkdomain=ti.com dmarc=pass fromdomain=ti.com); spf=pass (google.com: domain of linux-kernel+bounces-161760-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-161760-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id ge22-20020a05622a5c9600b0043ae0014fd3si2231485qtb.117.2024.04.28.22.55.29 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 28 Apr 2024 22:55:29 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-161760-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=izBL91tb; arc=pass (i=1 spf=pass spfdomain=ti.com dkim=pass dkdomain=ti.com dmarc=pass fromdomain=ti.com); spf=pass (google.com: domain of linux-kernel+bounces-161760-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-161760-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 5EA251C216CD for ; Mon, 29 Apr 2024 05:55:29 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A297A1118A; Mon, 29 Apr 2024 05:55:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="izBL91tb" Received: from fllv0016.ext.ti.com (fllv0016.ext.ti.com [198.47.19.142]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 451AFDF59; Mon, 29 Apr 2024 05:55:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.47.19.142 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714370112; cv=none; b=b+5C93ZiAdFeVEPOuF5DbAfpg0iENle4LH90RjdsH68nJorUmz6bqyQADa7yjv8wLH4QIkkUUtvKIImPOKPBZ7JoejYNVAh/OsU71tHerYdBh5luP5JWT30QWL51EVyxjRFjYX/L5s7O7e+/+WMSgHi3lyPy5z7nCbajnkq0b3k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714370112; c=relaxed/simple; bh=eX3H8wIHM0FiZFB+0x0D89Bzoy/mfyWPnroB4QnwF4I=; h=Message-ID:Date:MIME-Version:Subject:To:CC:References:From: In-Reply-To:Content-Type; b=DNPy06pBHtd/kW9heQvUH/2UTJSTm2s4OEAbayTnrWMbVlQLijLdl7HDbtRvPwrg9ZyeomKE3FCvXDHmy3fThYGsgBMm9LhuOUizyKpSYy2xwplnarPs5iU/ctcD2WYLRyctdPpFav34of1LcUvQFNS6iewA7TJA6qD7y1NrCx4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com; spf=pass smtp.mailfrom=ti.com; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b=izBL91tb; arc=none smtp.client-ip=198.47.19.142 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ti.com Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 43T5svHD072170; Mon, 29 Apr 2024 00:54:57 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1714370097; bh=gLzy6b8OJzfnnQC7tsrjEadostV0B9ZF2+TL8e/KtDY=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=izBL91tbOV2BdJNXdqzxBLj8xOluaLxal4ZexGf+deAsojZnQ9LqFekcG1WntIkCf NxDj2Xc2rNzXVFKBBU53YfdgmoRY6YgSaFrngJfLdI0U4M1wuC/fmB6kCuUQZMd7Fx 1jeALg5N44kJ7kAOL3rVrM1XXPBoPwarYgscj5zg= Received: from DFLE100.ent.ti.com (dfle100.ent.ti.com [10.64.6.21]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 43T5svsF071960 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Mon, 29 Apr 2024 00:54:57 -0500 Received: from DFLE112.ent.ti.com (10.64.6.33) by DFLE100.ent.ti.com (10.64.6.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Mon, 29 Apr 2024 00:54:57 -0500 Received: from lelvsmtp5.itg.ti.com (10.180.75.250) by DFLE112.ent.ti.com (10.64.6.33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Mon, 29 Apr 2024 00:54:56 -0500 Received: from [10.24.69.25] (danish-tpc.dhcp.ti.com [10.24.69.25]) by lelvsmtp5.itg.ti.com (8.15.2/8.15.2) with ESMTP id 43T5spBo097279; Mon, 29 Apr 2024 00:54:51 -0500 Message-ID: Date: Mon, 29 Apr 2024 11:24:50 +0530 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next] net: ti: icssg_prueth: Add SW TX / RX Coalescing based on hrtimers Content-Language: en-US To: Andrew Lunn CC: Dan Carpenter , Heiner Kallweit , Jan Kiszka , Diogo Ivo , Paolo Abeni , Jakub Kicinski , Eric Dumazet , "David S. Miller" , , , , , Vignesh Raghavendra , , Roger Quadros References: <20240424091823.1814136-1-danishanwar@ti.com> <98588a89-4970-4d75-be8a-ac410d77789f@lunn.ch> <1c5809f2-b69d-48d1-8c27-285f164ebeb8@ti.com> <2a4bea87-04bf-4373-8220-69650b435710@lunn.ch> From: MD Danish Anwar In-Reply-To: <2a4bea87-04bf-4373-8220-69650b435710@lunn.ch> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 On 25/04/24 6:19 pm, Andrew Lunn wrote: >> I did some benchmarking today with, >> 1. Default driver (without any IRQ coalescing enabled) >> 2. IRQ Coalescing (With this patch) >> 3. Default IRQ Coalescing (Suggested by you in the above patch) >> >> I have pasted the full logs at [1]. >> >> Below are the final numbers, >> >> ============================================================== >> Method | Tput_TX | CPU_TX | Tput_RX | CPU_RX | >> ============================================================== >> Default Driver 943 Mbps 31% 517 Mbps 38% | >> IRQ Coalescing (Patch) 943 Mbps 28% 518 Mbps 25% | >> Default IRQ Coalescing 942 Mbps 32% 521 Mbps 25% | >> ============================================================== >> >> I see that the performance number is more or less same for all three >> methods only the CPU load seems to be varying. The IRQ coalescing patch >> (using hrtimer) seems to improve the cpu load by 3-4% in TX and 13% in >> RX. Whereas the default method that you have suggested doesn't give any >> improvemnet in tx however cpu load improves in RX with the same amount >> as method 2. >> >> Please let me know if this patch is OK to you based on the benchmarking? > > It is good to include benchmark results in patches which claim to > improve performance. Please add the default and the patch version > results to the commit message. Sure, I will add the benchmarking numbers in commit message and send v2. > > The numbers show your more complex version does bring benefits, so it > is O.K. to use it. I just wounder how many other drivers would benefit > from a one line change. > > Andrew -- Thanks and Regards, Danish