diff options
| author | Derek Yerger <[email protected]> | 2016-03-11 17:31:18 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2016-03-11 22:09:09 -0800 |
| commit | 3b0ae7bee0777699eb066e8668d8a25d9215d38d (patch) | |
| tree | 0161784fb0a2373d61b8d1982d30b61a93d9aeef | |
| parent | 80f10bcc7f8c0ca4bc0ef61bdc001ec11d5991f9 (diff) | |
staging: ion: debugfs invalid gfp mask
The current code attempts assignment of -1 to an unsigned type. Note that
in a downstream function ion_page_pool_shrink this mask is only ever
evaluated against __GFP_HIGHMEM
(drivers/staging/android/ion/ion_page_pool.c, line 125).
Signed-off-by: Derek Yerger <[email protected]>
Reviewed-by: Laura Abbott <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -rwxr-xr-x | drivers/staging/android/ion/ion.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index 33b390e7ea31..1c872bdfddf6 100755 --- a/drivers/staging/android/ion/ion.c +++ b/drivers/staging/android/ion/ion.c @@ -1555,7 +1555,7 @@ static int debug_shrink_set(void *data, u64 val) struct shrink_control sc; int objs; - sc.gfp_mask = -1; + sc.gfp_mask = GFP_HIGHUSER; sc.nr_to_scan = val; if (!val) { @@ -1573,7 +1573,7 @@ static int debug_shrink_get(void *data, u64 *val) struct shrink_control sc; int objs; - sc.gfp_mask = -1; + sc.gfp_mask = GFP_HIGHUSER; sc.nr_to_scan = 0; objs = heap->shrinker.count_objects(&heap->shrinker, &sc); |