[Xen-devel] [qemu-mainline test] 113607: regressions - FAIL

2017-09-19 Thread osstest service owner
flight 113607 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/113607/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 113302

Re: [Xen-devel] [PATCH v3 01/15] docs: create Memory Bandwidth Allocation (MBA) feature document

2017-09-19 Thread Yi Sun
On 17-09-19 00:07:36, Jan Beulich wrote: > >>> Roger Pau Monné 09/18/17 7:21 PM >>> > >On Tue, Sep 05, 2017 at 05:32:23PM +0800, Yi Sun wrote: > >> +## Hardware perspective > >> + > >> + MBA defines a range of MSRs to support specifying a delay value (Thrtl) > >> per > >>

Re: [Xen-devel] [PATCH v3 01/15] docs: create Memory Bandwidth Allocation (MBA) feature document

2017-09-19 Thread Yi Sun
On 17-09-18 18:16:40, Roger Pau Monn� wrote: > On Tue, Sep 05, 2017 at 05:32:23PM +0800, Yi Sun wrote: > > +* xl interfaces: > > + > > + 1. `psr-mba-show [domain-id]`: > > Is this limited to domain-id, or one can also use the domain name? > Most of the xl commands accept either a domain-id or a

Re: [Xen-devel] [PATCH] vt-d: use two 32-bit writes to update DMAR fault address registers

2017-09-19 Thread Tian, Kevin
> From: Roger Pau Monné [mailto:roger@citrix.com] > Sent: Monday, September 18, 2017 5:10 PM > > On Mon, Sep 18, 2017 at 05:05:18PM +0800, Haozhong Zhang wrote: > > On 09/18/17 02:30 -0600, Jan Beulich wrote: > > > >>> On 18.09.17 at 10:18, wrote: > > > >> From: Jan

[Xen-devel] [xen-unstable-smoke test] 113612: tolerable all pass - PUSHED

2017-09-19 Thread osstest service owner
flight 113612 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/113612/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 13 migrate-support-checkfail never pass test-armhf-armhf-xl

Re: [Xen-devel] [PATCH v3 02/15] Rename PSR sysctl/domctl interfaces and xsm policy to make them be general

2017-09-19 Thread Yi Sun
On 17-09-19 09:03:38, Roger Pau Monn� wrote: > On Tue, Sep 05, 2017 at 05:32:24PM +0800, Yi Sun wrote: > > diff --git a/xen/include/public/domctl.h b/xen/include/public/domctl.h > > index 0669c31..a953157 100644 > > --- a/xen/include/public/domctl.h > > +++ b/xen/include/public/domctl.h > >

[Xen-devel] [linux-linus test] 113605: tolerable FAIL - PUSHED

2017-09-19 Thread osstest service owner
flight 113605 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/113605/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-i386-xl-qemut-win7-amd64 18 guest-start/win.repeat fail blocked in 113594

Re: [Xen-devel] [PATCH v3 04/15] x86: implement data structure and CPU init flow for MBA

2017-09-19 Thread Yi Sun
On 17-09-19 09:55:28, Roger Pau Monn� wrote: > On Tue, Sep 05, 2017 at 05:32:26PM +0800, Yi Sun wrote: > > This patch implements main data structures of MBA. > > > > Like CAT features, MBA HW info has cos_max which means the max thrtl > > register number, and thrtl_max which means the max

Re: [Xen-devel] [PATCH v2 04/24] xen/arm: mm: Redefine mfn_to_virt to use typesafe

2017-09-19 Thread Stefano Stabellini
On Sat, 16 Sep 2017, Julien Grall wrote: > Hi Stefano, > > On 09/16/2017 12:56 AM, Stefano Stabellini wrote: > > On Tue, 12 Sep 2017, Julien Grall wrote: > > > This add a bit more safety in the memory subsystem code. > > > > > > Signed-off-by: Julien Grall > > > --- > > >

[Xen-devel] [ovmf test] 113608: all pass - PUSHED

2017-09-19 Thread osstest service owner
flight 113608 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/113608/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 424a5ec33b3d5a842bff3f4695d0bd709c91a163 baseline version: ovmf

Re: [Xen-devel] [PATCH v2 24/24] xen/arm: mm: Use memory flags for modify_xen_mappings rather than custom one

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > This will help to consolidate the page-table code and avoid different > path depending on the action to perform. > > Signed-off-by: Julien Grall > Reviewed-by: Andre Przywara Much better now, thanks!

Re: [Xen-devel] [RFC] Unicore Subproject Proposal

2017-09-19 Thread Lars Kurth
Felipe, Simon, a quick note to let you know that the Advisory Board in today’s AB meeting decided to endorse your proposal. Let me know how you proceed: from my perspective, we can kick off a formal vote before you make modifications to the proposal, but I think it is better to post v2 first.

