diff options
author | Hans de Goede <hdegoede@redhat.com> | 2014-09-13 12:26:49 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-23 21:42:12 -0700 |
commit | 1ad7ed5af3d85d0d8b3cdc5a4b823272b85c46cf (patch) | |
tree | 7dbd23d73ce526fd2dcfe4478bbb830a3079c3c3 /fs/namei.c | |
parent | 102c00cb91f36f6f7afa6658b2436b04fb3d95b3 (diff) | |
download | talos-obmc-linux-1ad7ed5af3d85d0d8b3cdc5a4b823272b85c46cf.tar.gz talos-obmc-linux-1ad7ed5af3d85d0d8b3cdc5a4b823272b85c46cf.zip |
uas: Cleanup uas_log_cmd_state usage
Instead of doing:
uas_log_cmd_state(cmnd, __func__)
scmd_printk(KERN_ERR, cmnd, "error doing foo %d\n", err)
On error, resulting in 2 log calls for a single error, make uas_log_cmd_state
take a status code, and change calls like the above to:
uas_log_cmd_state(cmnd, "error doing foo", err)
Also change various sanity checks (which should never trigger) from:
"scmd_printk(KERN_ERR, cmnd, "sanity foo failed\n")" to calling the new
uas_log_cmd_state(), so that when they do trigger we get more info.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs/namei.c')
0 files changed, 0 insertions, 0 deletions