[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZWX0Dq6_-0NAFgSl@tiehlicka>
Date: Tue, 28 Nov 2023 15:07:10 +0100
From: Michal Hocko <mhocko@...e.com>
To: Gregory Price <gourry.memverge@...il.com>
Cc: linux-mm@...ck.org, linux-doc@...r.kernel.org,
linux-fsdevel@...r.kernel.org, linux-api@...r.kernel.org,
linux-arch@...r.kernel.org, linux-kernel@...r.kernel.org,
akpm@...ux-foundation.org, arnd@...db.de, tglx@...utronix.de,
luto@...nel.org, mingo@...hat.com, bp@...en8.de,
dave.hansen@...ux.intel.com, x86@...nel.org, hpa@...or.com,
tj@...nel.org, ying.huang@...el.com,
Gregory Price <gregory.price@...verge.com>
Subject: Re: [RFC PATCH 02/11] mm/mempolicy: swap cond reference counting
logic in do_get_mempolicy
On Wed 22-11-23 16:11:51, Gregory Price wrote:
[...]
> @@ -982,11 +991,11 @@ static long do_get_mempolicy(int *policy, nodemask_t *nmask,
> }
>
> out:
> - mpol_cond_put(pol);
> + mpol_put(pol);
> if (vma)
> mmap_read_unlock(mm);
> if (pol_refcount)
> - mpol_put(pol_refcount);
> + mpol_cond_put(pol_refcount);
Maybe I am just misreading the patch but pol_refcount should be always
NULL with this patch
> return err;
> }
>
> --
> 2.39.1
>
--
Michal Hocko
SUSE Labs
Powered by blists - more mailing lists