Re: [Xen-devel] [PATCH v2 18/24] xen/arm: mm: Rename and clarify AP[1] in the stage-1 page table

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > The description of AP[1] in Xen is based on testing rather than the ARM > ARM. > > Per the ARM ARM, on EL2 stage-1 page table, AP[1] is RES1 as the > translation regime applies to only one exception level (see D4.4.4 and > G4.6.1 in ARM DDI 0487B.a). >

Re: [Xen-devel] [PATCH v2 22/24] xen/arm: mm: Embed permission in the flags

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > Currently, it is not possible to specify the permission of a new > mapping. It would be necessary to use the function modify_xen_mappings > with a different set of flags. > > Introduce a couple of new flags for the permissions (Non-eXecutable, >

Re: [Xen-devel] [RFC] Unicore Subproject Proposal

2017-09-19 Thread Stefano Stabellini
On Mon, 18 Sep 2017, Felipe Huici wrote: > Hi Lars, all, > > [cc’ing authors of Erlang on Xen, HalVM and Rump]. > > Thanks everyone for all of the support and useful comments. We’ve > incorporated a number of them into a new version of the document (attached > and pasted at the bottom for

Re: [Xen-devel] [PATCH v2 23/24] xen/arm: mm: Handle permission flags when adding a new mapping

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > Currently, all the new mappings will be read-write non-executable. Allow the > caller to use other permissions. > > Signed-off-by: Julien Grall > > --- > Changes in v2: > - Switch the runtime check to a BUG_ON() Since

Re: [Xen-devel] [RFC] Unicore Subproject Proposal

2017-09-19 Thread Stefano Stabellini
On Wed, 20 Sep 2017, Lars Kurth wrote: > Felipe, Simon, > a quick note to let you know that the Advisory Board in today’s AB meeting > decided to endorse your proposal. > Let me know how you proceed: from my perspective, we can kick off a formal > vote before you make modifications to the

Re: [Xen-devel] [PATCH v2 20/24] xen/arm: mm: Rename 'ai' into 'flags' in create_xen_entries

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > The parameter 'ai' is used either for attribute index or for > permissions. Follow-up patch will rework that parameters to carry more > information. So rename the parameter to 'flags'. > > Signed-off-by: Julien Grall > Reviewed-by:

Re: [Xen-devel] [PATCH v2 21/24] xen/arm: page: Describe the layout of flags used to update page tables

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > Currently, the flags used to update page tables (i.e PAGE_HYPERVISOR_*) > only contains the memory attribute index. Follow-up patches will add > more information in it. So document the current layout. > > At the same time introduce PAGE_AI_MASK to get

Re: [Xen-devel] [PATCH v2 16/24] xen/arm: page: Use ARMv8 naming to improve readability

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > This is based on the Linux ARMv8 naming scheme (see arch/arm64/mm/proc.S). > Each > type will contain "NORMAL" or "DEVICE" to make clear whether each attribute > targets device or normal memory. > > Signed-off-by: Julien Grall >

Re: [Xen-devel] [PATCH v2 00/24] xen/arm: Memory subsystem clean-up

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > Hi all, > > This patch series contains clean-up for the ARM memory subsystem in > preparation > of reworking the page tables handling. > > A branch with the patches can be found on xenbits: > >

Re: [Xen-devel] [PATCH v2 19/24] xen/arm: Switch to SYS_STATE_boot just after end_boot_allocator()

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > We should consider the early boot period to end when we stop using the > boot allocator. This is inline with x86 and will be helpful to know > whether we should allocate memory from the boot allocator or xenheap. > > Signed-off-by: Julien Grall

Re: [Xen-devel] [PATCH v2 14/24] xen/arm: page: Use directly BUFFERABLE and drop DEV_WC

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > DEV_WC is only used for PAGE_HYPERVISOR_WC and does not bring much > improvement. > > Signed-off-by: Julien Grall > Reviewed-by: Andre Przywara > > --- > > Changes in v2: > - Remove DEV_WC

Re: [Xen-devel] [PATCH v2 13/24] xen/arm: page: Remove unused attributes DEV_NONSHARED and DEV_CACHED

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > They were imported from non-LPAE Linux, but Xen is LPAE only. It is time > to do some clean-up in the memory attribute and keep only what make > sense for Xen. Follow-up patch will do more clean-up. > > Also, update the comment saying our attribute

Re: [Xen-devel] [PATCH v2 17/24] xen/arm: page: Clean-up the definition of MAIRVAL

2017-09-19 Thread Stefano Stabellini
On Tue, 12 Sep 2017, Julien Grall wrote: > Currently MAIRVAL is defined in term of MAIR0VAL and MAIR1VAL which are > both hardcoded value. This makes quite difficult to understand the value > written in both registers. > > Rework the definition by using value of each attribute shifted by their >

[Xen-devel] [ovmf test] 113590: all pass - PUSHED

2017-09-19 Thread osstest service owner
flight 113590 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/113590/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 91cc526b15ffbbbdec5a57906596f37e059f80be baseline version: ovmf

Re: [Xen-devel] [PATCH] xen: credit2: fix spinlock irq-safety violation

2017-09-19 Thread Wei Liu
On Tue, Sep 19, 2017 at 04:11:28AM +0200, Dario Faggioli wrote: > In commit ad4b3e1e9df34 ("xen: credit2: implement > utilization cap") xfree() was being called (for > deallocating the budget replenishment timer, during > domain destruction) inside an IRQ disabled critical > section. > > That

Re: [Xen-devel] How to prepare the COLO test environment

2017-09-19 Thread Dario Faggioli
On Mon, 2017-09-18 at 18:52 +, Zhang Chen wrote: > 山本真吾 于2017年9月18日周一 下午6:27写道: > > I have read the following documents: > > > > COLO - Coarse Grain Lock Stepping > > https://wiki.xen.org/wiki/COLO_-_Coarse_Grain_Lock_Stepping > > > > Mailing list > >

Re: [Xen-devel] [PATCH v3 01/15] docs: create Memory Bandwidth Allocation (MBA) feature document

2017-09-19 Thread Jan Beulich
>>> Roger Pau Monné 09/18/17 7:21 PM >>> >On Tue, Sep 05, 2017 at 05:32:23PM +0800, Yi Sun wrote: >> +## Hardware perspective >> + >> + MBA defines a range of MSRs to support specifying a delay value (Thrtl) >> per >> + COS, with details below. >> + >> + ``` >> +

Re: [Xen-devel] [PATCH v3 02/15] Rename PSR sysctl/domctl interfaces and xsm policy to make them be general

2017-09-19 Thread Roger Pau Monné
On Tue, Sep 05, 2017 at 05:32:24PM +0800, Yi Sun wrote: > diff --git a/xen/include/public/domctl.h b/xen/include/public/domctl.h > index 0669c31..a953157 100644 > --- a/xen/include/public/domctl.h > +++ b/xen/include/public/domctl.h > @@ -37,7 +37,7 @@ > #include "hvm/save.h" > #include

Re: [Xen-devel] [PATCH v7 12/12] x86/hvm/ioreq: add a new mappable resource type...

2017-09-19 Thread Paul Durrant
> -Original Message- > From: Roger Pau Monne > Sent: 18 September 2017 17:18 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; Stefano Stabellini > ; Andrew Cooper ; > Ian Jackson ;

Re: [Xen-devel] [PATCH v2] xen: grant-table: Simplify get_paged_frame

2017-09-19 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Julien Grall > Sent: 18 September 2017 17:28 > To: xen-devel@lists.xen.org > Cc: sstabell...@kernel.org; Wei Liu ; George Dunlap > ; Andrew Cooper >

Re: [Xen-devel] [PATCH 05/22] libxl: #include "xentoolcore_internal.h"

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:42PM +0100, Ian Jackson wrote: > We are going to want to move something here. > > Signed-off-by: Ian Jackson Acked-by: Wei Liu ___ Xen-devel mailing list

[Xen-devel] [stage1-xen PATCH v2 08/11] README.md: Add CircleCI badge

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath Signed-off-by: Rajiv Ranganath Reviewed-by: Stefano Stabellini --- README.md |2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index 9ea6adf..e1cd40c 100644

[Xen-devel] [stage1-xen PATCH v2 10/11] BUILDING.md: Add Fedora instructions

2017-09-19 Thread Rajiv Ranganath
Signed-off-by: Rajiv Ranganath Reviewed-by: Stefano Stabellini --- BUILDING.md | 96 --- 1 file changed, 91 insertions(+), 5 deletions(-) diff --git a/BUILDING.md b/BUILDING.md index

[Xen-devel] [stage1-xen PATCH v2 09/11] build/fedora: Add `RUNNING_STAGE1_XEN.md`

2017-09-19 Thread Rajiv Ranganath
We are installing xen in `/opt/` directory. By keeping builds isolated in `/opt/` we can test multiple versions of xen during development. In production a simliar approach can be used to support multiple versions of xen along with a higher level tool that can do rolling updates and rollbacks.

[Xen-devel] [stage1-xen PATCH v2 11/11] README.md: Mention preview status

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath Signed-off-by: Rajiv Ranganath --- README.md |1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index e1cd40c..d1ae6b9 100644 --- a/README.md +++ b/README.md @@ -12,6 +12,7 @@ Stage1s

[Xen-devel] [stage1-xen PATCH v2 06/11] build/fedora: Add `xen-4.9.0-runit/*` scripts

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath Signed-off-by: Rajiv Ranganath --- build/fedora/xen-4.9.0-runit/setup.sh | 18 build/fedora/xen-4.9.0-runit/teardown.sh | 18

Re: [Xen-devel] [PATCH 06/22] tools: move CONTAINER_OF to xentoolcore_internal.h

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:43PM +0100, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 03/22] xentoolcore, _restrict_all: Introduce new library and implementation

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:40PM +0100, Ian Jackson wrote: > +int xentoolcore_restrict_all(uint32_t domid) { > +/* xxx lock */ This should be removed. With this fixed: Acked-by: Wei Liu ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH 04/22] tools: qemu-xen build: prepare to link against xentoolcore

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:41PM +0100, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [alsa-devel] [PATCH RESEND1 00/12] ALSA: vsnd: Add Xen para-virtualized frontend driver

2017-09-19 Thread Oleksandr Andrushchenko
Hi, all! We did some work on implementing the idea with feedback events from the backend to the frontend. Please see attached the changes to the existing sndif protocol [1]: 1. Introduced a new event channel from back to front 2. New event with number of bytes played/captured

Re: [Xen-devel] [PATCH v3 04/15] x86: implement data structure and CPU init flow for MBA

2017-09-19 Thread Roger Pau Monné
On Tue, Sep 05, 2017 at 05:32:26PM +0800, Yi Sun wrote: > This patch implements main data structures of MBA. > > Like CAT features, MBA HW info has cos_max which means the max thrtl > register number, and thrtl_max which means the max throttle value > (delay value). It also has a flag to

Re: [Xen-devel] [PATCH v2] xen: grant-table: Simplify get_paged_frame

2017-09-19 Thread Jan Beulich
>>> On 19.09.17 at 10:34, wrote: > I do wonder whether this function belongs in the grant table code though. > Getting the page from a (d, gfn) tuple is probably something that's needed in > a few places and hence putting the code in common/memory.c (with suitable >

Re: [Xen-devel] [PATCH] x86/domctl: Don't pause the whole domain if only getting vcpu state

2017-09-19 Thread Jan Beulich
>>> Razvan Cojocaru 09/18/17 7:05 PM >>> >On 09/18/2017 06:35 PM, Jan Beulich wrote: > On 12.09.17 at 15:53, wrote: >>> --- a/xen/arch/x86/domctl.c >>> +++ b/xen/arch/x86/domctl.c >>> @@ -625,6 +625,26 @@ long arch_do_domctl( >>>

Re: [Xen-devel] [PATCH v3 03/15] x86: rename 'cbm_type' to 'psr_type' to make it general

2017-09-19 Thread Roger Pau Monné
On Tue, Sep 05, 2017 at 05:32:25PM +0800, Yi Sun wrote: > This patch renames 'cbm_type' to 'psr_type' to make it be general. s/make it be general/generalize it/. > Then, we can reuse this for all psr allocation features. > > Signed-off-by: Yi Sun > Reviewed-by: Wei

Re: [Xen-devel] [PATCH 03/22] xentoolcore, _restrict_all: Introduce new library and implementation

2017-09-19 Thread Wei Liu
On Mon, Sep 18, 2017 at 05:08:18PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 03/22] xentoolcore, _restrict_all: Introduce new > library and implementation"): > > On Fri, Sep 15, 2017 at 07:48:40PM +0100, Ian Jackson wrote: > ... > > > +void

[Xen-devel] [stage1-xen PATCH v2 07/11] .circleci/config.yml: Add

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath Signed-off-by: Rajiv Ranganath --- .circleci/config.yml | 21 + 1 file changed, 21 insertions(+) create mode 100644 .circleci/config.yml diff --git a/.circleci/config.yml

[Xen-devel] [stage1-xen PATCH v2 03/11] build/fedora: Add `buildroot-Dockerfile`

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath Signed-off-by: Rajiv Ranganath Reviewed-by: Stefano Stabellini --- build/fedora/buildroot-Dockerfile | 113 + 1 file changed, 113 insertions(+)

[Xen-devel] [stage1-xen PATCH v2 05/11] build/fedora: Add `source_path.sh`

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath Signed-off-by: Rajiv Ranganath --- build/fedora/source_path.sh |3 +++ 1 file changed, 3 insertions(+) create mode 100755 build/fedora/source_path.sh diff --git a/build/fedora/source_path.sh

[Xen-devel] [stage1-xen PATCH v2 04/11] build/fedora: Add `run` and `components/*` scripts

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath In order to build stage1-xen we require three components - 1. Xen 2. Qemu 3. Rkt These components are built using scripts in `build/fedora/components/` directory. These scripts have constants (for example `XEN_PREFIX`) that can be used to

[Xen-devel] [stage1-xen PATCH v2 00/11] Add Fedora support

2017-09-19 Thread Rajiv Ranganath
Hi Stefano, I've made the following changes based on v1 series feedback. Kindly please review. build/fedora: Add `run` and `components/*` scripts - Updated commit message - Switched to Xen 4.9.0, Qemu 2.10.0, Rkt 1.28.1 build/fedora: Add `source_path.sh` - Updated paths to qemu and xen

[Xen-devel] [stage1-xen PATCH v2 02/11] build/fedora: Add `buildroot-README.md`

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath Signed-off-by: Rajiv Ranganath Reviewed-by: Stefano Stabellini --- build/fedora/buildroot-README.md | 50 ++ 1 file changed, 50 insertions(+) create

[Xen-devel] [stage1-xen PATCH v2 01/11] .gitignore: Add

2017-09-19 Thread Rajiv Ranganath
From: Rajiv M Ranganath Signed-off-by: Rajiv Ranganath Reviewed-by: Stefano Stabellini --- .gitignore |2 ++ 1 file changed, 2 insertions(+) create mode 100644 .gitignore diff --git a/.gitignore

[Xen-devel] [qemu-mainline test] 113586: regressions - FAIL

2017-09-19 Thread osstest service owner
flight 113586 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/113586/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-win7-amd64 18 guest-start/win.repeat fail REGR. vs. 113302

Re: [Xen-devel] [PATCH v7 03/12] tools/libxenforeignmemory: add support for resource mapping

2017-09-19 Thread Paul Durrant
> -Original Message- > From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] > Sent: 18 September 2017 17:16 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org > Subject: Re: [PATCH v7 03/12] tools/libxenforeignmemory: add support for > resource mapping > >

