Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp3570560imu; Fri, 18 Jan 2019 12:45:35 -0800 (PST) X-Google-Smtp-Source: ALg8bN5cE6PKzhK6cOXeMqBq/oYLt177RAgpE4I8hTujI/t78NXXtRRmTg2ezunQIMSr6SDH9h3L X-Received: by 2002:a63:5252:: with SMTP id s18mr19152184pgl.326.1547844335259; Fri, 18 Jan 2019 12:45:35 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1547844335; cv=none; d=google.com; s=arc-20160816; b=hIlqk52WUjI6aMgiU9glL0QmaLlQX+lQNnmbkpRS6gEv90pTZs2FRj3x+4j8Ogc6j0 LT80+41XhVDV8ZeIbI0Nmai+vhDhqNUji/PWJA93zI1TOSKyshcBUYDwcFw+yVuQSE5f tuBP/BAy8mkgyR3Z998ZwFPuZJb8RkZgp0XKMfIt+0nKvTVzZLOUyCy6IoaUtbHnWmyz 5DBQpugHrRtFhCWbRlYrgWSqYKlhlsiXtaN2V3WjI9TFa3yRKbfqFHZk5aP5VNGJktQ/ NiXAeSfg/YHs+7SpShzdqmcoXilj6WBLdp4KbZRpCpjy5q0lPZ+yFCNDhdKjDlyTjWZc awPw== 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=YLNrTBgrVls0O+klmSvZxlkJp97odg9A+eImbNX0qb8=; b=pRfDMxms5qy+rCW4JquC8cX0zWVlq/51cRgMvnhVIhAr5h/dnIRfSnfc3vkHVVFjNJ I2xQeh5PwDobqjxa3KrVU49pbub7C2fl2zqxJJd7llRAggnH5RPm3ojO3yFjAc5jcyZM 8aYdTkcUzIA1AtCuJhIvxBGhfmenIeacxQ8vpsmcW8m7rJ+PbLlaGNJewDObNGZ/zioP V7YJKhlIWYg2s/OzvEfzTLb7rS+JwpMrNAha3LU0SpMYfj4o0VWQvNnLl56p+OKuz5Zj 7I8S35RRkmLSR4aIvH34zavimJVgght2Fzv1TCV491vEIZ1jiPM/HH2FJXgaPWt0kQvO Jw4Q== 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f16si5813184pgb.140.2019.01.18.12.45.20; Fri, 18 Jan 2019 12:45:35 -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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729548AbfARUoD (ORCPT + 99 others); Fri, 18 Jan 2019 15:44:03 -0500 Received: from mga14.intel.com ([192.55.52.115]:41566 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729471AbfARUoC (ORCPT ); Fri, 18 Jan 2019 15:44:02 -0500 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 18 Jan 2019 12:44:02 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,492,1539673200"; d="scan'208";a="128950048" Received: from unknown (HELO localhost.localdomain) ([10.232.112.69]) by orsmga001.jf.intel.com with ESMTP; 18 Jan 2019 12:44:01 -0800 Date: Fri, 18 Jan 2019 13:42:51 -0700 From: Keith Busch To: "Rafael J. Wysocki" Cc: Linux Kernel Mailing List , ACPI Devel Maling List , Linux Memory Management List , Greg Kroah-Hartman , Dave Hansen , Dan Williams Subject: Re: [PATCHv4 05/13] Documentation/ABI: Add new node sysfs attributes Message-ID: <20190118204251.GI31543@localhost.localdomain> References: <20190116175804.30196-1-keith.busch@intel.com> <20190116175804.30196-6-keith.busch@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jan 17, 2019 at 12:41:19PM +0100, Rafael J. Wysocki wrote: > On Wed, Jan 16, 2019 at 6:59 PM Keith Busch wrote: > > > > Add entries for memory initiator and target node class attributes. > > > > Signed-off-by: Keith Busch > > I would recommend combining this with the previous patch, as the way > it is now I need to look at two patches at the time. :-) Will do. > > --- > > Documentation/ABI/stable/sysfs-devices-node | 25 ++++++++++++++++++++++++- > > 1 file changed, 24 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/ABI/stable/sysfs-devices-node b/Documentation/ABI/stable/sysfs-devices-node > > index 3e90e1f3bf0a..a9c47b4b0eee 100644 > > --- a/Documentation/ABI/stable/sysfs-devices-node > > +++ b/Documentation/ABI/stable/sysfs-devices-node > > @@ -90,4 +90,27 @@ Date: December 2009 > > Contact: Lee Schermerhorn > > Description: > > The node's huge page size control/query attributes. > > - See Documentation/admin-guide/mm/hugetlbpage.rst > > \ No newline at end of file > > + See Documentation/admin-guide/mm/hugetlbpage.rst > > + > > +What: /sys/devices/system/node/nodeX/classY/ > > +Date: December 2018 > > +Contact: Keith Busch > > +Description: > > + The node's relationship to other nodes for access class "Y". > > + > > +What: /sys/devices/system/node/nodeX/classY/initiator_nodelist > > +Date: December 2018 > > +Contact: Keith Busch > > +Description: > > + The node list of memory initiators that have class "Y" access > > + to this node's memory. CPUs and other memory initiators in > > + nodes not in the list accessing this node's memory may have > > + different performance. > > This does not follow the general "one value per file" rule of sysfs (I > know that there are other sysfs files with more than one value in > them, but it is better to follow this rule as long as that makes > sense). I think it actually does make sense to use "%*pbl" format here. The type we're displaying is a bit array, so this is one value for that attribute. There are many precedents for disapling bit arrays this way. The existing node attributes that we're appending to show cpu lists, and the top loevel shows node lists for "possible", "online", "has_memory", "has_cpu". This new attribute should fit well with exisiting similar attributes from this subsystem. > > + > > +What: /sys/devices/system/node/nodeX/classY/target_nodelist > > +Date: December 2018 > > +Contact: Keith Busch > > +Description: > > + The node list of memory targets that this initiator node has > > + class "Y" access. Memory accesses from this node to nodes not > > + in this list may have differet performance. > > -- > > Same here. > > And if you follow the recommendation given in the previous message > (add "initiators" and "targets" subdirs under "classX"), you won't > even need the two files above. I think that makes sense, I'll take try out this suggestion. > And, of course, the symlinks part needs to be documented as well. I > guess you can follow the > Documentation/ABI/testing/sysfs-devices-power_resources_D0 with that.