From: Scott Feldman Date: Thu, 11 Jun 2015 18:20:42 +0000 (-0700) Subject: switchdev: change BUG_ON to WARN for attr set failure case X-Git-Tag: v4.9.8~4072^2~13 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e9fdaec0e0d40f548c2b79e147c7ffd2809d2a64;p=platform%2Fkernel%2Flinux-rpi3.git switchdev: change BUG_ON to WARN for attr set failure case This particular BUG_ON condition was checking for attr set err in the COMMIT phase, which isn't expected (it's a driver bug if PREPARE phase is OK but COMMIT fails). But BUG_ON() is too strong for this case, so change to WARN(). BUG_ON() would be warranted if the system was corrupted beyond repair, but this is not the case here. Signed-off-by: Scott Feldman Signed-off-by: David S. Miller --- diff --git a/net/switchdev/switchdev.c b/net/switchdev/switchdev.c index 3e4b35a..f01d340 100644 --- a/net/switchdev/switchdev.c +++ b/net/switchdev/switchdev.c @@ -184,7 +184,8 @@ int switchdev_port_attr_set(struct net_device *dev, struct switchdev_attr *attr) attr->trans = SWITCHDEV_TRANS_COMMIT; err = __switchdev_port_attr_set(dev, attr); - BUG_ON(err); + WARN(err, "%s: Commit of attribute (id=%d) failed.\n", + dev->name, attr->id); return err; }