diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-08-06 19:04:48 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-08-06 21:01:15 -0400 |
commit | 0aa9a13d80bae1bb24956f6e3e2662b7242e0b41 (patch) | |
tree | c8cd64393cf574e3b9be1159e010c7de9e4e29f5 /mm/slub.c | |
parent | 8a7d9b4306258e092afaae3c663661d22bf91f5c (diff) |
mm, slub: fix some indenting in cmpxchg_double_slab()
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;".
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: David Rientjes <rientjes@google.com>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Cc: Christoph Lameter <cl@linux.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/slub.c')
-rw-r--r-- | mm/slub.c | 12 |
1 files changed, 6 insertions, 6 deletions
@@ -381,9 +381,9 @@ static inline bool __cmpxchg_double_slab(struct kmem_cache *s, struct page *page | |||
381 | defined(CONFIG_HAVE_ALIGNED_STRUCT_PAGE) | 381 | defined(CONFIG_HAVE_ALIGNED_STRUCT_PAGE) |
382 | if (s->flags & __CMPXCHG_DOUBLE) { | 382 | if (s->flags & __CMPXCHG_DOUBLE) { |
383 | if (cmpxchg_double(&page->freelist, &page->counters, | 383 | if (cmpxchg_double(&page->freelist, &page->counters, |
384 | freelist_old, counters_old, | 384 | freelist_old, counters_old, |
385 | freelist_new, counters_new)) | 385 | freelist_new, counters_new)) |
386 | return 1; | 386 | return 1; |
387 | } else | 387 | } else |
388 | #endif | 388 | #endif |
389 | { | 389 | { |
@@ -417,9 +417,9 @@ static inline bool cmpxchg_double_slab(struct kmem_cache *s, struct page *page, | |||
417 | defined(CONFIG_HAVE_ALIGNED_STRUCT_PAGE) | 417 | defined(CONFIG_HAVE_ALIGNED_STRUCT_PAGE) |
418 | if (s->flags & __CMPXCHG_DOUBLE) { | 418 | if (s->flags & __CMPXCHG_DOUBLE) { |
419 | if (cmpxchg_double(&page->freelist, &page->counters, | 419 | if (cmpxchg_double(&page->freelist, &page->counters, |
420 | freelist_old, counters_old, | 420 | freelist_old, counters_old, |
421 | freelist_new, counters_new)) | 421 | freelist_new, counters_new)) |
422 | return 1; | 422 | return 1; |
423 | } else | 423 | } else |
424 | #endif | 424 | #endif |
425 | { | 425 | { |