Re: [PATCH] Reduce ACPI resource conflict message to KERN_INFO

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 02 March 2010 04:45:01 pm Chase Douglas wrote:
> On Tue, 2010-03-02 at 16:14 -0700, Bjorn Helgaas wrote:
> > On Tuesday 02 March 2010 02:09:12 pm Chase Douglas wrote:
> > > -			printk("%sACPI: %s resource %s [0x%llx-0x%llx]"
> > > +			printk(KERN_INFO "ACPI: %s resource %s [0x%llx-0x%llx]"
> > >  			       " conflicts with ACPI region %s"
> > >  			       " [0x%llx-0x%llx]\n",
> > > -			       acpi_enforce_resources == ENFORCE_RESOURCES_LAX
> > > -			       ? KERN_WARNING : KERN_ERR,
> > >  			       ioport ? "I/O" : "Memory", res->name,
> > >  			       (long long) res->start, (long long) res->end,
> > >  			       res_list_elem->name,
> > 
> > Since you're touching this anyway, can you convert this to use %pR?
> 
> I see the potential usage of %p, but is the 'R' a typo? I haven't found
> it in man 3 fprintf nor in a google search.

Nope, see this description of the %p<x> kernel extensions:

http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob;f=lib/vsprintf.c;hb=51c24aaacaea90c8e87f1dec75a2ac7622b593f8#l915

There are several examples already in the tree, including
drivers/pci/setup-bus.c.

Bjorn
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux