From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [patch net-next RFC 0/2] fib4 offload: notifier to let hw to be aware of all prefixes Date: Mon, 19 Sep 2016 08:08:54 +0200 Message-ID: <20160919060854.GB1846@nanopsycho.orion> References: <1473163300-2045-1-git-send-email-jiri@resnulli.us> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Cc: netdev@vger.kernel.org, davem@davemloft.net, idosch@mellanox.com, eladr@mellanox.com, yotamg@mellanox.com, nogahf@mellanox.com, ogerlitz@mellanox.com, roopa@cumulusnetworks.com, nikolay@cumulusnetworks.com, linville@tuxdriver.com, tgraf@suug.ch, gospo@cumulusnetworks.com, sfeldma@gmail.com, ast@plumgrid.com, edumazet@google.com, hannes@stressinduktion.org, dsa@cumulusnetworks.com, jhs@mojatatu.com, vivien.didelot@savoirfairelinux.com, john.fastabend@intel.com, andrew@lunn.ch, ivecera@redhat.com To: Florian Fainelli Return-path: Received: from mail-wm0-f67.google.com ([74.125.82.67]:36005 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750700AbcISGI6 (ORCPT ); Mon, 19 Sep 2016 02:08:58 -0400 Received: by mail-wm0-f67.google.com with SMTP id b184so13114825wma.3 for ; Sun, 18 Sep 2016 23:08:58 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Sun, Sep 18, 2016 at 10:00:44PM CEST, f.fainelli@gmail.com wrote: >Le 06/09/2016 à 05:01, Jiri Pirko a écrit : >> From: Jiri Pirko >> >> This is RFC, unfinished. I came across some issues in the process so I would >> like to share those and restart the fib offload discussion in order to make it >> really usable. >> >> So the goal of this patchset is to allow driver to propagate all prefixes >> configured in kernel down HW. This is necessary for routing to work >> as expected. If we don't do that HW might forward prefixes known to kernel >> incorrectly. Take an example when default route is set in switch HW and there >> is an IP address set on a management (non-switch) port. >> >> Currently, only fibs related to the switch port netdev are offloaded using >> switchdev ops. This model is not extendable so the first patch introduces >> a replacement: notifier to propagate fib additions and removals to whoever >> interested. The second patch makes mlxsw to adopt this new way, registering >> one notifier block for each mlxsw (asic) instance. > >Instead of introducing another specialization of a notifier_block >implementation, could we somehow have a kernel-based netlink listener >which receives the same kind of event information from rtmsg_fib()? rtmsg_fib destination is userspace. The message format is netlink. I don't think it is wise to pass netlink messages inside kernel when we can pass nice structures. Lower overhead. This is how it is done in the rest of kernel. Not sure how your comment is related specifically to this patch. > >The reason is that having such a facility would hook directly onto >existing rtmsg_* calls that exist throughout the stack, and that seems >to scale better. >-- >Florian