Re: [Xen-devel] vTPM ownership problem

1. For xen 4.3.0, you’d better enable XSM security module. 2. share your vtpmmgr / vtpm instance log. 3. Clear vtpm / vtpmmgr disk, do it again. Quan From: xen-devel-boun...@lists.xen.org [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Zhenyang Feng Sent: Wednesday, September 09,

[Xen-devel] [PATCH] pci-attach: fix assertation

Run "xl pci-attach ", the 2nd time fails: xl: libxl_xshelp.c:209: libxl__xs_transaction_start: Assertion `!*t' failed. Aborted To fix that, initialize xs_transaction to avoid libxl__xs_transaction_start assertation error. Signed-off-by: Chunyan Liu --- tools/libxl/libxl_pci.c

[Xen-devel] [xen-4.1-testing baseline-only test] 37935: tolerable FAIL

This run is configured for baseline tests only. flight 37935 xen-4.1-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/37935/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-migrupgrade 1 build-check(1)

Re: [Xen-devel] [PATCH v2 4/4] tools/xen-mceinj: Pass in GPA when injecting through MSR_MCI_ADDR

On 2015/09/16 7:35, Haozhong Zhang wrote: > This patch removes the address translation in xen-mceinj which > translates the guest physical address passed-in through the argument of > '-p' to the host machine address. Instead, xen-mceinj now passes a flag > MC_MSRINJ_F_GPADDR to ask do_mca() in the

Re: [Xen-devel] [osstest test] 60719: tolerable FAIL - PUSHED

On Fri, 2015-09-11 at 21:56 -0600, Jim Fehlig wrote: > On 09/10/2015 10:40 AM, Ian Campbell wrote: > > On Thu, 2015-09-03 at 10:35 -0600, Jim Fehlig wrote: > > > > > I wonder if this has anything to do with migration V2? I noticed a > > > migration > > > regression a few days back, but later

Re: [Xen-devel] [PATCH v2 1/4] x86/mce: Fix code style

On 2015/09/16 7:35, Haozhong Zhang wrote: > Remove trailing whitespaces and fix indentations in mce.c and xen_mca.h. > > Signed-off-by: Haozhong Zhang Acked-by: Christoph Egger > --- > xen/arch/x86/cpu/mcheck/mce.c | 10 +- >

Re: [Xen-devel] [PATCH v6 16/18] vmx: Add some scheduler hooks for VT-d posted interrupts

> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, September 10, 2015 5:26 PM > To: Wu, Feng > Cc: Andrew Cooper; Dario Faggioli; George Dunlap; Tian, Kevin; > xen-devel@lists.xen.org; Keir Fraser > Subject: RE: [PATCH v6 16/18] vmx: Add some scheduler

[Xen-devel] [linux-3.10 test] 61983: regressions - FAIL

flight 61983 linux-3.10 real [real] http://logs.test-lab.xenproject.org/osstest/logs/61983/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qcow2 9 debian-di-install fail REGR. vs. 60670

Re: [Xen-devel] [Patch RFC 03/13] vt-d: Track the Device-TLB invalidation status in an invalidation table.

Hi Quan, The time of the mail is in a future. Can you configure your mail to report the correct time? On 16/09/2015 14:23, Quan Xu wrote: diff --git a/xen/include/xen/hvm/iommu.h b/xen/include/xen/hvm/iommu.h index 106e08f..28e7fc3 100644 --- a/xen/include/xen/hvm/iommu.h +++

Re: [Xen-devel] [Patch RFC 07/13] vt-d: If the qi_flag is Set, the domain's vCPUs are not allowed to

Hi Quan, On 16/09/2015 14:24, Quan Xu wrote: diff --git a/xen/arch/x86/x86_64/asm-offsets.c b/xen/arch/x86/x86_64/asm-offsets.c index 447c650..d26b026 100644 --- a/xen/arch/x86/x86_64/asm-offsets.c +++ b/xen/arch/x86/x86_64/asm-offsets.c @@ -116,6 +116,7 @@ void __dummy__(void) BLANK();

Re: [Xen-devel] [PATCH v2 2/4] tools/xen-mceinj: Fix code style

On Wed, Sep 16, 2015 at 08:45:58AM +0100, Wei Liu wrote: > On Wed, Sep 16, 2015 at 01:35:13PM +0800, Haozhong Zhang wrote: > > Remove trailing whitespaces in xen-mceinj.c. > > > > Signed-off-by: Haozhong Zhang > > Acked-by: Wei Liu > > Haozhong,

Re: [Xen-devel] [PATCH v2 2/4] tools/xen-mceinj: Fix code style

On Wed, Sep 16, 2015 at 03:52:39PM +0800, Haozhong Zhang wrote: > On Wed, Sep 16, 2015 at 08:45:58AM +0100, Wei Liu wrote: > > On Wed, Sep 16, 2015 at 01:35:13PM +0800, Haozhong Zhang wrote: > > > Remove trailing whitespaces in xen-mceinj.c. > > > > > > Signed-off-by: Haozhong Zhang

Re: [Xen-devel] [PATCH v2 2/4] tools/xen-mceinj: Fix code style

On Wed, Sep 16, 2015 at 08:58:41AM +0100, Wei Liu wrote: > On Wed, Sep 16, 2015 at 03:52:39PM +0800, Haozhong Zhang wrote: > > On Wed, Sep 16, 2015 at 08:45:58AM +0100, Wei Liu wrote: > > > On Wed, Sep 16, 2015 at 01:35:13PM +0800, Haozhong Zhang wrote: > > > > Remove trailing whitespaces in

Re: [Xen-devel] [PATCH v2 3/4] x86/mce: Translate passed-in GPA to host machine address

On 2015/09/16 7:35, Haozhong Zhang wrote: > This patch adds a new flag MC_MSRINJ_F_GPADDR to > xen_mc_msrinject.mcinj_flags, and makes do_mca() to translate the > guest physical address passed-in through > xen_mc_msrinject.mcinj_msr[i].value to the host machine address if > this flag is present. >

Re: [Xen-devel] [PATCH OSSTEST 3/4] Add support for selecting resources based on their properties.

On Tue, 2015-09-15 at 18:18 +0100, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST 3/4] Add support for selecting > resources based on their properties."): > > In particular for allocating hosts based on host properties. > > > > To do this we extend the hostflags syntax with

Re: [Xen-devel] [Patch RFC 11/13] vt-d: If the Device-TLB flush is still not completed when

Hi Quan, On 16/09/2015 14:24, Quan Xu wrote: to destroy virtual machine, schedule and wait on a waitqueue until the Device-TLB flush is completed. Signed-off-by: Quan Xu --- xen/common/domain.c | 10 ++ xen/drivers/passthrough/vtd/iommu.c | 9

Re: [Xen-devel] [PATCH] vtd/iommu: correct loglevel when check group divices

> From: Chen, Tiejun > Sent: Wednesday, September 16, 2015 9:00 AM > > Since commit 3848058e7dd6 (vtd/iommu: permit group devices to > passthrough in relaxed mode) is introduced, we always print > message as XENLOG_G_WARNING but its not correct in the case of > strict mode. So here is making this

Re: [Xen-devel] [PATCH v2 2/4] tools/xen-mceinj: Fix code style

On Wed, Sep 16, 2015 at 01:35:13PM +0800, Haozhong Zhang wrote: > Remove trailing whitespaces in xen-mceinj.c. > > Signed-off-by: Haozhong Zhang Acked-by: Wei Liu Haozhong, normally you should carry all the tags you received from the last round

Re: [Xen-devel] [PATCH v2 2/4] tools/xen-mceinj: Fix code style

On 2015/09/16 7:35, Haozhong Zhang wrote: > Remove trailing whitespaces in xen-mceinj.c. > > Signed-off-by: Haozhong Zhang Acked-by: Christoph Egger > --- > tools/tests/mce-test/tools/xen-mceinj.c | 66 > - > 1

Re: [Xen-devel] [PATCH OSSTEST 2/4] ts-hosts-allocate-Executive: add a label to loop over candidates

On Tue, 2015-09-15 at 18:04 +0100, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST 2/4] ts-hosts-allocate-Executive: > add a label to loop over candidates"): > > I'm going to want to "next CANDIDATE" from within a nested loop. > > I'm assuming you've checked that these are the only

Re: [Xen-devel] [PATCH v2 4/4] tools/xen-mceinj: Pass in GPA when injecting through MSR_MCI_ADDR

>>> On 16.09.15 at 07:35, wrote: > This patch removes the address translation in xen-mceinj which > translates the guest physical address passed-in through the argument of > '-p' to the host machine address. Instead, xen-mceinj now passes a flag > MC_MSRINJ_F_GPADDR to

Re: [Xen-devel] [Patch RFC 10/13] vt-d: Held on the removed page until the Device-TLB flush is completed.

Hi Quan, On 16/09/2015 14:24, Quan Xu wrote: Signed-off-by: Quan Xu --- xen/common/memory.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/xen/common/memory.c b/xen/common/memory.c index 61bb94c..4b2def5 100644 ---

Re: [Xen-devel] [PATCH 1/4] x86/EPT: always return proper order value from ept_get_entry()

> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Tuesday, September 15, 2015 3:31 PM > > This is so that callers can determine what range of address space would > get altered by a corresponding "set". > > Signed-off-by: Jan Beulich Acked-by: Kevin Tian

Re: [Xen-devel] [v2][PATCH] vtd/iommu: correct loglevel when check group divices

> From: Chen, Tiejun > Sent: Wednesday, September 16, 2015 3:39 PM > > Since commit 3848058e7dd6 (vtd/iommu: permit group devices to > passthrough in relaxed mode) is introduced, we always print > message as XENLOG_G_WARNING but its not correct in the case of > strict mode. So here is making this

Re: [Xen-devel] [PATCH] pci-attach: fix assertation

On Wed, 2015-09-16 at 14:16 +0800, Chunyan Liu wrote: For the subject I prefer to avoid "fix " style messages. In this case something like "libxl: ensure xs transaction is initialised in libxl__device_pci_add_xenstore" would be better. > Run "xl pci-attach ", the 2nd time fails: > xl:

[Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

Since commit 191b3f3344ee ("p2m/ept: enable PML in p2m-ept for log-dirty"), the A and D bits of EPT paging entries are set unconditionally, regardless of whether PML is enabled or not. This causes a regression in Xen 4.6 on some processors due to Intel Errata AVR41 -- HVM guests get severe memory

[Xen-devel] [PATCH] x86/sysctl: Don't clobber memory if NCAPINTS > ARRAY_SIZE(pi->hw_cap)

There is no current problem, as both NCAPINTS and pi->hw_cap are 8 entries, but the limit should be calculated appropriately so as to avoid hypervisor stack corruption if the two do get out of sync. Signed-off-by: Andrew Cooper --- CC: Jan Beulich

[Xen-devel] [v2][PATCH] vtd/iommu: correct loglevel when check group divices

Since commit 3848058e7dd6 (vtd/iommu: permit group devices to passthrough in relaxed mode) is introduced, we always print message as XENLOG_G_WARNING but its not correct in the case of strict mode. So here is making this message depending on the specific mode. CC: Yang Zhang

Re: [Xen-devel] [PATCH v2 2/4] tools/xen-mceinj: Fix code style

On Wed, Sep 16, 2015 at 03:52:39PM +0800, Haozhong Zhang wrote: > On Wed, Sep 16, 2015 at 08:45:58AM +0100, Wei Liu wrote: > > On Wed, Sep 16, 2015 at 01:35:13PM +0800, Haozhong Zhang wrote: > > > Remove trailing whitespaces in xen-mceinj.c. > > > > > > Signed-off-by: Haozhong Zhang

Re: [Xen-devel] [PATCH v6 11/29] xen/x86: add bitmap of enabled emulated devices

>>> On 04.09.15 at 14:08, wrote: > --- a/tools/libxl/libxl_x86.c > +++ b/tools/libxl/libxl_x86.c > @@ -7,8 +7,12 @@ int libxl__arch_domain_prepare_config(libxl__gc *gc, >libxl_domain_config *d_config, >

Re: [Xen-devel] [Patch RFC 08/13] vt-d: Held on the freed page until the Device-TLB flush is completed.

Hi Quan, On 16/09/2015 14:24, Quan Xu wrote: diff --git a/xen/include/xen/hvm/iommu.h b/xen/include/xen/hvm/iommu.h index e40fc7b..5dc0033 100644 --- a/xen/include/xen/hvm/iommu.h +++ b/xen/include/xen/hvm/iommu.h Same remarks as the previous patches for the fields, prototype and macros. @@

[Xen-devel] [PATCH OSSTEST v2 1/4] ts-hosts-allocate-Executive: Allow dry-run

Provide a new -n command line option which causes no allocations to be done, for debugging. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- ts-hosts-allocate-Executive | 10 ++ 1 file changed, 10 insertions(+) diff --git

[Xen-devel] [PATCH OSSTEST v2 3/4] Add support for selecting resources based on their properties.

In particular for allocating hosts based on host properties. To do this we extend the hostflags syntax with "condition:arg1:arg2". This specifies that the candidate host must pass the condition given the arguments. Each "condition" is a new module in the Osstest::ResourceCondition namespace. For

[Xen-devel] [PATCH OSSTEST v2 0/4] Avoid running Linux on hosts for the given version lacks drivers

As discussed in the thread at http://lists.xen.org/archives/html/xen-devel/2015-09/msg01067.html some older versions of Linux lack drivers for hardware in some of our test hosts, meaning in particular that the linux-3.4 branch is now stuck trying to run all of its jobs on hosts which linux-3.4.y

[Xen-devel] [PATCH OSSTEST v2 2/4] ts-hosts-allocate-Executive: add a label to loop over candidates

I'm going to want to "next CANDIDATE" from within a nested loop. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- ts-hosts-allocate-Executive | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

Re: [Xen-devel] [PATCH v2 4/4] tools/xen-mceinj: Pass in GPA when injecting through MSR_MCI_ADDR

On Wed, Sep 16, 2015 at 01:35:15PM +0800, Haozhong Zhang wrote: > This patch removes the address translation in xen-mceinj which > translates the guest physical address passed-in through the argument of > '-p' to the host machine address. Instead, xen-mceinj now passes a flag > MC_MSRINJ_F_GPADDR

Re: [Xen-devel] [PATCH v6 12/29] xen/x86: allow disabling the emulated local apic

>>> On 04.09.15 at 14:08, wrote: > --- a/xen/arch/x86/hvm/svm/svm.c > +++ b/xen/arch/x86/hvm/svm/svm.c > @@ -1035,6 +1035,7 @@ static void noreturn svm_do_resume(struct vcpu *v) > struct vmcb_struct *vmcb = v->arch.hvm_svm.vmcb; > bool_t debug_state =

Re: [Xen-devel] [PATCH v6 11/29] xen/x86: add bitmap of enabled emulated devices

>>> On 04.09.15 at 14:08, wrote: > --- a/xen/include/asm-x86/domain.h > +++ b/xen/include/asm-x86/domain.h > @@ -387,8 +387,21 @@ struct arch_domain > bool_t mem_access_emulate_enabled; > > struct monitor_write_data *event_write_data; > + > +/* Emulated

[Xen-devel] [PATCH OSSTEST v2 4/4] make-flight: Add a minimum linux version requirement to all linux-* branches

We have some hosts in the colo which are not supported by older Linux versions. Add a suitable hostflag using the new resource conditions syntax to cause this to occur. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- mfi-common | 11

Re: [Xen-devel] [PATCH v2 0/4] Fix tools/xen-mceinj to handle >=4GB guest memory

Haozhong Zhang writes ("[PATCH v2 0/4] Fix tools/xen-mceinj to handle >=4GB guest memory"): > The existing xen-mceinj can not inject MCE through MSR_MCI_ADDR to a > domain w/ more than 4GB memory, e.g. if domain 0 has more than 4GB > memory, the execution of the command > xen-mceinj -d 0

Re: [Xen-devel] [PATCH 3/4 RFC] x86/p2m: use large pages for MMIO mappings

Hi Jan, On 15/09/2015 08:34, Jan Beulich wrote: When mapping large BARs (e.g. the frame buffer of a graphics card) the overhead or establishing such mappings using onle 4k pages has, particularly after the XSA-125 fix, become unacceptable. Alter the XEN_DOMCTL_memory_mapping semantics once

Re: [Xen-devel] PCI Pass-through in Xen ARM: Draft 4

On 09/15/2015 11:58 AM, Jaggi, Manish wrote: > > > On Thursday 10 September 2015 06:42 AM, Julien Grall wrote: >> Hi Manish, > Hi Julien, sorry for late response.. >> >> On 13/08/2015 10:42, Manish Jaggi wrote: >>>3.2.Mapping between streamID - deviceID - pci sbdf - requesterID >>>

Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

Quan Xu writes ("[Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device"): > Introduction > Thanks for your submission. JOOI why did you CC me ? I did a quick scan of these patches and they don't seem to have any tools impact. I would prefer not to be CC'd unless

[Xen-devel] [qemu-upstream-4.4-testing test] 61986: regressions - FAIL

flight 61986 qemu-upstream-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/61986/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-xend 5 xen-build fail REGR. vs. 60565

Re: [Xen-devel] [PATCH] pci-attach: fix assertation

Ian Campbell writes ("Re: [PATCH] pci-attach: fix assertation"): > Ian, tools/libxl/libxl_internal.h says to initialised the transaction to 0 > rather than XBT_NULL explicitly -- I don't think there is a specific reason > for that though? It says: /* Transaction functions, best used together. *

Re: [Xen-devel] [OSSTEST PATCH 02/27] Executive: cs-job-create: Check that OSSTEST_FLIGHT is not set

Ian Campbell writes ("Re: [OSSTEST PATCH 02/27] Executive: cs-job-create: Check that OSSTEST_FLIGHT is not set"): > On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > > If OSSTEST_FLIGHT is set on entry to JobDB::Executive::flight_create, > > fail. This obviously represents some kind of

Re: [Xen-devel] [OSSTEST PATCH 07/27] memoise: New utility

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > Give this a GPLv2+ licence so that we can move it into some other > FLOSS package later. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell [...] > +#You should have received a

Re: [Xen-devel] [OSSTEST PATCH 13/27] cr-ensure-disk-space: Look at referring flights

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > Previously the flight to delete was simply the one with the lowest > flight number. Now we sort flights not by their own flight number, > but by the highest flight number of any referencing flight. > > This means that flights whose builds

[Xen-devel] [xen-4.2-testing baseline-only test] 37936: regressions - FAIL

This run is configured for baseline tests only. flight 37936 xen-4.2-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/37936/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-winxpsp3 6

Re: [Xen-devel] [OSSTEST PATCH 15/27] cr-ensure-disk-space: Take the flights db lock

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > This eliminates the race with cs-bisection-step (and other > flight-construction tools which might reuse previous flights, provided > that they also do not pass previous flight numbers from hand to hand > with the db unlocked). > >

[Xen-devel] [PATCH OSSTEST v4] ms-flights-summary: Produce an HTML report of all active flights

Jobs are categorised by a new ->Job field. This is added by ts-hosts-alllocate-Executive and propagated by the planner after recent patches. It contains $flight.$job. Jobs which do not include this are anonymous and are listed separately, using the resource name and info field (if present) as the

[Xen-devel] [PATCH] xen/xsm: Shut up GCC 5.1.1 warnings.

policydb.c: In function ‘user_read’: policydb.c:1443:26: error: ‘buf[2]’ may be used uninitialized in this function [-Werror=maybe-uninitialized] usrdatum->bounds = le32_to_cpu(buf[2]); ^ cc1: all warnings being treated as errors Which makes no sense, as : 1).

Re: [Xen-devel] [OSSTEST PATCH 03/27] mg-list-all-branches: Suppress `maintjobs'

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > This is not a real branch; it's a cron working directory, only. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing

Re: [Xen-devel] [OSSTEST PATCH 05/27] standalone: Set very long SQLite3 busy timeout in Perl

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > Without this, big standalone-generate-dump-flight-runvars jobs may > trying to serialise so much work that SQLite3 times out. And we are > about to introduce an optimisation which makes this much more likely. > > In standalone mode we

Re: [Xen-devel] [OSSTEST PATCH 02/27] Executive: cs-job-create: Check that OSSTEST_FLIGHT is not set

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > If OSSTEST_FLIGHT is set on entry to JobDB::Executive::flight_create, > fail. This obviously represents some kind of mistake. > > One effect is that standalone-generate-dump-flight-runvars won't fill > an Executive database with test

Re: [Xen-devel] [OSSTEST PATCH 04/27] standalone: Do not blunder on after errors

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > ./standalone's with_logging function would _log_ errors, but it > wouldn't exit immediately. As a result, the script would blunder on. > > Normally it wouldn't do very much more since most of the with_logging > calls are the last thing it

[Xen-devel] [OSSTEST PATCH 13/27] cr-ensure-disk-space: Look at referring flights

Previously the flight to delete was simply the one with the lowest flight number. Now we sort flights not by their own flight number, but by the highest flight number of any referencing flight. This means that flights whose builds are being reused are kept as long as the reusing flights. This

[Xen-devel] [OSSTEST PATCH 20/27] make-flight: Allow separate specification of pre-built Xen vs others

Make it possible to separately specify - reuse previous flight's builds of xen.git - reuse previous flight's builds of all other stuff Specifically: * Match the 4th argument to make-flight with case rather than if. If it contains a slash, the first part is for Xen and the second part

[Xen-devel] [OSSTEST PATCH 17/27] mg-show-flight-runvars: Decorate synth runvar names with ~

Make mg-show-flight-runvars -a append ~ to the names of synth runvars. (This is consistent with the new syntax in cs-job-create.) We do this by editing $row->[1] (and $colws[1]) so we can avoid disturbing the general column format calcuation and printing; this means we have to copy $row (because

[Xen-devel] [OSSTEST PATCH 06/27] cr-daily-branch: Honour AP_FETCH_PFX

This environment variable is prefixed to all cr-daily-branch's command lines involving ap-fetch. We are going to use this for memoisation. Signed-off-by: Ian Jackson --- cr-daily-branch | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git

[Xen-devel] [OSSTEST PATCH 27/27] crontab: Enable xen-unstable-smoke branch every 3h at <1mod3>:51 UTC

Signed-off-by: Ian Jackson --- crontab |1 + 1 file changed, 1 insertion(+) diff --git a/crontab b/crontab index 436645b..b9d4ad1 100644 --- a/crontab +++ b/crontab @@ -2,6 +2,7 @@ PATH=/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin

[Xen-devel] [OSSTEST PATCH 15/27] cr-ensure-disk-space: Take the flights db lock

This eliminates the race with cs-bisection-step (and other flight-construction tools which might reuse previous flights, provided that they also do not pass previous flight numbers from hand to hand with the db unlocked). Signed-off-by: Ian Jackson ---

[Xen-devel] [OSSTEST PATCH 25/27] ap-*: Be able to fetch and push xen.git#smoked

The branches and push gates are now: xen.git#staging -[xen-unstable-smoke]-> #smoked -[xen-unstable]-> #master Deployment note: When this passes the osstest self-push-gate, the main xen-unstable flight will start using smoked as an input. Therefore, until the new cronjob is installed to run the

[Xen-devel] [OSSTEST PATCH 00/27] xen.git#staging smoke tests

This is (finally!) my series to provide fast smoke tests of xen-unstable. The first two thirds of the series shaves a number of yaks I encountered on the way. Many of these (particularly the ones up to patch 11 or so) are generally useful and could perhaps go in before the rest. One outstanding

[Xen-devel] [OSSTEST PATCH 19/27] ts-host-alloc-Executive: Honour various hostalloc_* runvars

We honour hostalloc_maxbonus_variation hostalloc_bonus_previousfail hostalloc_bonus_sharereuse and make them default to their previous values. Signed-off-by: Ian Jackson --- ts-hosts-allocate-Executive | 13 + 1 file changed, 9 insertions(+), 4

[Xen-devel] [OSSTEST PATCH 09/27] mfi-common: Use job_create_build for build-*-prev

This causes it to call job_create_build_filter_callback but has no ultimate functional change. Signed-off-by: Ian Jackson --- mfi-common |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mfi-common b/mfi-common index 991baf5..aaaeac9 100644 ---

[Xen-devel] [OSSTEST PATCH 05/27] standalone: Set very long SQLite3 busy timeout in Perl

Without this, big standalone-generate-dump-flight-runvars jobs may trying to serialise so much work that SQLite3 times out. And we are about to introduce an optimisation which makes this much more likely. In standalone mode we probably don't care much about this timeout at all. (It might even

[Xen-devel] [OSSTEST PATCH 07/27] memoise: New utility

Give this a GPLv2+ licence so that we can move it into some other FLOSS package later. Signed-off-by: Ian Jackson --- memoise | 56 1 file changed, 56 insertions(+) create mode 100755 memoise diff --git

[Xen-devel] [OSSTEST PATCH 26/27] cri-common: Support xen-unstable-smoke

Add this branch to select_xenbranch. This works like xen-unstable in most respects. We are going to sort out the push gate ref plumbing in xen.git in the next osstest patch. Also, use a branch-settings file to set its resource priority to -20 to make it run ahead of anything else automatic.

[Xen-devel] [OSSTEST PATCH 08/27] standalone-generate-dump-flight-runvars: memoise ap-fetch

This makes it _much_ faster. Also you can say AP_FETCH_MEMO_KEEP=1 to make it reuse old information, which is useful for making comparisons. For a further speed improvement, use `eatmydata'. This is not the default because it risks corruption of `standalone.db' which is used for other purposes

[Xen-devel] [OSSTEST PATCH 02/27] Executive: cs-job-create: Check that OSSTEST_FLIGHT is not set

If OSSTEST_FLIGHT is set on entry to JobDB::Executive::flight_create, fail. This obviously represents some kind of mistake. One effect is that standalone-generate-dump-flight-runvars won't fill an Executive database with test flights. Signed-off-by: Ian Jackson ---

Re: [Xen-devel] [OSSTEST PATCH 01/27] docs: Document OSSTEST_RESOURCE_PRIORITY values in README.planner

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 06/27] cr-daily-branch: Honour AP_FETCH_PFX

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > This environment variable is prefixed to all cr-daily-branch's command > lines involving ap-fetch. We are going to use this for memoisation. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell

Re: [Xen-devel] [OSSTEST PATCH 10/27] ap-fetch-version*: Unwrap xen-unstable lines

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > Whitespace and \ change only. Makes the next patch a bit clearer. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel

Re: [Xen-devel] [OSSTEST PATCH 11/27] ap-push: Use refs/heads/ for destinations

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > When the destination is a branch, specify refs/heads/ explicitly. > This makes ap-push work even if the ref does not yet exist on the > destination. > > There is no functional change for an existing installation pushing to > an existing

Re: [Xen-devel] [Patch RFC 07/13] vt-d: If the qi_flag is Set, the domain's vCPUs are not allowed to

> -Original Message- > From: Julien Grall [mailto:julien.gr...@citrix.com] > Sent: Wednesday, September 16, 2015 5:44 PM > To: Xu, Quan; andrew.coop...@citrix.com; Dong, Eddie; ian.campb...@citrix.com; > ian.jack...@eu.citrix.com; jbeul...@suse.com; Nakajima, Jun; k...@xen.org; > Tian,

Re: [Xen-devel] [PATCH OSSTEST v2 3/4] Add support for selecting resources based on their properties.

On Wed, 2015-09-16 at 15:00 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [PATCH OSSTEST v2 3/4] Add support for > selecting resources based on their properties."): > > On Wed, 2015-09-16 at 14:50 +0100, Ian Jackson wrote: > > > As a matter of taste, I would have had propname_check throw

Re: [Xen-devel] [PATCH] pci-attach: fix assertation

On Wed, Sep 16, 2015 at 09:25:25AM +0100, Ian Campbell wrote: > On Wed, 2015-09-16 at 14:16 +0800, Chunyan Liu wrote: > > For the subject I prefer to avoid "fix " style messages. In this > case something like "libxl: ensure xs transaction is initialised in > libxl__device_pci_add_xenstore" would

[Xen-devel] [OSSTEST PATCH 04/27] standalone: Do not blunder on after errors

./standalone's with_logging function would _log_ errors, but it wouldn't exit immediately. As a result, the script would blunder on. Normally it wouldn't do very much more since most of the with_logging calls are the last thing it does - but the exit status would be wrong (0, from echo). As a

[Xen-devel] [OSSTEST PATCH 03/27] mg-list-all-branches: Suppress `maintjobs'

This is not a real branch; it's a cron working directory, only. Signed-off-by: Ian Jackson --- mg-list-all-branches |2 ++ 1 file changed, 2 insertions(+) diff --git a/mg-list-all-branches b/mg-list-all-branches index 1549f81..87703c7 100755 ---

[Xen-devel] [OSSTEST PATCH 01/27] docs: Document OSSTEST_RESOURCE_PRIORITY values in README.planner

Signed-off-by: Ian Jackson --- README.planner |7 +++ 1 file changed, 7 insertions(+) diff --git a/README.planner b/README.planner index 21b3415..dfa623e 100644 --- a/README.planner +++ b/README.planner @@ -84,6 +84,13 @@ no resources actually get allocated),

[Xen-devel] [OSSTEST PATCH 11/27] ap-push: Use refs/heads/ for destinations

When the destination is a branch, specify refs/heads/ explicitly. This makes ap-push work even if the ref does not yet exist on the destination. There is no functional change for an existing installation pushing to an existing branch. But for a hypothetical new installation, this would be

[Xen-devel] [OSSTEST PATCH 14/27] cr-ensure-disk-space: Make main loop body into a subroutine

No functional change. Signed-off-by: Ian Jackson --- cr-ensure-disk-space | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/cr-ensure-disk-space b/cr-ensure-disk-space index 0314e7a..8c5a708 100755 --- a/cr-ensure-disk-space +++

[Xen-devel] [OSSTEST PATCH 10/27] ap-fetch-version*: Unwrap xen-unstable lines

Whitespace and \ change only. Makes the next patch a bit clearer. Signed-off-by: Ian Jackson --- ap-fetch-version |3 +-- ap-fetch-version-old |3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/ap-fetch-version b/ap-fetch-version index

[Xen-devel] [OSSTEST PATCH 22/27] make-flight: Run job_create_test_filter_callback on true job name

job_create_test would pass $job to job_create_test_filter_callback but then later maybe append -xsm to it. Fix this. No functional change for existing in-tree code because all existing tests of the $job end in *. Signed-off-by: Ian Jackson --- mfi-common |7

[Xen-devel] [OSSTEST PATCH 12/27] cr-ensure-disk-space: Honour -D

Provide DEBUG and use db_prepare. For now that is the only debugging output. Signed-off-by: Ian Jackson --- cr-ensure-disk-space |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/cr-ensure-disk-space b/cr-ensure-disk-space index

[Xen-devel] [OSSTEST PATCH 16/27] cs-job-create: Permit creation of `synth' runvars

This will be useful if we want to set hostalloc_* runvars. Normally we would want to set those only on flights generated by make-flight. Flights generated by cs-bisection-step or cs-adjust-flight ought not to copy them, because cs-bisection-step makes its own arrangements for host specification,

[Xen-devel] [OSSTEST PATCH 23/27] make-flight: Provide xen-unstable-smoke branch

This contains a very limited set of jobs: build-amd64 build-armhf test-amd64-amd64-libvirt test-amd64-amd64-xl-qemuu-debianhvm-i386 test-armhf-armhf-xl The debianhvm job exists only in this flight, and is generated by having branch_debianhvm_arch return i386 instead of amd64.

Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

Julien Grall writes ("Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device"): > On 16/09/15 11:46, Ian Jackson wrote: > > JOOI why did you CC me ? I did a quick scan of these patches and they > > don't seem to have any tools impact. I would prefer not to be CC'd >

Re: [Xen-devel] [OSSTEST PATCH 09/27] mfi-common: Use job_create_build for build-*-prev

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > This causes it to call job_create_build_filter_callback but has no > ultimate functional change. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell

Re: [Xen-devel] [OSSTEST PATCH 07/27] memoise: New utility

Ian Campbell writes ("Re: [OSSTEST PATCH 07/27] memoise: New utility"): > These days http://www.gnu.org/licenses/gpl-howto.en.html recommends: > > You should have received a copy of the GNU General Public License > along with Foobar. If not, see . I will

Re: [Xen-devel] [PATCH OSSTEST v2 3/4] Add support for selecting resources based on their properties.

On Wed, 2015-09-16 at 14:50 +0100, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST v2 3/4] Add support for selecting > resources based on their properties."): > > In particular for allocating hosts based on host properties. > > Acked-by: Ian Jackson > > > >

