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.

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

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

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

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

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

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

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

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

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

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

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,

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

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

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

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 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

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.

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

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

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

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 *), len, pos); > +rc = read(config_fd, (uint8_t *), len); > } while (rc < 0 && (errno == EINTR

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(>real_device, offset, (uint8_t > *)); A bit ugly, and it relies on the host being little endian. > +break; > +case 2: rc =

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

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

[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 @@

[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

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

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

[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++ ) -

<    1   2