Re: [Xen-devel] [v2][PATCH] xen/vtd/iommu: permit group devices to passthrough in relaxed mode

2015-09-14 Thread Tian, Kevin
> From: Chen, Tiejun > Sent: Monday, September 14, 2015 2:25 PM > > > OK, that explanation is fine to me as long as it's made clear no > > security guarantee once admin uses 'relax' for any domain. Tiejun > > could you resend patch with right warning/error type? > > > > Sure, but a little bit mak

Re: [Xen-devel] [v2][PATCH] xen/vtd/iommu: permit group devices to passthrough in relaxed mode

2015-09-14 Thread Tian, Kevin
> From: Chen, Tiejun > Sent: Wednesday, September 09, 2015 10:00 AM > > Currently we don't allow passing through any group devices which are > sharing same RMRR entry since it would break security among VMs. And > indeed, we expect we can figure out a better way to handle this kind > of case compl

[Xen-devel] [PATCH] x86/p2m: fix mismatched unlock

2015-09-14 Thread Jan Beulich
Luckily, due to gfn_unlock() currently mapping to p2m_unlock(), this is only a cosmetic issue right now. Signed-off-by: Jan Beulich --- Despite its cosmetic nature I think it would be better to also fix this in 4.6. --- a/xen/arch/x86/mm/p2m.c +++ b/xen/arch/x86/mm/p2m.c @@ -912,7 +912,7 @@ stat

[Xen-devel] [seabios test] 61803: tolerable FAIL - PUSHED

2015-09-14 Thread osstest service owner
flight 61803 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/61803/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 17 guest-stop fail like 61525 Tests which did not succeed, but a

Re: [Xen-devel] [PATCH v3 1/4] x86: Support enable CDP by boot parameter and add get CDP status

2015-09-14 Thread Chao Peng
On Mon, Sep 14, 2015 at 11:27:04AM +0800, He Chen wrote: > @@ -1165,9 +1165,9 @@ This option can be specified more than once (up to 8 > times at present). > > `= ` > > ### psr (Intel) > -> `= List of ( cmt: | rmid_max: | cat: | > cos_max: )` > +> `= List of ( cmt: | rmid_max: | cat: | > cos_

Re: [Xen-devel] [PATCH] x86/p2m: fix mismatched unlock

2015-09-14 Thread Wei Liu
On Mon, Sep 14, 2015 at 01:13:31AM -0600, Jan Beulich wrote: > Luckily, due to gfn_unlock() currently mapping to p2m_unlock(), this is > only a cosmetic issue right now. > > Signed-off-by: Jan Beulich Reviewed-by: Wei Liu > --- > Despite its cosmetic nature I think it would be better to also f

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-14 Thread Roger Pau Monné
El 07/09/15 a les 17.33, Julien Grall ha escrit: > Hi all, > > ARM64 Linux is supporting both 4KB and 64KB page granularity. Although, Xen > hypercall interface and PV protocol are always based on 4KB page granularity. > > Any attempt to boot a Linux guest with 64KB pages enabled will result to a

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Shannon Zhao
On 2015/9/11 23:45, Daniel Kiper wrote: > On Fri, Sep 11, 2015 at 03:30:15PM +0200, Ard Biesheuvel wrote: >> On 11 September 2015 at 15:14, Stefano Stabellini >> wrote: >>> On Fri, 11 Sep 2015, Daniel Kiper wrote: On Thu, Sep 10, 2015 at 05:23:02PM +0100, Mark Rutland wrote: >>> C) When

Re: [Xen-devel] [PATCH v4 0/5] sched: credit2: introduce per-vcpu hard and soft affinity

2015-09-14 Thread Dario Faggioli
On Sun, 2015-07-12 at 22:13 -1000, Justin T. Weaver wrote: > Hello, > Hey Justin! Sorry a ton for being so late again. I'm now back at work after travelling and some vacations. I'll be reviewing this ASAP, right after having done a bit of catch up with the email backlog accumulated during this

Re: [Xen-devel] [PATCH for-4.6] libxl: handle read-only drives with qemu-xen

2015-09-14 Thread Wei Liu
On Fri, Sep 11, 2015 at 06:00:29PM +0100, Stefano Stabellini wrote: > The current libxl code doesn't deal with read-only drives at all. > > Upstream QEMU and qemu-xen only support read-only cdrom drives: make > sure to specify "readonly=on" for cdrom drives and return error in case > the user requ

Re: [Xen-devel] [PATCH v5 2/2] xen/arm: support gzip compressed kernels

2015-09-14 Thread Ian Campbell
On Fri, 2015-09-11 at 17:20 +0100, Stefano Stabellini wrote: > > > > + > > > +return 0; > > > +} > > > + > > > #ifdef CONFIG_ARM_64 > > > /* > > > * Check if the image is a 64-bit Image. > > > @@ -463,6 +522,22 @@ int kernel_probe(struct kernel_info *info) > > > printk("Loading ra

Re: [Xen-devel] [PATCH v3 2/4] x86: add domctl cmd to set/get CDP code/data CBM

2015-09-14 Thread Chao Peng
On Mon, Sep 14, 2015 at 11:27:05AM +0800, He Chen wrote: > CDP extends CAT and provides the capacity to control L3 code & data > cache. With CDP, one COS correspond to two CMBs(code & data). cbm_type ^s > is added to support distinguish different CBM operation. B

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Ard Biesheuvel
On 14 September 2015 at 10:42, Shannon Zhao wrote: [..] > > It only needs to apply following patch to fix a bug in Linux kernel when > mapping EFI_MEMORY_RUNTIME memory. > Could you explain why you think efi_virtmap_init() should fail if there are no EFI_MEMORY_RUNTIME regions? The absence of s

