Re: [Xen-devel] [PATCH v4 7/7] xen/x86: support XENPF_settime64

2015-11-13 Thread David Vrabel
On 12/11/15 17:30, Stefano Stabellini wrote: > Try XENPF_settime64 first, if it is not available fall back to > XENPF_settime32. > > No need to call __current_kernel_time() when all the info needed are > already passed via the struct timekeeper * argument. > > Return NOTIFY_BAD in case of

Re: [Xen-devel] [PATCH V2] libxl: relax readonly check introduced by XSA-142 fix

2015-11-13 Thread Stefano Stabellini
On Thu, 12 Nov 2015, Jim Fehlig wrote: > The fix for XSA-142 is quite a big hammer, rejecting readonly > disk configuration even when the requested backend is known to > support readonly. While it is true that qemu doesn't support > readonly for emulated IDE or AHCI disks > > $

Re: [Xen-devel] [PATCH] xen: sched: fix (ACPI S3) resume with cpupools with different schedulers.

2015-11-13 Thread Jan Beulich
>>> On 13.11.15 at 11:08, wrote: > Fix things by properly deallocating scheduler specific > data of the pCPU's pool scheduler during pCPU teardown, > and re-allocating them --always for during pCPU > bringup. Avoiding this was done for a reason iirc: What if one such

[Xen-devel] [PATCH 5/9] xen/arm: vgic: Re-order the register emulations to match the memory map

2015-11-13 Thread Julien Grall
It helps to find quickly whether we forgot to emulate a register or not. At the same time add the missing reserved/implementation defined registers. All other missing registers will be added in a follow-up if necessary. Note that only the distributor register map explicitely say the size of a

[Xen-devel] [PATCH 3/9] xen/arm: vgic: Properly emulate the full register

2015-11-13 Thread Julien Grall
The offset in the emulation is based on byte. As most of the registers are 64/32 bits, they will span over multiple bytes. However, the current emulation only care about the first offset. This will result to not emulate properly any access on the register with other offset. Introduce new macros

[Xen-devel] [PATCH 9/9] xen/arm: vgic-v3: Make clear that GICD_*SPI_* registers are reserved

2015-11-13 Thread Julien Grall
Our vGIC emulation have GICD_TYPER.MBIS set to 0 which means that GICD_*SPI_* registers are reserved. Implement them using the *_reserved labels. Also, implement thoses registers for the read part. Signed-off-by: Julien Grall --- xen/arch/arm/vgic-v3.c | 24

[Xen-devel] [PATCH 2/9] xen/arm: vgic-v3: Only emulate identification registers requested by the spec

2015-11-13 Thread Julien Grall
Most of the identification registers space contains implementation defined registers (see 8.1.13 in ARM IHI 0069A) and only GIC{D,R}_PIDR2 is required to be implemented. Currently the emulation of those registers mimic the ARM implementation, but it's untrue to say that we properly emulate a such

[Xen-devel] [PATCH 7/9] xen/arm: vgic-v3: Remove spurious return in GICR_INVALLR

2015-11-13 Thread Julien Grall
Signed-off-by: Julien Grall --- xen/arch/arm/vgic-v3.c | 1 - 1 file changed, 1 deletion(-) diff --git a/xen/arch/arm/vgic-v3.c b/xen/arch/arm/vgic-v3.c index 98104ba..c68afdb 100644 --- a/xen/arch/arm/vgic-v3.c +++ b/xen/arch/arm/vgic-v3.c @@ -248,7 +248,6 @@ static

[Xen-devel] [PATCH 6/9] xen/arm: vgic-v3: Emulate read to GICD_ICACTIVER

2015-11-13 Thread Julien Grall
The GICD_ICACTIVER registers are missing in the read emulation of the distributor. Call the common emulation for the whole range. Signed-off-by: Julien Grall --- xen/arch/arm/vgic-v3.c | 1 + 1 file changed, 1 insertion(+) diff --git a/xen/arch/arm/vgic-v3.c

[Xen-devel] [PATCH 1/9] xen/arm: vgic-v3: Use the correct offset GICR_IGRPMODR0

2015-11-13 Thread Julien Grall
The offset is 0x0D00 and not 0x0F80. Also re-order the definition to keep all the definitions ordered. Signed-off-by: Julien Grall --- xen/include/asm-arm/gic_v3_defs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Xen-devel] [PATCH 8/9] xen/arm: vgic-v3: Don't implement write-only register read as zero

2015-11-13 Thread Julien Grall
A read to a write only register is unknown. Use a memorable value to differentiate from an actual RAZ register. Signed-off-by: Julien Grall --- xen/arch/arm/vgic-v3.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git

[Xen-devel] [PATCH 0/9] xen/arm: Bunch of fixes for the vGIC emulation

2015-11-13 Thread Julien Grall
Hi all, The main point of this series is to fix the access to any register when the user doesn't write at the base offset of the registers. At the same, I took the opportunity to re-arrange the emulation and dropping any registers which doesn't exists or not required by the spec. This series is

[Xen-devel] [xen-unstable test] 64149: regressions - FAIL

2015-11-13 Thread osstest service owner
flight 64149 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/64149/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-arndale 6 xen-boot fail REGR. vs. 64035

Re: [Xen-devel] [PATCH v4 3/3] xen/arm: move ticks conversions function declarations to the header file

