From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Ahern Subject: Re: [PATCH RFC v2 net-next 03/25] netlink: introduce NLM_F_DUMP_PROPER_HDR flag Date: Tue, 2 Oct 2018 08:55:19 -0600 Message-ID: <61bdd006-98f8-18dd-9731-b64b34e35252@gmail.com> References: <20181002002851.5002-1-dsahern@kernel.org> <20181002002851.5002-4-dsahern@kernel.org> <20181002130614.77856ff8@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, davem@davemloft.net, christian@brauner.io, stephen@networkplumber.org To: Jiri Benc , David Ahern Return-path: Received: from mail-pl1-f193.google.com ([209.85.214.193]:45592 "EHLO mail-pl1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726118AbeJBVjK (ORCPT ); Tue, 2 Oct 2018 17:39:10 -0400 Received: by mail-pl1-f193.google.com with SMTP id y15-v6so1813593plr.12 for ; Tue, 02 Oct 2018 07:55:22 -0700 (PDT) In-Reply-To: <20181002130614.77856ff8@redhat.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 10/2/18 5:06 AM, Jiri Benc wrote: > On Mon, 1 Oct 2018 17:28:29 -0700, David Ahern wrote: >> Add a new flag, NLM_F_DUMP_PROPER_HDR, for userspace to indicate to the >> kernel that it believes it is sending the right header struct for the >> dump message type (ifinfomsg, ifaddrmsg, rtmsg, fib_rule_hdr, ...). > > Why is this limited to dumps? Other kind of netlink messages contain > the common struct, too. When introducing such mechanism, please make it > generic. Because all of the other requests -- NEW, DEL, and GET -- all seem to use and expect the right header. Dumps are the ones where the header is not looked at in most cases and for years iproute2 got away with sending the wrong one. > > Last time when we were discussing strict checking in netlink, it was > suggested to add a socket option instead of adding NLM flags[1]. > It makes a lot of sense: the number of flags is very limited and we'd > run out of them pretty fast. It's not just the header structure that > is currently checked sloppily. It's also attributes, flags in > attributes, etc. We can't assign a flag to all of them. > > You should also consider a different name for the flag: it should > reflect what the effect of the flag is. "Proper header" is not an > effect, it's a requirement for the message to pass. The effect is > enforced strict checking of the header. Proper means the correct header for the dump type is sent. You want take issue with a name suggest a different one.