Received: by 10.213.65.68 with SMTP id h4csp458632imn; Fri, 23 Mar 2018 08:13:47 -0700 (PDT) X-Google-Smtp-Source: AG47ELvqAB1a53nkKZSRa9d4doMl0uhwf8C+qKFtQmGKp6nepCZKpmTFNMnQ25/oAA6M3YWeXFyw X-Received: by 2002:a17:902:5409:: with SMTP id d9-v6mr17940609pli.176.1521818027524; Fri, 23 Mar 2018 08:13:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521818027; cv=none; d=google.com; s=arc-20160816; b=MnlvqMTUZRAzth0ipyNRsLgPPj5ZY5fwZYsKZin8qqYG56AD8cByZJJoTiBvKFRPH8 /NRZ2HUVWObgqtogkHtfKwQ1dabU9D/gYEweGoI7h/gc3ZeW3iukz0XlERafgsMCYQjT 9NOHuRl3E9X+4nq/F36Rm/CFKodbDVNaYMn0qULOmZGeDXSrSiUHomRW30dFut1fzRCC J1oGY7Bn5BLeFVNj8Zmn/fWrmgOgQB1AO6BQhZjcNQ6iHwpD4S+WNoE0s2CobKpRGA+Z lxFRpCiIt4KBQNApydcp1mV2dFNPfzdpINdWdIsNXGVl+4opR2K7qwFB8B61t2m4Qzpz ferQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :spamdiagnosticmetadata:spamdiagnosticoutput:content-language :accept-language:in-reply-to:references:message-id:date:thread-index :thread-topic:subject:cc:to:from:dkim-signature :arc-authentication-results; bh=VtKfI/nMqRpYi4/kNXxlkmU2UDsXxX8wtAeq6P+TmQs=; b=IHimGkkqog6Yu2hHPvaQAFX1QrrltVFwAcldewD8d57Emn1IgaIVOVZN9bK+0nglXG jCyL+zPLNAKrKiGBiSjwifI8AQPcgHdW5XdBSMUiZhh2akQlTpsQMth3+vrKaWDhpvod mYvka4pjp2ZaiFPSgcA+qfOuxN/ri1vJC3MLt8LcDS/OULQ1HLeJkITCgj4tt/zn5OVv 1ZE25Xm+COWT6+1s/SS179yHRhgF2I000uwu/pnNjgRPM6x7bAllgsOaIKPBpLKeasxR T1KZP8gn8SkAEjUL7JOyA8tzIM4DatwqQmn7YYKwEE3nFWy/SWkt6ZIFxX4JuO3Q1Q1U OjBw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@microsoft.com header.s=selector1 header.b=hHYJ1OAC; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=microsoft.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j4si6162262pgq.119.2018.03.23.08.13.26; Fri, 23 Mar 2018 08:13:47 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@microsoft.com header.s=selector1 header.b=hHYJ1OAC; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=microsoft.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751915AbeCWPMR (ORCPT + 99 others); Fri, 23 Mar 2018 11:12:17 -0400 Received: from mail-bl2nam02on0131.outbound.protection.outlook.com ([104.47.38.131]:24744 "EHLO NAM02-BL2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751747AbeCWPMP (ORCPT ); Fri, 23 Mar 2018 11:12:15 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=VtKfI/nMqRpYi4/kNXxlkmU2UDsXxX8wtAeq6P+TmQs=; b=hHYJ1OAC4FTo8HuvIqkL46KDoVvbrJ+ef1ZtM/B9PKPt9vdfFzPVni17N5HhWopmSUAYZZh8GnxO5/hMf1Sl+Q48fIl+hSAKVQEbgXU/miZouFiRpEk5aRrPPbnHwHvj1gIq1xUFGtdEJiVknHhMjDJqZDrLfh9O+VV5V7NeF4E= Received: from DM5PR2101MB0888.namprd21.prod.outlook.com (52.132.132.157) by DM5PR2101MB1015.namprd21.prod.outlook.com (52.132.133.37) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.631.1; Fri, 23 Mar 2018 15:12:12 +0000 Received: from DM5PR2101MB0888.namprd21.prod.outlook.com ([fe80::c096:fe9d:11a4:f9a0]) by DM5PR2101MB0888.namprd21.prod.outlook.com ([fe80::c096:fe9d:11a4:f9a0%3]) with mapi id 15.20.0631.007; Fri, 23 Mar 2018 15:12:12 +0000 From: KY Srinivasan To: Haiyang Zhang , Long Li , Stephen Hemminger , "James E . J . Bottomley" , "Martin K . Petersen" , "devel@linuxdriverproject.org" , "linux-scsi@vger.kernel.org" , "linux-kernel@vger.kernel.org" CC: Long Li Subject: RE: [PATCH 2/3] Netvsc: Use the vmbus functiton to calculate ring buffer percentage Thread-Topic: [PATCH 2/3] Netvsc: Use the vmbus functiton to calculate ring buffer percentage Thread-Index: AQHTwjxHXRWolAV9rECFIikUIQEk7KPd6ueAgAACz8A= Date: Fri, 23 Mar 2018 15:12:12 +0000 Message-ID: References: <20180323001538.31876-1-longli@linuxonhyperv.com> <20180323001538.31876-2-longli@linuxonhyperv.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [2601:600:a280:b50:78da:ddb7:fbbf:e153] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1;DM5PR2101MB1015;7:soLKs3ZexqlbRGH9JXCKqGKrPF9EhkT2M1FINTD9nBX07ZULQIuRztZgtmzPeUctXlPQuO8/mB+BT3MtZTqyQLwuxK3OHVbQ6j8Jdw/MxxXiC+A4rvWFn7Qvy8R8aFT/l5CvH8OBn5OymhnBbS/EQO0IXc0d1WPOoWz5mgSAMOFYG5M3itPiTXM+bH/jrHjL8wN0VRoZrtPnDiZvQ0ky5ewDdkwwR8Ms3DRWP0dcy2SEnDlqcW+vY6uw82Y1uYsg;20:jrZebNVQo6qfUn5Vl2Slnx+NRr75smoQu9RCFBKv8y19hxAEtwdOgN3BblGau6lFFQBhHcF2twDfFmrF/kH41IBuwmvmIzsSB0Cv3GrgsYxcw1/1W/fHD6QTcZ5Q8FA1OdYoOF7gpA93SclzhEo9mgNPo3r7PEDTufQVTylXFTY= x-ms-exchange-antispam-srfa-diagnostics: SOS; x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: 2671944f-fb45-49c4-6012-08d590d0746f x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(7020095)(4652020)(48565401081)(5600026)(4604075)(3008032)(2017052603328)(7193020);SRVR:DM5PR2101MB1015; x-ms-traffictypediagnostic: DM5PR2101MB1015: x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(28532068793085)(89211679590171)(9452136761055)(146099531331640); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(8211001083)(61425038)(6040522)(2401047)(5005006)(8121501046)(10201501046)(93006095)(93001095)(3002001)(3231221)(944501327)(52105095)(6055026)(61426038)(61427038)(6041310)(20161123564045)(20161123558120)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(6072148)(201708071742011);SRVR:DM5PR2101MB1015;BCL:0;PCL:0;RULEID:;SRVR:DM5PR2101MB1015; x-forefront-prvs: 0620CADDF3 x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(39860400002)(366004)(39380400002)(376002)(396003)(346002)(189003)(199004)(13464003)(8676002)(4326008)(1511001)(6116002)(81156014)(7736002)(102836004)(14454004)(106356001)(6436002)(105586002)(8990500004)(33656002)(3280700002)(8936002)(97736004)(11346002)(3660700001)(81166006)(25786009)(446003)(74316002)(2906002)(86612001)(5250100002)(2501003)(2201001)(305945005)(55016002)(6346003)(53936002)(99286004)(5660300001)(7696005)(10290500003)(86362001)(575784001)(478600001)(9686003)(46003)(2900100001)(6246003)(6506007)(76176011)(68736007)(229853002)(110136005)(316002)(10090500001)(107886003)(22452003)(53546011);DIR:OUT;SFP:1102;SCL:1;SRVR:DM5PR2101MB1015;H:DM5PR2101MB0888.namprd21.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;MX:1;A:1; received-spf: None (protection.outlook.com: microsoft.com does not designate permitted sender hosts) authentication-results: spf=none (sender IP is ) smtp.mailfrom=kys@microsoft.com; x-microsoft-antispam-message-info: bp28gn5ffXaUmwqEVpOiT+p6iwpXXk6pkQDWZNpzFv3dv+ftb5VTqmOdrq3nd1HqfLLxyFfwA57H+rlFF5ycGkIz0JtasjFnQh2JXeI1IazUr9nbVOuoFzIKKr0PHXRMIpm1hzOD4fLLN4JSN0iYgMM7Z3vIZ0IRtGDPDNmaN2Lfug6qepEPOJYNt/dq/sjeOMC71kLS1YgT5cM0pqB+Xl1WHiSwuFJ5bPYhNxtN5b67B7VuMgoqKVSg1Wjmu3NgBu9k62f/I+NLW8j5wKHFT7IslM3swcOsFwLQ8A0blCAcqEVemeIQTcyGiy+e8mo+33EHzBf5EuRaYHebNDj/dQ== spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2671944f-fb45-49c4-6012-08d590d0746f X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Mar 2018 15:12:12.2175 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR2101MB1015 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: Haiyang Zhang > Sent: Friday, March 23, 2018 8:01 AM > To: Long Li ; KY Srinivasan > ; Stephen Hemminger ; > James E . J . Bottomley ; Martin K . Petersen > ; devel@linuxdriverproject.org; linux- > scsi@vger.kernel.org; linux-kernel@vger.kernel.org > Cc: Long Li > Subject: RE: [PATCH 2/3] Netvsc: Use the vmbus functiton to calculate rin= g > buffer percentage >=20 >=20 >=20 > > -----Original Message----- > > From: Long Li > > Sent: Thursday, March 22, 2018 8:16 PM > > To: KY Srinivasan ; Haiyang Zhang > > ; Stephen Hemminger > ; > > James E . J . Bottomley ; Martin K . Petersen > > ; devel@linuxdriverproject.org; linux- > > scsi@vger.kernel.org; linux-kernel@vger.kernel.org > > Cc: Long Li > > Subject: [PATCH 2/3] Netvsc: Use the vmbus functiton to calculate ring > buffer > > percentage > > > > From: Long Li > > > > In Vmbus, we have defined a function to calculate available ring buffer > > percentage to write. > > > > Use that function and remove duplicate netvsc code. > > > > Signed-off-by: Long Li > > --- > > drivers/net/hyperv/netvsc.c | 17 +++-------------- > > drivers/net/hyperv/netvsc_drv.c | 3 --- > > 2 files changed, 3 insertions(+), 17 deletions(-) Why is the patch being sent to the scsi list and not to the network mailing= list and Dave Miller. K. Y > > > > diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hyperv/netvsc.c > index > > 0265d703eb03..8af0069e4d8c 100644 > > --- a/drivers/net/hyperv/netvsc.c > > +++ b/drivers/net/hyperv/netvsc.c > > @@ -31,7 +31,6 @@ > > #include > > #include > > #include > > -#include > > > > #include > > > > @@ -590,17 +589,6 @@ void netvsc_device_remove(struct hv_device > *device) > > #define RING_AVAIL_PERCENT_HIWATER 20 #define > > RING_AVAIL_PERCENT_LOWATER 10 > > > > -/* > > - * Get the percentage of available bytes to write in the ring. > > - * The return value is in range from 0 to 100. > > - */ > > -static u32 hv_ringbuf_avail_percent(const struct hv_ring_buffer_info > > *ring_info) -{ > > - u32 avail_write =3D hv_get_bytes_to_write(ring_info); > > - > > - return reciprocal_divide(avail_write * 100, netvsc_ring_reciprocal); > > -} > > - > > static inline void netvsc_free_send_slot(struct netvsc_device > *net_device, > > u32 index) > > { > > @@ -649,7 +637,8 @@ static void netvsc_send_tx_complete(struct > > netvsc_device *net_device, > > wake_up(&net_device->wait_drain); > > > > if (netif_tx_queue_stopped(netdev_get_tx_queue(ndev, q_idx)) > && > > - (hv_ringbuf_avail_percent(&channel->outbound) > > > RING_AVAIL_PERCENT_HIWATER || > > + (hv_get_avail_to_write_percent(&channel->outbound) > > > + RING_AVAIL_PERCENT_HIWATER || > > queue_sends < 1)) { > > netif_tx_wake_queue(netdev_get_tx_queue(ndev, q_idx)); > > ndev_ctx->eth_stats.wake_queue++; > > @@ -757,7 +746,7 @@ static inline int netvsc_send_pkt( > > struct netdev_queue *txq =3D netdev_get_tx_queue(ndev, packet- > >q_idx); > > u64 req_id; > > int ret; > > - u32 ring_avail =3D hv_ringbuf_avail_percent(&out_channel- > >outbound); > > + u32 ring_avail =3D > > +hv_get_avail_to_write_percent(&out_channel->outbound); > > > > nvmsg.hdr.msg_type =3D NVSP_MSG1_TYPE_SEND_RNDIS_PKT; > > if (skb) > > diff --git a/drivers/net/hyperv/netvsc_drv.c > b/drivers/net/hyperv/netvsc_drv.c > > index faea0be18924..b0b1c2fd2b7b 100644 > > --- a/drivers/net/hyperv/netvsc_drv.c > > +++ b/drivers/net/hyperv/netvsc_drv.c > > @@ -35,7 +35,6 @@ > > #include > > #include > > #include > > -#include > > > > #include > > #include > > @@ -55,7 +54,6 @@ static unsigned int ring_size __ro_after_init =3D 128= ; > > module_param(ring_size, uint, S_IRUGO); > MODULE_PARM_DESC(ring_size, > > "Ring buffer size (# of pages)"); unsigned int netvsc_ring_bytes > __ro_after_init; > > -struct reciprocal_value netvsc_ring_reciprocal __ro_after_init; > > > > static const u32 default_msg =3D NETIF_MSG_DRV | NETIF_MSG_PROBE | > > NETIF_MSG_LINK | NETIF_MSG_IFUP | > > @@ -2186,7 +2184,6 @@ static int __init netvsc_drv_init(void) > > ring_size); > > } > > netvsc_ring_bytes =3D ring_size * PAGE_SIZE; > > - netvsc_ring_reciprocal =3D reciprocal_value(netvsc_ring_bytes); > > > > ret =3D vmbus_driver_register(&netvsc_drv); > > if (ret) > > -- >=20 >=20 > Please also remove netvsc_ring_reciprocal from hyperv_net.h > Thanks. >=20 > Reviewed-by: Haiyang Zhang