Re: [Xen-devel] [v2][PATCH] xen/vtd/iommu: permit group devices to passthrough in relaxed mode

2015-09-14 Thread Wei Liu
On Mon, Sep 14, 2015 at 06:59:56AM +, Tian, Kevin wrote: > > From: Chen, Tiejun > > Sent: Wednesday, September 09, 2015 10:00 AM > > > > Currently we don't allow passing through any group devices which are > > sharing same RMRR entry since it would break security among VMs. And > > indeed, we

Re: [Xen-devel] [PATCH v3 3/4] tools: add tools support for Intel CDP

2015-09-14 Thread Chao Peng
On Mon, Sep 14, 2015 at 11:27:06AM +0800, He Chen wrote: > This is the xl/xc changes to support Intel Code/Data Prioritization. > CAT xl commands to set/get CBMs are extended to support CDP. > > Signed-off-by: He Chen > --- > #define LIBXL_HAVE_PSR_CAT 1 > + ? And you still need to broadcast the

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-14 Thread Ian Campbell
On Fri, 2015-09-11 at 17:28 +0200, Laszlo Ersek wrote: > [...] > For me that's not so clear-cut. OVMF is frequently used as a UEFI > development environment (it's better to brick a virtual machine than > your physical dev platform...) One flip side to this is that people often virtualize in order

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Mark Rutland
On Sat, Sep 12, 2015 at 12:36:55PM +0100, Daniel Kiper wrote: > On Fri, Sep 11, 2015 at 05:25:59PM +0100, Mark Rutland wrote: > > On Fri, Sep 11, 2015 at 01:46:43PM +0100, Daniel Kiper wrote: > > > On Thu, Sep 10, 2015 at 05:23:02PM +0100, Mark Rutland wrote: > > [...] > > > > > What's troublesom

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Shannon Zhao
On 2015/9/14 17:09, Ard Biesheuvel wrote: > On 14 September 2015 at 10:42, Shannon Zhao wrote: > [..] > >> >> It only needs to apply following patch to fix a bug in Linux kernel when >> mapping EFI_MEMORY_RUNTIME memory. >> > > Could you explain why you think efi_virtmap_init() should fail if

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Ard Biesheuvel
(snip some cc's) On 14 September 2015 at 11:31, Shannon Zhao wrote: > > > On 2015/9/14 17:09, Ard Biesheuvel wrote: >> On 14 September 2015 at 10:42, Shannon Zhao wrote: >> [..] >> >>> >>> It only needs to apply following patch to fix a bug in Linux kernel when >>> mapping EFI_MEMORY_RUNTIME mem

Re: [Xen-devel] [PATCH v2] arm/xen: Enable user access to the kernel before issuing a privcmd call

2015-09-14 Thread Stefano Stabellini
On Fri, 11 Sep 2015, Russell King - ARM Linux wrote: > On Fri, Sep 11, 2015 at 06:56:50PM +0100, Stefano Stabellini wrote: > > On Fri, 11 Sep 2015, Russell King - ARM Linux wrote: > > > On Fri, Sep 11, 2015 at 06:36:05PM +0100, Julien Grall wrote: > > > > On 11/09/15 18:32, Julien Grall wrote: > >

Re: [Xen-devel] [PATCH v3 4/4] docs: add document to introduce CDP command

2015-09-14 Thread Chao Peng
On Mon, Sep 14, 2015 at 11:27:07AM +0800, He Chen wrote: > Add new CDP option with CAT commands in xl interface man page. ^s to > Add description of CDP in xl-psr.markdown. > > Signed-off-by: He Chen > --- > docs/man/xl.pod.1 | 14 ++ > docs/misc/xl-psr.mar

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Ard Biesheuvel
On 14 September 2015 at 11:25, Mark Rutland wrote: > On Sat, Sep 12, 2015 at 12:36:55PM +0100, Daniel Kiper wrote: >> On Fri, Sep 11, 2015 at 05:25:59PM +0100, Mark Rutland wrote: >> > On Fri, Sep 11, 2015 at 01:46:43PM +0100, Daniel Kiper wrote: >> > > On Thu, Sep 10, 2015 at 05:23:02PM +0100, Ma

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Stefano Stabellini
On Mon, 14 Sep 2015, Mark Rutland wrote: > On Sat, Sep 12, 2015 at 12:36:55PM +0100, Daniel Kiper wrote: > > On Fri, Sep 11, 2015 at 05:25:59PM +0100, Mark Rutland wrote: > > > On Fri, Sep 11, 2015 at 01:46:43PM +0100, Daniel Kiper wrote: > > > > On Thu, Sep 10, 2015 at 05:23:02PM +0100, Mark Rutla

Re: [Xen-devel] [PATCH] x86/p2m: fix mismatched unlock

2015-09-14 Thread Andrew Cooper
On 14/09/15 08:13, Jan Beulich wrote: > Luckily, due to gfn_unlock() currently mapping to p2m_unlock(), this is > only a cosmetic issue right now. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen

Re: [Xen-devel] [PATCH] libxl: slightly refine pci-assignable-{add, remove} handling

2015-09-14 Thread George Dunlap
On Thu, Sep 10, 2015 at 1:36 PM, Jan Beulich wrote: > While it appears to be intentional for "xl pci-assignable-remove" to > not re-bind the original driver by default (requires the -r option), > permanently losing the information which driver was originally used > seems bad. Make "add; remove; ad

Re: [Xen-devel] [PATCH for-4.6] libxl: handle read-only drives with qemu-xen

2015-09-14 Thread Stefano Stabellini
On Mon, 14 Sep 2015, Wei Liu wrote: > On Fri, Sep 11, 2015 at 06:00:29PM +0100, Stefano Stabellini wrote: > > The current libxl code doesn't deal with read-only drives at all. > > > > Upstream QEMU and qemu-xen only support read-only cdrom drives: make > > sure to specify "readonly=on" for cdrom d

Re: [Xen-devel] [PATCH v3 0/4] Intel Code and Data Prioritization (CDP) feature enabling

2015-09-14 Thread Chao Peng
On Mon, Sep 14, 2015 at 11:27:03AM +0800, He Chen wrote: > Changes in v3: > - x86: remove redundant CDP field in cat_socket_enable (suggested by Chao) > - tools: simplify CBM setting function in tools (suggested by Jan) > - docs: Add boot parameter description (suggested by Chao & Ian) > - code sty

Re: [Xen-devel] [PATCH for-4.6] libxl: handle read-only drives with qemu-xen

2015-09-14 Thread Wei Liu
On Mon, Sep 14, 2015 at 10:49:00AM +0100, Stefano Stabellini wrote: > On Mon, 14 Sep 2015, Wei Liu wrote: > > On Fri, Sep 11, 2015 at 06:00:29PM +0100, Stefano Stabellini wrote: > > > The current libxl code doesn't deal with read-only drives at all. > > > > > > Upstream QEMU and qemu-xen only supp

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Ian Campbell
On Mon, 2015-09-14 at 11:43 +0200, Ard Biesheuvel wrote: > Xen will not boot the kernel via the stub, but directly. It needs to > supply a EFI like environment so that the kernel can boot via ACPI. > There is no reason whatsoever to mock up boot services or other pieces > of UEFI functionality tha

Re: [Xen-devel] [PULL 0/19] xen-2015-09-08-tag

2015-09-14 Thread Paolo Bonzini
On 10/09/2015 12:29, Stefano Stabellini wrote: > +if (lseek(config_fd, pos, SEEK_SET) != pos) { > +return -errno; > +} > do { > -rc = pread(config_fd, (uint8_t *)&val, len, pos); > +rc = read(config_fd, (uint8_t *)&val, len); > } while (rc < 0 && (errno =

Re: [Xen-devel] [PULL 21/29] xen/pt: Sync up the dev.config and data values.

2015-09-14 Thread Paolo Bonzini
On 10/09/2015 19:15, Stefano Stabellini wrote: > + > +switch (reg->size) { > +case 1: rc = xen_host_pci_get_byte(&s->real_device, offset, (uint8_t > *)&val); A bit ugly, and it relies on the host being little endian. > +break; > +case 2: rc = xen_host_pc

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Ard Biesheuvel
On 14 September 2015 at 11:57, Ian Campbell wrote: > On Mon, 2015-09-14 at 11:43 +0200, Ard Biesheuvel wrote: > >> Xen will not boot the kernel via the stub, but directly. It needs to >> supply a EFI like environment so that the kernel can boot via ACPI. >> There is no reason whatsoever to mock up

Re: [Xen-devel] [PATCH v2] arm/xen: Enable user access to the kernel before issuing a privcmd call

2015-09-14 Thread Russell King - ARM Linux
On Mon, Sep 14, 2015 at 10:36:55AM +0100, Stefano Stabellini wrote: > On Fri, 11 Sep 2015, Russell King - ARM Linux wrote: > > If you'd like your ack on it, please send one, I can still do that. > > Acked-by: Stefano Stabellini Sorry, that's a bit too late - I sent Linus the pull request on Satu

Re: [Xen-devel] [PATCH for-4.6] libxl: handle read-only drives with qemu-xen

2015-09-14 Thread Ian Jackson
Stefano Stabellini writes ("[PATCH for-4.6] libxl: handle read-only drives with qemu-xen"): > The current libxl code doesn't deal with read-only drives at all. > > Upstream QEMU and qemu-xen only support read-only cdrom drives: make > sure to specify "readonly=on" for cdrom drives and return erro

[Xen-devel] [PATCH OSSTEST 2/2] crontab-cambridge: Add distros-debian-stretch

2015-09-14 Thread Ian Campbell
I thought I'd done this ages ago... Signed-off-by: Ian Campbell --- crontab-cambridge | 1 + 1 file changed, 1 insertion(+) diff --git a/crontab-cambridge b/crontab-cambridge index 19669f0..c478203 100644 --- a/crontab-cambridge +++ b/crontab-cambridge @@ -9,6 +9,7 @@ MAILTO=ian.jack...@citrix.

[Xen-devel] [PATCH OSSTEST 1/2] crontab-cambridge: Change the days when we run a given distro-debian suite

2015-09-14 Thread Ian Campbell
The weekly CD images which are used by the snapshot flight are generated Sunday-Monday, so running that on a Saturday as we have been doing ensures that it will take at least two iterations/weeks to get any issues fixed. Also the current ordering of the existing releases made it hard to decide whe

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Ian Campbell
On Mon, 2015-09-14 at 12:02 +0200, Ard Biesheuvel wrote: > On 14 September 2015 at 11:57, Ian Campbell wrote: > > > or SetVirtualAddressMap/ConvertPointer, and > > > > These two are RTS, so in principal it could. > > > > (I'm not sure about ConvertPointer, is it useful for OS kernels, or > > ju

[Xen-devel] [PATCH] x86/PoD: use clear_domain_page()

2015-09-14 Thread Jan Beulich
Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm/p2m-pod.c +++ b/xen/arch/x86/mm/p2m-pod.c @@ -107,11 +107,7 @@ p2m_pod_cache_add(struct p2m_domain *p2m * promise to provide zero pages. So we scrub pages before using. */ for ( i = 0; i < (1 << order); i++ ) -{ -char *

[Xen-devel] [qemu-upstream-4.3-testing test] 61805: tolerable FAIL - PUSHED

2015-09-14 Thread osstest service owner
flight 61805 qemu-upstream-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/61805/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-xl-qemuu-debianhvm-amd64 19 guest-start/debianhvm.repeat fail in 61729 pass in 61805

Re: [Xen-devel] [PATCH] x86/PoD: use clear_domain_page()

2015-09-14 Thread Andrew Cooper
On 14/09/15 11:26, Jan Beulich wrote: > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] writing to read only scsi drives [and 1 more messages]

2015-09-14 Thread Ian Jackson
M A Young writes ("writing to read only scsi drives"): > I thought I would check here in case this is a new security issue but it > was reported at https://bugzilla.redhat.com/show_bug.cgi?id=1257893 that > in HVM guests it was possible to write to scsi devices (either specified > as sda etc. in

Re: [Xen-devel] [PATCH OSSTEST 2/2] crontab-cambridge: Add distros-debian-stretch

2015-09-14 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST 2/2] crontab-cambridge: Add distros-debian-stretch"): > I thought I'd done this ages ago... Acked-by: Ian Jackson Both patches. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-dev

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread George Dunlap
On Mon, Sep 14, 2015 at 4:48 AM, Juergen Gross wrote: > On 09/11/2015 04:41 PM, Ian Campbell wrote: >> >> On Fri, 2015-09-11 at 16:18 +0200, Juergen Gross wrote: >>> >>> On 09/11/2015 04:09 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 15:55 +0200, Juergen Gross wrote: > > On 09/

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Jan Beulich
>>> On 14.09.15 at 11:36, wrote: > On 14 September 2015 at 11:31, Shannon Zhao wrote: >> My understanding is that if there are no EFI_MEMORY_RUNTIME regions, it >> means we can't use runtime services and should not set the bit >> EFI_RUNTIME_SERVICES of efi.flags. But if efi_virtmap_init() return

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-14 Thread Julien Grall
Hi Roger, On 14/09/15 09:56, Roger Pau Monné wrote: > El 07/09/15 a les 17.33, Julien Grall ha escrit: >> Hi all, >> >> ARM64 Linux is supporting both 4KB and 64KB page granularity. Although, Xen >> hypercall interface and PV protocol are always based on 4KB page granularity. >> >> Any attempt to

Re: [Xen-devel] [v2][PATCH] xen/vtd/iommu: permit group devices to passthrough in relaxed mode

2015-09-14 Thread Jan Beulich
>>> On 14.09.15 at 08:24, wrote: >> OK, that explanation is fine to me as long as it's made clear no >> security guarantee once admin uses 'relax' for any domain. Tiejun >> could you resend patch with right warning/error type? >> > > Sure, but a little bit makes me confused when I'm trying to ad

Re: [Xen-devel] [PATCH] x86/p2m: fix mismatched unlock

2015-09-14 Thread George Dunlap
On Mon, Sep 14, 2015 at 8:13 AM, Jan Beulich wrote: > Luckily, due to gfn_unlock() currently mapping to p2m_unlock(), this is > only a cosmetic issue right now. > > Signed-off-by: Jan Beulich Reviewed-by: George Dunlap > --- > Despite its cosmetic nature I think it would be better to also fix

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread Juergen Gross
On 09/14/2015 12:36 PM, George Dunlap wrote: On Mon, Sep 14, 2015 at 4:48 AM, Juergen Gross wrote: On 09/11/2015 04:41 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 16:18 +0200, Juergen Gross wrote: On 09/11/2015 04:09 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 15:55 +0200, Juergen Gro

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-14 Thread Laszlo Ersek
On 09/12/15 01:06, Josh Triplett wrote: > On Fri, Sep 11, 2015 at 11:27:32PM +0200, Laszlo Ersek wrote: >> On 09/11/15 21:30, Josh Triplett wrote: >>> On Fri, Sep 11, 2015 at 05:28:06PM +0200, Laszlo Ersek wrote: Breaking Debian Wheezy's and BITS's GRUB is also bad, but the former is very

[Xen-devel] [OSSTEST PATCH] Executive: Abolish use of the `configdb'

2015-09-14 Thread Ian Jackson
This was a database used by networking infrastructure on the now-obsolete XenClient network in the Citrix Cambridge office (which used some management tools developed by Mythic Beasts). The production database in Cambridge no longer has the configdb, and both instances have `HostDB_Executive_NoCon

Re: [Xen-devel] [PATCH v6 00/31] Add ITS support

2015-09-14 Thread Vijay Kilari
On Wed, Sep 9, 2015 at 8:59 PM, Ian Campbell wrote: > On Mon, 2015-08-31 at 16:36 +0530, vijay.kil...@gmail.com wrote: >> Some Major TODOs: >>1) Avoid making vits_process_cmd() static in later point of time >>2) How to handle LPI that does not have LPI config table entry. >>3) Enable/

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-14 Thread Roger Pau Monné
Hello, El 14/09/15 a les 12.40, Julien Grall ha escrit: > Hi Roger, > > On 14/09/15 09:56, Roger Pau Monné wrote: >> El 07/09/15 a les 17.33, Julien Grall ha escrit: >>> Hi all, >>> >>> ARM64 Linux is supporting both 4KB and 64KB page granularity. Although, Xen >>> hypercall interface and PV prot

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-14 Thread Laszlo Ersek
On 09/14/15 11:22, Ian Campbell wrote: > On Fri, 2015-09-11 at 17:28 +0200, Laszlo Ersek wrote: >> > [...] >> For me that's not so clear-cut. OVMF is frequently used as a UEFI >> development environment (it's better to brick a virtual machine than >> your physical dev platform...) > > One flip sid

Re: [Xen-devel] [PATCH v2] arm/xen: Enable user access to the kernel before issuing a privcmd call

2015-09-14 Thread Stefano Stabellini
On Mon, 14 Sep 2015, Russell King - ARM Linux wrote: > On Mon, Sep 14, 2015 at 10:36:55AM +0100, Stefano Stabellini wrote: > > On Fri, 11 Sep 2015, Russell King - ARM Linux wrote: > > > If you'd like your ack on it, please send one, I can still do that. > > > > Acked-by: Stefano Stabellini > > S

Re: [Xen-devel] [OSSTEST PATCH] Executive: Abolish use of the `configdb'

2015-09-14 Thread Ian Campbell
On Mon, 2015-09-14 at 11:59 +0100, Ian Jackson wrote: > This was a database used by networking infrastructure on the > now-obsolete XenClient network in the Citrix Cambridge office (which > used some management tools developed by Mythic Beasts). > > The production database in Cambridge no longer h

Re: [Xen-devel] ANNOUNCEMENT: Xen 4.6 RC3

2015-09-14 Thread George Dunlap
On Wed, Sep 9, 2015 at 2:12 PM, Wei Liu wrote: > Hi all > > Xen 4.6 RC3 has been tagged. You can check out the tag 4.6.0-rc3 in xen.git. > > The tarball can be downloaded from: > > http://bits.xensource.com/oss-xen/release/4.6.0-rc3/xen-4.6.0-rc3.tar.gz > > Signature for tarball: > > http://bits.x

Re: [Xen-devel] [PATCH v6 00/31] Add ITS support

2015-09-14 Thread Julien Grall
Hi Vijay, On 14/09/15 12:00, Vijay Kilari wrote: > I will take care of A & B in next revision. > > Is there any further comments on this series?. I have not received > any comments on last few patches (patch #25 to patch#30). I haven't commented the rest of the series because I'm expecting to se

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread Ian Jackson
Juergen Gross writes ("Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API"): > On 09/14/2015 12:36 PM, George Dunlap wrote: > > Anyone want to look into the Linux source code to find out how big it > > will allow busnum / devnum to grow? > > drivers/usb/core/hcd.c is using a bitmap to find the ne

Re: [Xen-devel] ANNOUNCEMENT: Xen 4.6 RC3

2015-09-14 Thread Wei Liu
On Mon, Sep 14, 2015 at 12:11:47PM +0100, George Dunlap wrote: > On Wed, Sep 9, 2015 at 2:12 PM, Wei Liu wrote: > > Hi all > > > > Xen 4.6 RC3 has been tagged. You can check out the tag 4.6.0-rc3 in xen.git. > > > > The tarball can be downloaded from: > > > > http://bits.xensource.com/oss-xen/rele

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Ard Biesheuvel
On 14 September 2015 at 12:39, Jan Beulich wrote: On 14.09.15 at 11:36, wrote: >> On 14 September 2015 at 11:31, Shannon Zhao wrote: >>> My understanding is that if there are no EFI_MEMORY_RUNTIME regions, it >>> means we can't use runtime services and should not set the bit >>> EFI_RUNTIME

Re: [Xen-devel] ANNOUNCEMENT: Xen 4.6 RC3

2015-09-14 Thread Ian Jackson
Wei Liu writes ("Re: [Xen-devel] ANNOUNCEMENT: Xen 4.6 RC3"): > On Mon, Sep 14, 2015 at 12:11:47PM +0100, George Dunlap wrote: > > I realize they all point the same place, but shouldn't we ideally be > > using xenproject.org rather than xensource.com? Particularly as the > > latter hasn't actually

[Xen-devel] [OSSTEST PATCH] cs-bisection-step: Cope with graph-out (testids) containing ( ) etc.

2015-09-14 Thread Ian Jackson
cr-try-bisect launders / in the testid but relies on other characters being handled appropriately by cs-bisection-step. So for example it can pass graph-out=/home/logs/results/bisect/linux-linus/test-armhf-armhf-xl-arndale.leak-check--basis(8) But cs-bisection step foolishly assumed that the

Re: [Xen-devel] [PATCH v5 7/9] x86/intel_pstate: add a booting param to select the driver to load

2015-09-14 Thread Jan Beulich
>>> On 14.09.15 at 04:42, wrote: > By default, the old P-state driver (acpi-freq) is used. Adding > "intel_pstate" to the Xen booting param list to enable the > use of intel_pstate. However, if intel_pstate is enabled on a > machine which does not support the driver (e.g. Nehalem), the > old P-sta

Re: [Xen-devel] [PATCH] x86/PoD: use clear_domain_page()

2015-09-14 Thread George Dunlap
On Mon, Sep 14, 2015 at 11:26 AM, Jan Beulich wrote: > Signed-off-by: Jan Beulich Acked-by: George Dunlap > > --- a/xen/arch/x86/mm/p2m-pod.c > +++ b/xen/arch/x86/mm/p2m-pod.c > @@ -107,11 +107,7 @@ p2m_pod_cache_add(struct p2m_domain *p2m > * promise to provide zero pages. So we scrub p

Re: [Xen-devel] ANNOUNCEMENT: Xen 4.6 RC3

2015-09-14 Thread Ian Campbell
On Mon, 2015-09-14 at 12:13 +0100, Wei Liu wrote: > On Mon, Sep 14, 2015 at 12:11:47PM +0100, George Dunlap wrote: > > On Wed, Sep 9, 2015 at 2:12 PM, Wei Liu wrote: > > > Hi all > > > > > > Xen 4.6 RC3 has been tagged. You can check out the tag 4.6.0-rc3 in > > > xen.git. > > > > > > The tarbal

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-14 Thread Julien Grall
On 14/09/15 12:04, Roger Pau Monné wrote: > Hello, > > El 14/09/15 a les 12.40, Julien Grall ha escrit: >> Hi Roger, >> >> On 14/09/15 09:56, Roger Pau Monné wrote: >>> El 07/09/15 a les 17.33, Julien Grall ha escrit: Hi all, ARM64 Linux is supporting both 4KB and 64KB page granular

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread Juergen Gross
On 09/14/2015 01:12 PM, Ian Jackson wrote: Juergen Gross writes ("Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API"): On 09/14/2015 12:36 PM, George Dunlap wrote: Anyone want to look into the Linux source code to find out how big it will allow busnum / devnum to grow? drivers/usb/core/hcd.

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-14 Thread Arnd Bergmann
On Monday 14 September 2015 13:04:59 Roger Pau Monné wrote: > > TBH, I'm expecting a small impact to the performance. It would be hard > > to get the exactly the same performance as today if we keep the helpers > > to avoid the backend dealing himself with the splitting and page > > granularity. >

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Jan Beulich
>>> On 14.09.15 at 13:16, wrote: > (I still think not using SetVirtualAddressMap() at all > would be the best approach for arm64, but unfortunately, most of my > colleagues disagree with me) Any reasons they have? I'm curious because we run x86 Xen without using this function ... Jan _

Re: [Xen-devel] [PATCH v5 7/9] x86/intel_pstate: add a booting param to select the driver to load

2015-09-14 Thread Wang, Wei W
On 14/09/2015 19:18, Jan Beulich wrote: >>> On 14.09.15 at 04:42, wrote: > By default, the old P-state driver (acpi-freq) is used. Adding > "intel_pstate" to the Xen booting param list to enable the use of > intel_pstate. However, if intel_pstate is enabled on a machine which > does not suppor

[Xen-devel] [qemu-upstream-4.5-testing test] 61809: regressions - FAIL

2015-09-14 Thread osstest service owner
flight 61809 qemu-upstream-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/61809/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-vhd9 debian-di-install fail REGR. vs. 60577 test-am

[Xen-devel] [ovmf test] 61817: regressions - FAIL

2015-09-14 Thread osstest service owner
flight 61817 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/61817/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 60869 test-amd64-i386-xl-qemuu-ovm

[Xen-devel] __XEN_LATEST_INTERFACE_VERSION__ is still 0x00040600 in staging

2015-09-14 Thread Razvan Cojocaru
Hello, I've found this out by accident, since I have some code that does some #ifdef tricks based on __XEN_LATEST_INTERFACE_VERSION__, but while running staging ("Xen version 4.7-unstable") it seems that in xen/xen-compat.h we still have: #define __XEN_LATEST_INTERFACE_VERSION__ 0x00040600 Is th

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-14 Thread Roger Pau Monné
El 14/09/15 a les 13.21, Julien Grall ha escrit: > On 14/09/15 12:04, Roger Pau Monné wrote: >> Hello, >> >> El 14/09/15 a les 12.40, Julien Grall ha escrit: >>> Hi Roger, >>> >>> On 14/09/15 09:56, Roger Pau Monné wrote: El 07/09/15 a les 17.33, Julien Grall ha escrit: > Hi all, > >>>

Re: [Xen-devel] __XEN_LATEST_INTERFACE_VERSION__ is still 0x00040600 in staging

2015-09-14 Thread Jan Beulich
>>> On 14.09.15 at 13:53, wrote: > I've found this out by accident, since I have some code that does some > #ifdef tricks based on __XEN_LATEST_INTERFACE_VERSION__, but while > running staging ("Xen version 4.7-unstable") it seems that in > xen/xen-compat.h we still have: > > #define __XEN_LATEST

Re: [Xen-devel] __XEN_LATEST_INTERFACE_VERSION__ is still 0x00040600 in staging

2015-09-14 Thread Razvan Cojocaru
On 09/14/2015 03:13 PM, Jan Beulich wrote: On 14.09.15 at 13:53, wrote: >> I've found this out by accident, since I have some code that does some >> #ifdef tricks based on __XEN_LATEST_INTERFACE_VERSION__, but while >> running staging ("Xen version 4.7-unstable") it seems that in >> xen/xen-c

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Daniel Kiper
On Mon, Sep 14, 2015 at 10:25:19AM +0100, Mark Rutland wrote: > On Sat, Sep 12, 2015 at 12:36:55PM +0100, Daniel Kiper wrote: > > On Fri, Sep 11, 2015 at 05:25:59PM +0100, Mark Rutland wrote: > > > On Fri, Sep 11, 2015 at 01:46:43PM +0100, Daniel Kiper wrote: > > > > On Thu, Sep 10, 2015 at 05:23:0

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-14 Thread Ian Campbell
On Mon, 2015-09-14 at 13:07 +0200, Laszlo Ersek wrote: > Debian Wheezy is not very old, it's only a year older than RHEL7 (May > > 2013 > > vs June 2014) and only a bit older than two years in absolute terms. It is > > also the subject of an LTS effort, which extends its lifetime to 2018. > > (*)

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Daniel Kiper
On Mon, Sep 14, 2015 at 11:43:27AM +0200, Ard Biesheuvel wrote: > On 14 September 2015 at 11:25, Mark Rutland wrote: > > On Sat, Sep 12, 2015 at 12:36:55PM +0100, Daniel Kiper wrote: > >> On Fri, Sep 11, 2015 at 05:25:59PM +0100, Mark Rutland wrote: > >> > On Fri, Sep 11, 2015 at 01:46:43PM +0100,

Re: [Xen-devel] Dom0 crash with apache bench (ab)

2015-09-14 Thread Christoffer Dall
On Fri, Jul 31, 2015 at 03:17:56PM +0200, Christoffer Dall wrote: > On Fri, Jul 31, 2015 at 12:28 PM, David Vrabel > wrote: > > > On 31/07/15 11:24, Stefano Stabellini wrote: > > > This is a Linux Dom0 crash on x86 (Dell PowerEdge R320, Xeon E5-2450), > > > CC'ing relevant people. As you can see

Re: [Xen-devel] [PATCH] xen: use correct type for HYPERVISOR_memory_op()

2015-09-14 Thread Juergen Gross
Ping? On 09/04/2015 02:50 PM, Juergen Gross wrote: HYPERVISOR_memory_op() is defined to return an "int" value. This is wrong, as the Xen hypervisor will return "long". The sub-function XENMEM_maximum_reservation returns the maximum number of pages for the current domain. An int will overflow fo

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-14 Thread Julien Grall
On 14/09/15 13:08, Roger Pau Monné wrote: >> I'd like to see a basic support of 64KB page granularity upstream before >> starting to think about performance improvement. And there is still a >> lot to do. > > I wasn't actually thinking of this as a performance improvement, but > rather as a way of

[Xen-devel] [PATCH] libxc: remove useless stuff from domain builder

2015-09-14 Thread Juergen Gross
Remove unused fields from the domain builder and associated functions. Signed-off-by: Juergen Gross --- tools/libxc/include/xc_dom.h | 2 -- tools/python/xen/lowlevel/xc/xc.c | 8 ++-- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/tools/libxc/include/xc_dom.h b/tools/li

Re: [Xen-devel] [PATCH] xen: use correct type for HYPERVISOR_memory_op()

2015-09-14 Thread Julien Grall
Hi Juergen, On 04/09/15 13:50, Juergen Gross wrote: > HYPERVISOR_memory_op() is defined to return an "int" value. This is > wrong, as the Xen hypervisor will return "long". > > The sub-function XENMEM_maximum_reservation returns the maximum > number of pages for the current domain. An int will ov

Re: [Xen-devel] [PATCH v6 00/31] Add ITS support

2015-09-14 Thread Vijay Kilari
On Mon, Sep 14, 2015 at 4:39 PM, Julien Grall wrote: > Hi Vijay, > > On 14/09/15 12:00, Vijay Kilari wrote: >> I will take care of A & B in next revision. >> >> Is there any further comments on this series?. I have not received >> any comments on last few patches (patch #25 to patch#30). > > I hav

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Ard Biesheuvel
On 14 September 2015 at 14:28, Daniel Kiper wrote: > On Mon, Sep 14, 2015 at 11:43:27AM +0200, Ard Biesheuvel wrote: >> On 14 September 2015 at 11:25, Mark Rutland wrote: >> > On Sat, Sep 12, 2015 at 12:36:55PM +0100, Daniel Kiper wrote: >> >> On Fri, Sep 11, 2015 at 05:25:59PM +0100, Mark Rutlan

[Xen-devel] [Patch V2] xen: use correct type for HYPERVISOR_memory_op()

2015-09-14 Thread Juergen Gross
HYPERVISOR_memory_op() is defined to return an "int" value. This is wrong, as the Xen hypervisor will return "long". The sub-function XENMEM_maximum_reservation returns the maximum number of pages for the current domain. An int will overflow for a domain configured with 8TB of memory or more. Cor

[Xen-devel] [PATCH] xen/arm: correct comment in enlighten.c

2015-09-14 Thread Juergen Gross
Correct a comment in arch/arm/xen/enlighten.c referencing a wrong source file. Signed-off-by: Juergen Gross --- arch/arm/xen/enlighten.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c index eeeab07..5421706 100644 --- a/arc

Re: [Xen-devel] [PATCH] libxc: remove useless stuff from domain builder

2015-09-14 Thread Wei Liu
On Mon, Sep 14, 2015 at 02:54:09PM +0200, Juergen Gross wrote: > Remove unused fields from the domain builder and associated functions. > > Signed-off-by: Juergen Gross > --- > tools/libxc/include/xc_dom.h | 2 -- > tools/python/xen/lowlevel/xc/xc.c | 8 ++-- > 2 files changed, 2 insert

Re: [Xen-devel] [PATCH] libxc: remove useless stuff from domain builder

2015-09-14 Thread Juergen Gross
On 09/14/2015 03:23 PM, Wei Liu wrote: On Mon, Sep 14, 2015 at 02:54:09PM +0200, Juergen Gross wrote: Remove unused fields from the domain builder and associated functions. Signed-off-by: Juergen Gross --- tools/libxc/include/xc_dom.h | 2 -- tools/python/xen/lowlevel/xc/xc.c | 8 ++---

Re: [Xen-devel] [PATCH] libxc: remove useless stuff from domain builder

2015-09-14 Thread Wei Liu
On Mon, Sep 14, 2015 at 03:29:57PM +0200, Juergen Gross wrote: > On 09/14/2015 03:23 PM, Wei Liu wrote: > >On Mon, Sep 14, 2015 at 02:54:09PM +0200, Juergen Gross wrote: > >>Remove unused fields from the domain builder and associated functions. > >> > >>Signed-off-by: Juergen Gross > >>--- > >> t

Re: [Xen-devel] [PATCH] xen/arm: correct comment in enlighten.c

2015-09-14 Thread Stefano Stabellini
On Mon, 14 Sep 2015, Juergen Gross wrote: > Correct a comment in arch/arm/xen/enlighten.c referencing a wrong > source file. > > Signed-off-by: Juergen Gross Acked-by: Stefano Stabellini > arch/arm/xen/enlighten.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch

[Xen-devel] [PATCH] xen/keyhandler: Rework keyhandler infrastructure

2015-09-14 Thread Andrew Cooper
struct keyhandler does not contain much information, and requires a lot of boilerplate to use. It is far more convenient to have register_keyhandler() take each piece of information a parameter, especially when introducing temporary debugging keyhandlers. This in turn allows struct keyhandler its

[Xen-devel] [PATCH] xl: tighten parsing of "irq" and "iomem" list elements

2015-09-14 Thread Jan Beulich
While "ioport" list element parsing already validates that the entire input string got consumed, its two siblings so far didn't. Signed-off-by: Jan Beulich --- a/tools/libxl/xl_cmdimpl.c +++ b/tools/libxl/xl_cmdimpl.c @@ -1730,7 +1730,7 @@ static void parse_config_data(const char

Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters

2015-09-14 Thread Daniel Kiper
On Mon, Sep 14, 2015 at 03:09:34PM +0200, Ard Biesheuvel wrote: > On 14 September 2015 at 14:28, Daniel Kiper wrote: > > On Mon, Sep 14, 2015 at 11:43:27AM +0200, Ard Biesheuvel wrote: > >> On 14 September 2015 at 11:25, Mark Rutland wrote: > >> > On Sat, Sep 12, 2015 at 12:36:55PM +0100, Daniel

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread George Dunlap
On Mon, Sep 14, 2015 at 12:12 PM, Ian Jackson wrote: > Juergen Gross writes ("Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API"): >> On 09/14/2015 12:36 PM, George Dunlap wrote: >> > Anyone want to look into the Linux source code to find out how big it >> > will allow busnum / devnum to grow? >

Re: [Xen-devel] [Patch V2] xen: use correct type for HYPERVISOR_memory_op()

2015-09-14 Thread Boris Ostrovsky
On 09/14/2015 09:13 AM, Juergen Gross wrote: HYPERVISOR_memory_op() is defined to return an "int" value. This is wrong, as the Xen hypervisor will return "long". The sub-function XENMEM_maximum_reservation returns the maximum number of pages for the current domain. An int will overflow for a d

Re: [Xen-devel] [PATCH v3 1/4] x86: Support enable CDP by boot parameter and add get CDP status

2015-09-14 Thread Jan Beulich
>>> On 14.09.15 at 05:27, wrote: > --- a/xen/arch/x86/psr.c > +++ b/xen/arch/x86/psr.c > @@ -21,9 +21,16 @@ > > #define PSR_CMT(1<<0) > #define PSR_CAT(1<<1) > +#define PSR_CDP(1<<2) > > struct psr_cat_cbm { > -uint64_t cbm; > +union { > +uint64_t cbm

[Xen-devel] [PATCH for 4.6] x86/mm: make {set, clear}_identity_p2m_mapping() work for HVM guests as well

2015-09-14 Thread Andrew Cooper
From: Malcolm Crossley For Intel IGD passthrough, the guest driver programs the device to DMA to/from its RMRR. c/s 619ecf8 "make {set,clear}_identity_p2m_mapping() work for PV guests" was incomplete for pre-Broadwell systems which did not support shared EPT. The correct check to use is iommu_u

Re: [Xen-devel] [PATCH v3 2/4] x86: add domctl cmd to set/get CDP code/data CBM

2015-09-14 Thread Jan Beulich
>>> On 14.09.15 at 05:27, wrote: > --- a/xen/arch/x86/psr.c > +++ b/xen/arch/x86/psr.c > @@ -288,14 +288,39 @@ int psr_get_cat_l3_info(unsigned int socket, uint32_t > *cbm_len, > return 0; > } > > -int psr_get_l3_cbm(struct domain *d, unsigned int socket, uint64_t *cbm) > +int psr_get_l3_

  1   2   >