Re: [Xen-devel] [PATCH v3 05/15] x86: implement get hw info flow for MBA

2017-09-19 Thread Roger Pau Monné
On Tue, Sep 05, 2017 at 05:32:27PM +0800, Yi Sun wrote: > This patch implements get HW info flow for MBA including its callback > function and sysctl interface. > > Signed-off-by: Yi Sun > --- > v3: > - replace 'PSR_VAL_TYPE_MBA' to 'PSR_TYPE_MBA_THRTL'. >

Re: [Xen-devel] [PATCH v3 06/15] x86: implement get value interface for MBA

2017-09-19 Thread Roger Pau Monné
On Tue, Sep 05, 2017 at 05:32:28PM +0800, Yi Sun wrote: > This patch implements get value domctl interface for MBA. > > Signed-off-by: Yi Sun > Reviewed-by: Roger Pau Monné > --- > v3: > - change 'PSR_VAL_TYPE_MBA' to 'PSR_TYPE_MBA_THRTL'. >

Re: [Xen-devel] [PATCH v2 2/5] libxl: enable per-VCPU extratime flag for RTDS

2017-09-19 Thread Dario Faggioli
On Fri, 2017-09-15 at 12:01 -0400, Meng Xu wrote: > On Wed, Sep 13, 2017 at 8:16 PM, Dario Faggioli > wrote: > > > > > I'm ok with what it is in this patch, although I feel that we can > > > kill the > > >  if (scinfo->extratime != > > >

