Received: by 2002:ac0:8845:0:0:0:0:0 with SMTP id g63csp560611img; Thu, 28 Feb 2019 04:18:40 -0800 (PST) X-Google-Smtp-Source: AHgI3IaAB6d7MuFBiUGbVeL9cbKz/c/vbRGkH5cpyuYdTE6AfyOCj3xQrcWlUfll1509dE4ztFHF X-Received: by 2002:a65:40c5:: with SMTP id u5mr8096684pgp.275.1551356320256; Thu, 28 Feb 2019 04:18:40 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1551356320; cv=none; d=google.com; s=arc-20160816; b=Bb1jAJ11HQrgVGSaJWjkUmnyARfYy0F6XAvi5aAI/WfQ8MXTwNB9CUzucmrkg1fOwU 7BglzShj5ImNRl7SAZ0UuoQpY2KVPDgYcSrEuLS+oXCtETl+4Cxgbh68X+gjNrELpZXz xh1ApoTsuWy8qXK1vAgupgwDGYkXdrroRlFRL+hv0beiG8F9bci5vWEvpDNca9hNab2R RU5FStz8qmT6hQbwCI4VPrryteToJVDkBYWaRg6T31CVeaeAOfTD1fzUrm/yLGhPjvUW suQOsehAADpC1JvPk3HNiKGShhdHq5G87fAsqhkk+/kYwiSy1BBeYNN70ZFdlyMYMTOK oJiA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=zlRV+Xpewi2YGgLpwspHNg91ydUHHy4Vmzi7Nw6lK+k=; b=tDrUbRogLurBGzKdVm6mssmjvaOMYjBAKwfglOfHLu+BfXMWrCeCQJBomJmoeRPWhf 4het0XKsyOAirv+8La4mrOTFL0VJE/MfyM18lTQGoa1sfreB47uJNdyWTvSyy/aWd/ND d1d3HYzG9xURT8fEaJ0ydKTtPEUAEnEom0rSihUVmtvLfAsHfba25XXzggenCZT70tRq InSFc/7/fdJ8p8wFVJgTpPCr0j2eXy6LOfRhOjdtJEhy9b52wPnAv/yATQbDdPFZb9Sl gro95utAnnjGSg4ZRSxcjdUarmy6CTjJHVzbMFKSKycmGTj0rdUrQt7TbjXnlOBjbozW omsg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r11si17270609plo.400.2019.02.28.04.18.24; Thu, 28 Feb 2019 04:18:40 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731978AbfB1LdH (ORCPT + 99 others); Thu, 28 Feb 2019 06:33:07 -0500 Received: from outbound-smtp02.blacknight.com ([81.17.249.8]:40944 "EHLO outbound-smtp02.blacknight.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725945AbfB1LdH (ORCPT ); Thu, 28 Feb 2019 06:33:07 -0500 Received: from mail.blacknight.com (pemlinmail05.blacknight.ie [81.17.254.26]) by outbound-smtp02.blacknight.com (Postfix) with ESMTPS id 46393988D3 for ; Thu, 28 Feb 2019 11:33:05 +0000 (UTC) Received: (qmail 11447 invoked from network); 28 Feb 2019 11:33:05 -0000 Received: from unknown (HELO techsingularity.net) (mgorman@techsingularity.net@[37.228.225.79]) by 81.17.254.9 with ESMTPSA (AES256-SHA encrypted, authenticated); 28 Feb 2019 11:33:05 -0000 Date: Thu, 28 Feb 2019 11:33:03 +0000 From: Mel Gorman To: Andrey Ryabinin Cc: Andrew Morton , Johannes Weiner , Vlastimil Babka , Rik van Riel , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Michal Hocko Subject: Re: [PATCH v2 2/4] mm: remove zone_lru_lock() function access ->lru_lock directly Message-ID: <20190228113303.GE9565@techsingularity.net> References: <20190228083329.31892-1-aryabinin@virtuozzo.com> <20190228083329.31892-2-aryabinin@virtuozzo.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: <20190228083329.31892-2-aryabinin@virtuozzo.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 28, 2019 at 11:33:27AM +0300, Andrey Ryabinin wrote: > We have common pattern to access lru_lock from a page pointer: > zone_lru_lock(page_zone(page)) > > Which is silly, because it unfolds to this: > &NODE_DATA(page_to_nid(page))->node_zones[page_zonenum(page)]->zone_pgdat->lru_lock > while we can simply do > &NODE_DATA(page_to_nid(page))->lru_lock > > Remove zone_lru_lock() function, since it's only complicate things. > Use 'page_pgdat(page)->lru_lock' pattern instead. > > Signed-off-by: Andrey Ryabinin > Acked-by: Vlastimil Babka > Cc: Johannes Weiner > Cc: Michal Hocko > Cc: Rik van Riel > Cc: Mel Gorman Acked-by: Mel Gorman -- Mel Gorman SUSE Labs