From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jakub Kicinski Subject: Re: [PATCH net-next 0/3] nfp: flower: speed up stats update loop Date: Wed, 10 Oct 2018 08:08:14 -0700 Message-ID: <20181010080814.3c28c3d6@cakuba.netronome.com> References: <20181009015736.30268-1-jakub.kicinski@netronome.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: David Miller , Linux Netdev List , oss-drivers@netronome.com To: Or Gerlitz Return-path: Received: from mail-qt1-f194.google.com ([209.85.160.194]:36142 "EHLO mail-qt1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726649AbeJJWaz (ORCPT ); Wed, 10 Oct 2018 18:30:55 -0400 Received: by mail-qt1-f194.google.com with SMTP id u34-v6so6018844qth.3 for ; Wed, 10 Oct 2018 08:08:19 -0700 (PDT) In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Wed, 10 Oct 2018 16:40:56 +0300, Or Gerlitz wrote: > On Tue, Oct 9, 2018 at 4:58 AM Jakub Kicinski > wrote: > > > Given that our statistic IDs are already array indices, and considering > > each statistic is only 24B in size, we decided to forego the use > > 8B packet + 8B bytes --> 16B -- does your FW/HW provide last use? how do > you express this lastuse value in host jiffies? 24B is the size of the structure we keep in the driver, which costs host memory. The message from FW contains this: struct nfp_fl_stats_frame { __be32 stats_con_id; __be32 pkt_count; __be64 byte_count; __be64 stats_cookie; };