diff options
author | Bjorn Helgaas <bhelgaas@google.com> | 2014-02-26 13:25:56 -0500 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-02-26 16:42:09 -0500 |
commit | d19cb803a2ff85d1b64b9628e1aec2aa76a9260b (patch) | |
tree | ad782893ca50522d4285df1e523c5020f7095bfe /lib | |
parent | 5edb93b89f6cc3089ee283656555e7a9ad36a8a0 (diff) |
vsprintf: Add support for IORESOURCE_UNSET in %pR
Sometimes we have a struct resource where we know the type (MEM/IO/etc.)
and the size, but we haven't assigned address space for it. The
IORESOURCE_UNSET flag is a way to indicate this situation. For these
"unset" resources, the start address is meaningless, so print only the
size, e.g.,
- pci 0000:0c:00.0: reg 184: [mem 0x00000000-0x00001fff 64bit]
+ pci 0000:0c:00.0: reg 184: [mem size 0x2000 64bit]
For %pr (printing with raw flags), we still print the address range,
because %pr is mostly used for debugging anyway.
Thanks to Fengguang Wu <fengguang.wu@intel.com> for suggesting
resource_size().
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/vsprintf.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/lib/vsprintf.c b/lib/vsprintf.c index 185b6d300ebc..5e2cf6f342f8 100644 --- a/lib/vsprintf.c +++ b/lib/vsprintf.c | |||
@@ -719,10 +719,15 @@ char *resource_string(char *buf, char *end, struct resource *res, | |||
719 | specp = &mem_spec; | 719 | specp = &mem_spec; |
720 | decode = 0; | 720 | decode = 0; |
721 | } | 721 | } |
722 | p = number(p, pend, res->start, *specp); | 722 | if (decode && res->flags & IORESOURCE_UNSET) { |
723 | if (res->start != res->end) { | 723 | p = string(p, pend, "size ", str_spec); |
724 | *p++ = '-'; | 724 | p = number(p, pend, resource_size(res), *specp); |
725 | p = number(p, pend, res->end, *specp); | 725 | } else { |
726 | p = number(p, pend, res->start, *specp); | ||
727 | if (res->start != res->end) { | ||
728 | *p++ = '-'; | ||
729 | p = number(p, pend, res->end, *specp); | ||
730 | } | ||
726 | } | 731 | } |
727 | if (decode) { | 732 | if (decode) { |
728 | if (res->flags & IORESOURCE_MEM_64) | 733 | if (res->flags & IORESOURCE_MEM_64) |