Messages in this thread | | | Subject | Re: [PATCH v17 1/3] proc: add /proc/<pid>/arch_status | From | "Enrico Weigelt, metux IT consult" <> | Date | Thu, 25 Apr 2019 12:11:57 +0200 |
| |
On 25.04.19 03:50, Li, Aubrey wrote:
>>> +>>> +config PROC_PID_ARCH_STATUS>>> + bool "Enable /proc/<pid>/arch_status file">>>> Why is this switchable? x86 selects it if PROC_FS is enabled and all other>> architectures are absolutely not interested in this.> > Above and this, I was trying to avoid an empty arch_file on other architectures.> In previous proposal the entry only exists on the platform with AVX512. Why not making it depend on those archs that actually support it ?
> In that way proc_task_arch_status() should be defined in asm/processor.h,> but proc_task_arch_status() has four parameters, I don't want unnecessary> "struct pid_namespace *ns" and "struct pid *pid" leaked into arch headers,> so I defined task_arch_status(m, task) to avoid that.
--mtx
-- Enrico Weigelt, metux IT consult Free software and Linux embedded engineering info@metux.net -- +49-151-27565287
| |