From patchwork Sat Dec 1 00:23:05 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Matteo Croce X-Patchwork-Id: 1006269 X-Patchwork-Delegate: bpf@iogearbox.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=redhat.com Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 436Bmy14TGz9s9G for ; Sat, 1 Dec 2018 11:23:14 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726586AbeLALeT (ORCPT ); Sat, 1 Dec 2018 06:34:19 -0500 Received: from mail-wm1-f67.google.com ([209.85.128.67]:50252 "EHLO mail-wm1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725867AbeLALeS (ORCPT ); Sat, 1 Dec 2018 06:34:18 -0500 Received: by mail-wm1-f67.google.com with SMTP id 125so595027wmh.0 for ; Fri, 30 Nov 2018 16:23:10 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=5/bHV9VcgwIxAF6RjVffiNnbLV0J0u9xZU4SiuiNI9I=; b=ghJLdyOKBqTJz8WZnU4/Kj3+mrXsV0qDCJ44/dRa6lwREnDE2FTqTwLV2z88HI1E43 n/e9ORi7apqxnIkpVNReNOSHNZxzcG+BZuiP++C3bVIxKVNz/f2X4quWRo+uMIlxsIQ4 3n33l+xrsvntXg7G7WxCDQDfBZP+EnD7kCWJ07rVAJpE8jJ89yDzblqLGp8UaAxjWT/9 Dwa/gRNrOqvryShfihixQiqE97ms0PKmiI97xtIuOPOpgaB59xTftj3f7y8LoEb4kZnt NbUNUM52YIUbNGCHVK27XfyRzymkg8IDy2f14HvJByY3XveGp2YFZuNxdhBhxRobyoXJ JEGw== X-Gm-Message-State: AA+aEWbrc44hpbpvFsenCh6Zw5hNHV3boeJvPcl7GqcoF4ML6e+Is805 B/NaKKyEZKz31jSQVybt3m9U/Lb1fNE= X-Google-Smtp-Source: AFSGD/U6jVJtBQuSNnN+elJTzHAD3JSKWT6vqRigKcmXFW9Y4OTjj20lqa25CmA2zBA5x4EXzjuPRQ== X-Received: by 2002:a1c:be11:: with SMTP id o17mr681166wmf.111.1543623789490; Fri, 30 Nov 2018 16:23:09 -0800 (PST) Received: from raver.teknoraver.net (net-93-66-170-114.cust.vodafonedsl.it. [93.66.170.114]) by smtp.gmail.com with ESMTPSA id t187-v6sm429542wmt.45.2018.11.30.16.23.08 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 30 Nov 2018 16:23:08 -0800 (PST) From: Matteo Croce To: netdev@vger.kernel.org Cc: Alexei Starovoitov , Daniel Borkmann , John Fastabend , Y Song Subject: [PATCH bpf-next v2 1/2] samples: bpf: improve xdp1 example Date: Sat, 1 Dec 2018 01:23:05 +0100 Message-Id: <20181201002306.25834-2-mcroce@redhat.com> X-Mailer: git-send-email 2.19.1 In-Reply-To: <20181201002306.25834-1-mcroce@redhat.com> References: <20181201002306.25834-1-mcroce@redhat.com> MIME-Version: 1.0 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Store only the total packet count for every protocol, instead of the whole per-cpu array. Use bpf_map_get_next_key() to iterate the map, instead of looking up all the protocols. Signed-off-by: Matteo Croce --- samples/bpf/xdp1_user.c | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/samples/bpf/xdp1_user.c b/samples/bpf/xdp1_user.c index b02c531510ed..4f3d824fc044 100644 --- a/samples/bpf/xdp1_user.c +++ b/samples/bpf/xdp1_user.c @@ -34,26 +34,24 @@ static void int_exit(int sig) static void poll_stats(int map_fd, int interval) { unsigned int nr_cpus = bpf_num_possible_cpus(); - const unsigned int nr_keys = 256; - __u64 values[nr_cpus], prev[nr_keys][nr_cpus]; - __u32 key; + __u64 values[nr_cpus], prev[UINT8_MAX] = { 0 }; int i; - memset(prev, 0, sizeof(prev)); - while (1) { + __u32 key = UINT32_MAX; + sleep(interval); - for (key = 0; key < nr_keys; key++) { + while (bpf_map_get_next_key(map_fd, &key, &key) != -1) { __u64 sum = 0; assert(bpf_map_lookup_elem(map_fd, &key, values) == 0); for (i = 0; i < nr_cpus; i++) - sum += (values[i] - prev[key][i]); - if (sum) + sum += values[i]; + if (sum > prev[key]) printf("proto %u: %10llu pkt/s\n", - key, sum / interval); - memcpy(prev[key], values, sizeof(values)); + key, (sum - prev[key]) / interval); + prev[key] = sum; } } }