Re: [tip.git#x86/debug] Re: x86/dumpstack: Combine some printk()s

2016-04-04 Thread Sedat Dilek
On Sat, Apr 2, 2016 at 1:12 PM, Thomas Gleixner wrote: > On Sat, 2 Apr 2016, Sedat Dilek wrote: > >> Hi, >> >> I was looking through tip Git tree... >> >> Why didn't you use... >> >> + IS_ENABLED(CONFIG_DEBUG_PAGEALLOC) ? " DEBUG_PAGEALLOC" : "", >> >> ...instead of... >> >> +

Re: [tip.git#x86/debug] Re: x86/dumpstack: Combine some printk()s

2016-04-04 Thread Sedat Dilek
On Sat, Apr 2, 2016 at 1:12 PM, Thomas Gleixner wrote: > On Sat, 2 Apr 2016, Sedat Dilek wrote: > >> Hi, >> >> I was looking through tip Git tree... >> >> Why didn't you use... >> >> + IS_ENABLED(CONFIG_DEBUG_PAGEALLOC) ? " DEBUG_PAGEALLOC" : "", >> >> ...instead of... >> >> +

Re: [tip.git#x86/debug] Re: x86/dumpstack: Combine some printk()s

2016-04-02 Thread Thomas Gleixner
On Sat, 2 Apr 2016, Sedat Dilek wrote: > Hi, > > I was looking through tip Git tree... > > Why didn't you use... > > + IS_ENABLED(CONFIG_DEBUG_PAGEALLOC) ? " DEBUG_PAGEALLOC" : "", > > ...instead of... > > + debug_pagealloc_enabled() ? " DEBUG_PAGEALLOC" : "", > > ...for consistency

Re: [tip.git#x86/debug] Re: x86/dumpstack: Combine some printk()s

2016-04-02 Thread Thomas Gleixner
On Sat, 2 Apr 2016, Sedat Dilek wrote: > Hi, > > I was looking through tip Git tree... > > Why didn't you use... > > + IS_ENABLED(CONFIG_DEBUG_PAGEALLOC) ? " DEBUG_PAGEALLOC" : "", > > ...instead of... > > + debug_pagealloc_enabled() ? " DEBUG_PAGEALLOC" : "", > > ...for consistency

[tip.git#x86/debug] Re: x86/dumpstack: Combine some printk()s

2016-04-01 Thread Sedat Dilek
Hi, I was looking through tip Git tree... Why didn't you use... + IS_ENABLED(CONFIG_DEBUG_PAGEALLOC) ? " DEBUG_PAGEALLOC" : "", ...instead of... + debug_pagealloc_enabled() ? " DEBUG_PAGEALLOC" : "", ...for consistency reasons? Regards, - Sedat - [1]

[tip.git#x86/debug] Re: x86/dumpstack: Combine some printk()s

2016-04-01 Thread Sedat Dilek
Hi, I was looking through tip Git tree... Why didn't you use... + IS_ENABLED(CONFIG_DEBUG_PAGEALLOC) ? " DEBUG_PAGEALLOC" : "", ...instead of... + debug_pagealloc_enabled() ? " DEBUG_PAGEALLOC" : "", ...for consistency reasons? Regards, - Sedat - [1]