Re: [Xen-devel] [PATCH] Refactor ioreq server for better performance

2015-06-30 Thread Yu, Zhang
Thanks you, Paul. On 6/29/2015 8:12 PM, Paul Durrant wrote: -Original Message- From: Yu Zhang [mailto:yu.c.zh...@linux.intel.com] Sent: 26 June 2015 11:30 To: xen-de...@lists.xenproject.org; Paul Durrant; Andrew Cooper; jbeul...@suse.com; Kevin Tian; zhiyuan...@intel.com Subject: [PATCH]

[Xen-devel] [PATCH v9 2/8] xen/arm: Add functions of mapping between vCPUID and virtual affinity

2015-06-30 Thread Chen Baozi
From: Chen Baozi GICv3 restricts that the maximum number of CPUs in affinity 0 (one cluster) is 16. (See the note of 'Bits[15:0]' in '5.7.29 ICC_SGI0R_EL1 ICC_SGI1R_EL1 and ICC_ASGI1R_EL1, GICv3 Architecture Specification') That is to say the upper 4 bits of affinity 0 is unused. Current implemen

[Xen-devel] [PATCH v9 0/8] Support more than 8 vcpus on arm64 with GICv3

2015-06-30 Thread Chen Baozi
From: Chen Baozi Currently the number of vcpus on arm64 with GICv3 is limited up to 8 due to the fixed size of redistributor mmio region. Increasing the size makes the number expand to 16 because of AFF0 restriction on GICv3. To create a guest up to 128 vCPUs, which is the maxium number that GIC-

[Xen-devel] [PATCH v9 3/8] xen/arm: Use the new functions for vCPUID/vaffinity transformation

2015-06-30 Thread Chen Baozi
From: Chen Baozi There are 3 places to change: * Initialise vMPIDR value in vcpu_initialise() * Find the vCPU from vMPIDR affinity information when accessing GICD registers in vGIC * Find the vCPU from vMPIDR affinity information when booting with vPSCI in vGIC - Both PSCI 0.1 and PSCI 0.2

[Xen-devel] [PATCH v9 1/8] xen/arm: gic-v3: Increase the size of GICR in address space for guest

2015-06-30 Thread Chen Baozi
From: Chen Baozi Currently it only supports up to 8 vCPUs. Increase the region to hold up to 128 vCPUs, which is the maximum number that GIC-500 supports. Signed-off-by: Chen Baozi Reviewed-by: Julien Grall Acked-by: Ian Campbell --- xen/include/public/arch-arm.h | 4 ++-- 1 file changed, 2

[Xen-devel] [PATCH v9 4/8] xen/arm: Use AFF1 when translating ICC_SGI1R_EL1 to cpumask

2015-06-30 Thread Chen Baozi
From: Chen Baozi The old unsigned long type of vcpu_mask can only express 64 cpus at the most, which might not be enough for the guest which used vGICv3. We introduce a new struct sgi_target for the target cpu list of SGI, which holds the affinity path information (only level 1 at the moment). Fo

[Xen-devel] [PATCH v9 5/8] tools/libxl: Set 'reg' of cpu node equal to MPIDR affinity for domU

2015-06-30 Thread Chen Baozi
From: Chen Baozi According to ARM CPUs bindings, the reg field should match the MPIDR's affinity bits. We will use AFF0 and AFF1 when constructing the reg value of the guest at the moment, for it is enough for the current max vcpu number. Signed-off-by: Chen Baozi Reviewed-by: Julien Grall Ack

[Xen-devel] [PATCH v9 6/8] xen/arm: Set 'reg' of cpu node for dom0 to match MPIDR's affinity

2015-06-30 Thread Chen Baozi
From: Chen Baozi According to ARM CPUs bindings, the reg field should match the MPIDR's affinity bits. We will use AFF0 and AFF1 when constructing the reg value of the guest at the moment, for it is enough for the current max vcpu number. Signed-off-by: Chen Baozi Acked-by: Ian Campbell Review

[Xen-devel] [PATCH v9 8/8] xen/arm64: increase MAX_VIRT_CPUS to 128 on arm64

2015-06-30 Thread Chen Baozi
From: Chen Baozi After we have increased the size of GICR in address space for guest and made use of both AFF0 and AFF1 in (v)MPIDR, we are now able to support up to 4096 vCPUs in theory. However, it will cost 512M address space for GICR region, which is unnecessarily big at the moment. Consideri

[Xen-devel] [PATCH v9 7/8] xen/arm: make domain_max_vcpus return value from vgic_ops

2015-06-30 Thread Chen Baozi
From: Chen Baozi Each vGIC driver supports different maximum numbers of vCPU. For example, GICv2 is limited to 8 vCPUs, while GICv3 can support up to 4096 vCPUs if we use both AFF0 and AFF1. Thus, domain_max_vcpus should depend on not only MAX_VIRT_CPUS but also the version of vGIC that the guest

Re: [Xen-devel] [PATCH v2 00/15] xen/arm: Add support for GICv2 on GICv3

2015-06-30 Thread Chen Baozi
Hi Julien, On Fri, Jun 26, 2015 at 10:34:14AM +0100, Julien Grall wrote: > Hi all, > > This patch series adds support for GICv2 on GICv3. This feature is available > only when the GICv3 hardware is compatible with GICv2. > > When it's the case, the same interface is provided in order to use a >