Re: [Xen-devel] [OSSTEST PATCH 07/27] memoise: New utility

On Wed, 2015-09-16 at 14:58 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [OSSTEST PATCH 07/27] memoise: New utility"): > > These days http://www.gnu.org/licenses/gpl-howto.en.html recommends: > > > > You should have received a copy of the GNU General Public License > > along with

Re: [Xen-devel] Missing copyright in the Xen header files

On Thu, Sep 10, 2015 at 17:05 +, Lars Kurth wrote: > > > On 10/09/2015 17:26, "Roger Pau Monné" wrote: > > >CCing Lars (the community manager). > > > >El 09/09/15 a les 14.11, Mike Belopuhov ha escrit: > >> Hi, > >> > >> We're in the process of porting Xen PVHVM

Re: [Xen-devel] [OSSTEST PATCH 14/27] cr-ensure-disk-space: Make main loop body into a subroutine

On Wed, 2015-09-16 at 14:35 +0100, Ian Jackson wrote: > No functional change. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] PCI Pass-through in Xen ARM: Draft 4

On 15/09/15 19:58, Jaggi, Manish wrote: >> I can see 2 different solutions: >> 1) Let DOM0 pass the first requester ID when registering the bus >>Pros: >> * Less per-platform code in Xen >>Cons: >> * Assume that the requester ID are contiguous. (Is it really a

Re: [Xen-devel] Notes from Xen BoF at Debconf15

On Tue, 2015-09-08 at 10:24 +0100, Ian Campbell wrote: > Embedding in xen.git > > > We are much better about providing ways to use system-supplied > components these days (since 4.4) and Debian uses them. > > Waldi noted that iPXE did not have such an option. Since that iPXE

Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

> -Original Message- > From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] > Sent: Wednesday, September 16, 2015 6:47 PM > To: Xu, Quan > Cc: andrew.coop...@citrix.com; Dong, Eddie; ian.campb...@citrix.com; > ian.jack...@eu.citrix.com; jbeul...@suse.com; Nakajima, Jun; k...@xen.org; >

Re: [Xen-devel] [PATCH] pci-attach: fix assertation

On Wed, 2015-09-16 at 14:39 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [PATCH] pci-attach: fix assertation"): > > Ian, tools/libxl/libxl_internal.h says to initialised the transaction > > to 0 > > rather than XBT_NULL explicitly -- I don't think there is a specific > > reason > > for

[Xen-devel] [OSSTEST PATCH 24/27] make-flight: mfi-common: Honour $global_runvars

Provide a way for the main script to set some runvars on all jobs. We expect this to be mostly set with +=. Signed-off-by: Ian Jackson --- mfi-common |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mfi-common b/mfi-common index

  1   2   >