diff options
author | Aditya Pakki <pakki001@umn.edu> | 2019-11-17 14:28:36 -0600 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-11-18 17:08:20 -0800 |
commit | 60f5c4aaae452ae9252128ef7f9ae222aa70c569 (patch) | |
tree | a073029d67d0683fcaec8f1f6437ae44de2fcc39 /net/atm | |
parent | 34e59836565e36fade1464e054a3551c1a0364be (diff) | |
download | linux-60f5c4aaae452ae9252128ef7f9ae222aa70c569.tar.gz linux-60f5c4aaae452ae9252128ef7f9ae222aa70c569.tar.bz2 linux-60f5c4aaae452ae9252128ef7f9ae222aa70c569.zip |
net: atm: Reduce the severity of logging in unlink_clip_vcc
In case of errors in unlink_clip_vcc, the logging level is set to
pr_crit but failures in clip_setentry are handled by pr_err().
The patch changes the severity consistent across invocations.
Signed-off-by: Aditya Pakki <pakki001@umn.edu>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/atm')
-rw-r--r-- | net/atm/clip.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/atm/clip.c b/net/atm/clip.c index a7972da7235d..294cb9efe3d3 100644 --- a/net/atm/clip.c +++ b/net/atm/clip.c @@ -89,7 +89,7 @@ static void unlink_clip_vcc(struct clip_vcc *clip_vcc) struct clip_vcc **walk; if (!entry) { - pr_crit("!clip_vcc->entry (clip_vcc %p)\n", clip_vcc); + pr_err("!clip_vcc->entry (clip_vcc %p)\n", clip_vcc); return; } netif_tx_lock_bh(entry->neigh->dev); /* block clip_start_xmit() */ @@ -109,10 +109,10 @@ static void unlink_clip_vcc(struct clip_vcc *clip_vcc) error = neigh_update(entry->neigh, NULL, NUD_NONE, NEIGH_UPDATE_F_ADMIN, 0); if (error) - pr_crit("neigh_update failed with %d\n", error); + pr_err("neigh_update failed with %d\n", error); goto out; } - pr_crit("ATMARP: failed (entry %p, vcc 0x%p)\n", entry, clip_vcc); + pr_err("ATMARP: failed (entry %p, vcc 0x%p)\n", entry, clip_vcc); out: netif_tx_unlock_bh(entry->neigh->dev); } |