Re: [net-next PATCH 2/3] octeontx2-af: Add devlink health reporters for NPA

Hi Willem,

-----Original Message-----
From: Willem de Bruijn <willemdebruijn.kernel@gmail.com>
Sent: Tuesday, November 3, 2020 7:21 PM
To: George Cherian <gcherian@marvell.com>
Cc: Network Development <redacted>; linux-kernel <linux-
kernel@vger.kernel.org>; Jakub Kicinski [off-list ref]; David Miller
[off-list ref]; Sunil Kovvuri Goutham
[off-list ref]; Linu Cherian [off-list ref];
Geethasowjanya Akula [off-list ref]; masahiroy@kernel.org
Subject: [EXT] Re: [net-next PATCH 2/3] octeontx2-af: Add devlink health
reporters for NPA

quoted

quoted

quoted

 static int rvu_devlink_info_get(struct devlink *devlink, struct

devlink_info_req *req,

quoted

                                struct netlink_ext_ack *extack)  { @@
-53,7 +483,8 @@ int rvu_register_dl(struct rvu *rvu)
        rvu_dl->dl = dl;
        rvu_dl->rvu = rvu;
        rvu->rvu_dl = rvu_dl;
-       return 0;
+
+       return rvu_health_reporters_create(rvu);

when would this be called with rvu->rvu_dl == NULL?

During initialization.

This is the only caller, and it is only reached if rvu_dl is non-zero.

Yes!!! I got it, will address it in v2.

Regards
-George