From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Travis Subject: Re: [PATCH 4/8] SGI x86_64 UV: Limit the number of ACPI messages Date: Tue, 27 Oct 2009 08:27:35 -0700 Message-ID: <4AE71167.7060506@sgi.com> References: <20091023233743.439628000@alcatraz.americas.sgi.com> <20091023233754.668088000@alcatraz.americas.sgi.com> <1256354987.17875.6.camel@dc7800.home> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1256354987.17875.6.camel@dc7800.home> Sender: linux-kernel-owner@vger.kernel.org To: Bjorn Helgaas Cc: Ingo Molnar , Thomas Gleixner , Andrew Morton , Jack Steiner , Zhang Rui , Len Brown , Thomas Renninger , Alexey Dobriyan , Myron Stowe , Feng Tang , Suresh Siddha , Yinghai Lu , linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-acpi@vger.kernel.org Bjorn Helgaas wrote: ... > > I know we print way too much stuff for every processor, but again, I'd > rather see all CPUs or none. I think there's a little more value in > this one than the cooling device one (probably because I do a lot of > platform bringup), but it could certainly be made KERN_DEBUG and/or > combined with another processor discovery line. Is this more acceptable? Thanks, Mike --- SGI x86_64 UV: Limit the number of ACPI messages Limit number of ACPI messages of the form: [ 0.000000] ACPI: LSAPIC (acpi_id[0x00] lsapic_id[0x00] lsapic_eid[0x00] enabled) [ 99.638655] processor ACPI0007:00: registered as cooling_device0 Cc: Zhang Rui Cc: Len Brown Cc: Thomas Renninger Cc: Bjorn Helgaas Cc: Alexey Dobriyan Cc: Myron Stowe Cc: Feng Tang Cc: Suresh Siddha Cc: Yinghai Lu Cc: linux-acpi@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Mike Travis --- drivers/acpi/fan.c | 2 +- drivers/acpi/processor_core.c | 3 +-- drivers/acpi/tables.c | 13 ++++++++----- 3 files changed, 10 insertions(+), 8 deletions(-) --- linux.orig/drivers/acpi/fan.c +++ linux/drivers/acpi/fan.c @@ -267,7 +267,7 @@ goto end; } - dev_info(&device->dev, "registered as cooling_device%d\n", cdev->id); + dev_dbg(&device->dev, "registered as cooling_device%d\n", cdev->id); device->driver_data = cdev; result = sysfs_create_link(&device->dev.kobj, --- linux.orig/drivers/acpi/processor_core.c +++ linux/drivers/acpi/processor_core.c @@ -845,8 +845,7 @@ goto err_power_exit; } - dev_info(&device->dev, "registered as cooling_device%d\n", - pr->cdev->id); + dev_dbg(&device->dev, "registered as cooling_device%d\n", pr->cdev->id); result = sysfs_create_link(&device->dev.kobj, &pr->cdev->device.kobj, --- linux.orig/drivers/acpi/tables.c +++ linux/drivers/acpi/tables.c @@ -170,11 +170,14 @@ case ACPI_MADT_TYPE_LOCAL_SAPIC: { struct acpi_madt_local_sapic *p = - (struct acpi_madt_local_sapic *)header; - printk(KERN_INFO PREFIX - "LSAPIC (acpi_id[0x%02x] lsapic_id[0x%02x] lsapic_eid[0x%02x] %s)\n", - p->processor_id, p->id, p->eid, - (p->lapic_flags & ACPI_MADT_ENABLED) ? "enabled" : "disabled"); + (struct acpi_madt_local_sapic *)header; + + printk(KERN_DEBUG PREFIX + "LSAPIC (acpi_id[0x%02x] " + "lsapic_id[0x%02x] lsapic_eid[0x%02x] %s)\n", + p->processor_id, p->id, p->eid, + (p->lapic_flags & ACPI_MADT_ENABLED) ? + "enabled" : "disabled"); } break;