Commit 248a2204 authored by Shuah Khan's avatar Shuah Khan Committed by Greg Kroah-Hartman

usbip: stub: stop printing kernel pointer addresses in messages

Remove and/or change debug, info. and error messages to not print
kernel pointer addresses.
Signed-off-by: default avatarShuah Khan <shuahkh@osg.samsung.com>
Cc: stable <stable@vger.kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 8272d099
...@@ -237,11 +237,12 @@ void stub_device_cleanup_urbs(struct stub_device *sdev) ...@@ -237,11 +237,12 @@ void stub_device_cleanup_urbs(struct stub_device *sdev)
struct stub_priv *priv; struct stub_priv *priv;
struct urb *urb; struct urb *urb;
dev_dbg(&sdev->udev->dev, "free sdev %p\n", sdev); dev_dbg(&sdev->udev->dev, "Stub device cleaning up urbs\n");
while ((priv = stub_priv_pop(sdev))) { while ((priv = stub_priv_pop(sdev))) {
urb = priv->urb; urb = priv->urb;
dev_dbg(&sdev->udev->dev, "free urb %p\n", urb); dev_dbg(&sdev->udev->dev, "free urb seqnum %lu\n",
priv->seqnum);
usb_kill_urb(urb); usb_kill_urb(urb);
kmem_cache_free(stub_priv_cache, priv); kmem_cache_free(stub_priv_cache, priv);
......
...@@ -211,9 +211,6 @@ static int stub_recv_cmd_unlink(struct stub_device *sdev, ...@@ -211,9 +211,6 @@ static int stub_recv_cmd_unlink(struct stub_device *sdev,
if (priv->seqnum != pdu->u.cmd_unlink.seqnum) if (priv->seqnum != pdu->u.cmd_unlink.seqnum)
continue; continue;
dev_info(&priv->urb->dev->dev, "unlink urb %p\n",
priv->urb);
/* /*
* This matched urb is not completed yet (i.e., be in * This matched urb is not completed yet (i.e., be in
* flight in usb hcd hardware/driver). Now we are * flight in usb hcd hardware/driver). Now we are
...@@ -252,8 +249,8 @@ static int stub_recv_cmd_unlink(struct stub_device *sdev, ...@@ -252,8 +249,8 @@ static int stub_recv_cmd_unlink(struct stub_device *sdev,
ret = usb_unlink_urb(priv->urb); ret = usb_unlink_urb(priv->urb);
if (ret != -EINPROGRESS) if (ret != -EINPROGRESS)
dev_err(&priv->urb->dev->dev, dev_err(&priv->urb->dev->dev,
"failed to unlink a urb %p, ret %d\n", "failed to unlink a urb # %lu, ret %d\n",
priv->urb, ret); priv->seqnum, ret);
return 0; return 0;
} }
......
...@@ -88,7 +88,7 @@ void stub_complete(struct urb *urb) ...@@ -88,7 +88,7 @@ void stub_complete(struct urb *urb)
/* link a urb to the queue of tx. */ /* link a urb to the queue of tx. */
spin_lock_irqsave(&sdev->priv_lock, flags); spin_lock_irqsave(&sdev->priv_lock, flags);
if (sdev->ud.tcp_socket == NULL) { if (sdev->ud.tcp_socket == NULL) {
usbip_dbg_stub_tx("ignore urb for closed connection %p", urb); usbip_dbg_stub_tx("ignore urb for closed connection\n");
/* It will be freed in stub_device_cleanup_urbs(). */ /* It will be freed in stub_device_cleanup_urbs(). */
} else if (priv->unlinking) { } else if (priv->unlinking) {
stub_enqueue_ret_unlink(sdev, priv->seqnum, urb->status); stub_enqueue_ret_unlink(sdev, priv->seqnum, urb->status);
...@@ -190,8 +190,8 @@ static int stub_send_ret_submit(struct stub_device *sdev) ...@@ -190,8 +190,8 @@ static int stub_send_ret_submit(struct stub_device *sdev)
/* 1. setup usbip_header */ /* 1. setup usbip_header */
setup_ret_submit_pdu(&pdu_header, urb); setup_ret_submit_pdu(&pdu_header, urb);
usbip_dbg_stub_tx("setup txdata seqnum: %d urb: %p\n", usbip_dbg_stub_tx("setup txdata seqnum: %d\n",
pdu_header.base.seqnum, urb); pdu_header.base.seqnum);
usbip_header_correct_endian(&pdu_header, 1); usbip_header_correct_endian(&pdu_header, 1);
iov[iovnum].iov_base = &pdu_header; iov[iovnum].iov_base = &pdu_header;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment