| Message ID | 20111107044750.GB4361@in.ibm.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
On 11/06/2011 10:47 PM, Ananth N Mavinakayanahalli wrote: > The Processor Identification Register (PIR) on powerpc provides > information to decode the processor identification tag. Decoding > this information platform specfic. > > Export PIR data via sysfs. > > (Powerpc manuals state this register is 'optional'. I am not sure > though if there are any Linux supported powerpc platforms that > don't have it. Code in the kernel referencing PIR isn't under > a platform ifdef). Those references are in platform-specific files, under #ifdef CONFIG_SMP, often in areas that would only be executed in the presence of multiple CPUs (e.g. secondary release). The reference in misc_32.S is inside #ifdef CONFIG_KEXEC and is fairly recent -- it may not have been tested on these systems. I don't see PIR (other than in the acronym definition section) in manuals for UP-only cores such as e300, 8xx, and 750. What use does userspace have for this? If you want to return the currently executing CPU (which unless you're pinned could change as soon as the value is read...), why not just return smp_processor_id() or hard_smp_processor_id()? -Scott
Index: linux-3.1/arch/powerpc/kernel/sysfs.c =================================================================== --- linux-3.1.orig/arch/powerpc/kernel/sysfs.c +++ linux-3.1/arch/powerpc/kernel/sysfs.c @@ -177,11 +177,13 @@ SYSFS_PMCSETUP(mmcra, SPRN_MMCRA); SYSFS_PMCSETUP(purr, SPRN_PURR); SYSFS_PMCSETUP(spurr, SPRN_SPURR); SYSFS_PMCSETUP(dscr, SPRN_DSCR); +SYSFS_PMCSETUP(pir, SPRN_PIR); static SYSDEV_ATTR(mmcra, 0600, show_mmcra, store_mmcra); static SYSDEV_ATTR(spurr, 0600, show_spurr, NULL); static SYSDEV_ATTR(dscr, 0600, show_dscr, store_dscr); static SYSDEV_ATTR(purr, 0600, show_purr, store_purr); +static SYSDEV_ATTR(pir, 0400, show_pir, NULL); unsigned long dscr_default = 0; EXPORT_SYMBOL(dscr_default); @@ -394,6 +396,8 @@ static void __cpuinit register_cpu_onlin sysdev_create_file(s, &attr_dscr); #endif /* CONFIG_PPC64 */ + sysdev_create_file(s, &attr_pir); + cacheinfo_cpu_online(cpu); } @@ -464,6 +468,8 @@ static void unregister_cpu_online(unsign sysdev_remove_file(s, &attr_dscr); #endif /* CONFIG_PPC64 */ + sysdev_remove_file(s, &attr_pir); + cacheinfo_cpu_offline(cpu); }
The Processor Identification Register (PIR) on powerpc provides information to decode the processor identification tag. Decoding this information platform specfic. Export PIR data via sysfs. (Powerpc manuals state this register is 'optional'. I am not sure though if there are any Linux supported powerpc platforms that don't have it. Code in the kernel referencing PIR isn't under a platform ifdef). Signed-off-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com> --- arch/powerpc/kernel/sysfs.c | 6 ++++++ 1 file changed, 6 insertions(+)