Re: [Xen-devel] [PATCH 0/4] xen: sched: optimize exclusive pinning and soft-affinity checking

2017-09-19 Thread Dario Faggioli
On Mon, 2017-09-18 at 20:42 +0100, Anshul Makkar wrote: > > On 9/15/17 6:35 PM, Dario Faggioli wrote: > > Hello, > > > > Dario Faggioli (4): > >    xen: sched: introduce 'adjust_affinity' hook. > >    xen: sched: optimize exclusive pinning case (Credit1 & 2) > >    xen: sched:

Re: [Xen-devel] Booting signed xen.efi through shim

2017-09-19 Thread Tamas K Lengyel
On Mon, Sep 18, 2017 at 2:58 AM, Jan Beulich wrote: On 14.09.17 at 18:20, wrote: >> Of course, you can grab them from here: >> https://drive.google.com/drive/folders/0B5duyI9SzNtWaXE0cjM1QzZJbVk?usp=shar >> ing > > So the dumps of the two (using my

Re: [Xen-devel] [PATCH] xen, arm64: drop dummy lookup_address()

2017-09-19 Thread Stefano Stabellini
On Tue, 19 Sep 2017, Boris Ostrovsky wrote: > On 09/18/2017 06:35 PM, Tycho Andersen wrote: > > This is unused, and conflicts with the definition that we'll add for XPFO. > > > > Signed-off-by: Tycho Andersen > > CC: Boris Ostrovsky > > CC: Juergen

