Re: [PATCH 1/1 linux-next] pnfs/flexfiles: use swap() in ff_layout_sort_mirrors()

2015-06-13 Thread Fabian Frederick
> On 12 June 2015 at 19:17 Joe Perches wrote: > > > On Fri, 2015-06-12 at 18:58 +0200, Fabian Frederick wrote: > > Use kernel.h macro definition. > > A lot of these conversions seem to be for bubble sorts. > Maybe it'd be useful to add a generic one somewhere. Of course but those swap()

Re: [PATCH 1/1 linux-next] pnfs/flexfiles: use swap() in ff_layout_sort_mirrors()

2015-06-13 Thread Fabian Frederick
On 12 June 2015 at 19:17 Joe Perches j...@perches.com wrote: On Fri, 2015-06-12 at 18:58 +0200, Fabian Frederick wrote: Use kernel.h macro definition. A lot of these conversions seem to be for bubble sorts. Maybe it'd be useful to add a generic one somewhere. Of course but those swap()

Re: [PATCH 1/1 linux-next] pnfs/flexfiles: use swap() in ff_layout_sort_mirrors()

2015-06-12 Thread Joe Perches
On Fri, 2015-06-12 at 18:58 +0200, Fabian Frederick wrote: > Use kernel.h macro definition. A lot of these conversions seem to be for bubble sorts. Maybe it'd be useful to add a generic one somewhere. > diff --git a/fs/nfs/flexfilelayout/flexfilelayout.c >

[PATCH 1/1 linux-next] pnfs/flexfiles: use swap() in ff_layout_sort_mirrors()

2015-06-12 Thread Fabian Frederick
Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick --- fs/nfs/flexfilelayout/flexfilelayout.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/fs/nfs/flexfilelayout/flexfilelayout.c

[PATCH 1/1 linux-next] pnfs/flexfiles: use swap() in ff_layout_sort_mirrors()

2015-06-12 Thread Fabian Frederick
Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/nfs/flexfilelayout/flexfilelayout.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/fs/nfs/flexfilelayout/flexfilelayout.c

Re: [PATCH 1/1 linux-next] pnfs/flexfiles: use swap() in ff_layout_sort_mirrors()

2015-06-12 Thread Joe Perches
On Fri, 2015-06-12 at 18:58 +0200, Fabian Frederick wrote: Use kernel.h macro definition. A lot of these conversions seem to be for bubble sorts. Maybe it'd be useful to add a generic one somewhere. diff --git a/fs/nfs/flexfilelayout/flexfilelayout.c b/fs/nfs/flexfilelayout/flexfilelayout.c