From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753191Ab2G0DhS (ORCPT ); Thu, 26 Jul 2012 23:37:18 -0400 Received: from g1t0028.austin.hp.com ([15.216.28.35]:34792 "EHLO g1t0028.austin.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752963Ab2G0DhB (ORCPT ); Thu, 26 Jul 2012 23:37:01 -0400 Message-ID: <1343359930.3010.542.camel@misato.fc.hp.com> Subject: Re: [PATCH v3 1/4] ACPI: Add acpi_pr_() interfaces From: Toshi Kani To: Joe Perches Cc: Bjorn Helgaas , lenb@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, isimatu.yasuaki@jp.fujitsu.com, liuj97@gmail.com, srivatsa.bhat@linux.vnet.ibm.com, prarit@redhat.com, imammedo@redhat.com, vijaymohan.pandarathil@hp.com Date: Thu, 26 Jul 2012 21:32:10 -0600 In-Reply-To: <1343339854.17538.50.camel@joe2Laptop> References: <1343257978-7085-1-git-send-email-toshi.kani@hp.com> <1343257978-7085-2-git-send-email-toshi.kani@hp.com> <1343336330.3010.496.camel@misato.fc.hp.com> <20120726213730.GA2149@google.com> <1343338982.17538.45.camel@joe2Laptop> <1343339854.17538.50.camel@joe2Laptop> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3 (3.2.3-1.fc16) Content-Transfer-Encoding: 7bit Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > > >> > > >> 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, > > >> + struct printf_spec spec, const char *fmt) > [] > > Yes. I'm hesitant about this approach in general, because I don't > > think printing the ACPI path is something we should be doing often. > > It's not like a struct resource or a MAC address, where there are > > dozens or hundreds of users. I really think we should only print ACPI > > paths in one or two places, so adding a %p extension would waste a > > letter and encourage the wrong behavior. > > I don't much care for adding ACPI specific calls to vsprintf > as acpi is supposed to be OS generic anyway. > > I don't think there's anything wrong with Toshi's approach. > Anyone that looks for speed in a logging message is looking > for an oddly fitting thing. Tracing sure, but logging? Fully agreed! One cannot use printk in performance path. Thanks, -Toshi > I also don't see anything wrong with renaming it to just > acpi_, but that's a different discussion. > > cheers, Joe > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/