From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754860Ab2H3SZZ (ORCPT ); Thu, 30 Aug 2012 14:25:25 -0400 Received: from perches-mx.perches.com ([206.117.179.246]:42638 "EHLO labridge.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753528Ab2H3SZW (ORCPT ); Thu, 30 Aug 2012 14:25:22 -0400 Message-ID: <1346351121.2103.36.camel@joe2Laptop> Subject: Re: [PATCH 0/5] dev_ and dynamic_debug cleanups From: Joe Perches To: Jim Cromie Cc: Andrew Morton , netdev@vger.kernel.org, Greg Kroah-Hartman , "David S. Miller" , Jason Baron , Kay Sievers , linux-kernel@vger.kernel.org Date: Thu, 30 Aug 2012 11:25:21 -0700 In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.2- 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-08-30 at 11:43 -0600, Jim Cromie wrote: > On Sun, Aug 26, 2012 at 5:25 AM, Joe Perches wrote: > > The recent commit to fix dynamic_debug was a bit unclean. > > Neaten the style for dynamic_debug. > > Reduce the stack use of message logging that uses netdev_printk > > Add utility functions dev_printk_emit and dev_vprintk_emit for /dev/kmsg. > > > > Joe Perches (5): > > dev_dbg/dynamic_debug: Update to use printk_emit, optimize stack > > netdev_printk/dynamic_netdev_dbg: Directly call printk_emit > > netdev_printk/netif_printk: Remove a superfluous logging colon > > dev: Add dev_vprintk_emit and dev_printk_emit > > device and dynamic_debug: Use dev_vprintk_emit and dev_printk_emit > > > > Ive tested this on 2 builds differing only by DYNAMIC_DEBUG > It works for me on x86-64 > > However, I just booted a non-dyndbg build on x86-32, and got this. > > > root@voyage:~# dmesg | grep Unknown > mac80211: Unknown symbol __dynamic_dev_dbg (err 0) > mac80211: Unknown symbol __dynamic_pr_debug (err 0) > scx200_acb: Unknown symbol __dynamic_dev_dbg (err 0) > scx200_acb: Unknown symbol __dynamic_pr_debug (err 0) > hwmon: Unknown symbol __dynamic_dev_dbg (err 0) > nsc_gpio: Unknown symbol __dynamic_dev_dbg (err 0) > nsc_gpio: Unknown symbol __dynamic_dev_dbg (err 0) > root@voyage:~# > > It may be my error, will investigate asap, > but wanted to report now. Thanks, but I don't know how this is possible. There are no 32/64 component tests in this code.