| From: Donet Tom <donettom@linux.ibm.com> |
| Subject: mm/mempolicy: use the already fetched local variable |
| Date: Sat, 17 Feb 2024 01:31:33 -0600 |
| |
| Avoid doing a per cpu read and use the local variable thisnid. IMHO |
| this also makes the code more readable. |
| |
| Link: https://lkml.kernel.org/r/9c3f7b743477560d1c5b12b8c111a584a2cc92ee.1708097962.git.donettom@linux.ibm.com |
| Signed-off-by: Aneesh Kumar K.V (IBM) <aneesh.kumar@kernel.org> |
| Signed-off-by: Donet Tom <donettom@linux.ibm.com> |
| Cc: Andrea Arcangeli <aarcange@redhat.com> |
| Cc: Ben Widawsky <ben.widawsky@intel.com> |
| Cc: Dan Williams <dan.j.williams@intel.com> |
| Cc: Dave Hansen <dave.hansen@linux.intel.com> |
| Cc: Feng Tang <feng.tang@intel.com> |
| Cc: "Huang, Ying" <ying.huang@intel.com> |
| Cc: Hugh Dickins <hughd@google.com> |
| Cc: Ingo Molnar <mingo@redhat.com> |
| Cc: Johannes Weiner <hannes@cmpxchg.org> |
| Cc: Kefeng Wang <wangkefeng.wang@huawei.com> |
| Cc: Matthew Wilcox (Oracle) <willy@infradead.org> |
| Cc: Mel Gorman <mgorman@suse.de> |
| Cc: Michal Hocko <mhocko@kernel.org> |
| Cc: Mike Kravetz <mike.kravetz@oracle.com> |
| Cc: Peter Zijlstra (Intel) <peterz@infradead.org> |
| Cc: Rik van Riel <riel@surriel.com> |
| Cc: Suren Baghdasaryan <surenb@google.com> |
| Cc: Vlastimil Babka <vbabka@suse.cz> |
| Signed-off-by: Andrew Morton <akpm@linux-foundation.org> |
| --- |
| |
| mm/mempolicy.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/mm/mempolicy.c~mm-mempolicy-use-the-already-fetched-local-variable |
| +++ a/mm/mempolicy.c |
| @@ -2780,7 +2780,7 @@ int mpol_misplaced(struct folio *folio, |
| if (node_isset(curnid, pol->nodes)) |
| goto out; |
| z = first_zones_zonelist( |
| - node_zonelist(numa_node_id(), GFP_HIGHUSER), |
| + node_zonelist(thisnid, GFP_HIGHUSER), |
| gfp_zone(GFP_HIGHUSER), |
| &pol->nodes); |
| polnid = zone_to_nid(z->zone); |
| _ |