2015-11-13 Thread Julien Grall
Hi Stefano, On 12/11/15 17:46, Stefano Stabellini wrote: > This is just a cleanup, not required at the moment. > > Signed-off-by: Stefano Stabellini Reviewed-by: Julien Grall > --- > xen/arch/arm/vtimer.c |3 --- >

[Xen-devel] [PATCH 5/5] Scripts to create and delete xen-scsiback nodes in Linux target framework

2015-11-13 Thread Olaf Hering
Just to make them public, not meant for merging: The scripts used during development to create a bunch of SCSI devices in dom0 using the Linux target framework. targetcli3 and rtslib3 is used. A patch is required for python-rtslib: http://article.gmane.org/gmane.linux.scsi.target.devel/8146

[Xen-devel] [PATCH 1/5] vscsiif.h: fix WWN notation for p-dev property

2015-11-13 Thread Olaf Hering
The pvops kernel expects either "naa.WWN:LUN" or "h:c:t:l" in the p-dev property. Add the missing :LUN part to the comment. Signed-off-by: Olaf Hering Acked-by: Ian Campbell Cc: Ian Campbell Cc: Ian Jackson

[Xen-devel] [OSSTEST PATCH 19/24] Osstest/Testsupport.pm: change target's default kernkind to 'pvops'

2015-11-13 Thread Ian Jackson
This is safe only if no existing flights would be affected. (That is, the meaning of no existing sets of runvars would be changed.) To check whether this would make any difference I did some database searches. Since any time target_kernkind_check is called it sets a corresponding `console'

[Xen-devel] [OSSTEST PATCH 15/24] Nested HVM: Provide test-nested recipe

2015-11-13 Thread Ian Jackson
From: Robert Ho Signed-off-by: Robert Ho Signed-off-by: Ian Jackson Tested-by: Robert Ho Acked-by: Ian Campbell --- v14: ts-nested-setup command line syntax updated. v15:

[Xen-devel] [OSSTEST PATCH 23/24] Serial: Add new serial method object for `guest' type

2015-11-13 Thread Ian Jackson
From: Robert Ho L1 guests' serial ports are owned by qemu in L0. We can send them debug keys by writing to the qemu pipe. (xl debug-key looks like it would be useful but it actually sends debug keys to the hypervisor of the host it is running on. We want to send the debug

[Xen-devel] [OSSTEST PATCH 09/24] await_tcp(): Run check_ip on each loop iteration

2015-11-13 Thread Ian Jackson
From: Robert Ho await_tcp is often invoked after a reboot. In this situation the target's IP address may change. If this happens while await_tcp is running, we would continue to poll the old IP address. Fix this by running target_check_ip on each iteration. Signed-off-by:

[Xen-devel] [OSSTEST PATCH 06/24] Nested hosts: Provide PDU power method

2015-11-13 Thread Ian Jackson
From: Robert Ho This `guest' power method uses VM create/destroy. It is automatically used for nested hosts. It would not make much sense to configure it manually. For nested host/guest, its power on/off method shall be its host invoke $(toolstack)->create/destroy method.

[Xen-devel] [OSSTEST PATCH 14/24] Nested HVM: Provide ts-nested-setup to help make L1 usable as a host

2015-11-13 Thread Ian Jackson
From: Robert Ho * Provide the L1 with some storage for its own guests' disks * Install some packages in the L1 * Optionally, set a runvar defining the L1 for the rest of the job The recipe is going to run ts-xen-install etc. Signed-off-by: longtao.pang

[Xen-devel] [OSSTEST PATCH 16/24] Nested HVM: Add test job to appropriate flights

2015-11-13 Thread Ian Jackson
From: Robert Ho Signed-off-by: longtao.pang Signed-off-by: Robert Ho Signed-off-by: Ian Jackson Tested-by: Robert Ho --- v17: Use usual_debianhvm_image v14: Use default

[Xen-devel] [OSSTEST PATCH 20/24] Osstest/Testsupport.pm: use get_target_property() for some host setup

2015-11-13 Thread Ian Jackson
From: Robert Ho For nested cases, nested host can inherit its host's property for dhcp watch setup and ether_prefix property setup. Signed-off-by: Robert Ho Tested-by: Robert Ho Acked-by: Ian Campbell ---

[Xen-devel] [OSSTEST PATCH 13/24] sg-run-job: Provide infrastructure for layers of nesting

2015-11-13 Thread Ian Jackson
Provides nested-layer-descend, which can be called in an individual test job at the appropriate point (after the L1 has been set up). The inner host is a guest of the outer host; powering it off means destroying it. Putting the poweroff at this point in the loop, rather than in per-host-finish,

[Xen-devel] [OSSTEST PATCH 03/24] selecthost: Minor cleanups

2015-11-13 Thread Ian Jackson
Document the syntax for $ident. Log the ident as well as the selected hostname. Signed-off-by: Ian Jackson Tested-by: Robert Ho Acked-by: Ian Campbell --- v17: Fix typo in doc message. v14: New patch ---

[Xen-devel] [OSSTEST PATCH v17 00/24] Nested HVM

2015-11-13 Thread Ian Jackson
This is, I think, a fully-working series to support Nested HVM testing in osstest. It can be found here: git://xenbits.xen.org/people/iwj/osstest.git http://xenbits.xen.org/git-http/people/iwj/osstest.git in base.nested-hvm.v17..wip.nested-hvm.v17 a 01 cs-adjust-flight: Add some missing

Re: [Xen-devel] [OSSTEST PATCH 02/24] cs-adjust-flight: Allow adjusting "this" flight

2015-11-13 Thread Ian Campbell
On Fri, 2015-11-13 at 12:03 +, Ian Jackson wrote: > This allows cs-adjust-flight to be run by hand to adjust runvars, in a > flight being used with hand-invocation of ./ts-* scripts. > > Signed-off-by: Ian Jackson > Tested-by: Ian Jackson

Re: [Xen-devel] [OSSTEST PATCH 04/24] make-flight: Break out usual_debianhvm_image and honour DEBIAN_IMAGE_VERSION

2015-11-13 Thread Ian Campbell
On Fri, 2015-11-13 at 12:03 +, Ian Jackson wrote: > No functional change.  (Verified for xen-unstable with > standalone-generate-dump-flight-runvars.) > > Signed-off-by: Ian Jackson > Tested-by: Ian Jackson Acked-by: Ian Campbell

Re: [Xen-devel] [OSSTEST PATCH 23/24] Serial: Add new serial method object for `guest' type

