From patchwork Mon Nov 11 09:49:43 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Simon Marchi (Code Review)" X-Patchwork-Id: 35787 Received: (qmail 91202 invoked by alias); 11 Nov 2019 09:49:52 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 91136 invoked by uid 89); 11 Nov 2019 09:49:52 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3 autolearn=ham version=3.3.1 spammy=HContent-Transfer-Encoding:8bit X-HELO: mx1.osci.io X-Gerrit-PatchSet: 1 Date: Mon, 11 Nov 2019 04:49:43 -0500 From: "Florian Weimer (Code Review)" To: libc-alpha@sourceware.org Cc: Florian Weimer Message-ID: Auto-Submitted: auto-generated X-Gerrit-MessageType: newchange Subject: [review] linux: Add comment on affinity set sizes to tst-skeleton-affinity.c X-Gerrit-Change-Id: Ic6ec48f75f3a0576d3121befd04531382c92afb4 X-Gerrit-Change-Number: 610 X-Gerrit-ChangeURL: X-Gerrit-Commit: 79e59d7213987bcabe0b389079362542830e992c References: Reply-To: fweimer@redhat.com, fweimer@redhat.com, libc-alpha@sourceware.org MIME-Version: 1.0 Content-Disposition: inline User-Agent: Gerrit/3.0.3-76-gf8b6da0ab5 Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/610 ...................................................................... linux: Add comment on affinity set sizes to tst-skeleton-affinity.c Change-Id: Ic6ec48f75f3a0576d3121befd04531382c92afb4 --- M sysdeps/unix/sysv/linux/tst-skeleton-affinity.c 1 file changed, 5 insertions(+), 0 deletions(-) diff --git a/sysdeps/unix/sysv/linux/tst-skeleton-affinity.c b/sysdeps/unix/sysv/linux/tst-skeleton-affinity.c index 32fb94c..9d1f27f 100644 --- a/sysdeps/unix/sysv/linux/tst-skeleton-affinity.c +++ b/sysdeps/unix/sysv/linux/tst-skeleton-affinity.c @@ -264,6 +264,11 @@ struct conf conf; setup_conf (&conf); + /* Note: The CPU set size in bits can be less than the CPU count + (and the maximum test CPU) because the userspace interface rounds + up the bit count, and the rounded-up buffer size is passed into + the kernel. The kernel does not know that some of the buffer are + actually padding, and writes data there. */ printf ("info: Detected CPU set size (in bits): %d\n", conf.set_size); printf ("info: Maximum test CPU: %d\n", conf.last_cpu); if (conf.set_size < 0 || conf.last_cpu < 0)