Re: getting processor numbers

[Ulrich Drepper <drepper@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1905 bytes --]

Andrew Morton wrote:
> Did we mean to go off-list?

Oops, no, pressed the wrong button.

>> Andrew Morton wrote:
>>> So I'd have thought that in general an application should be querying its
>>> present affinity mask - something like sched_getaffinity()?  That fixes the
>>> CPU hotplug issues too, of course.
>> Does it really?
>>
>> My recollection is that the affinity masks of running processes is not
>> updated on hotplugging.  Is this addressed?
> 
> ah, yes, you're correct.
> 
> Inside a cpuset:
> 
>   sched_setaffinity() is constrained to those CPUs which are in the
>   cpuset.
> 
>   If a cpu if on/offlined we update each cpuset's cpu mask appropriately
>   but we do not update all the tasks presently running in the cpuset.
> 
> Outside a cpuset:
> 
>   sched_setaffinity() is constrained to all possible cpus
> 
>   We don't update each task's cpus_allowed when a CPU is removed.
> 
> 
> I think we trivially _could_ update each tasks's cpus_allowed mask when a
> CPU is removed, actually.

I think it has to be done.  But that's not so trivial.  What happens if
all the CPUs a process was supposed to be runnable on vanish.
Shouldn't, if no affinity mask is defined, new processors be added?  I
agree that if the process has a defined affinity mask no new processors
should be added _automatically_.


>> If yes, sched_getaffinity is a solution until the NUMA topology
>> framework can provide something better.  Even without a popcnt
>> instruction in the CPU (64-bit albeit) it's twice as fast as the the
>> stat() method proposed.
> 
> I'm surprised - I'd have expected sched_getaffinity() to be vastly quicker
> that doing fileystem operations.

You mean because it's only a factor of two?  Well, it's not once you
count the whole overhead.

-- 
➧ Ulrich Drepper ➧ Red Hat, Inc. ➧ 444 Castro St ➧ Mountain View, CA ❖


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 251 bytes --]