From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f46.google.com (mail-qv1-f46.google.com [209.85.219.46]) by mail.toke.dk (Postfix) with ESMTPS id 5314C9D89C8 for ; Tue, 3 Jan 2023 23:35:43 +0100 (CET) Received: by mail-qv1-f46.google.com with SMTP id o17so18437191qvn.4 for ; Tue, 03 Jan 2023 14:35:42 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=NL6C9QSnwD/h2JAvk9Oq/i2/KLbEVi58KRA8l/TSh7k=; b=gp86IeBOuXS2Dl5NNCEGV9D9eayx2ahhu7nGqvTjTmUj4A/nGzWxF5JN92FLzOaLJJ ge/5q2sU5go+skVeIzLy91og2ts9inXzEM9/78Es+SBxEGxGR1MSZBQlFhfO1qH3CNIe +wwesYk7Lq628i/0wrWY2K6B2la4LtX48oEDPAqwueWzwsRfQEJFNX3oYRX/NgDesWZ4 ATPBOsf/PF27DK51WaJyyNt6zX8WJPQQ+1/v1EdkUvWskVLvNG+80y3BeUpvELogh+3T ZDkyTdR2FuZqMKmHEH5Lb2Dt9P5l1R3DWU/5+bjb5jsXM2T49RymxGYHN3YBq5AygvCZ 37sg== X-Gm-Message-State: AFqh2kq0vXOTbTwSkwKx9U9GL6vp6j3WoeJvXlkd2wK1c3F/uX/MmY00 Zsaivjw++7J5oqsexEcNGGQ= X-Google-Smtp-Source: AMrXdXt3Fs5IX6BBC5vBxy+FnOUCncQTH3qM+ePZwc24UelHUHwd3waQ2jC5NgYZb+tgbd9GzqP6CQ== X-Received: by 2002:a0c:ef03:0:b0:530:e35d:8e82 with SMTP id t3-20020a0cef03000000b00530e35d8e82mr55664707qvr.9.1672785341619; Tue, 03 Jan 2023 14:35:41 -0800 (PST) Received: from maniforge.lan ([2620:10d:c091:480::1:7c6c]) by smtp.gmail.com with ESMTPSA id ay34-20020a05620a17a200b006b929a56a2bsm22847066qkb.3.2023.01.03.14.35.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Jan 2023 14:35:41 -0800 (PST) Date: Tue, 3 Jan 2023 16:35:40 -0600 From: David Vernet To: Stanislav Fomichev Message-ID: References: <20221220222043.3348718-1-sdf@google.com> <20221220222043.3348718-8-sdf@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/2.2.9 (2022-11-12) Message-ID-Hash: Z22QLL4EINGVJRDVV2Z4S6D7PYAZOZGA X-Message-ID-Hash: Z22QLL4EINGVJRDVV2Z4S6D7PYAZOZGA X-MailFrom: dcvernet@gmail.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: bpf@vger.kernel.org, ast@kernel.org, daniel@iogearbox.net, andrii@kernel.org, martin.lau@linux.dev, song@kernel.org, yhs@fb.com, john.fastabend@gmail.com, kpsingh@kernel.org, haoluo@google.com, jolsa@kernel.org, David Ahern , Jakub Kicinski , Willem de Bruijn , Jesper Dangaard Brouer , Anatoly Burakov , Alexander Lobakin , Magnus Karlsson , Maryam Tahhan , xdp-hints@xdp-project.net, netdev@vger.kernel.org X-Mailman-Version: 3.3.7 Precedence: list Subject: [xdp-hints] Re: [PATCH bpf-next v5 07/17] bpf: XDP metadata RX kfuncs List-Id: XDP hardware hints design discussion Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: On Tue, Jan 03, 2023 at 02:23:59PM -0800, Stanislav Fomichev wrote: > On Tue, Dec 27, 2022 at 12:33 PM David Vernet wrote: > > > > On Tue, Dec 20, 2022 at 02:20:33PM -0800, Stanislav Fomichev wrote: > > > > Hey Stanislav, > > > > [...] > > > > > diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h > > > index aad12a179e54..b41d18490595 100644 > > > --- a/include/linux/netdevice.h > > > +++ b/include/linux/netdevice.h > > > @@ -74,6 +74,7 @@ struct udp_tunnel_nic_info; > > > struct udp_tunnel_nic; > > > struct bpf_prog; > > > struct xdp_buff; > > > +struct xdp_md; > > > > > > void synchronize_net(void); > > > void netdev_set_default_ethtool_ops(struct net_device *dev, > > > @@ -1618,6 +1619,11 @@ struct net_device_ops { > > > bool cycles); > > > }; > > > > > > +struct xdp_metadata_ops { > > > + int (*xmo_rx_timestamp)(const struct xdp_md *ctx, u64 *timestamp); > > > + int (*xmo_rx_hash)(const struct xdp_md *ctx, u32 *hash); > > > +}; > > > + > > > /** > > > * enum netdev_priv_flags - &struct net_device priv_flags > > > * > > > @@ -2050,6 +2056,7 @@ struct net_device { > > > unsigned int flags; > > > unsigned long long priv_flags; > > > const struct net_device_ops *netdev_ops; > > > + const struct xdp_metadata_ops *xdp_metadata_ops; > > > > You need to document this field above the struct, or the docs build will > > complain: > > > > SPHINX htmldocs --> > > > > make[2]: Nothing to be done for 'html'. > > Using sphinx_rtd_theme theme > > source directory: networking > > ./include/linux/netdevice.h:2371: warning: Function parameter or > > member 'xdp_metadata_ops' not described in 'net_device' > > > > > int ifindex; > > > unsigned short gflags; > > > unsigned short hard_header_len; > > Thanks, I will try to actually build the doc. Last time I tried it > took too long and I gave up :-) FYI the docs build system supports building specific subdirectories, e.g.: make -j SPHINXDIRS="networking" htmldocs That should take O(seconds) instead of O(timeout) :-) - David