[Xen-devel] [linux-4.1 test] 113603: tolerable FAIL - PUSHED

2017-09-19 Thread osstest service owner
flight 113603 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/113603/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemuu-win7-amd64 18 guest-start/win.repeat fail blocked in 112503

[Xen-devel] [xen-unstable-smoke test] 113610: tolerable all pass - PUSHED

2017-09-19 Thread osstest service owner
flight 113610 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/113610/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 13 migrate-support-checkfail never pass test-armhf-armhf-xl

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

2017-09-19 Thread osstest service owner
flight 113602 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/113602/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 15 guest-saverestorefail REGR. vs. 113387 Tests which are

[Xen-devel] [PATCH v4 1/1] public/io/netif.h: add gref mapping control messages

2017-09-19 Thread Joao Martins
Adds 3 messages to allow guest to let backend keep grants mapped, such that 1) guests allowing fast recycling of pages can avoid doing grant ops for those cases, or otherwise 2) preferring copies over grants and 3) always using a fixed set of pages for network I/O. The three control ring messages

[Xen-devel] [PATCH v4 0/1] netif: staging grants for I/O requests

2017-09-19 Thread Joao Martins
Hey, This is v4 taking into consideration all comments received from v3 (changelog in the first patch). The specification is right after the diffstat. Reference implementation also here (on top of net-next): https://github.com/jpemartins/linux.git xen-net-stg-gnts-v3 Cheers, Joao Martins (1):