Re: [Xen-devel] [PATCH v4] run QEMU as non-root

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:06 +0100, Stefano Stabellini wrote: > On Thu, 25 Jun 2015, Ian Campbell wrote: > > On Mon, 2015-06-01 at 17:46 +0100, Stefano Stabellini wrote: > > > Try to use "xen-qemudepriv-$domname" first, then > > > "xen-qemudepriv-domid$domid", finally "xen-qemudepriv-shared" and roo

Re: [Xen-devel] fix building error

2015-06-30 Thread Ian Campbell
On Tue, 2015-06-30 at 14:10 +0800, Wen Congyang wrote: It would be usual do describe here which commit caused the breakage (i.e. removed the parameter). Also please make the subject something more descriptive such as "remove now unnecessary gc from libxl__async_exec_start calls" e.g. Subject: rem

Re: [Xen-devel] [Xen-users] xen physical address(paddr)and machine address (maddr)

2015-06-30 Thread xinyue
Thanks very much,I think it‘s very helpful and I'll try it. Does "for an arbitrary guest page it is unlikely to be from the xenheap" means that some guest pages can't be mapped to xen heap page? And I wonder in HVM doms with EPT supported, is there any difference if I won't monitor guest domain

Re: [Xen-devel] [Xen-users] xen physical address(paddr)and machine address (maddr)

2015-06-30 Thread xinyue
Sorry for the unproper behaviors.  2015-06-29, Mon, 20:57:41 ,Ian Campbell : On Mon, 2015-06-29 at 18:35 +0800, xinyue wrote: Please don't top post. Please have another read of hte Etiquette section of http://wiki.xen.org/wiki/Asking_Xen_Devel_Questions . > Thanks very much,I think it‘s very he

[Xen-devel] [linux-3.14 test] 58978: regressions - FAIL

2015-06-30 Thread osstest service user
flight 58978 linux-3.14 real [real] http://logs.test-lab.xenproject.org/osstest/logs/58978/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-amd64-xsm 6 xen-boot fail REGR. vs. 58861 test-amd64-amd64-xl-qe

Re: [Xen-devel] [v4][PATCH 11/19] tools: introduce some new parameters to set rdm policy

2015-06-30 Thread Ian Campbell
On Tue, 2015-06-30 at 11:08 +0800, Chen, Tiejun wrote: > >> > Having read all these docs I now know what all the options are, but I > still don't really know what I should write. I think an example or two > of real world usage would be helpful. > >>> > >>> Here I picked some code fra

Re: [Xen-devel] [PATCH v3 1/6] libxl: allow /local/domain/$LIBXL_TOOLSTACK_DOMID/device-model/$DOMID to be written by $DOMID

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:52 +0100, Stefano Stabellini wrote: > On Thu, 25 Jun 2015, Ian Campbell wrote: > > On Wed, 2015-06-10 at 11:09 +0100, Stefano Stabellini wrote: > > > The device model is going to restrict its xenstore connection to $DOMID > > > level. > > > > Am I correct in concluding tha

Re: [Xen-devel] [PATCH v3 2/6] libxl: do not add a vkb backend to hvm guests

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:59 +0100, Stefano Stabellini wrote: > On Thu, 25 Jun 2015, Ian Campbell wrote: > > On Tue, 2015-06-16 at 16:39 +0100, Stefano Stabellini wrote: > > > On Tue, 16 Jun 2015, Wei Liu wrote: > > > > On Wed, Jun 10, 2015 at 11:09:50AM +0100, Stefano Stabellini wrote: > > > > > Wh

Re: [Xen-devel] fix building error

2015-06-30 Thread Wen Congyang
On 06/30/2015 04:26 PM, Ian Campbell wrote: > On Tue, 2015-06-30 at 14:10 +0800, Wen Congyang wrote: > > It would be usual do describe here which commit caused the breakage > (i.e. removed the parameter). Also please make the subject something > more descriptive such as "remove now unnecessary gc

[Xen-devel] [PATCH v2] remove now unnecessary gc from libxl__async_exec_start calls

2015-06-30 Thread Wen Congyang
These were removed in commit f5f8400f. Signed-off-by: Wen Congyang --- tools/libxl/libxl_netbuffer.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/libxl/libxl_netbuffer.c b/tools/libxl/libxl_netbuffer.c index ff2d6c7..107e867 100644 --- a/tools/libxl/libxl_netbuff

Re: [Xen-devel] [PATCH v3 3/6] [WIP] libxl: xsrestrict QEMU

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 19:07 +0100, Stefano Stabellini wrote: > On Thu, 25 Jun 2015, Ian Campbell wrote: > > On Wed, 2015-06-10 at 11:09 +0100, Stefano Stabellini wrote: > > > Check whether QEMU supports the xsrestrict option, by parsing its --help > > > output. Store the result on xenstore for futu

