Messages in this thread | | | From | Shakeel Butt <> | Date | Thu, 29 Oct 2020 09:01:37 -0700 | Subject | Re: [PATCH v2] mm: memcontrol: Simplify the mem_cgroup_page_lruvec |
| |
On Thu, Oct 29, 2020 at 2:08 AM Michal Hocko <mhocko@suse.com> wrote: > > On Wed 28-10-20 11:50:13, Muchun Song wrote: > [...] > > -struct lruvec *mem_cgroup_page_lruvec(struct page *page, struct pglist_data *pgdat) > > +static struct lruvec * > > +__mem_cgroup_node_lruvec(struct mem_cgroup *memcg, struct pglist_data *pgdat, > > + int nid) > > I thought I have made it clear that this is not a good approach. Please > do not repost new version without that being addressed. If there are any > questions then feel free to ask for details.
You can get nid from pgdat (pgdat->node_id) and also pgdat from nid (NODE_DATA(nid)), so, __mem_cgroup_node_lruvec() only need one of them as parameter.
| |