From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.4 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 05910C4338F for ; Fri, 30 Jul 2021 21:47:40 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id C680D60560 for ; Fri, 30 Jul 2021 21:47:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230364AbhG3Vro (ORCPT ); Fri, 30 Jul 2021 17:47:44 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40058 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230020AbhG3Vrn (ORCPT ); Fri, 30 Jul 2021 17:47:43 -0400 Received: from mail-wr1-x42a.google.com (mail-wr1-x42a.google.com [IPv6:2a00:1450:4864:20::42a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2849AC061765 for ; Fri, 30 Jul 2021 14:47:38 -0700 (PDT) Received: by mail-wr1-x42a.google.com with SMTP id b11so7668978wrx.6 for ; Fri, 30 Jul 2021 14:47:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=isovalent-com.20150623.gappssmtp.com; s=20150623; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=i46g8UG3DHAsLBiviNjkwsX1JyJdrAOzSZ3O/djol10=; b=n0PH2Lho/h9UcCl/xHfLGmDzNNo2zkXMvVcr0GMJE4TddTv+dRVaj+aVKxYmWiQ3Fp RINXJ8rmx+ZFCcS8tcyWsC/8wNQpRiHTbV7NHmQdqigCuSjLPJ8maoj2dmSlwyP1JOdM oS6nsVrcooWiASjrAi0GdF50S6cxdWWciDMX9zbuFSOh/mPMhgLPAR3r73Pd0l4S0EUJ D2iitfPad7qOWILYjZJ+OLSMMgQhHxQ85taG9IvUj8+60aU3NdybOhD/Z2gMRV/LBltc QEQ7w3VAK1iGZfRUqHHSxQE7BEukM2VMvXTj309nwJRzncQbMqlghm2vll5G18NCkhvY 6o8w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=i46g8UG3DHAsLBiviNjkwsX1JyJdrAOzSZ3O/djol10=; b=cBbXJ+J8U5mjH9AFxGDosrZye2AjbeZVb4vnTk7QYu87QmBFZA/0TEGzkXmAb949qg 7eXXHFKtRDVNSz7kBUq/CNGQ/4b094oHzfrryOiaImPMQ5b1UosrOv9QkoWe2fJtiSPG tW4kC9Ag1laUz932/vGqc+MtK1zyuobUXfM2sBKMs3mVf3IAKyqlMDjZN8rU70zrrOYr vqCij7KPfB5nFdUfRRHhcul8obYIaHQD67Psm1LMNlF4SCZaHW/W1zwsbiq1bQYR9Tba GrYR4TMYd/RbJK8ecB++r0QZ8/GgdRZ65EEB5562iVKJkk4S4B03JVp2v9RVQ84b4pau wrjA== X-Gm-Message-State: AOAM533Je42JfW2OxBoEbpUmL4fCvCf1OBF/RWbT3fXzA0rQx0vu0XRY VlKhGofXMMQ45+js1XA2ZNdbT0baHx7bEQ0c X-Google-Smtp-Source: ABdhPJyQOFXhqIkCFOWUlCK4u8olBcohNoXcCPsXEOzescIhNAK8TXLqF/AvCtVBboBJRq7bqEEVaw== X-Received: by 2002:a05:6000:1149:: with SMTP id d9mr5524580wrx.26.1627681656407; Fri, 30 Jul 2021 14:47:36 -0700 (PDT) Received: from [192.168.1.8] ([149.86.68.125]) by smtp.gmail.com with ESMTPSA id d15sm2847727wrn.28.2021.07.30.14.47.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 30 Jul 2021 14:47:35 -0700 (PDT) Subject: Re: [PATCH bpf-next 3/7] tools: bpftool: complete and synchronise attach or map types To: Andrii Nakryiko Cc: Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Networking , bpf References: <20210729162932.30365-1-quentin@isovalent.com> <20210729162932.30365-4-quentin@isovalent.com> From: Quentin Monnet Message-ID: <7c8e25b4-1546-a411-ef21-ae6cd09477df@isovalent.com> Date: Fri, 30 Jul 2021 22:47:34 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.12.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: bpf@vger.kernel.org 2021-07-30 11:52 UTC-0700 ~ Andrii Nakryiko > On Thu, Jul 29, 2021 at 9:29 AM Quentin Monnet wrote: >> >> Update bpftool's list of attach type names to tell it about the latest >> attach types, or the "ringbuf" map. Also update the documentation, help >> messages, and bash completion when relevant. >> >> These missing items were reported by the newly added Python script used >> to help maintain consistency in bpftool. >> >> Signed-off-by: Quentin Monnet >> --- >> .../bpftool/Documentation/bpftool-prog.rst | 2 +- >> tools/bpf/bpftool/bash-completion/bpftool | 5 +- >> tools/bpf/bpftool/common.c | 76 ++++++++++--------- >> tools/bpf/bpftool/prog.c | 4 +- >> 4 files changed, 47 insertions(+), 40 deletions(-) >> >> diff --git a/tools/bpf/bpftool/common.c b/tools/bpf/bpftool/common.c >> index 1828bba19020..b47797cac64f 100644 >> --- a/tools/bpf/bpftool/common.c >> +++ b/tools/bpf/bpftool/common.c >> @@ -31,42 +31,48 @@ >> #endif >> >> const char * const attach_type_name[__MAX_BPF_ATTACH_TYPE] = { >> - [BPF_CGROUP_INET_INGRESS] = "ingress", >> - [BPF_CGROUP_INET_EGRESS] = "egress", >> - [BPF_CGROUP_INET_SOCK_CREATE] = "sock_create", >> - [BPF_CGROUP_INET_SOCK_RELEASE] = "sock_release", >> - [BPF_CGROUP_SOCK_OPS] = "sock_ops", >> - [BPF_CGROUP_DEVICE] = "device", >> - [BPF_CGROUP_INET4_BIND] = "bind4", >> - [BPF_CGROUP_INET6_BIND] = "bind6", >> - [BPF_CGROUP_INET4_CONNECT] = "connect4", >> - [BPF_CGROUP_INET6_CONNECT] = "connect6", >> - [BPF_CGROUP_INET4_POST_BIND] = "post_bind4", >> - [BPF_CGROUP_INET6_POST_BIND] = "post_bind6", >> - [BPF_CGROUP_INET4_GETPEERNAME] = "getpeername4", >> - [BPF_CGROUP_INET6_GETPEERNAME] = "getpeername6", >> - [BPF_CGROUP_INET4_GETSOCKNAME] = "getsockname4", >> - [BPF_CGROUP_INET6_GETSOCKNAME] = "getsockname6", >> - [BPF_CGROUP_UDP4_SENDMSG] = "sendmsg4", >> - [BPF_CGROUP_UDP6_SENDMSG] = "sendmsg6", >> - [BPF_CGROUP_SYSCTL] = "sysctl", >> - [BPF_CGROUP_UDP4_RECVMSG] = "recvmsg4", >> - [BPF_CGROUP_UDP6_RECVMSG] = "recvmsg6", >> - [BPF_CGROUP_GETSOCKOPT] = "getsockopt", >> - [BPF_CGROUP_SETSOCKOPT] = "setsockopt", >> + [BPF_CGROUP_INET_INGRESS] = "ingress", >> + [BPF_CGROUP_INET_EGRESS] = "egress", >> + [BPF_CGROUP_INET_SOCK_CREATE] = "sock_create", >> + [BPF_CGROUP_INET_SOCK_RELEASE] = "sock_release", >> + [BPF_CGROUP_SOCK_OPS] = "sock_ops", >> + [BPF_CGROUP_DEVICE] = "device", >> + [BPF_CGROUP_INET4_BIND] = "bind4", >> + [BPF_CGROUP_INET6_BIND] = "bind6", >> + [BPF_CGROUP_INET4_CONNECT] = "connect4", >> + [BPF_CGROUP_INET6_CONNECT] = "connect6", >> + [BPF_CGROUP_INET4_POST_BIND] = "post_bind4", >> + [BPF_CGROUP_INET6_POST_BIND] = "post_bind6", >> + [BPF_CGROUP_INET4_GETPEERNAME] = "getpeername4", >> + [BPF_CGROUP_INET6_GETPEERNAME] = "getpeername6", >> + [BPF_CGROUP_INET4_GETSOCKNAME] = "getsockname4", >> + [BPF_CGROUP_INET6_GETSOCKNAME] = "getsockname6", >> + [BPF_CGROUP_UDP4_SENDMSG] = "sendmsg4", >> + [BPF_CGROUP_UDP6_SENDMSG] = "sendmsg6", >> + [BPF_CGROUP_SYSCTL] = "sysctl", >> + [BPF_CGROUP_UDP4_RECVMSG] = "recvmsg4", >> + [BPF_CGROUP_UDP6_RECVMSG] = "recvmsg6", >> + [BPF_CGROUP_GETSOCKOPT] = "getsockopt", >> + [BPF_CGROUP_SETSOCKOPT] = "setsockopt", >> >> - [BPF_SK_SKB_STREAM_PARSER] = "sk_skb_stream_parser", >> - [BPF_SK_SKB_STREAM_VERDICT] = "sk_skb_stream_verdict", >> - [BPF_SK_SKB_VERDICT] = "sk_skb_verdict", >> - [BPF_SK_MSG_VERDICT] = "sk_msg_verdict", >> - [BPF_LIRC_MODE2] = "lirc_mode2", >> - [BPF_FLOW_DISSECTOR] = "flow_dissector", >> - [BPF_TRACE_RAW_TP] = "raw_tp", >> - [BPF_TRACE_FENTRY] = "fentry", >> - [BPF_TRACE_FEXIT] = "fexit", >> - [BPF_MODIFY_RETURN] = "mod_ret", >> - [BPF_LSM_MAC] = "lsm_mac", >> - [BPF_SK_LOOKUP] = "sk_lookup", >> + [BPF_SK_SKB_STREAM_PARSER] = "sk_skb_stream_parser", >> + [BPF_SK_SKB_STREAM_VERDICT] = "sk_skb_stream_verdict", >> + [BPF_SK_SKB_VERDICT] = "sk_skb_verdict", >> + [BPF_SK_MSG_VERDICT] = "sk_msg_verdict", >> + [BPF_LIRC_MODE2] = "lirc_mode2", >> + [BPF_FLOW_DISSECTOR] = "flow_dissector", >> + [BPF_TRACE_RAW_TP] = "raw_tp", >> + [BPF_TRACE_FENTRY] = "fentry", >> + [BPF_TRACE_FEXIT] = "fexit", >> + [BPF_MODIFY_RETURN] = "mod_ret", >> + [BPF_LSM_MAC] = "lsm_mac", >> + [BPF_SK_LOOKUP] = "sk_lookup", >> + [BPF_TRACE_ITER] = "trace_iter", >> + [BPF_XDP_DEVMAP] = "xdp_devmap", >> + [BPF_XDP_CPUMAP] = "xdp_cpumap", >> + [BPF_XDP] = "xdp", >> + [BPF_SK_REUSEPORT_SELECT] = "sk_skb_reuseport_select", >> + [BPF_SK_REUSEPORT_SELECT_OR_MIGRATE] = "sk_skb_reuseport_select_or_migrate", >> }; >> > > you are ruining Git blaming abilities for purely aesthetic reasons, > which are not good enough reasons, IMO. Please don't do this, this > nice alignment is nice, but definitely not necessary. So whatever is > longer then the "default indentation", just add another tab or two and > be done with it. I think I've been asked to do the opposite in the past. And I wouldn't be ruining much in the current case, the array has been moved from another location and nearly all types were added here in the same commit. But OK. > That way we can actually see what was added in this > patch, btw. On this one I agree :)