Re: [Xen-devel] [OSSTEST PATCH 01/13] ts-leak-check: Report leaked loop block devices

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > These show up as leaked processes `[loop1]' etc. too, but that is an > implementation detail, and it is clearer to show them separately. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell _

Re: [Xen-devel] fix building error

2015-06-30 Thread Ian Jackson
Wen Congyang writes ("Re: fix building error"): > On 06/30/2015 04:26 PM, Ian Campbell wrote: > > On Tue, 2015-06-30 at 14:10 +0800, Wen Congyang wrote: > > > > It would be usual do describe here which commit caused the breakage > > (i.e. removed the parameter). Also please make the subject someth

Re: [Xen-devel] [OSSTEST PATCH 02/13] Test Linux 4.1

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > The kernel.org stable git branch layout is the same for 4.1 as for > 3.x, so simply extend the pattern, and add the branch to BRANCHES. > > DEPLOYMENT NOTE: After this patch is acked, but just before it is > pushed to pretest, run (this versi

Re: [Xen-devel] [OSSTEST PATCH 03/13] Reporting: sg-report-flight: Fix undefined value in $worstrow[]

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > The fallback entry in the worstrow search, used when a job has not > been run at all, was wrong. Also, fix the doc comment which induced > the mistake. > > Signed-off-by: Ian Jackson > --- > sg-report-flight |4 ++-- > 1 file changed,

Re: [Xen-devel] [OSSTEST PATCH 04/13] Reporting: In report_run_getinfo, abstract code for returning

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > This will allow us to change the function's API in one place (plus all > the call sites). > > No functional change in this patch. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xe

Re: [Xen-devel] [PATCH v3 1/6] libxl: allow /local/domain/$LIBXL_TOOLSTACK_DOMID/device-model/$DOMID to be written by $DOMID

2015-06-30 Thread Ian Jackson
Stefano Stabellini writes ("Re: [PATCH v3 1/6] libxl: allow /local/domain/$LIBXL_TOOLSTACK_DOMID/device-model/$DOMID to be written by $DOMID"): > On Thu, 25 Jun 2015, Ian Campbell wrote: > > Am I correct in concluding that only oxenstored supports XS_RESTRICT? I > > don't see it in C xenstored at

Re: [Xen-devel] [OSSTEST PATCH 05/13] Reporting: report_run_getinfo produces ColourAttr

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > This includes the bgcolour attribute name and the quotes. This will > make it possible for this function to sometimes not set a background > for the whole table cell in the future (which will be part of putting > multiple step results in each

Re: [Xen-devel] [OSSTEST PATCH 06/13] Reporting: sg-report-job-history: Honour --limit

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > The global $limit was erroneously shadowed by a local in > processjobbranch. Fix this, and move the erroneously hardcoded value > to the global default. > > No functional change if --limit was not specified, which it is not for > any of our

Re: [Xen-devel] [OSSTEST PATCH 07/13] Reporting: report_run_getinfo produces Content

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > Previously we produced Summary, and the call sites were confused about > whether it had already been html-escaped. Now we produce something > which is explicitly already html. > > No functional change except that some sg-report-host-history

Re: [Xen-devel] [PATCH V5 2/7] libxl_read_file_contents: add new entry to read sysfs file

2015-06-30 Thread Ian Jackson
Chun Yan Liu writes ("Re: [PATCH V5 2/7] libxl_read_file_contents: add new entry to read sysfs file"): > >>> On 6/29/2015 at 06:52 PM, in message > <21905.9050.452111.208...@mariner.uk.xensource.com>, Ian Jackson > wrote: > > Chun Yan Liu writes ("Re: [PATCH V5 2/7] libxl_read_file_contents: add

Re: [Xen-devel] [OSSTEST PATCH 08/13] Reporting: Break out report_blessingscond

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > The returned SQL is a self-contained expression, and does not require > additional bind parameters. To spot SQL quoting problems, we die if > a blessing is not reasonable. > > The use sites of the $blessingcond in sg-report-flight are adjust

Re: [Xen-devel] [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS

2015-06-30 Thread Ian Jackson
Meng Xu writes ("Re: [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS"): > Thank you very much for your explanation! > > 2015-06-29 2:33 GMT-07:00 Dario Faggioli : ... > > (the subject changes a bit, depending on the actual tests results, but > > that's the patter. > > > Yes. I noticed

Re: [Xen-devel] [OSSTEST PATCH 09/13] Reporting sg-report-job-history: support --max-flight

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > Using report_blessingscond instead of the open-coded map. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-deve

Re: [Xen-devel] [OSSTEST PATCH 10/13] Reporting: sg-report-host-history: Support --max-flight

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > We can't use report_blessingscond because we handle blessings > differently in sg-report-host-history (since we want the history of > even adhoc etc. flights). > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell __

Re: [Xen-devel] [PATCH 12/35] libxl: events: Make timeout and async exec setup take an ao, not a gc

2015-06-30 Thread Ian Jackson
Wen Congyang writes ("Re: [PATCH 12/35] libxl: events: Make timeout and async exec setup take an ao, not a gc"): > On 06/26/2015 01:44 AM, Ian Jackson wrote: > > All the callers follow the obvious patterns and therefore supply the > > ao's gc to libxl__async_exec_start and the timeout setup functi

Re: [Xen-devel] [OSSTEST PATCH 11/13] Reporting: In history tables, show multiple failure steps

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > In report_run_getinfo, used by sg-report-job-history and > sg-report-host-history, make provision for reporting multiple failed > steps. > > The failed steps are put side by side (in a single table cell). The > cell background is set by the

Re: [Xen-devel] [PATCH v2] remove now unnecessary gc from libxl__async_exec_start calls

2015-06-30 Thread Ian Jackson
Wen Congyang writes ("[PATCH v2] remove now unnecessary gc from libxl__async_exec_start calls"): > These were removed in commit f5f8400f. Acked-by: Ian Jackson And I have committed it (with `libxl:' added to the front of the Subject). Thanks, Ian. _

