diff options
| author | Nick Desaulniers <nick.desaulniers@gmail.com> | 2017-07-06 15:36:50 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-07-06 16:24:31 -0700 |
| commit | f2f43e566a02a3bdde0a65e6a2e88d707c212a29 (patch) | |
| tree | 7607b53a29c2cfaf45860ac033cb96903ea93c9a /mm/nobootmem.c | |
| parent | ac34ceaf1cdb34e8c34c0873ee562e9df6087cbc (diff) | |
| download | op-kernel-dev-f2f43e566a02a3bdde0a65e6a2e88d707c212a29.zip op-kernel-dev-f2f43e566a02a3bdde0a65e6a2e88d707c212a29.tar.gz | |
mm/vmscan.c: fix unsequenced modification and access warning
Clang and its -Wunsequenced emits a warning
mm/vmscan.c:2961:25: error: unsequenced modification and access to 'gfp_mask' [-Wunsequenced]
.gfp_mask = (gfp_mask = current_gfp_context(gfp_mask)),
^
While it is not clear to me whether the initialization code violates the
specification (6.7.8 par 19 (ISO/IEC 9899) looks like it disagrees) the
code is quite confusing and worth cleaning up anyway. Fix this by
reusing sc.gfp_mask rather than the updated input gfp_mask parameter.
Link: http://lkml.kernel.org/r/20170510154030.10720-1-nick.desaulniers@gmail.com
Signed-off-by: Nick Desaulniers <nick.desaulniers@gmail.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/nobootmem.c')
0 files changed, 0 insertions, 0 deletions
