Re: [PATCH] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Wei Yang <richard.weiyang@xxxxxxxxx>
Subject: Re: [PATCH] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()
From: Michal Hocko <mhocko@xxxxxxxx>
Date: Wed, 28 Nov 2018 13:34:06 +0100
Cc: Dave Hansen <dave.hansen@xxxxxxxxx>, akpm@xxxxxxxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx
In-reply-to: <20181128084729.jozab2gaej5vh7ig@master>
References: <20181127023630.9066-1-richard.weiyang@gmail.com> <20181127062514.GJ12455@dhcp22.suse.cz> <3356e00d-9135-12ef-a53f-49d815b8fbfc@intel.com> <20181128010112.5tv7tpe3qeplzy6d@master> <20181128084729.jozab2gaej5vh7ig@master>
User-agent: Mutt/1.10.1 (2018-07-13)

On Wed 28-11-18 08:47:29, Wei Yang wrote:
[...]
> The mem_section[root] still has a chance to face the contention here.

_If_ that is really the case then we need a dedicated lock rather than
rely on pgdat which doesn't even make sense for sparsemem internal.

-- 
Michal Hocko
SUSE Labs

References:
- [PATCH] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()
  - From: Wei Yang
- Re: [PATCH] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()
  - From: Michal Hocko
- Re: [PATCH] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()
  - From: Dave Hansen
- Re: [PATCH] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()
  - From: Wei Yang
- Re: [PATCH] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()
  - From: Wei Yang

Prev by Date: Re: [PATCH v2 5/5] mm, memory_hotplug: Refactor shrink_zone/pgdat_span
Next by Date: Re: [PATCH v2 5/5] mm, memory_hotplug: Refactor shrink_zone/pgdat_span
Previous by thread: Re: [PATCH] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()
Next by thread: [PATCH v2] mm, sparse: drop pgdat_resize_lock in sparse_add/remove_one_section()
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]