Re: [Xen-devel] [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS

2015-06-30 Thread Ian Campbell
On Tue, 2015-06-30 at 10:11 +0100, Ian Jackson wrote: > Meng Xu writes ("Re: [PATCH v2] OSSTest: stop testing SEDF, start testing > RTDS"): > > Thank you very much for your explanation! > > > > 2015-06-29 2:33 GMT-07:00 Dario Faggioli : > ... > > > (the subject changes a bit, depending on the act

Re: [Xen-devel] [PATCH v2] remove now unnecessary gc from libxl__async_exec_start calls

2015-06-30 Thread Andrew Cooper
On 30/06/15 09:55, Wen Congyang wrote: > These were removed in commit f5f8400f. > > Signed-off-by: Wen Congyang > --- > tools/libxl/libxl_netbuffer.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tools/libxl/libxl_netbuffer.c b/tools/libxl/libxl_netbuffer.c > index

Re: [Xen-devel] [OSSTEST PATCH 05/13] Reporting: report_run_getinfo produces ColourAttr

2015-06-30 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 05/13] Reporting: report_run_getinfo produces ColourAttr"): > On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: ... > > - return { Summary => $summary, Colour => $colour }; > > + return { > > + Summary => $summary, > > + ColourAttr => "

Re: [Xen-devel] [OSSTEST PATCH 08/13] Reporting: Break out report_blessingscond

2015-06-30 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 08/13] Reporting: Break out report_blessingscond"): > Acked-by: Ian Campbell Thanks. > BTW, I noticed that your valid blessing regex here includes "-", which > is not allowed elsewhere, which breaks "./standalone make-flight -f > $branch $branch" for our

Re: [Xen-devel] [PATCH V5 2/7] libxl_read_file_contents: add new entry to read sysfs file

2015-06-30 Thread Chun Yan Liu
>>> On 6/30/2015 at 05:08 PM, in message <21906.23698.778991.627...@mariner.uk.xensource.com>, Ian Jackson wrote: > Chun Yan Liu writes ("Re: [PATCH V5 2/7] libxl_read_file_contents: add new > entry to read sysfs file"): > > >>> On 6/29/2015 at 06:52 PM, in message > > <21905.9050.452111.20

Re: [Xen-devel] [PATCH v3 COLOPre 01/26] tools/libxl: rename libxl__domain_suspend to libxl__domain_save

2015-06-30 Thread Yang Hongyang
On 06/29/2015 11:43 PM, Ian Campbell wrote: On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: Rename libxl__domain_suspend() to libxl__domain_save() since it actually do the save domain work. In response to Ian J's query on the last version Andrew said: The terminology used b

Re: [Xen-devel] [OSSTEST PATCH 13/13] Reporting: sg-report-flight: Put overall pushgate stats in email

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > This adds something like this (would have appeared in 37638): > > Last test of basis37629 2015-06-24 21:36:10 Z4 days > Testing same since37635 2015-06-28 03:05:56 Z1 days4 attempts > Failing since 37630 2

Re: [Xen-devel] [OSSTEST PATCH 05/13] Reporting: report_run_getinfo produces ColourAttr

2015-06-30 Thread Ian Campbell
On Tue, 2015-06-30 at 10:25 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [OSSTEST PATCH 05/13] Reporting: report_run_getinfo > produces ColourAttr"): > > On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > ... > > > - return { Summary => $summary, Colour => $colour }; > > > + return {

Re: [Xen-devel] [PATCH v3 COLOPre 04/26] tools/libxl: move remus code into libxl_remus.c

2015-06-30 Thread Yang Hongyang
On 06/29/2015 11:48 PM, Ian Campbell wrote: On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: Do the following things: - There are 2 checkpoint callbacks, rename them to: libxl__remus_domain_{save/restore}_checkpoint_callback - Moving remus callbacks into libxl_remus.c, export followi

Re: [Xen-devel] [v4][PATCH 11/19] tools: introduce some new parameters to set rdm policy

2015-06-30 Thread Chen, Tiejun
The code fragment didn't answer my question either, but that's not really the point. Do you mean I should write two example, respectively? I meant an example or two of actual concrete uses (ideally the most common ones) of the rdm parameter and what they mean in practical terms. What about t

Re: [Xen-devel] [OSSTEST PATCH 12/13] Reporting: Provide report_find_push_age_info

2015-06-30 Thread Ian Campbell
On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > This utility function provides information about how far behind the > push gate is right now. > > I have tested it with this rune in the Cambridge instance: > > perl -we 'use Osstest::Executive; use Osstest; use Data::Dumper; open > DEBUG

Re: [Xen-devel] [OSSTEST PATCH 03/13] Reporting: sg-report-flight: Fix undefined value in $worstrow[]

