qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: David Hildenbrand <david@redhat.com>
To: Markus Armbruster <armbru@redhat.com>
Cc: qemu-devel@nongnu.org, "Michal Privoznik" <mprivozn@redhat.com>,
	"Igor Mammedov" <imammedo@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"Eduardo Habkost" <eduardo@habkost.net>,
	"Dr . David Alan Gilbert" <dgilbert@redhat.com>,
	"Eric Blake" <eblake@redhat.com>,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Stefan Weil" <sw@weilnetz.de>
Subject: Re: [PATCH v1 3/7] util: Introduce ThreadContext user-creatable object
Date: Thu, 29 Sep 2022 18:05:08 +0200	[thread overview]
Message-ID: <f4389387-078f-3bad-bc2f-1d03362f59fb@redhat.com> (raw)
In-Reply-To: <87wn9mqsdq.fsf@pond.sub.org>

>>>> +##
>>>> +# @ThreadContextProperties:
>>>> +#
>>>> +# Properties for thread context objects.
>>>> +#
>>>> +# @cpu-affinity: the CPU affinity for all threads created in the thread
>>>> +#                context (default: QEMU main thread affinity)
>>>> +#
>>>> +# Since: 7.2
>>>> +##
>>>> +{ 'struct': 'ThreadContextProperties',
>>>> +  'data': { '*cpu-affinity': ['uint16'] } }
>>>
>>> I understand this is a list of affinities.  What I poor ignorant me
>>> doesn't understand is the meaning of the list index.  Or in other words,
>>> the list maps some range [0:N] to affinities, but what are the numbers
>>> being mapped there?
>>
>> Assume you have 8 physical CPUs.
>>
>> $ lscpu
>> ...
>>
>> NUMA:
>>    NUMA node(s):          1
>>    NUMA node0 CPU(s):     0-7
>> ...
>>
>> You will provide the CPU IDs here, for example as in patch #7 example:
>>
>> qemu-system-x86_64 -m 1G \
>>   -object thread-context,id=tc1,cpu-affinity=3-4 \
>>   -object memory-backend-ram,id=pc.ram,size=1G,prealloc=on,prealloc-threads=2,prealloc-context=tc1
>> \
>>   -machine memory-backend=pc.ram \
>>   -S -monitor stdio -sandbox enable=on,resourcecontrol=deny
>>
>>
>> Details about CPU affinities in general can be found in the man page of taskset:
>>
>> https://man7.org/linux/man-pages/man1/taskset.1.html
> 
> Is @cpu-affinity a set of CPU numbers?

Yes! For now I added to the description:

...
     General information about CPU affinities can be found in the man page of
     taskset:
         CPU affinity is a scheduler property that "bonds" a process to a given
         set of CPUs on the system. The Linux scheduler will honor the given CPU
         affinity and the process will not run on any other CPUs.
...
     A simple QEMU example to set the CPU affinity to CPU 0,1,6,7 would be:
         qemu-system-x86_64 -S \
           -object thread-context,id=tc1,cpu-affinity=0-1,cpu-affinity=6-7
     
     And we can query it via HMP/QMP:
         (qemu) qom-get tc1 cpu-affinity
         [
             0,
             1,
             6,
             7
         ]
     
     But note that due to dynamic library loading this example will not work
     before we actually make use of thread_context_create_thread() in QEMU
     code, because the type will otherwise not get registered.



> 
>> Please let me know how I can further clarify this, that would help, thanks!
> 
> What happens when you try to create a thread context object with CPU
> affinities on a host system that doesn't support CPU affinities?

qemu_thread_set_affinity() will fail and setting the attribute will result
in a "Setting CPU affinity failed".

Thanks!

-- 
Thanks,

David / dhildenb



  reply	other threads:[~2022-09-29 16:58 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-28 16:45 [PATCH v1 0/7] hostmem: NUMA-aware memory preallocation using ThreadContext David Hildenbrand
2022-09-28 16:45 ` [PATCH v1 1/7] util: Cleanup and rename os_mem_prealloc() David Hildenbrand
2022-09-28 16:45 ` [PATCH v1 2/7] util: Introduce qemu_thread_set_affinity() and qemu_thread_get_affinity() David Hildenbrand
2022-09-28 16:45 ` [PATCH v1 3/7] util: Introduce ThreadContext user-creatable object David Hildenbrand
2022-09-29 11:12   ` Markus Armbruster
2022-09-29 11:18     ` David Hildenbrand
2022-09-29 12:25       ` Markus Armbruster
2022-09-29 16:05         ` David Hildenbrand [this message]
2022-09-28 16:45 ` [PATCH v1 4/7] util: Add write-only "node-affinity" property for ThreadContext David Hildenbrand
2022-09-29 11:13   ` Markus Armbruster
2022-09-30  9:17     ` David Hildenbrand
2022-09-28 16:45 ` [PATCH v1 5/7] util: Make qemu_prealloc_mem() optionally consume a ThreadContext David Hildenbrand
2022-09-28 16:45 ` [PATCH v1 6/7] hostmem: Allow for specifying a ThreadContext for preallocation David Hildenbrand
2022-09-28 16:45 ` [PATCH v1 7/7] vl: Allow ThreadContext objects to be created before the sandbox option David Hildenbrand

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f4389387-078f-3bad-bc2f-1d03362f59fb@redhat.com \
    --to=david@redhat.com \
    --cc=armbru@redhat.com \
    --cc=berrange@redhat.com \
    --cc=dgilbert@redhat.com \
    --cc=eblake@redhat.com \
    --cc=eduardo@habkost.net \
    --cc=imammedo@redhat.com \
    --cc=mprivozn@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=sw@weilnetz.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).