Commit ea44c1d6 authored by Bjorn Helgaas's avatar Bjorn Helgaas Committed by Linus Torvalds

PNP: fix formatting of dbg_pnp_show_resources() output

Each resource should be printed on its own line, so start snprintf'ing
at the beginning of the buffer every time through the loop.

Also, use scnprintf() rather than snprintf() when building up the
buffer to print.  scnprintf() returns the number of characters actually
written into the buffer (not including the trailing NULL).

snprintf() returns the number of characters that *would be* written,
assuming everything would fit in the buffer.  That's nice if we want to
resize the buffer to make sure everything fits, but in this case, I
just want to keep from overflowing the buffer, and it's OK if the
output is truncated.

Using snprintf() meant that my "len" could grow to be more than the
the buffer size, which makes "sizeof(buf) - len" negative, which causes
this alarming WARN_ON:
    http://marc.info/?l=linux-kernel&m=121736480005656&w=2

More useful snprintf/scnprintf discussion:
    http://lwn.net/Articles/69419/Signed-off-by: default avatarBjorn Helgaas <bjorn.helgaas@hp.com>
Reported-by: default avatarPete Clements <clem@clem.clem-digital.net>
Cc: Rene Herman <rene.herman@keyaccess.nl>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 61a2d07d
...@@ -77,7 +77,7 @@ void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc) ...@@ -77,7 +77,7 @@ void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc)
{ {
#ifdef DEBUG #ifdef DEBUG
char buf[128]; char buf[128];
int len = 0; int len;
struct pnp_resource *pnp_res; struct pnp_resource *pnp_res;
struct resource *res; struct resource *res;
...@@ -89,9 +89,10 @@ void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc) ...@@ -89,9 +89,10 @@ void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc)
dev_dbg(&dev->dev, "%s: current resources:\n", desc); dev_dbg(&dev->dev, "%s: current resources:\n", desc);
list_for_each_entry(pnp_res, &dev->resources, list) { list_for_each_entry(pnp_res, &dev->resources, list) {
res = &pnp_res->res; res = &pnp_res->res;
len = 0;
len += snprintf(buf + len, sizeof(buf) - len, " %-3s ", len += scnprintf(buf + len, sizeof(buf) - len, " %-3s ",
pnp_resource_type_name(res)); pnp_resource_type_name(res));
if (res->flags & IORESOURCE_DISABLED) { if (res->flags & IORESOURCE_DISABLED) {
dev_dbg(&dev->dev, "%sdisabled\n", buf); dev_dbg(&dev->dev, "%sdisabled\n", buf);
...@@ -101,18 +102,18 @@ void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc) ...@@ -101,18 +102,18 @@ void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc)
switch (pnp_resource_type(res)) { switch (pnp_resource_type(res)) {
case IORESOURCE_IO: case IORESOURCE_IO:
case IORESOURCE_MEM: case IORESOURCE_MEM:
len += snprintf(buf + len, sizeof(buf) - len, len += scnprintf(buf + len, sizeof(buf) - len,
"%#llx-%#llx flags %#lx", "%#llx-%#llx flags %#lx",
(unsigned long long) res->start, (unsigned long long) res->start,
(unsigned long long) res->end, (unsigned long long) res->end,
res->flags); res->flags);
break; break;
case IORESOURCE_IRQ: case IORESOURCE_IRQ:
case IORESOURCE_DMA: case IORESOURCE_DMA:
len += snprintf(buf + len, sizeof(buf) - len, len += scnprintf(buf + len, sizeof(buf) - len,
"%lld flags %#lx", "%lld flags %#lx",
(unsigned long long) res->start, (unsigned long long) res->start,
res->flags); res->flags);
break; break;
} }
dev_dbg(&dev->dev, "%s\n", buf); dev_dbg(&dev->dev, "%s\n", buf);
...@@ -144,66 +145,67 @@ void dbg_pnp_show_option(struct pnp_dev *dev, struct pnp_option *option) ...@@ -144,66 +145,67 @@ void dbg_pnp_show_option(struct pnp_dev *dev, struct pnp_option *option)
struct pnp_dma *dma; struct pnp_dma *dma;
if (pnp_option_is_dependent(option)) if (pnp_option_is_dependent(option))
len += snprintf(buf + len, sizeof(buf) - len, len += scnprintf(buf + len, sizeof(buf) - len,
" dependent set %d (%s) ", " dependent set %d (%s) ",
pnp_option_set(option), pnp_option_set(option),
pnp_option_priority_name(option)); pnp_option_priority_name(option));
else else
len += snprintf(buf + len, sizeof(buf) - len, " independent "); len += scnprintf(buf + len, sizeof(buf) - len,
" independent ");
switch (option->type) { switch (option->type) {
case IORESOURCE_IO: case IORESOURCE_IO:
port = &option->u.port; port = &option->u.port;
len += snprintf(buf + len, sizeof(buf) - len, "io min %#llx " len += scnprintf(buf + len, sizeof(buf) - len, "io min %#llx "
"max %#llx align %lld size %lld flags %#x", "max %#llx align %lld size %lld flags %#x",
(unsigned long long) port->min, (unsigned long long) port->min,
(unsigned long long) port->max, (unsigned long long) port->max,
(unsigned long long) port->align, (unsigned long long) port->align,
(unsigned long long) port->size, port->flags); (unsigned long long) port->size, port->flags);
break; break;
case IORESOURCE_MEM: case IORESOURCE_MEM:
mem = &option->u.mem; mem = &option->u.mem;
len += snprintf(buf + len, sizeof(buf) - len, "mem min %#llx " len += scnprintf(buf + len, sizeof(buf) - len, "mem min %#llx "
"max %#llx align %lld size %lld flags %#x", "max %#llx align %lld size %lld flags %#x",
(unsigned long long) mem->min, (unsigned long long) mem->min,
(unsigned long long) mem->max, (unsigned long long) mem->max,
(unsigned long long) mem->align, (unsigned long long) mem->align,
(unsigned long long) mem->size, mem->flags); (unsigned long long) mem->size, mem->flags);
break; break;
case IORESOURCE_IRQ: case IORESOURCE_IRQ:
irq = &option->u.irq; irq = &option->u.irq;
len += snprintf(buf + len, sizeof(buf) - len, "irq"); len += scnprintf(buf + len, sizeof(buf) - len, "irq");
if (bitmap_empty(irq->map.bits, PNP_IRQ_NR)) if (bitmap_empty(irq->map.bits, PNP_IRQ_NR))
len += snprintf(buf + len, sizeof(buf) - len, len += scnprintf(buf + len, sizeof(buf) - len,
" <none>"); " <none>");
else { else {
for (i = 0; i < PNP_IRQ_NR; i++) for (i = 0; i < PNP_IRQ_NR; i++)
if (test_bit(i, irq->map.bits)) if (test_bit(i, irq->map.bits))
len += snprintf(buf + len, len += scnprintf(buf + len,
sizeof(buf) - len, sizeof(buf) - len,
" %d", i); " %d", i);
} }
len += snprintf(buf + len, sizeof(buf) - len, " flags %#x", len += scnprintf(buf + len, sizeof(buf) - len, " flags %#x",
irq->flags); irq->flags);
if (irq->flags & IORESOURCE_IRQ_OPTIONAL) if (irq->flags & IORESOURCE_IRQ_OPTIONAL)
len += snprintf(buf + len, sizeof(buf) - len, len += scnprintf(buf + len, sizeof(buf) - len,
" (optional)"); " (optional)");
break; break;
case IORESOURCE_DMA: case IORESOURCE_DMA:
dma = &option->u.dma; dma = &option->u.dma;
len += snprintf(buf + len, sizeof(buf) - len, "dma"); len += scnprintf(buf + len, sizeof(buf) - len, "dma");
if (!dma->map) if (!dma->map)
len += snprintf(buf + len, sizeof(buf) - len, len += scnprintf(buf + len, sizeof(buf) - len,
" <none>"); " <none>");
else { else {
for (i = 0; i < 8; i++) for (i = 0; i < 8; i++)
if (dma->map & (1 << i)) if (dma->map & (1 << i))
len += snprintf(buf + len, len += scnprintf(buf + len,
sizeof(buf) - len, sizeof(buf) - len,
" %d", i); " %d", i);
} }
len += snprintf(buf + len, sizeof(buf) - len, " (bitmask %#x) " len += scnprintf(buf + len, sizeof(buf) - len, " (bitmask %#x) "
"flags %#x", dma->map, dma->flags); "flags %#x", dma->map, dma->flags);
break; break;
} }
dev_dbg(&dev->dev, "%s\n", buf); dev_dbg(&dev->dev, "%s\n", buf);
......
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