Re: [PATCH] trace/ftrace.c: remove redundant swap function

2015-10-01 Thread Steven Rostedt
On Wed, 30 Sep 2015 21:43:38 +0200 Rasmus Villemoes wrote: > On Wed, Sep 09 2015, Rasmus Villemoes wrote: > > ping Both your patches look good. I applied both of them to my tree for the 4.4 window. I'll post them to linux-next after they've gone through my test suite. But currently my tests

Re: [PATCH] trace/ftrace.c: remove redundant swap function

2015-10-01 Thread Steven Rostedt
On Wed, 30 Sep 2015 21:43:38 +0200 Rasmus Villemoes wrote: > On Wed, Sep 09 2015, Rasmus Villemoes wrote: > > ping Both your patches look good. I applied both of them to my tree for the 4.4 window. I'll post them to linux-next after they've

Re: [PATCH] trace/ftrace.c: remove redundant swap function

2015-09-30 Thread Rasmus Villemoes
On Wed, Sep 09 2015, Rasmus Villemoes wrote: ping > To cover the common case of sorting an array of pointers, Daniel > Wagner recently modified the library sort() to use a specific swap > function for size==8, in addition to the size==4 case which was > already handled. Since sizeof(long) is

Re: [PATCH] trace/ftrace.c: remove redundant swap function

2015-09-30 Thread Rasmus Villemoes
On Wed, Sep 09 2015, Rasmus Villemoes wrote: ping > To cover the common case of sorting an array of pointers, Daniel > Wagner recently modified the library sort() to use a specific swap > function for size==8, in addition to the size==4 case which was > already

[PATCH] trace/ftrace.c: remove redundant swap function

2015-09-09 Thread Rasmus Villemoes
To cover the common case of sorting an array of pointers, Daniel Wagner recently modified the library sort() to use a specific swap function for size==8, in addition to the size==4 case which was already handled. Since sizeof(long) is either 4 or 8, ftrace_swap_ips() is redundant and we can just

[PATCH] trace/ftrace.c: remove redundant swap function

2015-09-09 Thread Rasmus Villemoes
To cover the common case of sorting an array of pointers, Daniel Wagner recently modified the library sort() to use a specific swap function for size==8, in addition to the size==4 case which was already handled. Since sizeof(long) is either 4 or 8, ftrace_swap_ips() is redundant and we can just