2015-11-13 Thread Ian Campbell
On Fri, 2015-11-13 at 12:03 +, Ian Jackson wrote: > From: Robert Ho > > L1 guests' serial ports are owned by qemu in L0.  We can send them > debug keys by writing to the qemu pipe. > > (xl debug-key looks like it would be useful but it actually sends > debug keys to the

Re: [Xen-devel] [PATCHv3] 00/28] Kconfig conversion

2015-11-13 Thread Julien Grall
Hi Doug, On 12/11/15 22:54, Doug Goldstein wrote: > The following series is a follow on to the Kconfig conversion patch series. > There are still more components to convert however this is the bare minimal > to get everything working and get the options out of the existing makefiles. > > The

[Xen-devel] [PATCH 0/5] libbxl: add support for pvscsi, iteration 6

2015-11-13 Thread Olaf Hering
Note: just a rebase close to "RELEASE-4.6.0", with no substantial changes Port vscsi=[] and scsi-{attach,detach,list} commands from xend to libxl. libvirt uses its existing SCSI support: http://lists.xenproject.org/archives/html/xen-devel/2015-04/msg02963.html targetcli/rtslib has to be aware

[Xen-devel] [PATCH 3/5] libxl: add support for vscsi

2015-11-13 Thread Olaf Hering
Port pvscsi support from xend to libxl: vscsi=['pdev,vdev{,options}'] xl scsi-attach xl scsi-detach xl scsi-list Signed-off-by: Olaf Hering Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell

[Xen-devel] [PATCH 4/5] vscsiif.h: add some notes about xenstore layout

2015-11-13 Thread Olaf Hering
Signed-off-by: Olaf Hering Acked-by: Ian Campbell Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan ---

[Xen-devel] [PATCH 2/5] docs: add vscsi to xenstore-paths.markdown

2015-11-13 Thread Olaf Hering
Signed-off-by: Olaf Hering Acked-by: Ian Campbell Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan ---

[Xen-devel] [OSSTEST PATCH 22/24] Serial: Factor out Osstest::Serial::keys_real

2015-11-13 Thread Ian Jackson
The sympathy and xenuse serial modules had too much in common. Factor out the common code, which is now responsible for - knowledge of the Xen console switch - splitting strings up into individual keys - timing decisions - error trapping and logging This new class is an abstract base

[Xen-devel] [OSSTEST PATCH 01/24] cs-adjust-flight: Add some missing doc comment info

2015-11-13 Thread Ian Jackson
Signed-off-by: Ian Jackson Acked-by: Ian Campbell --- v14: New patch --- cs-adjust-flight |8 1 file changed, 8 insertions(+) diff --git a/cs-adjust-flight b/cs-adjust-flight index 4bfef48..d70abf7 100755 --- a/cs-adjust-flight

[Xen-devel] [OSSTEST PATCH 02/24] cs-adjust-flight: Allow adjusting "this" flight

2015-11-13 Thread Ian Jackson
This allows cs-adjust-flight to be run by hand to adjust runvars, in a flight being used with hand-invocation of ./ts-* scripts. Signed-off-by: Ian Jackson Tested-by: Ian Jackson --- v17: Better documentation of running: flight status v14:

[Xen-devel] [OSSTEST PATCH 17/24] ts-xen-install: Properly handle hosts without a static IP address

