On Thu, May 12, 2022 at 10:17:13AM -0700, Roopa Prabhu wrote: > > On 5/12/22 09:47, Jakub Kicinski wrote: > > On Thu, 12 May 2022 09:22:17 -0700 Roopa Prabhu wrote: > > > On 5/12/22 02:55, Alaa Mohamed wrote: > > > > This patch adds extack msg support to vxlan_fdb_delete and vxlan_fdb_parse. > > > > extack is used to propagate meaningful error msgs to the user of vxlan > > > > fdb netlink api > > > > > > > > Signed-off-by: Alaa Mohamed <eng.alaamohamedsoliman.am@xxxxxxxxx> > > Also the patches don't apply to net-next, again. > > that's probably because the patches were already applied. Ido just told me > abt it also > > https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git/commit/?id=5dd6da25255a9d64622c693b99d7668da939a980 > > I have requested Alaa send an incremental fix (offline). > > thanks > Alaa, when you send the incremental fixup patch requested by Roopa, could you please also: - properly align the extack argument of the ocelot_port_fdb_del() function - properly align the extack argument of the vxlan_fdb_delete() function - properly align the "DST, VNI, ifindex and port are mutually exclusive with NH_ID" string to the open parenthesis of the NL_SET_ERR_MSG macro in vxlan_fdb_parse(). - make actual use of the extack in ice_fdb_add() and ice_fdb_del(), and remove the __always_unused from ice_fdb_add(). Thank you.