Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Toshi Kani
On Thu, 2012-07-26 at 14:57 -0700, Joe Perches wrote: > On Thu, 2012-07-26 at 15:50 -0600, Bjorn Helgaas wrote: > > On Thu, Jul 26, 2012 at 3:43 PM, Joe Perches wrote: > > > On Thu, 2012-07-26 at 15:37 -0600, Bjorn Helgaas wrote: > > >> PNP0C01:00: new device for \_SB_.PCI0.ISA_.MBIO > > >> >

Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Toshi Kani
On Thu, 2012-07-26 at 15:50 -0600, Bjorn Helgaas wrote: > On Thu, Jul 26, 2012 at 3:43 PM, Joe Perches wrote: > > On Thu, 2012-07-26 at 15:37 -0600, Bjorn Helgaas wrote: > >> PNP0C01:00: new device for \_SB_.PCI0.ISA_.MBIO > >> > >> I fiddled with this a while ago; it would look something like

Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Joe Perches
On Thu, 2012-07-26 at 15:50 -0600, Bjorn Helgaas wrote: > On Thu, Jul 26, 2012 at 3:43 PM, Joe Perches wrote: > > On Thu, 2012-07-26 at 15:37 -0600, Bjorn Helgaas wrote: > >> PNP0C01:00: new device for \_SB_.PCI0.ISA_.MBIO > >> > >> I fiddled with this a while ago; it would look something like

Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Toshi Kani
On Thu, 2012-07-26 at 15:37 -0600, Bjorn Helgaas wrote: > On Thu, Jul 26, 2012 at 02:58:50PM -0600, Toshi Kani wrote: > > On Thu, 2012-07-26 at 13:22 -0600, Bjorn Helgaas wrote: > > > On Wed, Jul 25, 2012 at 5:12 PM, Toshi Kani wrote: > > > > This patch introduces acpi_pr_(), where is a kernel >

Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Bjorn Helgaas
On Thu, Jul 26, 2012 at 3:43 PM, Joe Perches wrote: > On Thu, 2012-07-26 at 15:37 -0600, Bjorn Helgaas wrote: >> PNP0C01:00: new device for \_SB_.PCI0.ISA_.MBIO >> >> I fiddled with this a while ago; it would look something like this: > [] >> +static noinline_for_stack >> +char *acpi_name_stri

Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Joe Perches
On Thu, 2012-07-26 at 15:37 -0600, Bjorn Helgaas wrote: > PNP0C01:00: new device for \_SB_.PCI0.ISA_.MBIO > > I fiddled with this a while ago; it would look something like this: [] > +static noinline_for_stack > +char *acpi_name_string(char *buf, char *end, acpi_handle handle, > +

Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Bjorn Helgaas
On Thu, Jul 26, 2012 at 02:58:50PM -0600, Toshi Kani wrote: > On Thu, 2012-07-26 at 13:22 -0600, Bjorn Helgaas wrote: > > On Wed, Jul 25, 2012 at 5:12 PM, Toshi Kani wrote: > > > This patch introduces acpi_pr_(), where is a kernel > > > message level such as err/warn/info, to support improved log

Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Toshi Kani
On Thu, 2012-07-26 at 13:22 -0600, Bjorn Helgaas wrote: > On Wed, Jul 25, 2012 at 5:12 PM, Toshi Kani wrote: > > This patch introduces acpi_pr_(), where is a kernel > > message level such as err/warn/info, to support improved logging > > messages for ACPI, esp. in hotplug operations. acpi_pr_()

Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-26 Thread Bjorn Helgaas
On Wed, Jul 25, 2012 at 5:12 PM, Toshi Kani wrote: > This patch introduces acpi_pr_(), where is a kernel > message level such as err/warn/info, to support improved logging > messages for ACPI, esp. in hotplug operations. acpi_pr_() > appends "ACPI" prefix and ACPI object path to the messages. T

[PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces

2012-07-25 Thread Toshi Kani
This patch introduces acpi_pr_(), where is a kernel message level such as err/warn/info, to support improved logging messages for ACPI, esp. in hotplug operations. acpi_pr_() appends "ACPI" prefix and ACPI object path to the messages. This improves diagnostics in hotplug operations since it iden