From: Vijaya Kumar K
On x86, for the pages mapped with PAGE_HYPERVISOR attribute
non-leaf page tables are allocated with valid pte entries.
and with MAP_SMALL_PAGES attribute only non-leaf page tables are
allocated with invalid (valid bit set to 0) pte entries.
However on arm this is not the case
Thanks to all for the comments! I've implemented most of the changes
recommended here in the v2 review. I should have a new patch set ready
this week (with an updated soft affinity patch as well). I'll just
address the comments where you asked for my feedback...
>> rqd_avgload = rqd->
This patch inserts some boot data initilization in xen/arch and
function definition in xen/include.
Signed-off-by: Uma Sharma
---
diff --git a/xen/arch/x86/setup.c b/xen/arch/x86/setup.c
index d316212..4f3ce15 100644
--- a/xen/arch/x86/setup.c
+++ b/xen/arch/x86/setup.c
@@ -1259,6 +1259,11 @@ voi
I wonder if libxl__device_nextid is robust enough to handle holes. If
there are 3 devices (0,1,2) of type X, and device #0 or #1 is removed
with "X-detach", wouldn't the code return nextid==2? The result would be
that the caller will write to the wrong path.
I have not veified this theory in pract
The maximum of SW-IOMMU is limited to 2^11*128 = 256K.
And the size of IO_TLB_DEFAULT_SIZE is limited to (64UL<<20) 64M now.
While in different platform and different requirement this seems improper.
So modifing the IO_TLB_SEGSIZE to io_tlb_segsize and IO_TLB_DEFAULT_SIZE
to io_tlb_default_size whi
This patch inserts runqueue_per_core code.
And also makes generic selection for runqueue by using boot paarmeter.
Signed-off-by: Uma Sharma
---
diff --git a/xen/common/sched_credit2.c b/xen/common/sched_credit2.c
index ad0a5d4..2075e70 100644
--- a/xen/common/sched_credit2.c
+++ b/xen/common/sch
El 09/03/15 a les 4.30, Tian, Kevin ha escrit:
>> From: Roger Pau Monne [mailto:roger@citrix.com]
>> Sent: Friday, February 27, 2015 7:34 PM
>>
>> iommu_share_p2m_table should not prevent PVH guests from using a shared
>> page
>> table between the IOMMU and EPT. Clean the code by removing the a
Hi David,
Recently I met an issue which is likely related with this patch. It
happened when running block benchmark on domU, the backend was an iSCSI
disk connected to dom0. I got below panic at put_page_testzero() on
dom0, at that time the ixgbe network card was freeing skb pages in
__skb_frag_un
On Sun, 2015-03-08 at 10:29 +, xen.org wrote:
> flight 35957 xen-unstable real [real]
> http://www.chiark.greenend.org.uk/~xensrcts/logs/35957/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be run:
> test-amd64-amd64-rumpuserxen-amd64
On 09/03/15 09:09, Bob Liu wrote:
> Hi David,
>
> Recently I met an issue which is likely related with this patch. It
> happened when running block benchmark on domU, the backend was an iSCSI
> disk connected to dom0. I got below panic at put_page_testzero() on
> dom0, at that time the ixgbe netwo
On Fri, 2015-03-06 at 16:50 +, George Dunlap wrote:
> > +libxl_device_usb *
> > +libxl_device_usb_assignable_list(libxl_ctx *ctx, int *num)
>
> I'm a bit ambivalent about this one. For people using xl, "lsusb"
> should be just fine. Is anyone building a toolstack on top of libxl
> directly g
On Fri, 2015-03-06 at 20:03 +, Korupol, Naveen (EXT) wrote:
> I am using J6 evm.
What is one of them?
> played with couple different u-boots
Does your u-boot launch the kernel (AKA Xen in this case) in NS.HYP
mode?
> If there are any probable fixes/hints for bringing up XEN on J6 evm.
Give
Il 09/03/2015 04:00, openlui ha scritto:
Hi:
I want to enable qemu-dm's tracing in XEN. I have built qemu with
"make debug=y tools" command, and found that during building, qemu is
configured with "trace_backend=stderr" option. However, there aren't
any trace log in /var/log/qemu/qemu-dm-
On Fri, 2015-03-06 at 21:24 +, Korupol, Naveen (EXT) wrote:
> I am a new to XEN and am trying to boot it on J6 EVM.
I've replied to your first posting, lets carry on in that thread.
___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.
On Mon, 2015-03-09 at 12:48 +0530, Uma Sharma wrote:
> This patch inserts some boot data initilization in xen/arch and
> function definition in xen/include.
> Signed-off-by: Uma Sharma
>
So, this is basically the same as patch 1 from this series of mine:
http://lists.xen.org/archives/html/xen-de
On Mon, Mar 09, 2015 at 02:27:46PM +0800, Chen, Tiejun wrote:
[...]
> >>+
> >>+FILE *f = fopen(pci_device_vendor_path, "r");
> >>+if (!f) {
> >>+LOGE(ERROR,
> >>+ "pci device "PCI_BDF" does not have vendor attribute",
> >>+ pcidev->domain, pcidev->bus, pcidev
On Mon, Mar 09, 2015 at 02:50:42PM +0800, Chen, Tiejun wrote:
> On 2015/3/6 20:59, Wei Liu wrote:
> >On Fri, Mar 06, 2015 at 05:18:36PM +0800, Chen, Tiejun wrote:
> >>On 2015/3/6 17:08, Tiejun Chen wrote:
> >>>Although we already have 'gfx_passthru' in b_info, this doesn' suffice
> >>>after we want
On Mon, Mar 09, 2015 at 02:45:36PM +0800, Chen, Tiejun wrote:
[...]
> >
> >>+exit (1);
> >>+}
> >>+} else if (!xlu_cfg_get_string(config, "gfx_passthru", &buf, 0)) {
> >>+if (libxl_gfx_passthru_kind_from_string(buf,
> >>&b_info->u.hvm.gfx_passthru_ki
On 03/09/2015 09:39 AM, Ian Campbell wrote:
> On Fri, 2015-03-06 at 16:50 +, George Dunlap wrote:
>>> +libxl_device_usb *
>>> +libxl_device_usb_assignable_list(libxl_ctx *ctx, int *num)
>>
>> I'm a bit ambivalent about this one. For people using xl, "lsusb"
>> should be just fine. Is anyone b
On Fri, Mar 06, 2015 at 06:20:57PM +0100, Dario Faggioli wrote:
> so update the relevant manpage and the example file.
>
> Signed-off-by: Dario Faggioli
> Cc: Ian Campbell
> Cc: Ian Jackson
> Cc: Stefano Stabellini
> Cc: Wei Liu
> Cc: Juergen Gross
> Cc: Meng Xu
Acked-by: Wei Liu
> ---
>
On Fri, Mar 06, 2015 at 06:21:07PM +0100, Dario Faggioli wrote:
> as it was not covering the '-n' option, which is present
> since d743a223 ("xl: add node-affinity to the output of
> `xl list`").
>
> Signed-off-by: Dario Faggioli
> Cc: Ian Campbell
> Cc: Ian Jackson
> Cc: Stefano Stabellini
>
(I'm putting the list back since I think the below may be of more
general interest)
On Sat, 2015-03-07 at 12:19 +0100, Fabio Canigliula wrote:
> >> Could you suggest me some boards based on ARMv7 (A7 and/or A15) or
> >> ARMv8 fully controllable and on which Xen 4.5 works? Any help would be
> >> gre
Hi Uma,
First of all, nice to see the patch here on xen-devel... good job! ;-P
A few things:
On Mon, 2015-03-09 at 12:48 +0530, Uma Sharma wrote:
> This patch inserts some boot data initilization in xen/arch and
> function definition in xen/include.
>
About this description. The fact that it add
At 02:03 + on 09 Mar (1425863009), Wu, Feng wrote:
>
>
> > -Original Message-
> > From: Tim Deegan [mailto:t...@xen.org]
> > Sent: Friday, March 06, 2015 5:44 PM
> > To: Wu, Feng
> > Cc: Jan Beulich; Zhang, Yang Z; Tian, Kevin; xen-devel@lists.xen.org
> > Subject: Re: [Xen-devel] VT-d
Hi Edgar,
On 09/03/2015 07:23, Edgar E. Iglesias wrote:
From: "Edgar E. Iglesias"
Replace the warning with an info message stating that the platform
is generic.
Suggested-by: Ian Campbell
Signed-off-by: Edgar E. Iglesias
Reviewed-by: Julien Grall
Regards,
--
Julien Grall
__
On Fri, Mar 06, 2015 at 06:21:15PM +0100, Dario Faggioli wrote:
> more specifically, the ones used as argument presence
> flags in `xl list'.
>
> Signed-off-by: Dario Faggioli
> Cc: Ian Campbell
> Cc: Ian Jackson
> Cc: Stefano Stabellini
> Cc: Wei Liu
Acked-by: Wei Liu
On Fri, Mar 06, 2015 at 06:21:23PM +0100, Dario Faggioli wrote:
> which, if provided, makes the command print a column
> with the name of the cpupool of the listed domain(s).
>
> Signed-off-by: Dario Faggioli
> Cc: Ian Campbell
> Cc: Ian Jackson
> Cc: Stefano Stabellini
> Cc: Wei Liu
> Cc: Ju
On Fri, Mar 06, 2015 at 06:21:32PM +0100, Dario Faggioli wrote:
> To add (removes) to (from) a cpupool all the pCPUs corresponding
> to the bits that are set in the passed bitmap.
>
> This is convenient and useful in order to implement, in xl,
> the possibility of specifying ranges of pCPUs to be
On Mon, 2015-03-09 at 10:17 +, George Dunlap wrote:
> On 03/09/2015 09:39 AM, Ian Campbell wrote:
> > On Fri, 2015-03-06 at 16:50 +, George Dunlap wrote:
> >>> +libxl_device_usb *
> >>> +libxl_device_usb_assignable_list(libxl_ctx *ctx, int *num)
> >>
> >> I'm a bit ambivalent about this one
On Sat, 7 Mar 2015, Bunny Mintoo wrote:
> Hi,
> I am totally new to Xen hence dropping this mail to get started with Xen
> development.
>
> I already have a fully functional Qemu implementation (along with kernel
> changes) to emulate my new x86 based Instruction Set
> Architecture (ISA). This I
Hi Edgar,
On 06/03/2015 03:08, Edgar E. Iglesias wrote:
+#define cuart_read(uart, off) readl((uart)->regs + (off))
+#define cuart_write(uart, off,val) writel((val), (uart)->regs + (off))
+
+static void cuart_interrupt(int irq, void *data, struct cpu_user_regs *regs)
+{
+struct
On 03/09/2015 05:30 PM, David Vrabel wrote:
> On 09/03/15 09:09, Bob Liu wrote:
>> Hi David,
>>
>> Recently I met an issue which is likely related with this patch. It
>> happened when running block benchmark on domU, the backend was an iSCSI
>> disk connected to dom0. I got below panic at put_page
On Fri, Mar 06, 2015 at 06:21:42PM +0100, Dario Faggioli wrote:
> in fact, right now, xl sub-commands 'cpupool-cpu-add' and
> 'cpupool-cpu-remove' only accept the specification of one
> pCPU to be added or removed to/from a cpupool.
>
> With this change, they can deal with ranges, like "4-8",
> or
On 05/03/2015 18:36, Ian Campbell wrote:
On Tue, 2015-03-03 at 14:45 +, Julien Grall wrote:
Hello Frediano,
On 03/03/15 11:19, Frediano Ziglio wrote:
This check allow to detect mail interrupt controller even if it does
main
not match one of the standard ones.
This allow boards with n
On 06/03/15 21:43, Daniel De Graaf wrote:
> This removes the FLASK_DEVELOP and FLASK_BOOTPARAM configuration
> parameters which have never been settable by users. Disabling the
> FLASK_DEVELOP configuration option has not produced a compiling
> hypervisor for some time, and the FLASK_BOOTPARAM opt
On 09/03/15 10:51, Bob Liu wrote:
>
> On 03/09/2015 05:30 PM, David Vrabel wrote:
>> On 09/03/15 09:09, Bob Liu wrote:
>>> Hi David,
>>>
>>> Recently I met an issue which is likely related with this patch. It
>>> happened when running block benchmark on domU, the backend was an iSCSI
>>> disk conn
On Mon, 2015-03-09 at 13:14 +1000, Edgar E. Iglesias wrote:
> On Fri, Mar 06, 2015 at 09:44:16AM +, Ian Campbell wrote:
> > On Fri, 2015-03-06 at 11:31 +1000, Edgar E. Iglesias wrote:
> > > On Thu, Mar 05, 2015 at 04:50:15PM +, Ian Campbell wrote:
> > > > On Thu, 2015-03-05 at 18:27 +1000,
On Mon, 9 Mar 2015, 신정섭 wrote:
> HI.
>
> I'm now modifying GIC in Xen 4.5.0.
>
> I have some question about lock of pending_irq struct in Xen ARM.
>
>
>
> pending_irq of SGI and PPI is stored in 'vcpu struct'. (IRQ num 0 ~ 31)
>
> pending_irq of SPI is stored in 'domain struct'. (IRQ num 3
On Fri, Mar 06, 2015 at 06:22:07PM +0100, Dario Faggioli wrote:
> instead manually free the elements of the list, which is
> exactly the purpose of the said function.
>
> Trade also a couple of 'return'-s with 'goto out'-s, which
> is more in line with libxl usage paradigm.
>
> Signed-off-by: Dar
On Fri, Mar 06, 2015 at 06:21:51PM +0100, Dario Faggioli wrote:
> instead of just list of single pCPUs or NUMA node IDs, as
> it happens right now.
>
> On the other hand, after this change, strings containing
> pCPUs and NUMA node ranges is supported. The syntax is the
> same one supported by the
On Fri, Mar 06, 2015 at 06:21:59PM +0100, Dario Faggioli wrote:
> with the rest of the file, where we return 1 on 0, rather
> than using libxl error codes.
>
While being consistent is good I'm not very sure if we should go for 0/1
rather than libxl error codes. I vaguely remember at some point we
On Fri, Mar 06, 2015 at 05:19:14PM +, Stefano Stabellini wrote:
> This reverts commit 2563bca11544361dc2afa5e20b5663e10a0715cb.
>
Missing S-o-B.
> Reviewed-by: Mike Latimer
> Tested-by: Mike Latimer
> ---
> tools/libxl/xl_cmdimpl.c | 17 +++--
> 1 file changed, 3 insertions(
On Fri, Mar 06, 2015 at 05:19:15PM +, Stefano Stabellini wrote:
[...]
> int libxl_wait_for_free_memory(libxl_ctx *ctx, uint32_t domid, uint32_t
> memory_kb, int wait_secs);
> -/* wait for the memory target of a domain to be reached */
> +/*
> + * Wait for the memory target of a domain to be
On Fri, 6 Mar 2015, David Vrabel wrote:
> On 06/03/15 17:51, Stefano Stabellini wrote:
> > On Fri, 6 Mar 2015, David Vrabel wrote:
> >> Auto-translated physmap guests (arm, arm64 and x86 PVHVM/PVH) map and
> >> unmap foreign GFNs using the same method (updating the physmap).
> >> Unify the two arm
On Mon, 2015-03-09 at 10:58 +, Wei Liu wrote:
> On Fri, Mar 06, 2015 at 06:21:51PM +0100, Dario Faggioli wrote:
> > ---
> > docs/man/xlcpupool.cfg.pod.5 | 22 +++---
> > tools/libxl/xl_cmdimpl.c | 17 ++---
> > 2 files changed, 33 insertions(+), 6 deletions
On 06/03/15 21:43, Daniel De Graaf wrote:
> This unifies the flask_enforcing and flask_enabled boot parameters into
> a single parameter with additional states. Defined options are:
>
> force - require policy to be loaded at boot time and enforce it
> late - bootloader policy is not used; later
On Mon, Mar 09, 2015 at 11:18:57AM +, Dario Faggioli wrote:
[...]
> > > +
> >
> > I suppose this is the old syntax?
> >
> It is.
>
> > I'm asking because I want to be sure we still support the old syntax. I
> > think we still support the old syntax from the look of the changes below
> > but
El 09/03/15 a les 12.02, David Vrabel ha escrit:
> On 09/03/15 10:51, Bob Liu wrote:
>>
>> On 03/09/2015 05:30 PM, David Vrabel wrote:
>>> On 09/03/15 09:09, Bob Liu wrote:
Hi David,
Recently I met an issue which is likely related with this patch. It
happened when running block
On Mon, 2015-03-09 at 11:01 +, Wei Liu wrote:
> On Fri, Mar 06, 2015 at 06:21:59PM +0100, Dario Faggioli wrote:
> > with the rest of the file, where we return 1 on 0, rather
> > than using libxl error codes.
> >
>
> While being consistent is good I'm not very sure if we should go for 0/1
> ra
On Mon, 9 Mar 2015, Wei Liu wrote:
> On Fri, Mar 06, 2015 at 05:19:14PM +, Stefano Stabellini wrote:
> > This reverts commit 2563bca11544361dc2afa5e20b5663e10a0715cb.
> >
>
> Missing S-o-B.
Do I really need to S-o-B a revert?
In any case:
Signed-off-by: Stefano Stabellini
> > Reviewed-b
On Mon, 2015-03-09 at 10:39 +, Wei Liu wrote:
> On Fri, Mar 06, 2015 at 06:21:32PM +0100, Dario Faggioli wrote:
> > --- a/tools/libxl/libxl.c
> > +++ b/tools/libxl/libxl.c
> > @@ -6343,17 +6343,31 @@ out:
> >
> > int libxl_cpupool_cpuadd(libxl_ctx *ctx, uint32_t poolid, int cpu)
> > {
> >
On Mon, Mar 09, 2015 at 08:58:38AM +0100, Olaf Hering wrote:
> I wonder if libxl__device_nextid is robust enough to handle holes. If
> there are 3 devices (0,1,2) of type X, and device #0 or #1 is removed
> with "X-detach", wouldn't the code return nextid==2? The result would be
> that the caller w
On Mon, 2015-03-09 at 10:51 +, Wei Liu wrote:
> On Fri, Mar 06, 2015 at 06:21:42PM +0100, Dario Faggioli wrote:
> > ---
> > docs/man/xl.pod.1| 25 ++--
> > tools/libxl/xl_cmdimpl.c | 94
> > --
> > 2 files changed, 62 insertions
On Mon, 2015-03-09 at 09:23 +, Ian Campbell wrote:
> On Sun, 2015-03-08 at 10:29 +, xen.org wrote:
> > flight 35957 xen-unstable real [real]
> > http://www.chiark.greenend.org.uk/~xensrcts/logs/35957/
> >
> > Regressions :-(
> >
> > Tests which did not succeed and are blocking,
> > includ
On 03/09/2015 07:11 AM, Justin Weaver wrote:
> +static int get_safe_pcpu(struct csched2_vcpu *svc)
> +{
>
I also don't like the name... __choose_cpu() maybe ?
>>>
>>> I'm not 100% happy with the name, but I think "get_safe_pcpu" is more
>>> descriptive than just "__choose_cpu".
>>>
On 09/03/15 10:33, Tim Deegan wrote:
> At 02:03 + on 09 Mar (1425863009), Wu, Feng wrote:
>>
>>> -Original Message-
>>> From: Tim Deegan [mailto:t...@xen.org]
>>> Sent: Friday, March 06, 2015 5:44 PM
>>> To: Wu, Feng
>>> Cc: Jan Beulich; Zhang, Yang Z; Tian, Kevin; xen-devel@lists.xen.o
>>> On 09.03.15 at 09:55, wrote:
> --- a/xen/common/sched_credit2.c
> +++ b/xen/common/sched_credit2.c
> @@ -165,6 +165,8 @@
>
> int opt_migrate_resist=500;
> integer_param("sched_credit2_migrate_resist", opt_migrate_resist);
> +static char __initdata opt_credit2_runqueue[10] = "socket";
> +st
On Mon, Mar 09, 2015 at 11:31:09AM +, Dario Faggioli wrote:
[...]
> > > --- a/tools/libxl/libxl.h
> > > +++ b/tools/libxl/libxl.h
> > > @@ -1456,8 +1456,12 @@ int libxl_cpupool_destroy(libxl_ctx *ctx, uint32_t
> > > poolid);
> > > int libxl_cpupool_rename(libxl_ctx *ctx, const char *name, uin
On Mon, Mar 09, 2015 at 12:47:51PM +0200, Julien Grall wrote:
> Hi Edgar,
Hi Julien,
>
> On 06/03/2015 03:08, Edgar E. Iglesias wrote:
> >+#define cuart_read(uart, off) readl((uart)->regs + (off))
> >+#define cuart_write(uart, off,val) writel((val), (uart)->regs + (off))
> >+
> >+
On Mon, Mar 09, 2015 at 11:09:13AM +, Ian Campbell wrote:
> On Mon, 2015-03-09 at 13:14 +1000, Edgar E. Iglesias wrote:
> > On Fri, Mar 06, 2015 at 09:44:16AM +, Ian Campbell wrote:
> > > On Fri, 2015-03-06 at 11:31 +1000, Edgar E. Iglesias wrote:
> > > > On Thu, Mar 05, 2015 at 04:50:15PM
On Mon, Mar 9, 2015 at 8:55 AM, Uma Sharma wrote:
> This patch inserts runqueue_per_core code.
> And also makes generic selection for runqueue by using boot paarmeter.
>
> Signed-off-by: Uma Sharma
Thanks Uma! Exciting to see this.
A couple of comments below.
> ---
>
> diff --git a/xen/common
>>> On 06.03.15 at 22:43, wrote:
> @@ -60,6 +61,26 @@ extern int ss_initialized;
>
> extern struct xsm_operations *original_ops;
>
> +static void __init parse_flask_param(char *s)
> +{
> +if ( !strcmp(s, "force") || !strcmp(s, "enforcing") )
So what's the point of allowing two values wit
On Fri, 6 Mar 2015, David Vrabel wrote:
> Make the IOCTL_PRIVCMD_MMAPBATCH_V2 (and older V1 version) map
> multiple frames at a time rather than one at a time, despite the pages
> being non-consecutive GFNs.
>
> xen_remap_foreign_mfn_array() is added which maps an array of GFNs
> (instead of a con
On 03/09/2015 07:22 PM, Roger Pau Monné wrote:
> El 09/03/15 a les 12.02, David Vrabel ha escrit:
>> On 09/03/15 10:51, Bob Liu wrote:
>>>
>>> On 03/09/2015 05:30 PM, David Vrabel wrote:
On 09/03/15 09:09, Bob Liu wrote:
> Hi David,
>
> Recently I met an issue which is likely rela
On Mon, 9 Mar 2015, Wei Liu wrote:
> On Fri, Mar 06, 2015 at 05:19:15PM +, Stefano Stabellini wrote:
> [...]
> > int libxl_wait_for_free_memory(libxl_ctx *ctx, uint32_t domid, uint32_t
> > memory_kb, int wait_secs);
> > -/* wait for the memory target of a domain to be reached */
> > +/*
> >
On Mon, 2015-03-09 at 12:41 +, Stefano Stabellini wrote:
> Given the very limited change in behaviour (in fact existing callers
> should work as-is) I thought that we could do without one. Also see:
> http://marc.info/?l=xen-devel&m=142563879603876&w=2.
I was also basing that on the big commen
If the cfg file includes root= but not extra= (nor cmdline=, which
supercedes both) then the command line will end up with an extra
"(null)" on it (at least with glibc's implementation of asprintf).
Signed-off-by: Ian Campbell
Cc: Chunyan Liu
---
tools/libxl/xl_cmdimpl.c |4 +++-
1 file cha
Hi all
This is version 7 of this series rebased on top of master.
This patch series implements virtual NUMA support for both PV and HVM guest.
That is, admin can configure via libxl what virtual NUMA topology the guest
sees.
This is the stage 1 (basic vNUMA support) and part of stage 2 (vNUMA-wa
Currently all in tree code doesn't set the superpage flag, I would just
remove superpage support if I can, but Konrad wants it retained for the
moment.
As I'm going to change the p2m_host array allocation, duplicate the code
snippet to allocate p2m_host array in this patch, so that we retain the
b
This function gets the machine E820 map and sanitize it according to PV
guest configuration.
This will be used in later patch. No functional change introduced in
this patch.
Signed-off-by: Wei Liu
Reviewed-by: Andrew Cooper
Reviewed-by: Dario Faggioli
Cc: Ian Campbell
Cc: Ian Jackson
Cc: Ele
This function is used to check whether vNUMA configuration (be it
auto-generated or supplied by user) is valid.
Define a new error code ERROR_VNUMA_CONFIG_INVALID.
The checks performed can be found in the comment of the function.
This vNUMA function (and future ones) is placed in a new file call
Add a new field p2m_size to keep track of the number of pages covered by
p2m. Change total_pages to p2m_size in functions which in fact need
the size of p2m.
This is needed because we are going to ditch the assumption that PV x86
has only one contiguous ram region. Originally the p2m size was alw
A vnode consists of one or more vmemranges (virtual memory range). One
example of multiple vmemranges is that there is a hole in one vnode.
Currently we haven't exported vmemrange interface to libxl user.
Vmemranges are generated during domain build, so we have relevant
structures in domain build
Make XENMEM_increase_reservation and XENMEM_populate_physmap
vNUMA-aware.
That is, if guest requests Xen to allocate memory for specific vnode,
Xen can translate vnode to pnode using vNUMA information of that guest.
XENMEMF_vnode is introduced for the guest to mark the node number is in
fact virt
Introduce a arch-independent routine to generate one vmemrange per
vnode. Also introduce arch-dependent routines for different
architectures because part of the process is arch-specific -- ARM has
yet have NUMA support and E820 is x86 only.
For those x86 guests who care about machine E820 map (i.e
A domain can contain several virtual NUMA nodes, hence we introduce an
array in libxl_domain_build_info.
libxl_vnode_info contains the size of memory in that node, the distance
from that node to every nodes, the underlying pnode and a bitmap of
vcpus.
Signed-off-by: Wei Liu
Reviewed-by: Dario Fa
>From libxc's point of view, it only needs to know vnode to pnode mapping
and size of each vnode to allocate memory accordingly. Add these fields
to xc_dom structure.
The caller might not pass in vNUMA information. In that case, a dummy
layout is generated for the convenience of libxc's allocation
Transform the user supplied vNUMA configuration into libxl internal
representations, and finally libxc representations. Check validity of
the configuration along the line.
Signed-off-by: Wei Liu
Reviewed-by: Dario Faggioli
Cc: Ian Campbell
Cc: Ian Jackson
Cc: Dario Faggioli
Cc: Elena Ufimtsev
Signed-off-by: Wei Liu
Reviewed-by: Dario Faggioli
Cc: Ian Campbell
Cc: Ian Jackson
---
Changes in v6:
1. Better description of the macro.
---
tools/libxl/libxl.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h
index e3d2ae8..97a7c9c 100644
The algorithm is more or less the same as the one used for PV guest.
Libxc gets hold of the mapping of vnode to pnode and size of each vnode
then allocate memory accordingly.
And then the function returns low memory end, high memory end and mmio
start to caller. Libxl needs those values to constru
1. Extend grammar of parser.
2. Adjust internal functions to accept XLU_ConfigValue instead of
char *.
Signed-off-by: Wei Liu
Cc: Ian Jackson
Cc: Ian Campbell
Acked-by: Ian Jackson
---
tools/libxl/libxlu_cfg.c | 30 +++---
tools/libxl/libxlu_cfg_i.h | 5 +++--
to
Move a while loop in xc_hvm_build_x86 one block to the right. No
functional change introduced.
Functional changes will be introduced in next patch.
Signed-off-by: Wei Liu
Cc: Ian Campbell
Cc: Ian Jackson
Cc: Dario Faggioli
Cc: Elena Ufimtseva
Acked-by: Ian Campbell
---
tools/libxc/xc_hvm_b
These APIs can be used to manipulate XLU_ConfigValue and XLU_ConfigList.
APIs introduced:
1. xlu_cfg_value_type
2. xlu_cfg_value_get_string
3. xlu_cfg_value_get_list
4. xlu_cfg_get_listitem2
Move some definitions from private header to public header as needed.
Signed-off-by: Wei Liu
Cc: Ian Jac
This patches does following things:
1. Properly define a XLU_ConfigList type. Originally it was defined to
be XLU_ConfigSetting.
2. Define XLU_ConfigValue type, which can be either a string or a list
of XLU_ConfigValue.
3. ConfigSetting now references XLU_ConfigValue. Originally it only
w
Signed-off-by: Wei Liu
Cc: Ian Campbell
Cc: Ian Jackson
Acked-by: Ian Campbell
---
Changes in v6:
1. Join two lines to make code more compact.
2. Use %zu and drop casting.
---
tools/libxl/xl_cmdimpl.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/tools/libxl/xl_cmdimpl.c b/to
This patch includes configuration options parser and documentation.
Please find the hunk to xl.cfg.pod.5 for more information.
Signed-off-by: Wei Liu
Cc: Ian Campbell
Cc: Ian Jackson
---
Changes in v7:
1. Fill in max_memkb with vNUMA memory settings.
2. Check vcpus specified in vnuma matches m
Originally only setting has line number recorded. Since we're moving to
more sophisticated API, record the location for individual value. It is
useful for error reporting.
Signed-off-by: Wei Liu
Cc: Ian Campbell
Cc: Ian Jackson
---
Changes in v7:
1. Use yylloc in empty rule.
1. Use YYLTYPE inst
Transform user supplied vNUMA configuration into libxl internal
representations then libxc representations. Check validity along the
line.
Libxc has more involvement in building vmemranges in HVM case compared
to PV case. The building of vmemranges is placed after xc_hvm_build
returns, because it
Disallow memory relocation when vNUMA is enabled, because relocated
memory ends up off node. Further more, even if we dynamically expand
node coverage in hvmloader, low memory and high memory may reside
in different physical nodes, blindly relocating low memory to high
memory gives us a sub-optimal
On Tue, Mar 3, 2015 at 5:13 PM, Julien Grall wrote:
> Hello Vijay,
>
> On 03/03/2015 03:55, Vijay Kilari wrote:
>>
>> On Mon, Mar 2, 2015 at 6:49 PM, Julien Grall
>> wrote:
>>>
>>> On 02/03/15 12:30, vijay.kil...@gmail.com wrote:
From: Vijaya Kumar K
Add ITS support for arm.
On Mon, Mar 09, 2015 at 12:41:25PM +, Stefano Stabellini wrote:
> On Mon, 9 Mar 2015, Wei Liu wrote:
> > On Fri, Mar 06, 2015 at 05:19:15PM +, Stefano Stabellini wrote:
> > [...]
> > > int libxl_wait_for_free_memory(libxl_ctx *ctx, uint32_t domid, uint32_t
> > > memory_kb, int wait_secs);
On Mon, Mar 09, 2015 at 12:48:56PM +, Ian Campbell wrote:
> If the cfg file includes root= but not extra= (nor cmdline=, which
> supercedes both) then the command line will end up with an extra
> "(null)" on it (at least with glibc's implementation of asprintf).
>
> Signed-off-by: Ian Campbell
Thank you :-)
Sure I have no problem, you can add second patch of this series.
I will work on the comments and will again resend the patch. Thank you.
Regards,
Uma Sharma
On Mon, Mar 9, 2015 at 3:57 PM, Dario Faggioli
wrote:
> Hi Uma,
>
> First of all, nice to see the patch here on xen-devel.
On Mon, 2015-03-09 at 12:18 +, George Dunlap wrote:
> On Mon, Mar 9, 2015 at 8:55 AM, Uma Sharma wrote:
> > --- a/xen/common/sched_credit2.c
> > +++ b/xen/common/sched_credit2.c
> > @@ -1935,15 +1938,36 @@ static void init_pcpu(const struct scheduler *ops,
> > int cpu)
> > return;
On Tue, Mar 3, 2015 at 10:58 PM, Stefano Stabellini
wrote:
> On Mon, 2 Mar 2015, vijay.kil...@gmail.com wrote:
>> From: Vijaya Kumar K
>>
>> This patch implements hw_irq_controller api's required
>> to handle LPI's.
>>
>> Signed-off-by: Vijaya Kumar K
>> ---
>> xen/arch/arm/gic-v3-its.c | 104
Thank you :-)
I will work on the things you mentioned and resend the patch.
It's great to work on patches. I was trying to figure out how to
change the code so it looks neat and now I have the answer. Thank you.
:-)
I will summarize the performance in cover patch.
Regards,
Uma Sharma
On Mon, Ma
On 09/03/15 12:25, Stefano Stabellini wrote:
> On Fri, 6 Mar 2015, David Vrabel wrote:
>> Make the IOCTL_PRIVCMD_MMAPBATCH_V2 (and older V1 version) map
>> multiple frames at a time rather than one at a time, despite the pages
>> being non-consecutive GFNs.
>>
>> xen_remap_foreign_mfn_array() is ad
>>> On 09.03.15 at 13:51, wrote:
> @@ -759,6 +784,14 @@ long do_memory_op(unsigned long cmd,
> XEN_GUEST_HANDLE_PARAM(void) arg)
> return start_extent;
> args.domain = d;
>
> +if ( construct_memop_from_reservation(d, &reservation, &args) )
> +{
> +
>>> On 09.03.15 at 07:59, wrote:
> From: Vijaya Kumar K
>
> On x86, for the pages mapped with PAGE_HYPERVISOR attribute
> non-leaf page tables are allocated with valid pte entries.
> and with MAP_SMALL_PAGES attribute only non-leaf page tables are
> allocated with invalid (valid bit set to 0) pt
On Mon, Mar 09, 2015 at 01:22:55PM +, Jan Beulich wrote:
> >>> On 09.03.15 at 13:51, wrote:
> > @@ -759,6 +784,14 @@ long do_memory_op(unsigned long cmd,
> > XEN_GUEST_HANDLE_PARAM(void) arg)
> > return start_extent;
> > args.domain = d;
> >
> > +if ( construct
1 - 100 of 185 matches
Mail list logo
