On Wed, Jan 07, Ian Campbell wrote:
> On Wed, 2015-01-07 at 16:49 +, Ian Jackson wrote:
> > Certainly removing this feature this late in the 4.5 release cycle is
> > not appropriate.
>
> I agree that faffing around with the initscripts/systemd units at the
> eleventh hour seem liable to leave
> -Original Message-
> From: Pang, LongtaoX
> Sent: Wednesday, January 07, 2015 11:53 AM
> To: 'Wei Liu'
> Cc: xen-devel@lists.xen.org; ian.jack...@eu.citrix.com;
> ian.campb...@citrix.com; Hu, Robert; Zheng, Di
> Subject: RE: [OSSTEST PATCH 3/4] Add nested testcase of installing L2 guest
On 2015/1/7 23:44, Konrad Rzeszutek Wilk wrote:
> On Wed, Jan 07, 2015 at 11:37:52PM +0800, Jiang Liu wrote:
>> On 2015/1/7 22:50, Konrad Rzeszutek Wilk wrote:
>>> On Wed, Jan 07, 2015 at 02:13:49PM +0800, Jiang Liu wrote:
Commit b81975eade8c ("x86, irq: Clean up irqdomain transition code")
>>
flight 33227 xen-4.2-testing real [real]
http://www.chiark.greenend.org.uk/~xensrcts/logs/33227/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-i386-xend-qemut-winxpsp3 5 xen-boot fail REGR. vs. 32291
test-amd64-amd64-
flight 33241 libvirt real [real]
http://www.chiark.greenend.org.uk/~xensrcts/logs/33241/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
build-armhf 3 host-install(3) broken REGR. vs. 32648
build-i386-libvirt
On Wed, Jan 07, 2015 at 09:43:44PM +, Ian Jackson wrote:
> Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to
> get CMT L3 event mask"):
> > Other culprits are xc_get_max_nodes(), xc_get_max_cpus(), 4 instances in
> > xc_psr.c and most things in xc_offline_page.c which
Hi,
I am trying to test the single-queue networking performance for
netback/netfront in upstream, my testing environment is as follows:
1. Using pkt-gen to send a single UDP flow from one host to a vm which runs
on another XEN host. Two hosts are connected with 10GE network (Intel 82599
On 08/01/2015 01:22, Christian Refvik wrote:
> Hello,
>
> My name is Christian and I am new to Xen development, and I've been
> struggling a bit. I'm trying to develop a device driver so that a Windows
> 2012 Server VM has a way to send about 10MB of data to a CentOS VM. There
> is no real de
branch xen-unstable
xen branch xen-unstable
job test-amd64-i386-xl-qemuu-winxpsp3
test windows-install
Tree: linux git://xenbits.xen.org/linux-pvops.git
Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git
Tree: qemuu g
Hello,
My name is Christian and I am new to Xen development, and I've been struggling
a bit. I'm trying to develop a device driver so that a Windows 2012 Server VM
has a way to send about 10MB of data to a CentOS VM. There is no real device
on the backend, I just need a way to constantly sen
On Wednesday, January 07, 2015 09:38:31 AM Ian Campbell wrote:
> That's exactly what I was about to suggest as I read the penultimate
> paragraph, i.e. keep waiting so long as some reasonable delta occurs on
> each iteration.
Thanks, Ian.
I wonder if there is a future-safe threshold on the amount
On Wed, Jan 7, 2015 at 2:51 AM, Jan Beulich wrote:
On 06.01.15 at 23:20, wrote:
>> On Mon, Jan 5, 2015 at 3:44 AM, Jan Beulich wrote:
>>> >>> Elena Ufimtseva 01/02/15 7:32 PM >>>
>>> >The last successful command is the reading status register of second IOMMU
>>> >unit:
>>> >
>>> >>> >./xen
Ping in case this mail is hidden after long holiday. :-)
> From: Tian, Kevin
> Sent: Friday, December 26, 2014 7:23 PM
>
> (please note some proposal is different from last sent version after more
> discussions. But I tried to summarize previous discussions and explained why
> we choose a differe
> From: Jan Beulich [mailto:jbeul...@suse.com]
> Sent: Wednesday, January 07, 2015 6:16 PM
>
> >>> On 23.12.14 at 07:52, wrote:
> >> From: Jan Beulich [mailto:jbeul...@suse.com]
> >> Sent: Friday, December 19, 2014 7:26 PM
> >>
> >> This can (and will) be legitimately the case when sharing page
On Tue, Jan 6, 2015 at 10:39 PM, Zhang, Yang Z
wrote:
> Elena Ufimtseva wrote on 2015-01-07:
> >
> >
> > On Mon, Jan 5, 2015 at 7:53 AM, Zhang, Yang Z
> > wrote:
> >
> >
> > Jan Beulich wrote on 2015-01-05:
> >
> > Elena Ufimtseva 01/02/15 7:32 PM >>>
>>> The
> > last s
Yea, I debugged through Harper's code and it seems like the error
response is coming from XenStore on Dom0. Ive been looking at it, and
OXenStored is running on Dom0. Of all the languages possible, they
decided to write xenstore in OCaml. I am not very good at OCaml and
dont know anyone that is, so
Wei Liu wrote:
> Jim, another idea: if those strings are likely to be wrong and in fact
> not used, can we just not print them?
>
IMO they are useful, particularly when they are correct :-). They allow
users to see which emulators are available and their complete path,
which in turn can be dir
Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get
CMT L3 event mask"):
> Other culprits are xc_get_max_nodes(), xc_get_max_cpus(), 4 instances in
> xc_psr.c and most things in xc_offline_page.c which appears to have
> static structures for domain context. The "plugga
On 16/12/14 19:33, Mihai Donțu wrote:
> Implemented xmem_pool_check(), xmem_pool_check_locked() and
> xmem_pool_check_unlocked() to verity the integrity of the TLSF matrix.
>
> Signed-off-by: Mihai Donțu
This review supersedes (and is adjusted accordingly for) the two
discussion threads which hap
With recent changes in p2m we now have legitimate cases when
p2m memory needs to be freed during early boot (i.e. before
slab is initialized).
Signed-off-by: Boris Ostrovsky
Reviewed-by: Juergen Gross
---
v2: Add __ref annotation
arch/x86/xen/p2m.c | 9 ++---
1 file changed, 6 insertions(
HVM guests have always been confined to using the domain callback
via (see HVM_PARAM_CALLBACK_IRQ) to receive event notifications.
This is usually an IOAPIC vector and is only used if the event
channel is bound to vcpu 0.
PVHVM Linux uses a pre-defined interrupt vector for the event
channel upcall
On Wed, 2015-01-07 at 10:54 -0500, Boris Ostrovsky wrote:
> On 01/07/2015 10:17 AM, Jan Beulich wrote:
> >> This is the same information (pxm -> node mapping ) that we provide in
> >> XEN_SYSCTL_topologyinfo (renamed in this series to
> >> XEN_SYSCTL_cputopoinfo). Given that I expect the two topolo
On Wed, 2015-01-07 at 16:49 +, Ian Jackson wrote:
> Certainly removing this feature this late in the 4.5 release cycle is
> not appropriate.
I agree that faffing around with the initscripts/systemd units at the
eleventh hour seem liable to leave us with a release where xenstored
doesn't start
On Tue, Jan 06, 2015 at 06:57:27PM +, David Vrabel wrote:
> From: Jenny Herbert
>
> The foreign page flag will be used by Xen guests to mark pages that
> have grant mappings of frames from other (foreign) guests.
>
> The foreign flag is an aliases the existing (Xen-specific) pinned
is an al
On 07/01/15 16:37, Ian Jackson wrote:
> Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to
> get CMT L3 event mask"):
>> On 07/01/15 11:12, Chao Peng wrote:
>>> +int xc_psr_cmt_get_l3_event_mask(xc_interface *xch, uint32_t *event_mask)
>>> +{
>>> +static int val = 0;
>
On Wed, 2015-01-07 at 16:37 +, Ian Jackson wrote:
> > As the result of the hypercall is going to be the same, the
> > worse that a race could achieve is a wasted hypercall.
>
> This kind of analysis is unfounded in the presence of modern compilers
> with aggressive optimisations. At the very
On 07/01/15 16:47, Ian Campbell wrote:
> On Wed, 2015-01-07 at 16:42 +, Julien Grall wrote:
>>> +
>>> +if ( chosen )
>>> +{
>>> +ret = dt_property_read_string(chosen, "stdout-path", &stdout);
>>> +if ( ret >= 0 )
>>> +{
>>> +pr
On 01/07/2015 11:16 AM, Imre Palik wrote:
From: "Palik, Imre"
In Dom0's the use of the TSC clocksource (whenever it is stable enough to
be used) instead of the Xen clocksource should not cause any issues, as
Dom0 VMs never live-migrated. The TSC clocksource is somewhat more
efficient than the
On Wed, Jan 07, 2015 at 04:52:00PM +, Ian Jackson wrote:
> Wei Liu writes ("[PATCH 2/2] libxl_internal: comment on domain userdata
> unlock function"):
> > Discuss why we need to unlink file path before closes fd.
>
> Acked-by: Ian Jackson
> subject to minor grammar complaint:
>
> Potential
>>> On 07.01.15 at 17:30, wrote:
> On Wed, 2015-01-07 at 17:16 +0100, Imre Palik wrote:
>> From: "Palik, Imre"
>>
>> In Dom0's the use of the TSC clocksource (whenever it is stable enough to
>> be used) instead of the Xen clocksource should not cause any issues, as
>> Dom0 VMs never live-migrate
On Wed, Jan 07, 2015 at 04:37:40PM +, Ian Jackson wrote:
> Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to
> get CMT L3 event mask"):
> > On 07/01/15 11:12, Chao Peng wrote:
> > > +int xc_psr_cmt_get_l3_event_mask(xc_interface *xch, uint32_t *event_mask)
> > > +{
>
>>> On 07.01.15 at 16:54, wrote:
> On 01/07/2015 10:17 AM, Jan Beulich wrote:
>> I personally think that exposing e.g. the firmware
>> determined (and hence hopefully stable across reboots) PXM would
>> be more reasonable.
>
> Again, the main argument that I see against using PXM values directly
On Wed, 2015-01-07 at 16:33 +, Ian Jackson wrote:
> for
> form's sake I don't want to just say "+1" without giving others an
> opportunity to (self-)nominate.
I'd assumed I could change my vote any time up to the deadline, but it's
also true that an existing +1 vote might have a chilling effec
Wei Liu writes ("[PATCH 1/2] libxl_internal: lock_carefd -> carefd"):
> lock_ prefix is redundant.
Acked-by: Ian Jackson
(for 4.6)
___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
Olaf Hering writes ("[PATCH 1/2] tools/hotplug: introduce XENSTORED_ARGS= in
sysconfig file."):
> It is already used in the runlevel script and the service file.
> It is supposed to replace XENSTORED_TRACE= boolean, which cant be easily
> supported in the xenstored.service file.
I don't think it
Wei Liu writes ("[PATCH 2/2] libxl_internal: comment on domain userdata unlock
function"):
> Discuss why we need to unlink file path before closes fd.
Acked-by: Ian Jackson
subject to minor grammar complaint:
Potential backport candidate for 4.5.1 ?
> diff --git a/tools/libxl/libxl_internal.c
On Wed, 2015-01-07 at 16:42 +, Julien Grall wrote:
> > +
> > +if ( chosen )
> > +{
> > +ret = dt_property_read_string(chosen, "stdout-path", &stdout);
> > +if ( ret >= 0 )
> > +{
> > +printk("Taking dtuart configuration from
>
On Wed, 2015-01-07 at 17:37 +0100, Olaf Hering wrote:
> It is already used in the runlevel script and the service file.
> It is supposed to replace XENSTORED_TRACE= boolean, which cant be easily
> supported in the xenstored.service file.
>
> Signed-off-by: Olaf Hering
> Cc: Ian Jackson
> Cc: Ste
Hi Ian,
On 07/01/15 15:31, Ian Campbell wrote:
> ePAPR v1.1 section 3.5 defines the /chosen/stdout-path property to
> refer to the device to be used for boot console output, so if no
> dtuart property is given try to use that instead. This will make Xen
> find a suitable console by default on DT p
Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get
CMT L3 event mask"):
> On 07/01/15 11:12, Chao Peng wrote:
> > +int xc_psr_cmt_get_l3_event_mask(xc_interface *xch, uint32_t *event_mask)
> > +{
> > +static int val = 0;
>
> This should be uint32_t rather than int
As suggested by IanC:
With these changes the existing XENSTORED_TRACE= sysconfig variable is
removed, in favor of a new XENSTORED_ARGS= variable. This variable is
already used in the code, but it was never put into the sysconfig files.
All this is to remove the illusion that XENSTORED_TRACE= will
It is already used in the runlevel script and the service file.
It is supposed to replace XENSTORED_TRACE= boolean, which cant be easily
supported in the xenstored.service file.
Signed-off-by: Olaf Hering
Cc: Ian Jackson
Cc: Stefano Stabellini
Cc: Ian Campbell
Cc: Wei Liu
---
tools/hotplug/L
The existing XENSTORED_TRACE= boolean cant be easily implemented in the
xenstored.service file. All additional options should be added to the
XENSTORED_ARGS= variable.
Signed-off-by: Olaf Hering
Cc: Ian Jackson
Cc: Stefano Stabellini
Cc: Ian Campbell
Cc: Wei Liu
---
tools/hotplug/Linux/init.
On Wed, 2015-01-07 at 16:31 +, Julien Grall wrote:
> On 07/01/15 15:31, Ian Campbell wrote:
> > I've retained the handling of ',' for compatibility, but I'm almost
> > inclined to just drop it, if not now then in a release or two.
>
> None of the current drivers support options. So I think we
Ian Campbell writes ("Re: [Xen-devel] Nominations for Xen 4.5 stable tree
maintainer."):
> I'm not sure how voting is supposed to proceed with multiple nominations
> (and with the deadline for nominations apparently being the same as for
> voting), but given that Jan has thrown his hat in the ring
On Wed, 2015-01-07 at 17:16 +0100, Imre Palik wrote:
> From: "Palik, Imre"
>
> In Dom0's the use of the TSC clocksource (whenever it is stable enough to
> be used) instead of the Xen clocksource should not cause any issues, as
> Dom0 VMs never live-migrated.
Is this still true given that dom0's
On 07/01/15 15:31, Ian Campbell wrote:
> I've retained the handling of ',' for compatibility, but I'm almost
> inclined to just drop it, if not now then in a release or two.
None of the current drivers support options. So I think we could drop
the support of ",".
Backport: I don't think #2 is req
flight 33192 seabios real [real]
http://www.chiark.greenend.org.uk/~xensrcts/logs/33192/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-i386-xl-qemut-winxpsp3 5 xen-boot fail REGR. vs. 32830
test-amd64-amd64-xl-qemuu
From: "Palik, Imre"
In Dom0's the use of the TSC clocksource (whenever it is stable enough to
be used) instead of the Xen clocksource should not cause any issues, as
Dom0 VMs never live-migrated. The TSC clocksource is somewhat more
efficient than the Xen paravirtualised clocksource, thus it sho
Hi Ian,
On 07/01/15 15:31, Ian Campbell wrote:
> - Don't log at all if console=dtuart (the default) was not present, in
> that case the user has asked for something else, no need for every
> other driver to tell them this.
> - Use "dtuart" in all other messages, rather than just "console" or
>
Hi Ian,
On 07/01/15 15:31, Ian Campbell wrote:
> Signed-off-by: Ian Campbell
Reviewed-by: Julien Grall
Regards,
> ---
> xen/drivers/char/dt-uart.c | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/xen/drivers/char/dt-uart.c b/xen/drivers/char/dt-uart.c
> index fa92b5c..
On Wed, 2015-01-07 at 10:37 -0500, Konrad Rzeszutek Wilk wrote:
> On Wed, Jan 07, 2015 at 03:32:15PM +, Ian Campbell wrote:
> > The first Xen technical call will be at:
> > Wed 14 Jan 17:00:00 GMT 201
> > `date -d @1421254800`
> >
> > See http://lists.xen.org/archives/html/xen-devel/20
On 01/07/2015 10:17 AM, Jan Beulich wrote:
On 07.01.15 at 15:55, wrote:
On 01/07/2015 04:21 AM, Jan Beulich wrote:
On 06.01.15 at 03:18, wrote:
+for ( ; ti->first_dev < ti->num_devs; ti->first_dev++ )
+{
+xen_sysctl_pcitopo_t pcitopo;
+struct pci_dev *
On 07/01/15 15:41, Jan Beulich wrote:
On 16.12.14 at 20:33, wrote:
>> Implemented xmem_pool_check(), xmem_pool_check_locked() and
>> xmem_pool_check_unlocked() to verity the integrity of the TLSF matrix.
>>
>> Signed-off-by: Mihai Donțu
> Andrew, Julien,
>
> having gone through the discussio
>>> On 07.01.15 at 16:34, wrote:
> On 01/07/2015 10:07 AM, Jan Beulich wrote:
> On 07.01.15 at 15:47, wrote:
>>> On 07/01/15 14:42, Boris Ostrovsky wrote:
I kept this field as an int to be able to store NUMA_NO_NODE which I
thought to be (int)-1.
But now I see that NUMA_NO
On Wed, Jan 07, 2015 at 11:37:52PM +0800, Jiang Liu wrote:
> On 2015/1/7 22:50, Konrad Rzeszutek Wilk wrote:
> > On Wed, Jan 07, 2015 at 02:13:49PM +0800, Jiang Liu wrote:
> >> Commit b81975eade8c ("x86, irq: Clean up irqdomain transition code")
> >> breaks xen IRQ allocation because xen_smp_prepar
>>> On 07.01.15 at 16:31, wrote:
> On 01/07/2015 10:06 AM, Jan Beulich wrote:
>>
Of course an additional question would be whether the node wouldn't
better go into struct arch_pci_dev - that depends on whether we
expect ARM to be using NUMA...
>>> Since we have CPU topology in commo
On Wed, Jan 07, 2015 at 03:27:15PM +, Ian Jackson wrote:
> Olaf Hering writes ("Re: [PATCH 7/7] tools/hotplug: add wrapper to start
> xenstored"):
> > If I recall correctly the point of the current 'sh -c "exec ..."' stunt
> > was to expand the XENSTORE variable from the sysconfig file. But th
>>> On 16.12.14 at 20:33, wrote:
> Implemented xmem_pool_check(), xmem_pool_check_locked() and
> xmem_pool_check_unlocked() to verity the integrity of the TLSF matrix.
>
> Signed-off-by: Mihai Donțu
Andrew, Julien,
having gone through the discussion following this patch submission
once again j
Signed-off-by: Ian Campbell
---
xen/drivers/char/dt-uart.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/xen/drivers/char/dt-uart.c b/xen/drivers/char/dt-uart.c
index fa92b5c..45a87a6 100644
--- a/xen/drivers/char/dt-uart.c
+++ b/xen/drivers/char/dt-uart.c
@@ -70,3 +70,13 @@ v
On 01/07/2015 10:06 AM, Jan Beulich wrote:
Of course an additional question would be whether the node wouldn't
better go into struct arch_pci_dev - that depends on whether we
expect ARM to be using NUMA...
Since we have CPU topology in common code I thought this would be
arch-independent as we
On 2015/1/7 22:50, Konrad Rzeszutek Wilk wrote:
> On Wed, Jan 07, 2015 at 02:13:49PM +0800, Jiang Liu wrote:
>> Commit b81975eade8c ("x86, irq: Clean up irqdomain transition code")
>> breaks xen IRQ allocation because xen_smp_prepare_cpus() doesn't invoke
>> setup_IO_APIC(), so no irqdomains create
On Wed, Jan 07, 2015 at 03:32:15PM +, Ian Campbell wrote:
> The first Xen technical call will be at:
> Wed 14 Jan 17:00:00 GMT 201
> `date -d @1421254800`
>
> See http://lists.xen.org/archives/html/xen-devel/2015-01/msg00414.html
> for more information on the call.
>
> Please let me k
On 01/07/2015 10:07 AM, Jan Beulich wrote:
On 07.01.15 at 15:47, wrote:
On 07/01/15 14:42, Boris Ostrovsky wrote:
I kept this field as an int to be able to store NUMA_NO_NODE which I
thought to be (int)-1.
But now I see that NUMA_NO_NODE is, in fact, 0xff but is promoted to
(int)-1 by pxm_to_
On Wed, Jan 07, 2015 at 10:27:37AM -0500, Konrad Rzeszutek Wilk wrote:
> >
> > And I think there was optimizations to oxenstored to support up to 1000 VMs
> > per host..
>
> .. and that.
>
Actually I'm not sure anymore about "up to 1000 VMs per host".. I was certain I
was reading about such n
The first Xen technical call will be at:
Wed 14 Jan 17:00:00 GMT 201
`date -d @1421254800`
See http://lists.xen.org/archives/html/xen-devel/2015-01/msg00414.html
for more information on the call.
Please let me know (CC-ing the list) any topics which you would like to
discuss. It might be
',' is a valid character in a device-tree path (see ePAPR v1.1 Table
2-1), in fact ',' is actually pretty common in node names.
Using ',' as a separator breaks for example on fast models. If you use
the full path (/smb/motherboard/iofpga@3,/uart@09) rather
than the alias then earlyprin
ePAPR v1.1 section 3.5 defines the /chosen/stdout-path property to
refer to the device to be used for boot console output, so if no
dtuart property is given try to use that instead. This will make Xen
find a suitable console by default on DT platforms which include this
property.
As it happens the
- Don't log at all if console=dtuart (the default) was not present, in
that case the user has asked for something else, no need for every
other driver to tell them this.
- Use "dtuart" in all other messages, rather than just "console" or
"uart".
- Be more explicit if we are exiting because dt
Before I put on my air tanks and go looking in my QUEUE-4.6 email folder
I wanted to start the year by doing some actual programming, and this
seemed like an afternoons hacking...
The two main changes here are:
* a bugfix to deal with DT paths which contain a common, which are
perfe
The x86 struct arch_shared_info field pfn_to_mfn_frame_list_list
currently contains the mfn of the top level page frame of the 3 level
p2m tree, which is used by the Xen tools during saving and restoring
(and live migration) of pv domains and for crash dump analysis. With
three levels of the p2m tr
On Tue, Jan 06, 2015 at 09:14:22PM +0200, Pasi Kärkkäinen wrote:
> On Tue, Jan 06, 2015 at 01:21:58PM -0500, Konrad Rzeszutek Wilk wrote:
> > On Tue, Jan 06, 2015 at 06:06:23PM +, Ian Jackson wrote:
> > > Konrad Rzeszutek Wilk writes ("[Xen-devel] [PATCH] README, xen/Makefile:
> > > Update to
Olaf Hering writes ("Re: [PATCH 7/7] tools/hotplug: add wrapper to start
xenstored"):
> If I recall correctly the point of the current 'sh -c "exec ..."' stunt
> was to expand the XENSTORE variable from the sysconfig file. But this
> approach leads to failures with SELinux because the socket passi
The x86 struct arch_shared_info field pfn_to_mfn_frame_list_list
currently contains the mfn of the top level page frame of the 3 level
p2m tree, which is used by the Xen tools during saving and restoring
(and live migration) of pv domains and for crash dump analysis. With
three levels of the p2m tr
On 07/01/15 15:18, Juergen Gross wrote:
> On 01/07/2015 03:47 PM, David Vrabel wrote:
>> The page table walk is only needed to distinguish between identity and
>> missing, both of which have INVALID_P2M_ENTRY.
>
> As get_phys_to_machine is called by __pfn_to_mfn() only which already
> checks for m
>>> On 06.01.15 at 14:41, wrote:
> On 06/01/15 02:18, Boris Ostrovsky wrote:
>> Instead of copying data for each field in xen_sysctl_topologyinfo separately
>> put cpu/socket/node into a single structure and do a single copy for each
>> processor.
>>
>> There is also no need to copy whole op to us
Discuss why we need to unlink file path before closes fd.
Signed-off-by: Wei Liu
Cc: Ian Campbell
Cc: Ian Jackson
---
tools/libxl/libxl_internal.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/tools/libxl/libxl_internal.c b/tools/libxl/libxl_internal.c
index 9d8025d..a7
lock_ prefix is redundant.
Signed-off-by: Wei Liu
Cc: Ian Campbell
Cc: Ian Jackson
---
tools/libxl/libxl_internal.c |6 +++---
tools/libxl/libxl_internal.h |2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/libxl/libxl_internal.c b/tools/libxl/libxl_internal.c
These changes were requested by Ian Campbell when relevant patches were posted
but they didn't warrant a resend at that time.
Wei Liu (2):
libxl_internal: lock_carefd -> carefd
libxl_internal: comment on domain userdata unlock function
tools/libxl/libxl_internal.c | 20 +---
On 01/07/2015 10:10 AM, David Vrabel wrote:
On 07/01/15 14:08, Boris Ostrovsky wrote:
With recent changes in p2m we now have legitimate cases when
p2m memory needs to be freed during early boot (i.e. before
slab is initialized).
Signed-off-by: Boris Ostrovsky
Applied to to stable/for-linus-3.
>>> On 07.01.15 at 15:55, wrote:
> On 01/07/2015 04:21 AM, Jan Beulich wrote:
> On 06.01.15 at 03:18, wrote:
>>> +for ( ; ti->first_dev < ti->num_devs; ti->first_dev++ )
>>> +{
>>> +xen_sysctl_pcitopo_t pcitopo;
>>> +struct pci_dev *pdev;
>>> +
>>> +
On 01/07/2015 03:47 PM, David Vrabel wrote:
The page table walk is only needed to distinguish between identity and
missing, both of which have INVALID_P2M_ENTRY.
As get_phys_to_machine is called by __pfn_to_mfn() only which already
checks for mfn == INVALID_P2M_ENTRY this optimization will have
On 07/01/15 14:08, Boris Ostrovsky wrote:
> With recent changes in p2m we now have legitimate cases when
> p2m memory needs to be freed during early boot (i.e. before
> slab is initialized).
>
> Signed-off-by: Boris Ostrovsky
Applied to to stable/for-linus-3.19, thanks.
If I understand correctl
On 01/07/2015 03:47 PM, David Vrabel wrote:
If the non-RAM regions in the e820 memory map are larger than the size
of the initial balloon, a BUG was triggered as the frames are remaped
beyond the limit of the linear p2m. The frames are remapped into the
initial balloon area (xen_extra_mem) but n
On 01/07/2015 03:47 PM, David Vrabel wrote:
This accounting is just used to print a diagnostic message that isn't
very useful.
Signed-off-by: David Vrabel
Reviewed-by: Juergen Gross
---
arch/x86/xen/setup.c | 27 +--
1 file changed, 9 insertions(+), 18 deletion
>>> On 07.01.15 at 15:45, wrote:
> On 01/07/2015 04:12 AM, Jan Beulich wrote:
> On 06.01.15 at 14:41, wrote:
>>> On 06/01/15 02:18, Boris Ostrovsky wrote:
Instead of copying data for each field in xen_sysctl_topologyinfo
separately
put cpu/socket/node into a single structure a
>>> On 07.01.15 at 15:47, wrote:
> On 07/01/15 14:42, Boris Ostrovsky wrote:
>> I kept this field as an int to be able to store NUMA_NO_NODE which I
>> thought to be (int)-1.
>>
>> But now I see that NUMA_NO_NODE is, in fact, 0xff but is promoted to
>> (int)-1 by pxm_to_node(). Given that there is
>>> On 07.01.15 at 15:42, wrote:
> On 01/07/2015 04:06 AM, Jan Beulich wrote:
> On 06.01.15 at 03:18, wrote:
>>> --- a/xen/include/xen/pci.h
>>> +++ b/xen/include/xen/pci.h
>>> @@ -56,6 +56,8 @@ struct pci_dev {
>>>
>>> u8 phantom_stride;
>>>
>>> +int node; /* NUMA node */
>>
On Wed, Jan 07, Konrad Rzeszutek Wilk wrote:
> On Wed, Jan 07, 2015 at 10:53:06AM +0100, Olaf Hering wrote:
> > Its my understanding that the reported SELinux failure is not only
> > related to the context= mount option, but also to the socket passing
> > from systemd.
>
> I couldn't spot any err
On Wed, Jan 07, 2015 at 10:15:39AM +, Jan Beulich wrote:
> >>> On 23.12.14 at 07:52, wrote:
> >> From: Jan Beulich [mailto:jbeul...@suse.com]
> >> Sent: Friday, December 19, 2014 7:26 PM
> >>
> >> This can (and will) be legitimately the case when sharing page tables
> >> with EPT (more of a
On Wed, Jan 07, 2015 at 10:53:06AM +0100, Olaf Hering wrote:
> On Mon, Jan 05, Konrad Rzeszutek Wilk wrote:
>
> > +Release Issues
> > +==
> > +
> > +While we did the utmost to get a release out, there are certain
> > +fixes which were not complete on time. As such please reference this
On Wed, Jan 07, 2015 at 10:49:38AM +0100, Olaf Hering wrote:
> On Tue, Jan 06, Ian Jackson wrote:
>
> > Olaf Hering writes ("[PATCH 7/7] tools/hotplug: add wrapper to start
> > xenstored"):
> > > The shell wrapper in xenstored.service does not handle XENSTORE_TRACE.
> > ...
> > > +XENSTORED_LIBEX
On 01/07/2015 04:21 AM, Jan Beulich wrote:
On 06.01.15 at 03:18, wrote:
--- a/xen/common/sysctl.c
+++ b/xen/common/sysctl.c
@@ -365,6 +365,66 @@ long do_sysctl(XEN_GUEST_HANDLE_PARAM(xen_sysctl_t)
u_sysctl)
}
break;
#ifdef HAS_PCI
+case XEN_SYSCTL_pcitopoinfo:
+{
+
On Wed, Jan 07, 2015 at 09:31:50AM +, Ian Campbell wrote:
> On Wed, 2015-01-07 at 10:23 +0100, Olaf Hering wrote:
> > On Tue, Jan 06, Ian Campbell wrote:
> >
> > > On Fri, 2014-12-19 at 12:25 +0100, Olaf Hering wrote:
> >
> > ...
> >
> > > Acked-by: Ian Campbell
> > >
> > > (on commit s/Ap
On Wed, Jan 07, 2015 at 02:13:49PM +0800, Jiang Liu wrote:
> Commit b81975eade8c ("x86, irq: Clean up irqdomain transition code")
> breaks xen IRQ allocation because xen_smp_prepare_cpus() doesn't invoke
> setup_IO_APIC(), so no irqdomains created for IOAPICs and
> mp_map_pin_to_irq() fails at the
This accounting is just used to print a diagnostic message that isn't
very useful.
Signed-off-by: David Vrabel
---
arch/x86/xen/setup.c | 27 +--
1 file changed, 9 insertions(+), 18 deletions(-)
diff --git a/arch/x86/xen/setup.c b/arch/x86/xen/setup.c
index dfd77de..66
If the non-RAM regions in the e820 memory map are larger than the size
of the initial balloon, a BUG was triggered as the frames are remaped
beyond the limit of the linear p2m. The frames are remapped into the
initial balloon area (xen_extra_mem) but not enough of this is
available.
Ensure enough
On 07/01/15 14:42, Boris Ostrovsky wrote:
> On 01/07/2015 04:06 AM, Jan Beulich wrote:
> On 06.01.15 at 03:18, wrote:
>>> @@ -618,7 +620,22 @@ ret_t do_physdev_op(int cmd,
>>> XEN_GUEST_HANDLE_PARAM(void) arg)
>>> }
>>> else
>>> pdev_info.is_virtfn = 0;
>>> -
The page table walk is only needed to distinguish between identity and
missing, both of which have INVALID_P2M_ENTRY.
Signed-off-by: David Vrabel
---
arch/x86/xen/p2m.c | 30 ++
1 file changed, 18 insertions(+), 12 deletions(-)
diff --git a/arch/x86/xen/p2m.c b/arc
The linear p2m changes in 3.19-rc1 is broken with some dom0
configurations. While trying to fix it I also noticed that
get_phys_to_machine() could be optimized for the common case.
David
___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.
1 - 100 of 178 matches
Mail list logo
