diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-05-14 19:16:44 +0300 |
|---|---|---|
| committer | Pekka Enberg <penberg@kernel.org> | 2014-05-15 12:14:54 +0300 |
| commit | 9892616242bf41fc270f22f478d3274bb0987798 (patch) | |
| tree | a20f784bc7597617a0a499f20e3e0bce3da4d8ae | |
| parent | 3fbd30959c3bcc3c053309606b8c711ee53c1882 (diff) | |
| download | linux-slab/next.tar.gz | |
mm: fix some indenting in cmpxchg_double_slab()slab/next
The return statement goes with the cmpxchg_double() condition so it
needs to be indented another tab.
Also these days the fashion is to line function parameters up, and it
looks nicer that way because then the "freelist_new" is not at the same
indent level as the "return 1;".
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
| -rw-r--r-- | mm/slub.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/mm/slub.c b/mm/slub.c index 2863505abe2c41..319695694599a2 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -382,9 +382,9 @@ static inline bool __cmpxchg_double_slab(struct kmem_cache *s, struct page *page defined(CONFIG_HAVE_ALIGNED_STRUCT_PAGE) if (s->flags & __CMPXCHG_DOUBLE) { if (cmpxchg_double(&page->freelist, &page->counters, - freelist_old, counters_old, - freelist_new, counters_new)) - return 1; + freelist_old, counters_old, + freelist_new, counters_new)) + return 1; } else #endif { @@ -418,9 +418,9 @@ static inline bool cmpxchg_double_slab(struct kmem_cache *s, struct page *page, defined(CONFIG_HAVE_ALIGNED_STRUCT_PAGE) if (s->flags & __CMPXCHG_DOUBLE) { if (cmpxchg_double(&page->freelist, &page->counters, - freelist_old, counters_old, - freelist_new, counters_new)) - return 1; + freelist_old, counters_old, + freelist_new, counters_new)) + return 1; } else #endif { |