From: Rebecca Schultz Zavin <[email protected]>

The high variable was sometimes used uninitialized

Signed-off-by: Rebecca Schultz Zavin <[email protected]>
[jstultz: modified patch to apply to staging directory]
Signed-off-by: John Stultz <[email protected]>
---
 drivers/staging/android/ion/ion_page_pool.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/android/ion/ion_page_pool.c 
b/drivers/staging/android/ion/ion_page_pool.c
index e59da3a..df6e29c 100644
--- a/drivers/staging/android/ion/ion_page_pool.c
+++ b/drivers/staging/android/ion/ion_page_pool.c
@@ -203,8 +203,7 @@ static int ion_page_pool_shrink(struct shrinker *shrinker,
        bool high;
        int nr_to_scan = sc->nr_to_scan;
 
-       if (sc->gfp_mask & __GFP_HIGHMEM)
-               high = true;
+       high = sc->gfp_mask & __GFP_HIGHMEM;
 
        if (nr_to_scan == 0)
                return ion_page_pool_total(high);
-- 
1.8.3.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to