Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-13 Thread Joonsoo Kim
On Thu, Dec 10, 2015 at 11:03:34AM +0100, Vlastimil Babka wrote: > On 12/10/2015 05:04 AM, Steven Rostedt wrote: > >On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: > >>Ccing, Steven to ask trace-cmd problem. > >> > >>I'd like to use %pgp in tracepoint output. It works well when I do >

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-13 Thread Joonsoo Kim
On Thu, Dec 10, 2015 at 11:03:34AM +0100, Vlastimil Babka wrote: > On 12/10/2015 05:04 AM, Steven Rostedt wrote: > >On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: > >>Ccing, Steven to ask trace-cmd problem. > >> > >>I'd like to use %pgp in tracepoint output. It works well when I do >

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-10 Thread James Hogan
On 9 December 2015 at 11:29, Arnd Bergmann wrote: > On Friday 04 December 2015 16:16:33 Vlastimil Babka wrote: >> --- a/include/linux/mmdebug.h >> +++ b/include/linux/mmdebug.h >> @@ -2,15 +2,20 @@ >> #define LINUX_MM_DEBUG_H 1 >> >> #include >> +#include >> +#include > > 8<- > Subject:

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-10 Thread Vlastimil Babka
On 12/10/2015 05:04 AM, Steven Rostedt wrote: On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: Ccing, Steven to ask trace-cmd problem. I'd like to use %pgp in tracepoint output. It works well when I do 'cat /sys/kernel/debug/tracing/trace' but not works well when I do './trace-cmd

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-10 Thread Vlastimil Babka
On 12/10/2015 04:51 AM, Steven Rostedt wrote: I should have been Cc'd on this as I'm the maintainer of a few of the files here that is being modified. Sorry about that. --- a/include/linux/trace_events.h +++ b/include/linux/trace_events.h @@ -15,16 +15,6 @@ struct tracer; struct dentry;

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-10 Thread Rasmus Villemoes
On Thu, Dec 10 2015, Steven Rostedt wrote: > On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: >> >> [page_ref:page_ref_unfreeze] bad op token & >> [page_ref:page_ref_set] bad op token & >> [page_ref:page_ref_mod_unless] bad op token & >> [page_ref:page_ref_mod_and_test] bad

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-10 Thread Vlastimil Babka
On 12/10/2015 04:51 AM, Steven Rostedt wrote: I should have been Cc'd on this as I'm the maintainer of a few of the files here that is being modified. Sorry about that. --- a/include/linux/trace_events.h +++ b/include/linux/trace_events.h @@ -15,16 +15,6 @@ struct tracer; struct dentry;

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-10 Thread Vlastimil Babka
On 12/10/2015 05:04 AM, Steven Rostedt wrote: On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: Ccing, Steven to ask trace-cmd problem. I'd like to use %pgp in tracepoint output. It works well when I do 'cat /sys/kernel/debug/tracing/trace' but not works well when I do './trace-cmd

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-10 Thread Rasmus Villemoes
On Thu, Dec 10 2015, Steven Rostedt wrote: > On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: >> >> [page_ref:page_ref_unfreeze] bad op token & >> [page_ref:page_ref_set] bad op token & >> [page_ref:page_ref_mod_unless] bad op token & >>

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-10 Thread James Hogan
On 9 December 2015 at 11:29, Arnd Bergmann wrote: > On Friday 04 December 2015 16:16:33 Vlastimil Babka wrote: >> --- a/include/linux/mmdebug.h >> +++ b/include/linux/mmdebug.h >> @@ -2,15 +2,20 @@ >> #define LINUX_MM_DEBUG_H 1 >> >> #include >> +#include >> +#include > >

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Joonsoo Kim
On Wed, Dec 09, 2015 at 11:04:56PM -0500, Steven Rostedt wrote: > On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: > > Ccing, Steven to ask trace-cmd problem. > > Thanks, I should have been Cc'd for the rest as well. > > > > > On Fri, Dec 04, 2015 at 04:16:33PM +0100, Vlastimil

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Steven Rostedt
On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: > Ccing, Steven to ask trace-cmd problem. Thanks, I should have been Cc'd for the rest as well. > > On Fri, Dec 04, 2015 at 04:16:33PM +0100, Vlastimil Babka wrote: > > In mm we use several kinds of flags bitfields that are sometimes

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Steven Rostedt
I should have been Cc'd on this as I'm the maintainer of a few of the files here that is being modified. On Fri, Dec 04, 2015 at 04:16:33PM +0100, Vlastimil Babka wrote: > In mm we use several kinds of flags bitfields that are sometimes printed for > debugging purposes, or exported to userspace

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Joonsoo Kim
Ccing, Steven to ask trace-cmd problem. On Fri, Dec 04, 2015 at 04:16:33PM +0100, Vlastimil Babka wrote: > In mm we use several kinds of flags bitfields that are sometimes printed for > debugging purposes, or exported to userspace via sysfs. To make them easier to > interpret independently on

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Vlastimil Babka
On 12/09/2015 12:29 PM, Arnd Bergmann wrote: > On Friday 04 December 2015 16:16:33 Vlastimil Babka wrote: >> --- a/include/linux/mmdebug.h >> +++ b/include/linux/mmdebug.h >> @@ -2,15 +2,20 @@ >> #define LINUX_MM_DEBUG_H 1 >> >> #include >> +#include >> +#include > > 8<- > Subject: mm:

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Arnd Bergmann
On Friday 04 December 2015 16:16:33 Vlastimil Babka wrote: > --- a/include/linux/mmdebug.h > +++ b/include/linux/mmdebug.h > @@ -2,15 +2,20 @@ > #define LINUX_MM_DEBUG_H 1 > > #include > +#include > +#include 8<- Subject: mm: fix generated/bounds.h The inclusion of linux/tracepoint.h

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Arnd Bergmann
On Friday 04 December 2015 16:16:33 Vlastimil Babka wrote: > --- a/include/linux/mmdebug.h > +++ b/include/linux/mmdebug.h > @@ -2,15 +2,20 @@ > #define LINUX_MM_DEBUG_H 1 > > #include > +#include > +#include 8<- Subject: mm: fix generated/bounds.h The inclusion of linux/tracepoint.h

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Vlastimil Babka
On 12/09/2015 12:29 PM, Arnd Bergmann wrote: > On Friday 04 December 2015 16:16:33 Vlastimil Babka wrote: >> --- a/include/linux/mmdebug.h >> +++ b/include/linux/mmdebug.h >> @@ -2,15 +2,20 @@ >> #define LINUX_MM_DEBUG_H 1 >> >> #include >> +#include >> +#include > > 8<- > Subject: mm:

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Steven Rostedt
On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: > Ccing, Steven to ask trace-cmd problem. Thanks, I should have been Cc'd for the rest as well. > > On Fri, Dec 04, 2015 at 04:16:33PM +0100, Vlastimil Babka wrote: > > In mm we use several kinds of flags bitfields that are sometimes

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Joonsoo Kim
Ccing, Steven to ask trace-cmd problem. On Fri, Dec 04, 2015 at 04:16:33PM +0100, Vlastimil Babka wrote: > In mm we use several kinds of flags bitfields that are sometimes printed for > debugging purposes, or exported to userspace via sysfs. To make them easier to > interpret independently on

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Steven Rostedt
I should have been Cc'd on this as I'm the maintainer of a few of the files here that is being modified. On Fri, Dec 04, 2015 at 04:16:33PM +0100, Vlastimil Babka wrote: > In mm we use several kinds of flags bitfields that are sometimes printed for > debugging purposes, or exported to userspace

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-09 Thread Joonsoo Kim
On Wed, Dec 09, 2015 at 11:04:56PM -0500, Steven Rostedt wrote: > On Thu, Dec 10, 2015 at 11:59:44AM +0900, Joonsoo Kim wrote: > > Ccing, Steven to ask trace-cmd problem. > > Thanks, I should have been Cc'd for the rest as well. > > > > > On Fri, Dec 04, 2015 at 04:16:33PM +0100, Vlastimil

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-05 Thread Rasmus Villemoes
On Fri, Dec 04 2015, Vlastimil Babka wrote: > In mm we use several kinds of flags bitfields that are sometimes printed for > debugging purposes, or exported to userspace via sysfs. To make them easier to > interpret independently on kernel version and config, we want to dump also the > symbolic

Re: [PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-05 Thread Rasmus Villemoes
On Fri, Dec 04 2015, Vlastimil Babka wrote: > In mm we use several kinds of flags bitfields that are sometimes printed for > debugging purposes, or exported to userspace via sysfs. To make them easier to > interpret independently on kernel version and config, we want to dump also

[PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-04 Thread Vlastimil Babka
In mm we use several kinds of flags bitfields that are sometimes printed for debugging purposes, or exported to userspace via sysfs. To make them easier to interpret independently on kernel version and config, we want to dump also the symbolic flag names. So far this has been done with repeated

[PATCH v2 1/3] mm, printk: introduce new format string for flags

2015-12-04 Thread Vlastimil Babka
In mm we use several kinds of flags bitfields that are sometimes printed for debugging purposes, or exported to userspace via sysfs. To make them easier to interpret independently on kernel version and config, we want to dump also the symbolic flag names. So far this has been done with repeated