From: Aaron Conole <aconole@redhat.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: davem@davemloft.net, axboe@kernel.dk,
linux-block@vger.kernel.org, jiri@resnulli.us,
netdev@vger.kernel.org, dev@openvswitch.org,
philipp.reisner@linbit.com, jmaloy@redhat.com,
christoph.boehmwalder@linbit.com, edumazet@google.com,
tipc-discussion@lists.sourceforge.net,
Jiri Pirko <jiri@nvidia.com>,
ying.xue@windriver.com, johannes@sipsolutions.net,
pabeni@redhat.com, drbd-dev@lists.linbit.com,
lars.ellenberg@linbit.com, jacob.e.keller@intel.com
Subject: Re: [ovs-dev] [PATCH net-next v3 03/10] genetlink: remove userhdr from struct genl_info
Date: Tue, 15 Aug 2023 08:49:08 -0400 [thread overview]
Message-ID: <f7twmxwxygr.fsf@redhat.com> (raw)
In-Reply-To: <20230814214723.2924989-4-kuba@kernel.org> (Jakub Kicinski's message of "Mon, 14 Aug 2023 14:47:16 -0700")
Jakub Kicinski <kuba@kernel.org> writes:
> Only three families use info->userhdr today and going forward
> we discourage using fixed headers in new families.
> So having the pointer to user header in struct genl_info
> is an overkill. Compute the header pointer at runtime.
>
> Reviewed-by: Johannes Berg <johannes@sipsolutions.net>
> Reviewed-by: Jiri Pirko <jiri@nvidia.com>
> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> ---
Seems the OVS side didn't change from v2 so still:
Reviewed-by: Aaron Conole <aconole@redhat.com>
WARNING: multiple messages have this Message-ID (diff)
From: Aaron Conole <aconole@redhat.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: axboe@kernel.dk, dev@openvswitch.org, jiri@resnulli.us,
netdev@vger.kernel.org, jacob.e.keller@intel.com,
philipp.reisner@linbit.com, jmaloy@redhat.com,
linux-block@vger.kernel.org, edumazet@google.com,
tipc-discussion@lists.sourceforge.net, lars.ellenberg@linbit.com,
Jiri Pirko <jiri@nvidia.com>,
ying.xue@windriver.com, johannes@sipsolutions.net,
pabeni@redhat.com, davem@davemloft.net,
drbd-dev@lists.linbit.com
Subject: Re: [Drbd-dev] [ovs-dev] [PATCH net-next v3 03/10] genetlink: remove userhdr from struct genl_info
Date: Tue, 15 Aug 2023 08:49:08 -0400 [thread overview]
Message-ID: <f7twmxwxygr.fsf@redhat.com> (raw)
In-Reply-To: <20230814214723.2924989-4-kuba@kernel.org> (Jakub Kicinski's message of "Mon, 14 Aug 2023 14:47:16 -0700")
Jakub Kicinski <kuba@kernel.org> writes:
> Only three families use info->userhdr today and going forward
> we discourage using fixed headers in new families.
> So having the pointer to user header in struct genl_info
> is an overkill. Compute the header pointer at runtime.
>
> Reviewed-by: Johannes Berg <johannes@sipsolutions.net>
> Reviewed-by: Jiri Pirko <jiri@nvidia.com>
> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> ---
Seems the OVS side didn't change from v2 so still:
Reviewed-by: Aaron Conole <aconole@redhat.com>
next prev parent reply other threads:[~2023-08-15 12:50 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-14 21:47 [PATCH net-next v3 00/10] genetlink: provide struct genl_info to dumps Jakub Kicinski
2023-08-14 21:47 ` [PATCH net-next v3 01/10] genetlink: push conditional locking into dumpit/done Jakub Kicinski
2023-08-14 21:47 ` [PATCH net-next v3 02/10] genetlink: make genl_info->nlhdr const Jakub Kicinski
2023-08-14 21:47 ` [PATCH net-next v3 03/10] genetlink: remove userhdr from struct genl_info Jakub Kicinski
2023-08-14 21:47 ` [Drbd-dev] " Jakub Kicinski
2023-08-15 12:49 ` Aaron Conole [this message]
2023-08-15 12:49 ` [Drbd-dev] [ovs-dev] " Aaron Conole
2023-08-14 21:47 ` [PATCH net-next v3 04/10] genetlink: add struct genl_info to struct genl_dumpit_info Jakub Kicinski
2023-08-14 21:47 ` [PATCH net-next v3 05/10] genetlink: use attrs from struct genl_info Jakub Kicinski
2023-08-15 21:46 ` kernel test robot
2023-08-14 21:47 ` [PATCH net-next v3 06/10] genetlink: add a family pointer to " Jakub Kicinski
2023-08-14 21:47 ` [PATCH net-next v3 07/10] genetlink: add genlmsg_iput() API Jakub Kicinski
2023-08-14 21:47 ` [PATCH net-next v3 08/10] netdev-genl: use struct genl_info for reply construction Jakub Kicinski
2023-08-14 21:47 ` [PATCH net-next v3 09/10] ethtool: netlink: simplify arguments to ethnl_default_parse() Jakub Kicinski
2023-08-14 21:47 ` [PATCH net-next v3 10/10] ethtool: netlink: always pass genl_info to .prepare_data Jakub Kicinski
2023-08-15 6:06 ` Jiri Pirko
2023-08-15 22:10 ` [PATCH net-next v3 00/10] genetlink: provide struct genl_info to dumps patchwork-bot+netdevbpf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f7twmxwxygr.fsf@redhat.com \
--to=aconole@redhat.com \
--cc=axboe@kernel.dk \
--cc=christoph.boehmwalder@linbit.com \
--cc=davem@davemloft.net \
--cc=dev@openvswitch.org \
--cc=drbd-dev@lists.linbit.com \
--cc=edumazet@google.com \
--cc=jacob.e.keller@intel.com \
--cc=jiri@nvidia.com \
--cc=jiri@resnulli.us \
--cc=jmaloy@redhat.com \
--cc=johannes@sipsolutions.net \
--cc=kuba@kernel.org \
--cc=lars.ellenberg@linbit.com \
--cc=linux-block@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=philipp.reisner@linbit.com \
--cc=tipc-discussion@lists.sourceforge.net \
--cc=ying.xue@windriver.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.