From 2b57ea169265fa4243e0ca7832469db89914e73a Mon Sep 17 00:00:00 2001 From: Povilas Kanapickas Date: Sun, 4 Aug 2024 17:16:19 +0300 Subject: [PATCH] test/syscalls/proc: Update comment about required_fields on x86 The previous comment has been no longer correct at least since f84a0134072cfd50496a98b125f7290484a8f51c, as conditional cpuinfo fields are now checked too. --- test/syscalls/linux/proc.cc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/test/syscalls/linux/proc.cc b/test/syscalls/linux/proc.cc index cf77088df..ec7528564 100644 --- a/test/syscalls/linux/proc.cc +++ b/test/syscalls/linux/proc.cc @@ -120,9 +120,10 @@ namespace { #endif /* SUID_DUMP_ROOT */ #if defined(__x86_64__) || defined(__i386__) -// This list of "required" fields is taken from reading the file -// arch/x86/kernel/cpu/proc.c and seeing which fields will be unconditionally -// printed by the kernel. +// This list of "required" fields consists of the set of fields that are printed +// unconditionally by the kernel (see arch/x86/kernel/cpu/proc.c) and a set of fields +// that are printed conditionally, but are present on common configurations +// (e.g. when CONFIG_SMP is set). static const char* required_fields[] = { "processor", "vendor_id",