From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753482AbdFWNbS (ORCPT ); Fri, 23 Jun 2017 09:31:18 -0400 Received: from s3.sipsolutions.net ([5.9.151.49]:49174 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751256AbdFWNbR (ORCPT ); Fri, 23 Jun 2017 09:31:17 -0400 Message-ID: <1498224668.2595.5.camel@sipsolutions.net> Subject: Re: [PATCH net-next v3 1/6] vxlan: refactor verification and application of configuration From: Johannes Berg To: Matthias Schiffer , Jiri Benc Cc: davem@davemloft.net, hannes@stressinduktion.org, pshelar@ovn.org, aduyck@mirantis.com, roopa@cumulusnetworks.com, netdev@vger.kernel.org, dev@openvswitch.org, linux-kernel@vger.kernel.org Date: Fri, 23 Jun 2017 15:31:08 +0200 In-Reply-To: References: <6e2c8bb77be42834f1da5a3ca79f5e455574a136.1497825555.git.mschiffer@universe-factory.net> <20170623105220.18561f34@griffin> <113c3fe1-f490-d92d-2b4a-69b7115d8115@universe-factory.net> <1498213402.2595.3.camel@sipsolutions.net> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.6-1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2017-06-23 at 14:02 +0200, Matthias Schiffer wrote: > > It seems though that rtnl_link_ops.newlink/changelink don't allow > passing the extack yet... how do we proceed here? Treewide change > (maybe by someone who knows their Coccinelle-fu?), or would the > introduction of new versions of the newlink and changelink fields be > more acceptable, so drivers can be moved to the new API one by one? I think treewide change is easy enough, this seems to work: @ops1@ identifier newfn, ops; @@ static struct rtnl_link_ops ops = { .newlink = newfn, ... }; @@ identifier ops1.newfn; identifier src_net, dev, tb, data; @@ -int newfn(struct net *src_net, struct net_device *dev, -    struct nlattr *tb[], struct nlattr *data[]) +int newfn(struct net *src_net, struct net_device *dev, +    struct nlattr *tb[], struct nlattr *data[], +    struct netlink_ext_ack *extack) {...} @ops2@ identifier chfn, ops; @@ static struct rtnl_link_ops ops = { .changelink = chfn, ... }; @@ identifier ops2.chfn; identifier dev, tb, data; @@ -int chfn(struct net_device *dev, -   struct nlattr *tb[], struct nlattr *data[]) +int chfn(struct net_device *dev, +   struct nlattr *tb[], struct nlattr *data[], +   struct netlink_ext_ack *extack) {...} I guess if there are any stragglers you'd find them by compile-testing :) johannes From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Berg Subject: Re: [PATCH net-next v3 1/6] vxlan: refactor verification and application of configuration Date: Fri, 23 Jun 2017 15:31:08 +0200 Message-ID: <1498224668.2595.5.camel@sipsolutions.net> References: <6e2c8bb77be42834f1da5a3ca79f5e455574a136.1497825555.git.mschiffer@universe-factory.net> <20170623105220.18561f34@griffin> <113c3fe1-f490-d92d-2b4a-69b7115d8115@universe-factory.net> <1498213402.2595.3.camel@sipsolutions.net> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: dev-yBygre7rU0TnMu66kgdUjQ@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, roopa-qUQiAmfTcIp+XZJcv9eMoEEOCMrvLtNR@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, aduyck-nYU0QVwCCFFWk0Htik3J/w@public.gmane.org, davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org To: Matthias Schiffer , Jiri Benc Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: ovs-dev-bounces-yBygre7rU0TnMu66kgdUjQ@public.gmane.org Errors-To: ovs-dev-bounces-yBygre7rU0TnMu66kgdUjQ@public.gmane.org List-Id: netdev.vger.kernel.org T24gRnJpLCAyMDE3LTA2LTIzIGF0IDE0OjAyICswMjAwLCBNYXR0aGlhcyBTY2hpZmZlciB3cm90 ZToKPiAKPiBJdCBzZWVtcyB0aG91Z2ggdGhhdCBydG5sX2xpbmtfb3BzLm5ld2xpbmsvY2hhbmdl bGluayBkb24ndCBhbGxvdwo+IHBhc3NpbmcgdGhlIGV4dGFjayB5ZXQuLi4gaG93IGRvIHdlIHBy b2NlZWQgaGVyZT8gVHJlZXdpZGUgY2hhbmdlCj4gKG1heWJlIGJ5IHNvbWVvbmUgd2hvIGtub3dz IHRoZWlyIENvY2NpbmVsbGUtZnU/KSwgb3Igd291bGQgdGhlCj4gaW50cm9kdWN0aW9uIG9mIG5l dyB2ZXJzaW9ucyBvZiB0aGUgbmV3bGluayBhbmQgY2hhbmdlbGluayBmaWVsZHMgYmUKPiBtb3Jl IGFjY2VwdGFibGUsIHNvIGRyaXZlcnMgY2FuIGJlIG1vdmVkIHRvIHRoZSBuZXcgQVBJIG9uZSBi eSBvbmU/CgpJIHRoaW5rIHRyZWV3aWRlIGNoYW5nZSBpcyBlYXN5IGVub3VnaCwgdGhpcyBzZWVt cyB0byB3b3JrOgoKQG9wczFACmlkZW50aWZpZXIgbmV3Zm4sIG9wczsKQEAKc3RhdGljIHN0cnVj dCBydG5sX2xpbmtfb3BzIG9wcyA9IHsKCS5uZXdsaW5rID0gbmV3Zm4sCi4uLgp9OwoKQEAKaWRl bnRpZmllciBvcHMxLm5ld2ZuOwppZGVudGlmaWVyIHNyY19uZXQsIGRldiwgdGIsIGRhdGE7CkBA Ci1pbnQgbmV3Zm4oc3RydWN0IG5ldCAqc3JjX25ldCwgc3RydWN0IG5ldF9kZXZpY2UgKmRldiwK LQnCoMKgwqBzdHJ1Y3QgbmxhdHRyICp0YltdLCBzdHJ1Y3QgbmxhdHRyICpkYXRhW10pCitpbnQg bmV3Zm4oc3RydWN0IG5ldCAqc3JjX25ldCwgc3RydWN0IG5ldF9kZXZpY2UgKmRldiwKKwnCoMKg wqBzdHJ1Y3QgbmxhdHRyICp0YltdLCBzdHJ1Y3QgbmxhdHRyICpkYXRhW10sCisJwqDCoMKgc3Ry dWN0IG5ldGxpbmtfZXh0X2FjayAqZXh0YWNrKQp7Li4ufQoKQG9wczJACmlkZW50aWZpZXIgY2hm biwgb3BzOwpAQApzdGF0aWMgc3RydWN0IHJ0bmxfbGlua19vcHMgb3BzID0gewoJLmNoYW5nZWxp bmsgPSBjaGZuLAouLi4KfTsKCkBACmlkZW50aWZpZXIgb3BzMi5jaGZuOwppZGVudGlmaWVyIGRl diwgdGIsIGRhdGE7CkBACi1pbnQgY2hmbihzdHJ1Y3QgbmV0X2RldmljZSAqZGV2LAotCcKgwqBz dHJ1Y3QgbmxhdHRyICp0YltdLCBzdHJ1Y3QgbmxhdHRyICpkYXRhW10pCitpbnQgY2hmbihzdHJ1 Y3QgbmV0X2RldmljZSAqZGV2LAorCcKgwqBzdHJ1Y3QgbmxhdHRyICp0YltdLCBzdHJ1Y3Qgbmxh dHRyICpkYXRhW10sCisJwqDCoHN0cnVjdCBuZXRsaW5rX2V4dF9hY2sgKmV4dGFjaykKey4uLn0K CkkgZ3Vlc3MgaWYgdGhlcmUgYXJlIGFueSBzdHJhZ2dsZXJzIHlvdSdkIGZpbmQgdGhlbSBieSBj b21waWxlLXRlc3RpbmcKOikKCmpvaGFubmVzCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fCmRldiBtYWlsaW5nIGxpc3QKZGV2QG9wZW52c3dpdGNoLm9yZwpo dHRwczovL21haWwub3BlbnZzd2l0Y2gub3JnL21haWxtYW4vbGlzdGluZm8vb3ZzLWRldgo=