|
 |
a9cca76 |
commit d072ef23b8ee6bcabc00beff0b5702e704a473cb
|
|
|
a9cca76 |
Author: Josh Boyer <jwboyer@redhat.com>
|
|
|
a9cca76 |
Date: Tue Aug 2 08:09:56 2011 -0400
|
|
|
a9cca76 |
|
|
|
a9cca76 |
usbnet/cdc_ncm: Don't use stack variables for DMA buffers
|
|
|
a9cca76 |
|
|
|
a9cca76 |
The cdc_ncm driver still has a few places where stack variables are passed
|
|
|
a9cca76 |
to the cdc_ncm_do_request function. This triggers a stack trace in
|
|
|
a9cca76 |
lib/dma-debug.c if the CONFIG_DEBUG_DMA_API option is set.
|
|
|
a9cca76 |
|
|
|
a9cca76 |
Adjust these calls to pass parameters that have been allocated with kzalloc.
|
|
|
a9cca76 |
|
|
|
a9cca76 |
Signed-off-by: Josh Boyer <jwboyer@redhat.com>
|
|
|
a9cca76 |
|
|
|
a9cca76 |
diff --git a/drivers/net/usb/cdc_ncm.c b/drivers/net/usb/cdc_ncm.c
|
|
|
a9cca76 |
index fd622a6..96dd386 100644
|
|
|
a9cca76 |
--- a/drivers/net/usb/cdc_ncm.c
|
|
|
a9cca76 |
+++ b/drivers/net/usb/cdc_ncm.c
|
|
|
a9cca76 |
@@ -260,23 +260,38 @@ static u8 cdc_ncm_setup(struct cdc_ncm_ctx *ctx)
|
|
|
a9cca76 |
req.wIndex = cpu_to_le16(iface_no);
|
|
|
a9cca76 |
|
|
|
a9cca76 |
if (flags & USB_CDC_NCM_NCAP_NTB_INPUT_SIZE) {
|
|
|
a9cca76 |
- struct usb_cdc_ncm_ndp_input_size ndp_in_sz;
|
|
|
a9cca76 |
+ struct usb_cdc_ncm_ndp_input_size *ndp_in_sz;
|
|
|
a9cca76 |
+
|
|
|
a9cca76 |
+ ndp_in_sz = kzalloc(sizeof(*ndp_in_sz), GFP_KERNEL);
|
|
|
a9cca76 |
+ if (!ndp_in_sz) {
|
|
|
a9cca76 |
+ err = -ENOMEM;
|
|
|
a9cca76 |
+ goto size_err;
|
|
|
a9cca76 |
+ }
|
|
|
a9cca76 |
|
|
|
a9cca76 |
req.wLength = 8;
|
|
|
a9cca76 |
- ndp_in_sz.dwNtbInMaxSize = cpu_to_le32(ctx->rx_max);
|
|
|
a9cca76 |
- ndp_in_sz.wNtbInMaxDatagrams =
|
|
|
a9cca76 |
+ ndp_in_sz->dwNtbInMaxSize = cpu_to_le32(ctx->rx_max);
|
|
|
a9cca76 |
+ ndp_in_sz->wNtbInMaxDatagrams =
|
|
|
a9cca76 |
cpu_to_le16(CDC_NCM_DPT_DATAGRAMS_MAX);
|
|
|
a9cca76 |
- ndp_in_sz.wReserved = 0;
|
|
|
a9cca76 |
- err = cdc_ncm_do_request(ctx, &req, &ndp_in_sz, 0, NULL,
|
|
|
a9cca76 |
+ ndp_in_sz->wReserved = 0;
|
|
|
a9cca76 |
+ err = cdc_ncm_do_request(ctx, &req, ndp_in_sz, 0, NULL,
|
|
|
a9cca76 |
1000);
|
|
|
a9cca76 |
+ kfree(ndp_in_sz);
|
|
|
a9cca76 |
} else {
|
|
|
a9cca76 |
- __le32 dwNtbInMaxSize = cpu_to_le32(ctx->rx_max);
|
|
|
a9cca76 |
+ __le32 *dwNtbInMaxSize;
|
|
|
a9cca76 |
+ dwNtbInMaxSize = kzalloc(sizeof(*dwNtbInMaxSize), GFP_KERNEL);
|
|
|
a9cca76 |
+ if (!dwNtbInMaxSize) {
|
|
|
a9cca76 |
+ err = -ENOMEM;
|
|
|
a9cca76 |
+ goto size_err;
|
|
|
a9cca76 |
+ }
|
|
|
a9cca76 |
+ *dwNtbInMaxSize = cpu_to_le32(ctx->rx_max);
|
|
|
a9cca76 |
|
|
|
a9cca76 |
req.wLength = 4;
|
|
|
a9cca76 |
- err = cdc_ncm_do_request(ctx, &req, &dwNtbInMaxSize, 0,
|
|
|
a9cca76 |
+ err = cdc_ncm_do_request(ctx, &req, dwNtbInMaxSize, 0,
|
|
|
a9cca76 |
NULL, 1000);
|
|
|
a9cca76 |
+ kfree(dwNtbInMaxSize);
|
|
|
a9cca76 |
}
|
|
|
a9cca76 |
|
|
|
a9cca76 |
+size_err:
|
|
|
a9cca76 |
if (err)
|
|
|
a9cca76 |
pr_debug("Setting NTB Input Size failed\n");
|
|
|
a9cca76 |
}
|
|
|
a9cca76 |
@@ -362,9 +377,15 @@ static u8 cdc_ncm_setup(struct cdc_ncm_ctx *ctx)
|
|
|
a9cca76 |
|
|
|
a9cca76 |
/* set Max Datagram Size (MTU) */
|
|
|
a9cca76 |
if (flags & USB_CDC_NCM_NCAP_MAX_DATAGRAM_SIZE) {
|
|
|
a9cca76 |
- __le16 max_datagram_size;
|
|
|
a9cca76 |
+ __le16 *max_datagram_size;
|
|
|
a9cca76 |
u16 eth_max_sz = le16_to_cpu(ctx->ether_desc->wMaxSegmentSize);
|
|
|
a9cca76 |
|
|
|
a9cca76 |
+ max_datagram_size = kzalloc(sizeof(*max_datagram_size), GFP_KERNEL);
|
|
|
a9cca76 |
+ if (!max_datagram_size) {
|
|
|
a9cca76 |
+ err = -ENOMEM;
|
|
|
a9cca76 |
+ goto max_dgram_err;
|
|
|
a9cca76 |
+ }
|
|
|
a9cca76 |
+
|
|
|
a9cca76 |
req.bmRequestType = USB_TYPE_CLASS | USB_DIR_IN |
|
|
|
a9cca76 |
USB_RECIP_INTERFACE;
|
|
|
a9cca76 |
req.bNotificationType = USB_CDC_GET_MAX_DATAGRAM_SIZE;
|
|
|
a9cca76 |
@@ -372,13 +393,15 @@ static u8 cdc_ncm_setup(struct cdc_ncm_ctx *ctx)
|
|
|
a9cca76 |
req.wIndex = cpu_to_le16(iface_no);
|
|
|
a9cca76 |
req.wLength = cpu_to_le16(2);
|
|
|
a9cca76 |
|
|
|
a9cca76 |
- err = cdc_ncm_do_request(ctx, &req, &max_datagram_size, 0, NULL,
|
|
|
a9cca76 |
+ err = cdc_ncm_do_request(ctx, &req, max_datagram_size, 0, NULL,
|
|
|
a9cca76 |
1000);
|
|
|
a9cca76 |
+
|
|
|
a9cca76 |
if (err) {
|
|
|
a9cca76 |
pr_debug("GET_MAX_DATAGRAM_SIZE failed, use size=%u\n",
|
|
|
a9cca76 |
CDC_NCM_MIN_DATAGRAM_SIZE);
|
|
|
a9cca76 |
+ kfree(max_datagram_size);
|
|
|
a9cca76 |
} else {
|
|
|
a9cca76 |
- ctx->max_datagram_size = le16_to_cpu(max_datagram_size);
|
|
|
a9cca76 |
+ ctx->max_datagram_size = le16_to_cpu(*max_datagram_size);
|
|
|
a9cca76 |
/* Check Eth descriptor value */
|
|
|
a9cca76 |
if (eth_max_sz < CDC_NCM_MAX_DATAGRAM_SIZE) {
|
|
|
a9cca76 |
if (ctx->max_datagram_size > eth_max_sz)
|
|
|
a9cca76 |
@@ -401,10 +424,12 @@ static u8 cdc_ncm_setup(struct cdc_ncm_ctx *ctx)
|
|
|
a9cca76 |
req.wValue = 0;
|
|
|
a9cca76 |
req.wIndex = cpu_to_le16(iface_no);
|
|
|
a9cca76 |
req.wLength = 2;
|
|
|
a9cca76 |
- max_datagram_size = cpu_to_le16(ctx->max_datagram_size);
|
|
|
a9cca76 |
+ *max_datagram_size = cpu_to_le16(ctx->max_datagram_size);
|
|
|
a9cca76 |
|
|
|
a9cca76 |
- err = cdc_ncm_do_request(ctx, &req, &max_datagram_size,
|
|
|
a9cca76 |
+ err = cdc_ncm_do_request(ctx, &req, max_datagram_size,
|
|
|
a9cca76 |
0, NULL, 1000);
|
|
|
a9cca76 |
+ kfree(max_datagram_size);
|
|
|
a9cca76 |
+max_dgram_err:
|
|
|
a9cca76 |
if (err)
|
|
|
a9cca76 |
pr_debug("SET_MAX_DATAGRAM_SIZE failed\n");
|
|
|
a9cca76 |
}
|