From: Dario Faggioli <dario.faggioli@citrix.com>
To: Li Yechen <lccycc123@gmail.com>
Cc: Keir Fraser <keir@xen.org>, Elena Ufimtseva <ufimtseva@gmail.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
George Dunlap <George.Dunlap@eu.citrix.com>,
Matt Wilson <msw@linux.com>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
Jan Beulich <JBeulich@suse.com>
Subject: Re: [PATCH RFC v2 1/7] xen/vNUMA: vNUMA support for PV guests.
Date: Thu, 3 Oct 2013 17:17:22 +0200 [thread overview]
Message-ID: <1380813442.14853.10.camel@Solace> (raw)
In-Reply-To: <CAP5+zHQshSvGmb2mUOzUjwoLRVy0vz22+BP4+N9heP_6sfHpbw@mail.gmail.com>
[-- Attachment #1.1: Type: text/plain, Size: 1317 bytes --]
On gio, 2013-10-03 at 20:27 +0800, Li Yechen wrote:
> Hi Elena,
> Thank you for your great work first :)
>
> In your patch, guest can use a hypercall name: "XENMEM_get_vnuma_info"
> to get the structure below:
> +struct vnuma_topology_info {
> + domid_t domid;
> + uint16_t nr_vnodes;
> + uint32_t _pad;
> + XEN_GUEST_HANDLE_64(uint) vdistance;
> + XEN_GUEST_HANDLE_64(uint) vcpu_to_vnode;
> + XEN_GUEST_HANDLE_64(vnuma_
> memblk_t) vnuma_memblks;
> +};
>
>
> I see that Xen have save a vnode_to_pnode[] array for mapping
>
Yes it does. We spoke about it quite a bit, and Xen seems to be the best
place where to store such info, considering how easy it is to keep it
there, and that it may turn out useful in future (and ballooning is a
lcear example of that).
> However, this hypercall won't give this mapping to Guest VM.
>
Not with this hypercall, no. Actually, did most of the people, when
reviewing your RFC patches, said that they prefer the mapping not
exposed at all to the guest?
Dario
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2013-10-03 15:17 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-13 8:49 [PATCH RFC v2 1/7] xen/vNUMA: vNUMA support for PV guests Elena Ufimtseva
2013-09-13 9:31 ` Andrew Cooper
2013-09-13 11:53 ` Jan Beulich
2013-09-13 11:00 ` Jan Beulich
2013-09-16 15:46 ` George Dunlap
2013-09-17 6:44 ` Elena Ufimtseva
2013-09-17 6:59 ` Elena Ufimtseva
2013-09-17 7:05 ` Jan Beulich
2013-09-17 7:11 ` Dario Faggioli
2013-09-17 7:19 ` Elena Ufimtseva
2013-09-17 9:04 ` Dario Faggioli
2013-10-03 12:27 ` Li Yechen
2013-10-03 15:17 ` Dario Faggioli [this message]
2013-10-03 15:33 ` Elena Ufimtseva
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=1380813442.14853.10.camel@Solace \
--to=dario.faggioli@citrix.com \
--cc=George.Dunlap@eu.citrix.com \
--cc=JBeulich@suse.com \
--cc=keir@xen.org \
--cc=lccycc123@gmail.com \
--cc=msw@linux.com \
--cc=stefano.stabellini@eu.citrix.com \
--cc=ufimtseva@gmail.com \
--cc=xen-devel@lists.xen.org \
/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).