Re: [PATCH] linux-user: Add missing /proc/cpuinfo fields for sparc

[Helge Deller <deller@gmx.de>]

On 1/9/25 01:40, Richard Henderson wrote:
> On 1/8/25 15:21, Helge Deller wrote:
>> My suggestion:
>> I change my patch to just add:
>>
>> +    dprintf(fd, "ncpus probed\t: %d\n", num_cpus);
>> +    dprintf(fd, "ncpus active\t: %d\n", num_cpus);
>> +    dprintf(fd, "State:\n");
>> +    for (i = 0; i < num_cpus; i++) {
>> +        dprintf(fd, "CPU%d:\t\t: online\n", i);
>> +    }
>>
>> Those were the lines which I was mostly interested in since
>> I found one application which wants to parse /proc/cpuinfo
>> to get number of CPUs in system.
>> Would that be acceptable (for now) ?
>
> Yes, thanks.  If only linux had added a prctl or something to not require parsing...

True, but most people forget about glibc providing sysconf(_SC_NPROCESSORS_ONLN)
as portable function so that people do not need to implement parsing
/proc/cpuinfo for each architecture themselves.

I'll respin the patch.

Helge