Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Borislav Petkov
On Fri, Aug 03, 2012 at 09:46:15AM -0700, Pandita, Vikram wrote: > I mostly work with ARM Soc - specifically on OMAP. SMP multi core > systems in ARM-v7 world started to show up only few years back - > unlike x86 world. This is exactly the thing: other SMP vendors have made it so far without emitt

Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Pandita, Vikram
Borislav/Kay On Fri, Aug 3, 2012 at 3:42 AM, Borislav Petkov wrote: > > On Fri, Aug 03, 2012 at 12:09:15PM +0200, Kay Sievers wrote: > > That would be needed, I guess. We easily have server systems with more > > than 255 CPUs. It will only be a matter of time, that the number of > > CPUs will inc

Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Borislav Petkov
On Fri, Aug 03, 2012 at 12:09:15PM +0200, Kay Sievers wrote: > That would be needed, I guess. We easily have server systems with more > than 255 CPUs. It will only be a matter of time, that the number of > CPUs will increase for everybody, I guess. Actually, AFAICR, we already have SGI machines bo

Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Kay Sievers
On Fri, Aug 3, 2012 at 11:56 AM, Pandita, Vikram wrote: > On Fri, Aug 3, 2012 at 2:48 AM, Kay Sievers wrote: >> On Fri, Aug 3, 2012 at 11:36 AM, Pandita, Vikram >> wrote: >>> On Fri, Aug 3, 2012 at 2:32 AM, Venu Byravarasu >>> wrote: >> As having Macro locally in the file of interest wou

Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Pandita, Vikram
Kay On Fri, Aug 3, 2012 at 2:48 AM, Kay Sievers wrote: > On Fri, Aug 3, 2012 at 11:36 AM, Pandita, Vikram > wrote: >> On Fri, Aug 3, 2012 at 2:32 AM, Venu Byravarasu >> wrote: > >>> As having Macro locally in the file of interest would serve the purpose, >>> Why to change the printk code? >>

Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Kay Sievers
On Fri, Aug 3, 2012 at 11:43 AM, Nikunj A Dadhania wrote: > On Fri, 3 Aug 2012 02:16:18 -0700, Vikram Pandita > wrote: >> +static size_t print_cpuid(u8 cpuid, char *buf) >> +{ >> + >> + if (!printk_cpuid) >> + return 0; >> + >> + if (!buf) >> + return 4; >> + >

Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Kay Sievers
On Fri, Aug 3, 2012 at 11:36 AM, Pandita, Vikram wrote: > On Fri, Aug 3, 2012 at 2:32 AM, Venu Byravarasu > wrote: >> As having Macro locally in the file of interest would serve the purpose, >> Why to change the printk code? > > As stated, the logic followed is exactly similar to well proven an

RE: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Venu Byravarasu
> -Original Message- > From: Pandita, Vikram [mailto:vikram.pand...@ti.com] > Sent: Friday, August 03, 2012 3:07 PM > To: Venu Byravarasu > Cc: gre...@linuxfoundation.org; k...@vrfy.org; linux- > ker...@vger.kernel.org; Mike Turquette; Vimarsh Zutshi > Subject: Re: [

Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Nikunj A Dadhania
On Fri, 3 Aug 2012 02:16:18 -0700, Vikram Pandita wrote: > From: Vikram Pandita > > Introduce config option to enable CPU id reporting for printk() calls. > > Example logs with this option enabled look like: > [1] [2.328613] usbcore: registered new interface driver libusual > [1] [2.3

Re: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Pandita, Vikram
>> To: Vikram Pandita; gre...@linuxfoundation.org; k...@vrfy.org >> Cc: linux-kernel@vger.kernel.org; Mike Turquette; Vimarsh Zutshi >> Subject: RE: [PATCH v2] printk: add option to print cpu id >> >> > -Original Message- >> > From: linux-kernel-ow...@vger.kern

RE: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Venu Byravarasu
r.kernel.org; Mike Turquette; Vimarsh Zutshi > Subject: RE: [PATCH v2] printk: add option to print cpu id > > > -Original Message- > > From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- > > ow...@vger.kernel.org] On Behalf Of Vikram Pandita > > S

RE: [PATCH v2] printk: add option to print cpu id

2012-08-03 Thread Venu Byravarasu
> -Original Message- > From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- > ow...@vger.kernel.org] On Behalf Of Vikram Pandita > Sent: Friday, August 03, 2012 2:46 PM > To: gre...@linuxfoundation.org; k...@vrfy.org > Cc: linux-kernel@vger.kernel.org; Vikram Pandita; Mike Turquet