Re: [Xen-devel] [PATCH v5 06/10] arm: smccc: handle SMCs according to SMCCC

2017-09-19 Thread Volodymyr Babchuk
Hi Julien, On 13.09.17 14:11, Julien Grall wrote: Hi, On 08/31/2017 09:09 PM, Volodymyr Babchuk wrote: +static void fill_uuid(struct cpu_user_regs *regs, const xen_uuid_t *u) Actually why do you pass a pointer for u? This requires every caller to introduce temporary variable because the

Re: [Xen-devel] [PATCH v2] xen: grant-table: Simplify get_paged_frame

2017-09-19 Thread Julien Grall
On 19/09/17 10:47, Paul Durrant wrote: -Original Message- From: Julien Grall [mailto:julien.gr...@arm.com] Sent: 19 September 2017 10:40 To: Jan Beulich ; Paul Durrant Cc: Andrew Cooper ; George Dunlap

Re: [Xen-devel] [PATCH v3 11/15] tools: implement the new xl get hw info interface

2017-09-19 Thread Roger Pau Monné
On Tue, Sep 05, 2017 at 05:32:33PM +0800, Yi Sun wrote: > This patch implements a new xl get HW info interface. A new argument > is added for psr-hwinfo command to get and show MBA HW info. > > Signed-off-by: Yi Sun > --- > v3: > - change the format string of printf

Re: [Xen-devel] [PATCH 03/22] xentoolcore, _restrict_all: Introduce new library and implementation

2017-09-19 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 03/22] xentoolcore, _restrict_all: Introduce new library and implementation"): > On Fri, Sep 15, 2017 at 07:48:40PM +0100, Ian Jackson wrote: > > +int xentoolcore_restrict_all(uint32_t domid) { > > +/* xxx lock */ > > This should be removed. Oops, missed one indeed.

Re: [Xen-devel] [PATCH 03/22] xentoolcore, _restrict_all: Introduce new library and implementation

2017-09-19 Thread Wei Liu
On Tue, Sep 19, 2017 at 11:47:12AM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 03/22] xentoolcore, _restrict_all: Introduce new > library and implementation"): > > On Fri, Sep 15, 2017 at 07:48:40PM +0100, Ian Jackson wrote: > > > +int xentoolcore_restrict_all(uint32_t domid) { > > >

Re: [Xen-devel] [PATCH 20/22] libxl: userlookup_helper_getpwnam rename and turn into a macro

2017-09-19 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 20/22] libxl: userlookup_helper_getpwnam rename and turn into a macro"): > On Fri, Sep 15, 2017 at 07:48:57PM +0100, Ian Jackson wrote: > > +#define DEFINE_USERLOOKUP_HELPER(NAME,SPEC_TYPE,STRUCTNAME,SYSCONF) > > \ > > Some of the "\"'s are misaligned. So

Re: [Xen-devel] [PATCH 10/22] xentoolcore_restrict_all: Implement for libxenforeignmemory

2017-09-19 Thread Wei Liu
On Tue, Sep 19, 2017 at 11:51:06AM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 10/22] xentoolcore_restrict_all: Implement for > libxenforeignmemory"): > > Same comment for xendevicemodel_restrict on idempotent applies here. > > Are you happy with me fixing this by changing the docs

Re: [Xen-devel] [PATCH v3 09/17] livepatch/arm[32, 64]: Modify livepatch_funcs