2015-06-30 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 03/13] Reporting: sg-report-flight: Fix undefined value in $worstrow[]"): > On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: ... > > diff --git a/sg-report-flight b/sg-report-flight > > index 6481521..117b609 100755 > > --- a/sg-report-flight > > +++ b

Re: [Xen-devel] [PATCH v2] remove now unnecessary gc from libxl__async_exec_start calls

2015-06-30 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2] remove now unnecessary gc from libxl__async_exec_start calls"): > On 30/06/15 09:55, Wen Congyang wrote: > > These were removed in commit f5f8400f. ... > > -rc = libxl__async_exec_start(gc, &dev->aodev.aes); > > +rc = libxl__async_exec_star

Re: [Xen-devel] [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS

2015-06-30 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS"): > On Tue, 2015-06-30 at 10:11 +0100, Ian Jackson wrote: > > I hope you are able to set up the appropriate email filter. > > Perhaps we could provide some X-osstest-* headers to help? Branch and > Flight are alr

Re: [Xen-devel] [PATCH V5 2/7] libxl_read_file_contents: add new entry to read sysfs file

2015-06-30 Thread Ian Jackson
Chun Yan Liu writes ("Re: [PATCH V5 2/7] libxl_read_file_contents: add new entry to read sysfs file"): > >>> On 6/30/2015 at 05:08 PM, in message > <21906.23698.778991.627...@mariner.uk.xensource.com>, Ian Jackson > wrote: > > Maybe sysfs will grow support for bigger files in the future. (Also,

Re: [Xen-devel] [OSSTEST PATCH 08/13] Reporting: Break out report_blessingscond

2015-06-30 Thread Ian Campbell
On Tue, 2015-06-30 at 10:31 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [OSSTEST PATCH 08/13] Reporting: Break out > report_blessingscond"): > > Acked-by: Ian Campbell > > Thanks. > > > BTW, I noticed that your valid blessing regex here includes "-", which > > is not allowed elsewhere

Re: [Xen-devel] [PATCH v3 COLOPre 06/26] libxl/save: Refactor libxl__domain_suspend_state

2015-06-30 Thread Yang Hongyang
On 06/30/2015 12:01 AM, Ian Campbell wrote: On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: [...] switch (type) { case LIBXL_DOMAIN_TYPE_HVM: { dss->hvm = 1; +dsps->hvm = 1; break; } case LIBXL_DOMAIN_TYPE_PV: dss->hvm = 0;

Re: [Xen-devel] [PATCH v2] remove now unnecessary gc from libxl__async_exec_start calls

2015-06-30 Thread Ian Campbell
On Tue, 2015-06-30 at 10:40 +0100, Ian Jackson wrote: > Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2] remove now unnecessary gc > from libxl__async_exec_start calls"): > > On 30/06/15 09:55, Wen Congyang wrote: > > > These were removed in commit f5f8400f. > ... > > > -rc = libxl__async_exe

Re: [Xen-devel] [PATCH v3 COLOPre 07/26] libxc/restore: fix error handle of process_record

2015-06-30 Thread Yang Hongyang
On 06/30/2015 12:07 AM, Ian Campbell wrote: On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: ITYM "handling" in the subject. And perhaps "change" rather than "fix"? If the err is RECORD_NOT_PROCESSED, and it is an optional record, restore will still fail. There're two options to fix t

Re: [Xen-devel] Fwd: [v3 12/15] vmx: posted-interrupt handling when vCPU is blocked

2015-06-30 Thread Dario Faggioli
On Tue, 2015-06-30 at 02:59 +, Wu, Feng wrote: > > Quoting the design document in patch 1: > > > > +Here is the scenario for the usage of the new global vector: > > + > > +1. vCPU0 is running on pCPU0 > > +2. vCPU0 is blocked and vCPU1 is currently running on pCPU0 > > +3. An external interru

Re: [Xen-devel] [PATCH v3 COLOPre 06/26] libxl/save: Refactor libxl__domain_suspend_state

2015-06-30 Thread Ian Campbell
On Tue, 2015-06-30 at 17:43 +0800, Yang Hongyang wrote: > > On 06/30/2015 12:01 AM, Ian Campbell wrote: > > On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > > [...] > >> switch (type) { > >> case LIBXL_DOMAIN_TYPE_HVM: { > >> dss->hvm = 1; > >> +dsps->hvm = 1

Re: [Xen-devel] [OSSTEST PATCH 13/13] Reporting: sg-report-flight: Put overall pushgate stats in email

2015-06-30 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 13/13] Reporting: sg-report-flight: Put overall pushgate stats in email"): > On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > > This adds something like this (would have appeared in 37638): > > > > Last test of basis37629 2015-06-24 21:36:10

Re: [Xen-devel] [PATCH] xen/events/fifo: Consume unprocessed events when a CPU dies

2015-06-30 Thread Ross Lagerwall
On 06/29/2015 02:32 PM, Boris Ostrovsky wrote: On 06/29/2015 06:19 AM, Ross Lagerwall wrote: On 06/19/2015 05:06 PM, David Vrabel wrote: On 19/06/15 17:02, Boris Ostrovsky wrote: On 06/19/2015 11:15 AM, Ross Lagerwall wrote: When a CPU is offlined, there may be unprocessed events on a port fo

Re: [Xen-devel] [OSSTEST PATCH 08/13] Reporting: Break out report_blessingscond

2015-06-30 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 08/13] Reporting: Break out report_blessingscond"): > On Tue, 2015-06-30 at 10:31 +0100, Ian Jackson wrote: > > IMO `-' is valid in blessings. Consider `commission-oseleta'. > > I'm being very confused -- I was thinking of flight names, not blessing > nam

Re: [Xen-devel] [PATCH v3 COLOPre 09/26] tools/libxl: introduce enum type libxl_checkpointed_stream

2015-06-30 Thread Yang Hongyang
On 06/30/2015 12:30 AM, Ian Campbell wrote: On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: introduce enum type libxl_checkpointed_stream in IDL. rename the last argument of migrate_receive from "remus" to "checkpointed" since the semantics of this parameter has changed. I don't see

Re: [Xen-devel] [OSSTEST PATCH 13/13] Reporting: sg-report-flight: Put overall pushgate stats in email

2015-06-30 Thread Ian Campbell
On Tue, 2015-06-30 at 10:50 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [OSSTEST PATCH 13/13] Reporting: sg-report-flight: > Put overall pushgate stats in email"): > > On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote: > > > This adds something like this (would have appeared in 37638)

[Xen-devel] [OSSTEST PATCH 11/14] Reporting: Break out report_blessingscond

2015-06-30 Thread Ian Jackson
The returned SQL is a self-contained expression, and does not require additional bind parameters. To spot SQL quoting problems, we die if a blessing is not reasonable. The use sites of the $blessingcond in sg-report-flight are adjusted to no longer pass @blessings to execute. No overall function

[Xen-devel] [OSSTEST PATCH 13/14] Reporting: Provide report_find_push_age_info

2015-06-30 Thread Ian Jackson
This utility function provides information about how far behind the push gate is right now. I have tested it with this rune in the Cambridge instance: perl -we 'use Osstest::Executive; use Osstest; use Data::Dumper; open DEBUG, ">&STDERR" or die $!; csreadconfig(); print Dumper report_find_pu

[Xen-devel] [OSSTEST PATCH 01/14] ts-leak-check: Report leaked loop block devices

2015-06-30 Thread Ian Jackson
These show up as leaked processes `[loop1]' etc. too, but that is an implementation detail, and it is clearer to show them separately. Signed-off-by: Ian Jackson Acked-by: Ian Campbell --- ts-leak-check |9 + 1 file changed, 9 insertions(+) diff --git a/ts-leak-check b/ts-leak-chec

[Xen-devel] [OSSTEST PATCH 08/14] Reporting: In report_run_getinfo, abstract code for returning

2015-06-30 Thread Ian Jackson
This will allow us to change the function's API in one place (plus all the call sites). No functional change in this patch. Signed-off-by: Ian Jackson Acked-by: Ian Campbell --- Osstest/Executive.pm | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/Os

[Xen-devel] [OSSTEST PATCH v2 00/14] Linux 4.1, and reporting improvements

2015-06-30 Thread Ian Jackson
Most of this series is improvements to html and email output. It includes the push gate age query library function I offered to Ian Campbell. Also thrown in is a patch to test the Linux 4.1 stable branch and a related cleanup. Most of these patches are already acked. I have reordered the series

[Xen-devel] [OSSTEST PATCH 10/14] Reporting: report_run_getinfo produces Content

2015-06-30 Thread Ian Jackson
Previously we produced Summary, and the call sites were confused about whether it had already been html-escaped. Now we produce something which is explicitly already html. No functional change except that some sg-report-host-history output is escaped as it ought to have been (but this is only rel

[Xen-devel] [OSSTEST PATCH 09/14] Reporting: report_run_getinfo produces ColourAttr

2015-06-30 Thread Ian Jackson
This includes the bgcolour attribute name and the quotes. This will make it possible for this function to sometimes not set a background for the whole table cell in the future (which will be part of putting multiple step results in each cell). No functional change. Signed-off-by: Ian Jackson Ac

[Xen-devel] [OSSTEST PATCH 06/14] Reporting: sg-report-job-history: support --max-flight

2015-06-30 Thread Ian Jackson
Using report_blessingscond instead of the open-coded map. Signed-off-by: Ian Jackson Acked-by: Ian Campbell --- sg-report-job-history |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/sg-report-job-history b/sg-report-job-history index 51cb5b6..3d02e3f 100755 --- a/s

[Xen-devel] [OSSTEST PATCH 12/14] Reporting: In history tables, show multiple failure steps

2015-06-30 Thread Ian Jackson
In report_run_getinfo, used by sg-report-job-history and sg-report-host-history, make provision for reporting multiple failed steps. The failed steps are put side by side (in a single table cell). The cell background is set by the job status, but the individual steps may vary their colour, as see

[Xen-devel] [OSSTEST PATCH 02/14] Test Linux 4.1

2015-06-30 Thread Ian Jackson
The kernel.org stable git branch layout is the same for 4.1 as for 3.x, so simply extend the pattern, and add the branch to BRANCHES. We move the linux-stable.git pattern to the end, and extend it to linux-[3-9].[0-9]*, to provide a bit more future-proofing. (There is no particular reason other t

[Xen-devel] [OSSTEST PATCH 03/14] ap-common: Abolish linux-3.0 special case

2015-06-30 Thread Ian Jackson
This is identical to the effect of the linux-[3-9] etc. glob. Signed-off-by: Ian Jackson --- v2: New patch in this version of the series. --- ap-common |4 1 file changed, 4 deletions(-) diff --git a/ap-common b/ap-common index 7ddeb9a..dfeab9e 100644 --- a/ap-common +++ b/ap-common @@

[Xen-devel] [OSSTEST PATCH 05/14] Reporting: sg-report-job-history: Honour --limit

2015-06-30 Thread Ian Jackson
The global $limit was erroneously shadowed by a local in processjobbranch. Fix this, and move the erroneously hardcoded value to the global default. No functional change if --limit was not specified, which it is not for any of our runs with the in-tree configs. Signed-off-by: Ian Jackson Acked-

[Xen-devel] [OSSTEST PATCH 04/14] Reporting: sg-report-flight: Fix undefined value in $worstrow[]

2015-06-30 Thread Ian Jackson
The fallback entry in the worstrow search, used when a job has not been run at all, was wrong. Also, fix the doc comment which induced the mistake, and add a comment about the contents of @worst. Signed-off-by: Ian Jackson --- v2: Added the comment about @worst. --- sg-report-flight |6

[Xen-devel] [OSSTEST PATCH 07/14] Reporting: sg-report-host-history: Support --max-flight

2015-06-30 Thread Ian Jackson
We can't use report_blessingscond because we handle blessings differently in sg-report-host-history (since we want the history of even adhoc etc. flights). Signed-off-by: Ian Jackson Acked-by: Ian Campbell --- sg-report-host-history |6 ++ 1 file changed, 6 insertions(+) diff --git a/s

[Xen-devel] [OSSTEST PATCH 14/14] Reporting: sg-report-flight: Put overall pushgate stats in email

2015-06-30 Thread Ian Jackson
This adds something like this (would have appeared in 37638): Last test of basis37629 2015-06-24 21:36:10 Z4 days Failing since 37630 2015-06-26 10:35:44 Z3 days9 attempts Testing same since37635 2015-06-28 03:05:56 Z1 days4 attempts or this (would hav

Re: [Xen-devel] [PATCH] Refactor ioreq server for better performance

2015-06-30 Thread Paul Durrant
> -Original Message- > From: Yu, Zhang [mailto:yu.c.zh...@linux.intel.com] > Sent: 30 June 2015 08:11 > To: Paul Durrant; xen-de...@lists.xenproject.org; Andrew Cooper; > jbeul...@suse.com; Kevin Tian; zhiyuan...@intel.com > Subject: Re: [Xen-devel] [PATCH] Refactor ioreq server for better

Re: [Xen-devel] [PATCH v3 COLOPre 12/26] tools/libxl: Update libxl_domain_unpause() to support qemu-xen

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > Currently, libxl__domain_unpause() only supports > qemu-xen-traditional. Update it to support qemu-xen. > We use libxl__domain_resume_device_model to unpause guest dm. There appears to be at least two significant semantic changes to the lib

Re: [Xen-devel] [PATCH v3 COLOPre 14/26] tools/libxl: export logdirty_init

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > We need to enable logdirty on secondary, so we export logdirty_init > for internal use. Rename it to libxl__logdirty_init. > > Signed-off-by: Yang Hongyang Acked-by: Ian Campbell ___ Xen-dev

Re: [Xen-devel] [PATCH v3 COLOPre 08/26] tools/libxc: support to resume uncooperative HVM guests

2015-06-30 Thread Wen Congyang
On 06/30/2015 12:27 AM, Ian Campbell wrote: > On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: >> From: Wen Congyang >> >> 1. suspend >> a. PVHVM and PV: we use the same way to suspend the guest(send the suspend > > space between "guest" and the open parenthesis please. > >>request to

Re: [Xen-devel] [PATCH v3 COLOPre 11/26] tools/libxl: introduce a new API libxl__domain_restore() to load qemu state

2015-06-30 Thread Yang Hongyang
On 06/30/2015 12:38 AM, Ian Campbell wrote: On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: Secondary vm is running in colo mode. So we will do the following things again and again: 1. suspend both primay vm and secondary vm 2. sync the state 3. resume both primary vm and secondary vm

Re: [Xen-devel] [PATCH v3 COLOPre 15/26] tools/libxl: Add back channel to allow migration target send data back

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > + * LIBXL_HAVE_DOMAIN_CREATE_RESTORE_SEND_FD 1 > + * > + * If this is defined, libxl_domain_create_restore()'s API has changed to > + * include a send_fd param which used for libxl migration back channel.

Re: [Xen-devel] [PATCH v3 COLOPre 06/26] libxl/save: Refactor libxl__domain_suspend_state

2015-06-30 Thread Yang Hongyang
On 06/30/2015 05:50 PM, Ian Campbell wrote: On Tue, 2015-06-30 at 17:43 +0800, Yang Hongyang wrote: On 06/30/2015 12:01 AM, Ian Campbell wrote: On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: [...] switch (type) { case LIBXL_DOMAIN_TYPE_HVM: { dss->hvm = 1; +

Re: [Xen-devel] [PATCH v3 COLOPre 16/26] tools/libx{l, c}: add back channel to libxc

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > We need to send secondary's dirty page pfns back to primary. In v2 Ian asked (<21888.2988.774072.32...@mariner.uk.xensource.com>): In the pdf http://www.socc2013.org/home/program/a3-dong.pdf?attredirects=0

Re: [Xen-devel] [v3 12/15] vmx: posted-interrupt handling when vCPU is blocked

2015-06-30 Thread Andrew Cooper
On 24/06/15 06:18, Feng Wu wrote: > @@ -1848,6 +1869,33 @@ static struct hvm_function_table __initdata > vmx_function_table = { > .enable_msr_exit_interception = vmx_enable_msr_exit_interception, > }; > > +/* > + * Handle VT-d posted-interrupt when VCPU is blocked. > + */ > +static void pi

Re: [Xen-devel] [PATCH] xen: new maintainer for the RTDS scheduler

2015-06-30 Thread George Dunlap
On Thu, Jun 25, 2015 at 1:44 PM, Dario Faggioli wrote: > Signed-off-by: Dario Faggioli > --- > Cc: George Dunlap > Cc: Meng Xu Not sure how I missed this one in my inbox: Acked-by: George Dunlap > --- > MAINTAINERS |5 + > 1 file changed, 5 insertions(+) > > diff --git a/MAINTAINER

[Xen-devel] [PATCH] x86/traps: Move early pagefault static data into __initdata

2015-06-30 Thread Andrew Cooper
Signed-off-by: Andrew Cooper CC: Jan Beulich --- xen/arch/x86/traps.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/arch/x86/traps.c b/xen/arch/x86/traps.c index ac62f20..7a5d911 100644 --- a/xen/arch/x86/traps.c +++ b/xen/arch/x86/traps.c @@ -1552,8 +1552,8 @@ v

Re: [Xen-devel] [PATCH v3 COLOPre 16/26] tools/libx{l, c}: add back channel to libxc

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > diff --git a/tools/libxl/libxl_save_callout.c > b/tools/libxl/libxl_save_callout.c > index 12cc57e..25817d6 100644 > --- a/tools/libxl/libxl_save_callout.c > +++ b/tools/libxl/libxl_save_callout.c > @@ -27,7 +27,7 @@ > */ > static void

Re: [Xen-devel] [PATCH v3 COLOPre 17/26] tools/libx{l, c}: introduce should_checkpoint callback

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > Under COLO, we are doing checkpoint on demand, if this > callback returns 1, we will take another checkpoint. > 0 indicates unexpected error. Is this checkpoint therefore expected to be blocking until another checkpoint is desired (or an er

Re: [Xen-devel] [PATCH v3 COLOPre 18/26] tools/libx{l, c}: add postcopy/suspend callback to restore side

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > Secondary(restore side) is running under COLO, we also need > postcopy/suspend callbacks. I wonder if this and the previous patch ought to be folded into the patches which introduce the actual use of the callback? > > Signed-off-by: Yang H

Re: [Xen-devel] [PATCH v3 COLOPre 19/26] libxc/migration: Specification update for DIRTY_BITMAP records

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > Used by secondary to send it's dirty bitmap to primary under COLO. This is the backchannel, right? It seems to me that this ought to be described more clearly as a separate stream in the opposite direction, rather than looking like just an

Re: [Xen-devel] [PATCH v3 COLOPre 21/26] tools/libxl: refactor write stream to support back channel

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > From: Wen Congyang Refactor how? (Tip: In general no change can be sufficiently described in just the subject line unless it is absolutely the most trivial possible one). It looks like mostly checkpoint_callback is renamed to write_recor

Re: [Xen-devel] Unable to unregister netdevice after "netfilter: bridge: forward IPv6 fragmented packets"

2015-06-30 Thread Julien Grall
Hi Florian, On 29/06/15 19:55, Florian Westphal wrote: > Julien Grall wrote: >> I tried to run the latest Linux tree >> (4a10a91756ef381bced7b88cfb9232f660b92d93) as DOM0 Xen. >> After destroying a guest using network, I got the following >> lines in the DOM0 kernel log: >> >> unregister_netdevic

Re: [Xen-devel] [PATCH v3 COLOPre 22/26] tools/libxl: refactor read stream to support back channel

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > From: Wen Congyang Same comments as previous patch.. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [v4][PATCH 03/19] xen/vtd: create RMRR mapping

2015-06-30 Thread George Dunlap
On Wed, Jun 24, 2015 at 8:33 AM, Jan Beulich wrote: On 24.06.15 at 09:26, wrote: >>> This would need to go into patch 2; I wonder whether folding that >> >> Yes. >> >>> and this one wouldn't be warranted, avoiding the former adding >> >> Are you saying to fold patch #2 and patch #3? But shou

Re: [Xen-devel] [PATCH v3 COLOPre 23/26] docs/libxl: Introduce COLO_CONTEXT to support migration v2 colo streams

2015-06-30 Thread Ian Campbell
On Thu, 2015-06-25 at 14:25 +0800, Yang Hongyang wrote: > From: Wen Congyang This seems to mix up forward and backward facing information in a single record? > Signed-off-by: Wen Congyang > --- > docs/specs/libxl-migration-stream.pandoc | 21 - > tools/libxl/libxl_sr_stream

  1   2   3   >