From: Dave Hansen <dave.hansen@intel.com>
To: "Song Bao Hua (Barry Song)" <song.bao.hua@hisilicon.com>,
"tiantao (H)" <tiantao6@hisilicon.com>,
"corbet@lwn.net" <corbet@lwn.net>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
Cc: "linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
Valentin Schneider <valentin.schneider@arm.com>,
Dave Hansen <dave.hansen@linux.intel.com>,
Daniel Bristot de Oliveira <bristot@redhat.com>
Subject: Re: [PATCH 1/2] CPU, NUMA topology ABIs: clarify the overflow issue of sysfs pagebuf
Date: Thu, 29 Apr 2021 15:38:39 -0700 [thread overview]
Message-ID: <fd78ac30-dd3b-a7d7-eae8-193b09a7d49a@intel.com> (raw)
In-Reply-To: <4bf6870f7f3942398e4d1fdaa42184c7@hisilicon.com>
On 4/29/21 3:32 PM, Song Bao Hua (Barry Song) wrote:
> $ strace numactl --hardware 2>&1 | grep cpu
> openat(AT_FDCWD, "/sys/devices/system/cpu",
> O_RDONLY|O_NONBLOCK|O_DIRECTORY|O_CLOEXEC) = 3
> openat(AT_FDCWD, "/sys/devices/system/node/node0/cpumap", O_RDONLY) = 3
> openat(AT_FDCWD, "/sys/devices/system/node/node1/cpumap", O_RDONLY) = 3
> openat(AT_FDCWD, "/sys/devices/system/node/node2/cpumap", O_RDONLY) = 3
> openat(AT_FDCWD, "/sys/devices/system/node/node3/cpumap", O_RDONLY) = 3
>
> If we move to binary, it means we have to change those applications.
I thought Greg was saying to using a sysfs binary attribute using
something like like sysfs_create_bin_file(). Those don't have the
PAGE_SIZE limitation. But, there's also nothing to keep us from spewing
nice human-readable text via the "binary" file.
We don't need to change the file format, just the internal kernel API
that we produce the files with.
next prev parent reply other threads:[~2021-04-29 22:38 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-29 7:03 [PATCH 0/2] clarify and cleanup CPU and NUMA topology ABIs Tian Tao
2021-04-29 7:03 ` [PATCH 1/2] CPU, NUMA topology ABIs: clarify the overflow issue of sysfs pagebuf Tian Tao
2021-04-29 14:21 ` Dave Hansen
2021-04-29 15:46 ` Greg KH
2021-04-29 21:08 ` Song Bao Hua (Barry Song)
2021-04-29 21:38 ` Dave Hansen
2021-04-29 22:32 ` Song Bao Hua (Barry Song)
2021-04-29 22:38 ` Dave Hansen [this message]
2021-04-29 22:48 ` Song Bao Hua (Barry Song)
2021-04-30 6:05 ` gregkh
2021-07-23 11:20 ` Song Bao Hua (Barry Song)
2021-07-23 11:28 ` gregkh
2021-07-23 12:48 ` Song Bao Hua (Barry Song)
2021-04-29 7:03 ` [PATCH 2/2] Documentation/ABI: Move the topology-related sysfs interface to the right place Tian Tao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fd78ac30-dd3b-a7d7-eae8-193b09a7d49a@intel.com \
--to=dave.hansen@intel.com \
--cc=bristot@redhat.com \
--cc=corbet@lwn.net \
--cc=dave.hansen@linux.intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=peterz@infradead.org \
--cc=rafael@kernel.org \
--cc=song.bao.hua@hisilicon.com \
--cc=tiantao6@hisilicon.com \
--cc=valentin.schneider@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).