2015-11-13 Thread Ian Jackson
From: Robert Ho Check IpStatic, and if it is not set, provide a dhcp stanza in /etc/network/interfaces, rather than an `inet static' one. This is necessary for L1 nested hosts, because they don't have a static IP address. In principle this makes matters more correct for

[Xen-devel] [OSSTEST PATCH 10/24] LVM: Break out lv_create

2015-11-13 Thread Ian Jackson
We are going to want to reuse this. lv_create doesn't (want to) take a $gho, but the $vg and $lv names directly (so that callers can use it when they don't have a suitable $gho whose $gho->{Lvdev} they want to use). In the one existing call site we pass $gho->{Vg} and $gho->{Lv} so that the

[Xen-devel] [OSSTEST PATCH 21/24] HVM guests: Use qemu "pipe:" for serial output logging

2015-11-13 Thread Ian Jackson
Modern qemu has the "pipe:/PATH" character driver. This opens /PATH.in for reading and /PATH.out for writing. In my tests, I found that: - contrary to the documentation, they do not need to be pipes (at least, /PATH.out can be a file) - but they must both already exist - qemu will

[Xen-devel] [OSSTEST PATCH 12/24] sg-run-job: Break out per-host-prep and per-host-finish

2015-11-13 Thread Ian Jackson
No functional change. We now call the per-host-ts finish steps unconditionally, rather than only if !$need_build_host, per-host-ts is (complicated) no-op if $need_build_host, since in that case $need_xen_hosts is {}. Signed-off-by: Ian Jackson Tested by: Robert Ho

[Xen-devel] [OSSTEST PATCH 08/24] target_check_ip: Rename and improve from guest_check_ip

2015-11-13 Thread Ian Jackson
Make this function suitable for running on targets with static IP addresses. (Ie, on physical hosts.) Accordingly, rename it and adjust all call sites. Signed-off-by: Ian Jackson Tested-by: Robert Ho Acked-by: Ian Campbell

[Xen-devel] [OSSTEST PATCH 04/24] make-flight: Break out usual_debianhvm_image and honour DEBIAN_IMAGE_VERSION

2015-11-13 Thread Ian Jackson
No functional change. (Verified for xen-unstable with standalone-generate-dump-flight-runvars.) Signed-off-by: Ian Jackson Tested-by: Ian Jackson --- v17: New patch. --- make-flight |2 +- mfi-common |5 + 2 files changed, 6

[Xen-devel] [OSSTEST PATCH 07/24] DhcpWatch::leases: Fix a reporting message

2015-11-13 Thread Ian Jackson
This talks about `guest_check_ip', but this code is now factored out into a method. Use the correct method name in reporting. Signed-off-by: Ian Jackson Tested-by: Robert Ho Acked-by: Ian Campbell --- v14: New patch ---

[Xen-devel] [OSSTEST PATCH 24/24] Serial::xenuse: Send xenuse output to /dev/null

2015-11-13 Thread Ian Jackson
Like sympathy, attaching via xenuse causes xenuse to send output from the host to its own stdout. But we don't want the ts-logs-capture stdout to contain this serial output, interleaved with its own log messages. We'll capture the whole serial log from the xenuse logfile. So redirect it to

[Xen-devel] [OSSTEST PATCH 18/24] ts-xen-install: networking: Rename `nodhcp' to `ensurebridge'

2015-11-13 Thread Ian Jackson
This function does not (now) always undo the DHCP configuration. Sometimes it leaves it. Its main function is to ensure that we have a bridge for use by guests. So rename the function. Signed-off-by: Ian Jackson Tested-by: Robert Ho Acked-by:

[Xen-devel] [OSSTEST PATCH 11/24] Toolstack::xl: Provide block_attach method

2015-11-13 Thread Ian Jackson
It is possible that this may work some of the time with xm, so I have taken no measures to prevent it running then. Signed-off-by: Ian Jackson Signed-off-by: Robert Hu Tested-by: Robert Hu Acked-by: Ian Campbell

[Xen-devel] [OSSTEST PATCH 05/24] selecthost: Support nested hosts (guests which are also hosts)