2017-09-19 Thread Julien Grall
Hi Konrad, On 19/09/17 01:35, Konrad Rzeszutek Wilk wrote: On Thu, Sep 14, 2017 at 02:20:42PM +0100, Julien Grall wrote: Hi Konrad, On 12/09/17 01:37, Konrad Rzeszutek Wilk wrote: This was found when porting livepatch-build-tools to ARM64/32. When livepatch-build-tools are built (and

Re: [Xen-devel] [PATCH v3 15/15] docs: add MBA description in docs

2017-09-19 Thread Roger Pau Monné
On Tue, Sep 05, 2017 at 05:32:37PM +0800, Yi Sun wrote: > This patch adds MBA description in related documents. > > Signed-off-by: Yi Sun > Acked-by: Wei Liu > --- > v2: > - state the value type shown by 'psr-mba-show'. For linear mode, >

[Xen-devel] [PATCH 2/2] xen: dont try setting max grants multiple times

2017-09-19 Thread Juergen Gross
Trying to call xengnttab_set_max_grants() with the same file handle might fail on some kernels, as this operation is allowed only once. This is a problem for the qdisk backend as blk_connect() can be called multiple times for a domain, e.g. in case grub-xen is being used to boot it. So instead

[Xen-devel] [PATCH 1/2] xen: add a global indicator for grant copy being available

2017-09-19 Thread Juergen Gross
The Xen qdisk backend needs to test whether grant copy operations is available in the kernel. Unfortunately this collides with using xengnttab_set_max_grants() on some kernels as this operation has to be the first one after opening the gnttab device. In order to solve this problem test for the

[Xen-devel] [PATCH 0/2] xen: fix gnttab handling with old dom0 kernels

2017-09-19 Thread Juergen Gross
Trying to start a domain with a qdisk backend on an old dom0 kernel (non-pvops xen kernel) will fail as xengnttab_set_max_grants() will succeed only if called directly after opening the gnttab device. These two patches address the issue by: - moving the test for availability of

Re: [Xen-devel] [Xen-users] UEFI Secure Boot Xen 4.9

2017-09-19 Thread Daniel Kiper
On Mon, Sep 18, 2017 at 11:24:15AM -0400, Tamas K Lengyel wrote: > On Tue, Sep 5, 2017 at 12:26 PM, Tamas K Lengyel > wrote: > > On Mon, Sep 4, 2017 at 6:40 AM, Daniel Kiper > > wrote: > >> On Wed, Aug 30, 2017 at 10:16:23AM -0600, Tamas K

Re: [Xen-devel] [PATCH 03/22] xentoolcore, _restrict_all: Introduce new library and implementation

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:40PM +0100, Ian Jackson wrote: > + > +int xentoolcore_restrict_all(uint32_t domid) { > +/* xxx lock */ > +int r; > +Xentoolcore__Active_Handle *ah; > + > +lock(); > +XENTOOLCORE_LIST_FOREACH(ah, , entry) { > +r = ah->restrict_callback(ah,

Re: [Xen-devel] [PATCH 11/22] xentoolcore_restrict_all: Declare problems due to no evtchn support

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:48PM +0100, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 10/22] xentoolcore_restrict_all: Implement for libxenforeignmemory

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:47PM +0100, Ian Jackson wrote: > +static int all_restrict_cb(Xentoolcore__Active_Handle *ah, uint32_t domid) { > +xenforeignmemory_handle *fmem = CONTAINER_OF(ah, *fmem, tc_ah); > + > +if (fmem->fd < 0) > +/* just in case */ > +return 0; > + >

Re: [Xen-devel] [PATCH 12/22] xentoolcore_restrict_all: "Implement" for xengnttab

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:49PM +0100, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2] xen: grant-table: Simplify get_paged_frame

2017-09-19 Thread Paul Durrant
> -Original Message- > From: Julien Grall [mailto:julien.gr...@arm.com] > Sent: 19 September 2017 10:40 > To: Jan Beulich ; Paul Durrant > > Cc: Andrew Cooper ; George Dunlap > ; Ian Jackson

Re: [Xen-devel] [PATCH 17/22] xl, libxl: Provide dm_restrict

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:54PM +0100, Ian Jackson wrote: > This functionality is still quite imperfect, but it will be useful in > certain restricted use cases. > > Signed-off-by: Ian Jackson Seeing this is mostly plumbing for QEMU and a technology preview

Re: [Xen-devel] [PATCH 22/22] RFC: tools: xentoolcore_restrict_all: use domid_t

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:59PM +0100, Ian Jackson wrote: > This is an RFC because it does not currently compile, because not all > the places that use xentoolcore have a definition of domid in scope! > > Signed-off-by: Ian Jackson If the places you mentioned are

Re: [Xen-devel] [PATCH v7 04/16] xen: add new domctl hypercall to set grant table resource limits

2017-09-19 Thread Juergen Gross
On 19/09/17 12:14, Paul Durrant wrote: >> -Original Message- >> From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of >> Juergen Gross >> Sent: 19 September 2017 10:59 >> To: xen-de...@lists.xenproject.org >> Cc: Juergen Gross ; sstabell...@kernel.org; Wei

Re: [Xen-devel] [PATCH v7 14/16] xen: make grant resource limits per domain

2017-09-19 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Juergen Gross > Sent: 19 September 2017 10:59 > To: xen-de...@lists.xenproject.org > Cc: Juergen Gross ; sstabell...@kernel.org; Wei Liu > ; George Dunlap

Re: [Xen-devel] [PATCH v3 12/15] tools: rename 'xc_psr_cat_type' to 'xc_psr_type'

2017-09-19 Thread Roger Pau Monné
On Tue, Sep 05, 2017 at 05:32:34PM +0800, Yi Sun wrote: > This patch renames 'xc_psr_cat_type' to 'xc_psr_type' so that > the structure name is common for all allocation features. > > Signed-off-by: Yi Sun > Acked-by: Wei Liu > Reviewed-by: Chao

Re: [Xen-devel] [PATCH v7 09/16] libxl: add max possible mfn to libxl_physinfo

2017-09-19 Thread Ian Jackson
Juergen Gross writes ("[PATCH v7 09/16] libxl: add max possible mfn to libxl_physinfo"): > Add the maximum possible mfn of the host to the libxl_physinfo > data structure. Acked-by: Ian Jackson ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v7 11/16] libxl: add libxl support for setting grant table resource limits

2017-09-19 Thread Ian Jackson
Juergen Gross writes ("[PATCH v7 11/16] libxl: add libxl support for setting grant table resource limits"): > Add new domain config items for setting the limits for the maximum > numbers of grant table frames and maptrack frames of a domain. Acked-by: Ian Jackson

Re: [Xen-devel] [PATCH] MAINTAINERS: Add public/arch-arm.h under the ARM subsystem

2017-09-19 Thread Wei Liu
On Tue, Sep 19, 2017 at 12:25:53PM +0100, Julien Grall wrote: > The header public/arch-arm.h contains mostly ARM specific code. Avoid CC > the "THE REST" maintainers on it. > > Signed-off-by: Julien Grall Acked-by: Wei Liu

Re: [Xen-devel] [PATCH v3] xen: grant-table: Simplify get_paged_frame

2017-09-19 Thread Wei Liu
On Tue, Sep 19, 2017 at 12:22:28PM +0100, Julien Grall wrote: > The implementation of get_paged_frame is currently different whether the > architecture support sharing memory or paging memory. Both > version are extremely similar so it is possible to consolidate in a > single implementation. > >

[Xen-devel] [PATCH] MAINTAINERS: Add public/arch-arm.h under the ARM subsystem

2017-09-19 Thread Julien Grall
The header public/arch-arm.h contains mostly ARM specific code. Avoid CC the "THE REST" maintainers on it. Signed-off-by: Julien Grall --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 77b1e11b8c..12dbad130c 100644 ---

Re: [Xen-devel] [PATCH v3] xen: grant-table: Simplify get_paged_frame

2017-09-19 Thread Wei Liu
On Tue, Sep 19, 2017 at 12:34:28PM +0100, Julien Grall wrote: > Would the below patch be fine? Yes. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 13/22] tools/xenstore: get_handle: use "goto err" error handling style

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:50PM +0100, Ian Jackson wrote: > Replace the ad-hoc exit clauses with the error handling style where > - local variables contain either things to be freed, or sentinels > - all error exits go via an "err" label which frees everything > > Signed-off-by: Ian

Re: [Xen-devel] [PATCH 14/22] tools/xenstore: get_handle: Allocate struct before opening fd

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:51PM +0100, Ian Jackson wrote: > Now we can also abolish the temporary local variable "fd" and simply > use h->fd. > > This ordering is necessary to be able to call > xentoolcore__register_active_handle sensibly. > > Signed-off-by: Ian Jackson

Re: [Xen-devel] [PATCH 19/22] libxl: libxl__dm_runas_helper: return pwd

2017-09-19 Thread Wei Liu
On Fri, Sep 15, 2017 at 07:48:56PM +0100, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v7 13/16] xen/arm: move arch specific grant table bits into grant_table.c

2017-09-19 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Juergen Gross > Sent: 19 September 2017 10:59 > To: xen-de...@lists.xenproject.org > Cc: Juergen Gross ; sstabell...@kernel.org; Wei Liu > ; George Dunlap

  1   2   3   >