2015-11-13 Thread Ian Jackson
We introduce a new syntax: instead of a hostname (which might appear in a command line argument to a ts-* script and hence be passed to selecthost, or which might be in a runvar), we now support :. Such `hosts' (let us refer to such a thing as an L1, although in principle further nesting may be

Re: [Xen-devel] [PATCH v5 2/6] xen/arm: vgic-v2: Handle correctly byte write in ITARGETSR

2015-11-13 Thread Stefano Stabellini
On Mon, 9 Nov 2015, Julien Grall wrote: > During a store, the byte is always in the low part of the register (i.e > [0:7]). > > We are incorrectly masking the register by using a shift of the byte > offset in the ITARGETSR while the byte is alwasy in r[0:7]. This will > result in a target list

Re: [Xen-devel] [OSSTEST PATCH 24/24] Serial::xenuse: Send xenuse output to /dev/null

2015-11-13 Thread Ian Campbell
On Fri, 2015-11-13 at 12:03 +, Ian Jackson wrote: > Like sympathy, attaching via xenuse causes xenuse to send output from > the host to its own stdout. > > But we don't want the ts-logs-capture stdout to contain this serial > output, interleaved with its own log messages.  We'll capture the >

[Xen-devel] [PATCH QEMU v2 2/2] xen: fix usage of xc_domain_create in domain builder

2015-11-13 Thread Roger Pau Monne
Due to the addition of HVMlite and the requirement to always provide a valid xc_domain_configuration_t, xc_domain_create now always takes an arch domain config, which can be NULL in order to mimic previous behaviour. Add a small stub called xen_domain_create that encapsulates the correct call to

Re: [Xen-devel] Uniform commands for booting xen

2015-11-13 Thread Ian Campbell
On Fri, 2015-11-13 at 12:04 +0300, Andrei Borzenkov wrote: > > How do you express modules other than kernel+initrd in that > > scheme, without grub needing to be aware of any new addition we > > may find necessary going forward? > > > > Are modules used by Xen self-identifying? Is it enough to

Re: [Xen-devel] [PATCHv3] 03/28] arm: drop now redefined CONFIG_64BIT

2015-11-13 Thread Jan Beulich
>>> On 12.11.15 at 23:54, wrote: > The switch to Kconfig provides variables prefixed with CONFIG_ with > results in a redefinition of this variable. So I can't see how this can be a separate patch: Either the redefinition causes a build failure after the previous patch (if

Re: [Xen-devel] Reproducible hang with cstate disabled

2015-11-13 Thread Jan Beulich
>>> On 13.11.15 at 06:47, wrote: > I saw xen hang after > > (XEN) HVM: HAP page sizes: 4kB, 2MB, 1GB > ( > > The line that's supposed to be there is > > (XEN) Brought up 24 CPUs > > After power cycling I went into the BIOS. In the BIOS, C-STATE was disabled. > I changed it to

Re: [Xen-devel] [RFC] libxl: relax readonly check introduced by XSA-142 fix

2015-11-13 Thread Ian Campbell
On Thu, 2015-11-12 at 10:53 -0700, Jim Fehlig wrote: > Ian Campbell wrote: > > On Thu, 2015-11-12 at 08:45 -0700, Jim Fehlig wrote: > > >   > > > > The commit message doesn't say anything about AHCI. Are AHCI disks > > > > actually emulated correctly by QEMU with readonly=on? > > > I just double

Re: [Xen-devel] [PATCH v5 1/9] libxc: reorganize domain builder guest memory allocator

2015-11-13 Thread Ian Campbell
On Thu, 2015-11-12 at 15:55 +, Wei Liu wrote: >  > Note that my Wheezy  PV installation is using grub2 so pvgrub doesn't > seem to be able to parse its config file FYI you can workaround this by installing the "pv-grub-menu" package (in wheezy-backports and from Jessie onwards) which provides

Re: [Xen-devel] Uniform commands for booting xen

2015-11-13 Thread Ian Campbell
On Fri, 2015-11-13 at 00:48 -0700, Jan Beulich wrote: > > > > On 12.11.15 at 18:09, wrote: > > On Thu, 2015-11-12 at 08:44 -0700, Jan Beulich wrote: > > > > > > On 12.11.15 at 14:41, wrote: > > > > Hello, all. I'd like to have set of commands that

Re: [Xen-devel] Uniform commands for booting xen

2015-11-13 Thread Andrei Borzenkov
On Fri, Nov 13, 2015 at 10:48 AM, Jan Beulich wrote: On 12.11.15 at 18:09, wrote: >> On Thu, 2015-11-12 at 08:44 -0700, Jan Beulich wrote: >>> > > > On 12.11.15 at 14:41, wrote: >>> > Hello, all. I'd like to have set of

Re: [Xen-devel] [PATCH v3 2/4] docs: Document a xenstore path for PV driver version information...

2015-11-13 Thread Ian Jackson
Paul Durrant writes ("[Xen-devel] [PATCH v3 2/4] docs: Document a xenstore path for PV driver version information..."): > diff --git a/docs/misc/xenstore-paths.markdown > b/docs/misc/xenstore-paths.markdown > index 71dfda2..cae5115 100644 > --- a/docs/misc/xenstore-paths.markdown > +++

Re: [Xen-devel] [PATCH XEN v5 19/23] tools/libs/call: Update some log messages to not refer to xc.

2015-11-13 Thread Andrew Cooper
On 09/11/15 12:00, Ian Campbell wrote: > Signed-off-by: Ian Campbell > Acked-by: Wei Liu I agree with the sentiment, but the error message refer to the libxencall entry API, so a developer can match it back to the call in their code. ~Andrew > ---

Re: [Xen-devel] [PATCH] x86/IO-APIC: fix setup of Xen internally used IRQs (take 2)

2015-11-13 Thread Andrew Cooper
On 13/11/15 15:41, Jan Beulich wrote: > ..., i.e. namely that of a PCI serial card with an IRQ above the > legacy range. This had got broken by the switch to cpumask_any() in > cpu_mask_to_apicid_phys(). Fix this by allowing all CPUs for that IRQ > (via setup_vector_irq() properly updating a

Re: [Xen-devel] [PATCH v5 3/6] xen/arm: vgic-v2: Don't ignore a write in ITARGETSR if one field is 0

2015-11-13 Thread Julien Grall
On 13/11/15 14:37, Stefano Stabellini wrote: > On Mon, 9 Nov 2015, Julien Grall wrote: >> +#define NR_TARGETS_PER_ITARGETSR4U >> +#define NR_BITS_PER_TARGET (32U / NR_TARGETS_PER_ITARGETSR) >> + >> +/* >> + * Store an ITARGETSR register. This function only deals with ITARGETSR8 >> + * and

Re: [Xen-devel] [PATCH XEN v5 02/23] tools: Refactor "xentoollog" into its own library

2015-11-13 Thread Ian Campbell
On Fri, 2015-11-13 at 15:05 +, Andrew Cooper wrote: > On 09/11/15 12:00, Ian Campbell wrote: > > diff --git a/tools/ocaml/Makefile.rules b/tools/ocaml/Makefile.rules > > index 1796060..45e71f2 100644 > > --- a/tools/ocaml/Makefile.rules > > +++ b/tools/ocaml/Makefile.rules > > @@ -1,18 +1,18

Re: [Xen-devel] [PATCH v3 0/4] docs: Document xenstore paths

2015-11-13 Thread Ian Jackson
Paul Durrant writes ("[Xen-devel] [PATCH v3 0/4] docs: Document xenstore paths"): > Patch #1 documents paths, some already in used by XenServer, which can be > used by guests to advertise contol capabilities. Thanks for these patches. I am pleased to see what for upstream Xen are new facilities

[Xen-devel] [PATCH v2] xen: sched: fix (ACPI S3) resume with cpupools with different schedulers.

2015-11-13 Thread Dario Faggioli
In fact, with 2 cpupools, one (the default) Credit and one Credit2 (with at least 1 pCPU in the latter), trying a (e.g., ACPI S3) suspend/resume crashes like this: (XEN) [ 150.587779] [ Xen-4.7-unstable x86_64 debug=y Not tainted ] (XEN) [ 150.587783] CPU:6 (XEN) [ 150.587786]

Re: [Xen-devel] [PATCH XEN v5 02/23] tools: Refactor "xentoollog" into its own library

2015-11-13 Thread Andrew Cooper
On 09/11/15 12:00, Ian Campbell wrote: > diff --git a/tools/ocaml/Makefile.rules b/tools/ocaml/Makefile.rules > index 1796060..45e71f2 100644 > --- a/tools/ocaml/Makefile.rules > +++ b/tools/ocaml/Makefile.rules > @@ -1,18 +1,18 @@ > -ifdef V > - ifeq ("$(origin V)", "command line") > -

[Xen-devel] [PATCH] x86: drop hard_smp_procssor_id()

2015-11-13 Thread Jan Beulich
... and use what it aliased to directly. Signed-off-by: Jan Beulich --- a/xen/arch/x86/cpu/common.c +++ b/xen/arch/x86/cpu/common.c @@ -162,13 +162,13 @@ static inline u32 _phys_pkg_id(u32 cpuid /* * cpuid returns the value latched in the HW at reset, not the APIC ID *

Re: [Xen-devel] [PATCH] x86: drop hard_smp_procssor_id()

2015-11-13 Thread Andrew Cooper
On 13/11/15 15:46, Jan Beulich wrote: > ... and use what it aliased to directly. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v3 0/4] docs: Document xenstore paths

2015-11-13 Thread Paul Durrant
> -Original Message- > From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] > Sent: 13 November 2015 16:10 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org > Subject: Re: [Xen-devel] [PATCH v3 0/4] docs: Document xenstore paths > > Paul Durrant writes ("[Xen-devel] [PATCH v3 0/4]

Re: [Xen-devel] [PATCH v3 2/4] docs: Document a xenstore path for PV driver version information...

2015-11-13 Thread Ian Jackson
Paul Durrant writes ("RE: [Xen-devel] [PATCH v3 2/4] docs: Document a xenstore path for PV driver version information..."): > [Ian J:] > > If it is to be used for bug workaround, it definitely needs to be more > > structured. > > No, that's not the intention. The intention is that the

Re: [Xen-devel] [PATCH XEN v5 09/23] tools: Refactor hypercall calling wrappers into libxencall.

2015-11-13 Thread Andrew Cooper
On 09/11/15 12:00, Ian Campbell wrote: > diff --git a/tools/libs/call/include/xencall.h > b/tools/libs/call/include/xencall.h > new file mode 100644 > index 000..a0b3591 > --- /dev/null > +++ b/tools/libs/call/include/xencall.h > @@ -0,0 +1,84 @@ > +/* > + * This library is free software; you

Re: [Xen-devel] [PATCH v5 4/6] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-11-13 Thread Stefano Stabellini
On Mon, 9 Nov 2015, Julien Grall wrote: > Xen is currently directly storing the value of GICD_ITARGETSR register > (for GICv2) and GICD_IROUTER (for GICv3) in the rank. This makes the > emulation of the registers access very simple but makes the code to get > the target vCPU for a given vIRQ more

[Xen-devel] [PATCH] x86/IO-APIC: adjust setting of destinations

2015-11-13 Thread Jan Beulich
setup_IO_APIC_irqs() runs before APs get brought up, so using desc->arch.cpu_mask as best risks it being either empty or having bits for CPUs other than the BP set. Just use the APIC ID of the only online CPU directly. Replace a stray hard_smp_processor_id() at once. Signed-off-by: Jan Beulich

Re: [Xen-devel] [PATCH v3 4/4] docs: Document xenstore paths for domain network address information

2015-11-13 Thread Paul Durrant
> -Original Message- > From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] > Sent: 13 November 2015 16:18 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; Ian Campbell; Jan Beulich; Keir (Xen.org); > Tim (Xen.org) > Subject: Re: [PATCH v3 4/4] docs: Document xenstore paths for

Re: [Xen-devel] [PATCH v3 0/4] docs: Document xenstore paths

2015-11-13 Thread Paul Durrant
> -Original Message- > From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] > Sent: 13 November 2015 16:20 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org > Subject: RE: [Xen-devel] [PATCH v3 0/4] docs: Document xenstore paths > > Paul Durrant writes ("RE: [Xen-devel] [PATCH v3

Re: [Xen-devel] [PATCH XEN v5 09/23] tools: Refactor hypercall calling wrappers into libxencall.

2015-11-13 Thread Roger Pau Monné
El 13/11/15 a les 17.57, Ian Campbell ha escrit: > On Fri, 2015-11-13 at 15:16 +, Ian Campbell wrote: >> >> On FreeBSD hypercall->retval is in the XEN_ERRNO_* space. But I think we >> probably want to map onto the FreeBSD ERRNO space, IIRC we decided on this >> approach, rather than converting

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-13 Thread Ian Campbell
On Wed, 2015-11-11 at 15:03 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH XEN v5 07/23] tools: Refactor > /dev/xen/gnt{dev,shr} wrappers into libxengnttab."): > > libxengnttab will provide a stable API and ABI for accessing the > > grant table devices. > > > > The functions are moved

Re: [Xen-devel] [PATCH] xen: sched: fix (ACPI S3) resume with cpupools with different schedulers.

2015-11-13 Thread Juergen Gross
On 13/11/15 11:08, Dario Faggioli wrote: > In fact, with 2 cpupools, one (the default) Credit and > one Credit2 (with at least 1 pCPU in the latter), trying > a (e.g., ACPI S3) suspend/resume crashes like this: > > (XEN) [ 150.587779] [ Xen-4.7-unstable x86_64 debug=y Not tainted > ]

Re: [Xen-devel] [PATCH v5 4/6] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-11-13 Thread Julien Grall
On 13/11/15 15:44, Stefano Stabellini wrote: > On Mon, 9 Nov 2015, Julien Grall wrote: >> -static struct vcpu *vgic_v3_get_target_vcpu(struct vcpu *v, unsigned int irq) >> -{ >> -struct vcpu *v_target; >> -struct vgic_irq_rank *rank = vgic_rank_irq(v, irq); >> +#define

[Xen-devel] build issue due to '-Werror=cast-align' on ARM (armhf) [was: Re: [libvirt test] 63528: regressions - FAIL]

2015-11-13 Thread Dario Faggioli
Hello, The Xen Project's automated test suite is failing at running its libvirt tests for a few time, like this: On Wed, 2015-11-04 at 09:04 +, osstest service owner wrote: > flight 63528 libvirt real [real] > http://logs.test-lab.xenproject.org/osstest/logs/63528/ > > Regressions :-( > >

[Xen-devel] 9p file system for xen

2015-11-13 Thread Linda
Hello, I worked this summer as an intern under Julien Grall and Wei Liu. My project was to develop a prototype/proof of concept xen front/back end for the 9p file system. I mostly hacked the virtio 9p system. This project was not complete, at the end of the summer. Julien said that

Re: [Xen-devel] [PATCH XEN v5 09/23] tools: Refactor hypercall calling wrappers into libxencall.

2015-11-13 Thread Andrew Cooper
On 09/11/15 12:00, Ian Campbell wrote: > diff --git a/tools/libs/call/linux.c b/tools/libs/call/linux.c > new file mode 100644 > index 000..906ca7e > --- /dev/null > +++ b/tools/libs/call/linux.c > @@ -0,0 +1,132 @@ > +/* > + * This library is free software; you can redistribute it and/or > +

Re: [Xen-devel] [PATCH XEN v5 14/23] tools: foreignmemory: provide xenforeignmemory_unmap.

2015-11-13 Thread Andrew Cooper
On 13/11/15 16:16, Andrew Cooper wrote: > On 09/11/15 12:00, Ian Campbell wrote: >> diff --git a/tools/libs/foreignmemory/linux.c >> b/tools/libs/foreignmemory/linux.c >> index 01cd42e..86a5a97 100644 >> --- a/tools/libs/foreignmemory/linux.c >> +++ b/tools/libs/foreignmemory/linux.c >> @@ -276,6

Re: [Xen-devel] [PATCH v3 2/4] docs: Document a xenstore path for PV driver version information...

2015-11-13 Thread Paul Durrant
> -Original Message- > From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] > Sent: 13 November 2015 16:12 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; Keir (Xen.org); Ian Campbell; Tim > (Xen.org); Jan Beulich > Subject: Re: [Xen-devel] [PATCH v3 2/4] docs: Document a xenstore

Re: [Xen-devel] [PATCH v3 4/4] docs: Document xenstore paths for domain network address information

2015-11-13 Thread Ian Jackson
Paul Durrant writes ("[PATCH v3 4/4] docs: Document xenstore paths for domain network address information"): > + ~/attr/vif/$DEVID/name = STRING [w] You need to explicitly state what this information is to be used for. This "name" is presumably useful for humans in the management UI ? You

[Xen-devel] [PATCH QEMU v3 2/2] xen: fix usage of xc_domain_create in domain builder

2015-11-13 Thread Roger Pau Monne
Due to the addition of HVMlite and the requirement to always provide a valid xc_domain_configuration_t, xc_domain_create now always takes an arch domain config, which can be NULL in order to mimic previous behaviour. Add a small stub called xen_domain_create that encapsulates the correct call to

Re: [Xen-devel] [PATCH v5 3/6] xen/arm: vgic-v2: Don't ignore a write in ITARGETSR if one field is 0

2015-11-13 Thread Stefano Stabellini
On Fri, 13 Nov 2015, Julien Grall wrote: > On 13/11/15 14:37, Stefano Stabellini wrote: > > On Mon, 9 Nov 2015, Julien Grall wrote: > >> +#define NR_TARGETS_PER_ITARGETSR4U > >> +#define NR_BITS_PER_TARGET (32U / NR_TARGETS_PER_ITARGETSR) > >> + > >> +/* > >> + * Store an ITARGETSR register.

Re: [Xen-devel] [PATCH XEN v5 13/23] tools: Refactor foreign memory mapping into libxenforeignmemory

2015-11-13 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH XEN v5 13/23] tools: Refactor foreign memory mapping into libxenforeignmemory"): > On Wed, 2015-11-11 at 15:13 +, Ian Jackson wrote: > Are you suggesting: > > void *xenforeignmemory_map(xenforeignmemory_handle *fmem, uint32_t dom, >    

Re: [Xen-devel] [PATCH v5 3/6] xen/arm: vgic-v2: Don't ignore a write in ITARGETSR if one field is 0

2015-11-13 Thread Julien Grall
On 13/11/15 15:05, Stefano Stabellini wrote: > On Fri, 13 Nov 2015, Julien Grall wrote: >> On 13/11/15 14:37, Stefano Stabellini wrote: >>> On Mon, 9 Nov 2015, Julien Grall wrote: +for ( i = 0; i < NR_TARGETS_PER_ITARGETSR; i++, offset++, virq++ ) >>> >>> offset is not needed in the loop

[Xen-devel] [PATCH] x86/IO-APIC: fix setup of Xen internally used IRQs (take 2)

2015-11-13 Thread Jan Beulich
..., i.e. namely that of a PCI serial card with an IRQ above the legacy range. This had got broken by the switch to cpumask_any() in cpu_mask_to_apicid_phys(). Fix this by allowing all CPUs for that IRQ (via setup_vector_irq() properly updating a booting CPU's vector_irq[], thus avoiding "No irq

Re: [Xen-devel] [PATCH v3 0/4] docs: Document xenstore paths

2015-11-13 Thread Ian Jackson
Paul Durrant writes ("RE: [Xen-devel] [PATCH v3 0/4] docs: Document xenstore paths"): > [Ian:] > > And the documents themselves should then clearly state what the > > behaviour should be on the part of a toolstack which finds the > > information missing, or set to a particular values. > > Yes, I

Re: [Xen-devel] [PATCH XEN v5 09/23] tools: Refactor hypercall calling wrappers into libxencall.

2015-11-13 Thread Ian Campbell
On Fri, 2015-11-13 at 18:16 +0100, Roger Pau Monné wrote: > El 13/11/15 a les 17.57, Ian Campbell ha escrit: > > On Fri, 2015-11-13 at 15:16 +, Ian Campbell wrote: > > > > > > On FreeBSD hypercall->retval is in the XEN_ERRNO_* space. But I think > > > we > > > probably want to map onto the

Re: [Xen-devel] [PATCH XEN v5 09/23] tools: Refactor hypercall calling wrappers into libxencall.

2015-11-13 Thread Ian Campbell
On Wed, 2015-11-11 at 15:31 +, Wei Liu wrote: > On Wed, Nov 11, 2015 at 03:08:13PM +, Ian Jackson wrote: > > Ian Campbell writes ("[PATCH XEN v5 09/23] tools: Refactor hypercall > > calling wrappers into libxencall."): > > > libxencall will provide a stable API and ABI for calling

Re: [Xen-devel] [PATCH XEN v5 09/23] tools: Refactor hypercall calling wrappers into libxencall.

2015-11-13 Thread Ian Campbell
On Wed, 2015-11-11 at 15:08 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH XEN v5 09/23] tools: Refactor hypercall > calling wrappers into libxencall."): > > libxencall will provide a stable API and ABI for calling hypercalls > > (although those hypercalls themselves may not have a

Re: [Xen-devel] [PATCH XEN v5 13/23] tools: Refactor foreign memory mapping into libxenforeignmemory

2015-11-13 Thread Ian Campbell
On Wed, 2015-11-11 at 15:13 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH XEN v5 13/23] tools: Refactor foreign memory > mapping into libxenforeignmemory"): > > libxenforeignmemory will provide a stable API and ABI for mapping > > foreign domain memory (subject to appropriate

Re: [Xen-devel] [PATCH XEN v5 14/23] tools: foreignmemory: provide xenforeignmemory_unmap.

2015-11-13 Thread Andrew Cooper
On 09/11/15 12:00, Ian Campbell wrote: > diff --git a/tools/libs/foreignmemory/linux.c > b/tools/libs/foreignmemory/linux.c > index 01cd42e..86a5a97 100644 > --- a/tools/libs/foreignmemory/linux.c > +++ b/tools/libs/foreignmemory/linux.c > @@ -276,6 +276,12 @@ void

Re: [Xen-devel] [PATCH] x86/IO-APIC: adjust setting of destinations

2015-11-13 Thread Andrew Cooper
On 13/11/15 15:44, Jan Beulich wrote: > setup_IO_APIC_irqs() runs before APs get brought up, so using > desc->arch.cpu_mask as best risks it being either empty or having bits > for CPUs other than the BP set. Just use the APIC ID of the only > online CPU directly. Replace a stray

Re: [Xen-devel] [PATCH v3 4/4] docs: Document xenstore paths for domain network address information

2015-11-13 Thread Ian Jackson
Paul Durrant writes ("RE: [PATCH v3 4/4] docs: Document xenstore paths for domain network address information"): > [Ian:] > > The primary (virtual hardware) VIF MAC address is already recorded by > > the toolstack in the PV protocol backend directory. > > > > What is this for ? > > The frontend

Re: [Xen-devel] [PATCH V2] libxl: relax readonly check introduced by XSA-142 fix

2015-11-13 Thread Jim Fehlig
On 11/12/2015 07:40 PM, Jim Fehlig wrote: > The fix for XSA-142 is quite a big hammer, rejecting readonly > disk configuration even when the requested backend is known to > support readonly. While it is true that qemu doesn't support > readonly for emulated IDE or AHCI disks > > $

  1   2   >