[Xen-devel] [xen-unstable test] 113001: tolerable trouble: blocked/broken/fail/pass

flight 113001 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/113001/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 7 xen-boot fail pass in 112992 test-amd64-i386-migrupgrade

[Xen-devel] [qemu-mainline test] 112999: regressions - trouble: blocked/broken/fail/pass

flight 112999 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/112999/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-xsm 6 xen-buildfail REGR. vs. 112869 Tests which are

[Xen-devel] [ovmf baseline-only test] 72052: all pass

This run is configured for baseline tests only. flight 72052 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72052/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 1cc0f69bbecc8a67969d6c1eae495ca95af6c63c baseline

[Xen-devel] [linux-4.9 test] 112996: regressions - trouble: blocked/broken/fail/pass

flight 112996 linux-4.9 real [real] http://logs.test-lab.xenproject.org/osstest/logs/112996/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 112863 Tests which

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

flight 113000 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/113000/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 1cc0f69bbecc8a67969d6c1eae495ca95af6c63c baseline version: ovmf

[Xen-devel] [xen-unstable-smoke test] 113003: tolerable trouble: broken/pass - PUSHED

flight 113003 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/113003/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a build-arm64-pvops 2

[Xen-devel] [xen-unstable baseline-only test] 72051: regressions - trouble: blocked/broken/fail/pass

This run is configured for baseline tests only. flight 72051 xen-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72051/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-win7-amd64 16

[Xen-devel] [linux-3.18 test] 112995: trouble: blocked/broken/fail/pass

flight 112995 linux-3.18 real [real] http://logs.test-lab.xenproject.org/osstest/logs/112995/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-pvops 3 capture-logs broken REGR. vs. 112102

[Xen-devel] [PATCH] xen: Drop asmlinkage everywhere

asmlinkage is defined as nothing on all architectures, and not used consistently anywhere, even in common code. Remove it all. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Stefano Stabellini CC: Julien Grall

Re: [Xen-devel] [PATCH] xen: use vMSI related #define-s from public interface

On Fri, Sep 01, 2017 at 10:25:42AM -0600, Jan Beulich wrote: > Xen and qemu having identical #define-s (with different names) is a > strong hint that these should have been part of the public interface > from the very start. Use them if they're available, falling back to > privately defined values

Re: [Xen-devel] [PATCH resend] domctl/x86: move vMSI related #define-s to public interface

On Fri, Sep 01, 2017 at 10:24:10AM -0600, Jan Beulich wrote: > Xen and qemu having identical #define-s (with different names) is a > strong hint that these should be part of the public interface, at the > same time making obvious that any change to the values in an interface > modification (and

Re: [Xen-devel] [PATCH v2 3/6] libxl:xl: add parsing code to parse "libxl_static_sshm" from xl config files

2017-09-02 0:03 GMT+08:00 Wei Liu : > On Sun, Aug 27, 2017 at 04:36:12PM +0800, Zhongze Liu wrote: >> Add the parsing utils for the newly introduced libxl_static_sshm struct >> to the libxl/libxlu_* family. And add realated parsing code in xl to >> parse the struct from xl

Re: [Xen-devel] [PATCH v2 5/6] libxl: support mapping static shared memory areas during domain creation

Hi Wei, Thanks for reviewing. 2017-09-02 0:23 GMT+08:00 Wei Liu : > On Sun, Aug 27, 2017 at 04:36:14PM +0800, Zhongze Liu wrote: >> Add libxl__sshm_add to map shared pages from one DomU to another, The mapping >> process involves the follwing steps: >> >> * Set defaults

Re: [Xen-devel] [PATCH 2/2] x86/mm: Use mfn_t for make_cr3()

On Wed, Aug 30, 2017 at 1:19 PM, Andrew Cooper wrote: > No functional change. > > Signed-off-by: Andrew Cooper Acked-by: George Dunlap > --- > CC: Jan Beulich > CC: Wei Liu

Re: [Xen-devel] [PATCH 1/2] x86/mm: Use mfn_t for new_guest_cr3()

On Wed, Aug 30, 2017 at 1:19 PM, Andrew Cooper wrote: > No functional change (as confirmed by diffing the assembly). > > Signed-off-by: Andrew Cooper Acked-by: George Dunlap > --- > CC: Jan Beulich

[Xen-devel] [PATCH] x86/mm: Consolidate all Xen L4 slot writing into init_xen_l4_slots()

Having all of this logic together makes it easier to follow Xen's virtual setup across the whole system. No practical changes to the resulting L4, although the logic has been rearanged to avoid rewriting some slots. This changes the zap_ro_mpt parameter to simply ro_mpt. Another side effect is

Re: [Xen-devel] [PATCH] libxc: increase maximum migration stream record length

On Thu, Aug 31, 2017 at 11:24:23AM +0100, Ian Jackson wrote: > Andrew Cooper writes ("Re: [PATCH] libxc: increase maximum migration stream > record length"): > > On 10/08/17 12:24, Juergen Gross wrote: > > > Today the maximum record lenth in a migration stream is 8MB. This > > > limits the size

Re: [Xen-devel] [PATCH] libxl: fix incremental parallel build

Jan Beulich writes ("Re: [PATCH] libxl: fix incremental parallel build"): > On 01.09.17 at 17:28, wrote: > > Do you mean parallel build in which two makes enter libxl? Is that > > possible? > > No, only a single entry into that subtree. Are you sure ? > > Why does libacpi

Re: [Xen-devel] [PATCH] libxl, xl: change p9 to p9s

On Wed, Aug 30, 2017 at 12:24:58PM +0100, Wei Liu wrote: > On Tue, Aug 29, 2017 at 12:19:01PM +0100, Wei Liu wrote: > > To match our naming convention. Since we released p9 one release ago, > > we need to define a macro inside libxl.h to indicate the change. > > > > Signed-off-by: Wei Liu

[Xen-devel] [OSSTEST PATCH 1/2] PERLLIB, @INC: Use BEGIN { }

unshifting @INC is only effective if done at compile time, so that it affects `use' directives. So "Perl: Put . on @INC" f261b07cbe96844dba5d8594c38149c0b179068a is simply broken This patch was autogenerated using the following rune: git-grep -l 'unshift @INC' | xargs perl -i~ -pe

[Xen-devel] [OSSTEST PATCH] authorized_keys: Look for ed25519 and ecdsa keys too

Reported-by: George Dunlap Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index 6e19b28..daeb3e0 100644 ---

Re: [Xen-devel] [PATCH resend] domctl/x86: move vMSI related #define-s to public interface

On Fri, Sep 01, 2017 at 10:24:10AM -0600, Jan Beulich wrote: > Xen and qemu having identical #define-s (with different names) is a > strong hint that these should be part of the public interface, at the > same time making obvious that any change to the values in an interface > modification (and

Re: [Xen-devel] [PATCH resend] domctl/x86: move vMSI related #define-s to public interface

On 01/09/17 17:24, Jan Beulich wrote: > Xen and qemu having identical #define-s (with different names) is a > strong hint that these should be part of the public interface, at the > same time making obvious that any change to the values in an interface > modification (and hence needs suitable

Re: [Xen-devel] [PATCH RFC] domctl: improve locking during domain destruction

On 01/09/17 17:18, Jan Beulich wrote: > There is no need to hold the global domctl lock while across > domain_kill() - the domain lock is fully sufficient here. > > Signed-off-by: Jan Beulich > --- > Sadly so far we haven't had any feedback on this change from the people > who

[Xen-devel] [PATCH] xen: use vMSI related #define-s from public interface

Xen and qemu having identical #define-s (with different names) is a strong hint that these should have been part of the public interface from the very start. Use them if they're available, falling back to privately defined values only when using older headers. Signed-off-by: Jan Beulich

Re: [Xen-devel] [PATCH v2 3/6] libxl:xl: add parsing code to parse "libxl_static_sshm" from xl config files

On Sun, Aug 27, 2017 at 04:36:12PM +0800, Zhongze Liu wrote: > Add the parsing utils for the newly introduced libxl_static_sshm struct > to the libxl/libxlu_* family. And add realated parsing code in xl to > parse the struct from xl config files. This is for the proposal "Allow > setting up shared

[Xen-devel] [PATCH resend] domctl/x86: move vMSI related #define-s to public interface

Xen and qemu having identical #define-s (with different names) is a strong hint that these should be part of the public interface, at the same time making obvious that any change to the values in an interface modification (and hence needs suitable care). Signed-off-by: Jan Beulich

Re: [Xen-devel] [PATCH v2 5/6] libxl: support mapping static shared memory areas during domain creation

On Sun, Aug 27, 2017 at 04:36:14PM +0800, Zhongze Liu wrote: > Add libxl__sshm_add to map shared pages from one DomU to another, The mapping > process involves the follwing steps: > > * Set defaults and check for further errors in the static_shm configs: > overlapping areas, invalid ranges,

[Xen-devel] [PATCH] domctl/x86: move vMSI related #define-s to public interface

Xen and qemu having identical #define-s (with different names) is a strong hint that these should be part of the public interface, at the same time making obvious that any change to the values in an interface modification (and hence needs suitable care). Signed-off-by: Jan Beulich

[Xen-devel] [PATCH RFC] domctl: improve locking during domain destruction

There is no need to hold the global domctl lock while across domain_kill() - the domain lock is fully sufficient here. Signed-off-by: Jan Beulich --- Sadly so far we haven't had any feedback on this change from the people who observed an issue apparently resulting from heavy

Re: [Xen-devel] [PATCH v3] xen: Implement hypercall for tracing of program counters

>>> On 11.08.17 at 17:25, wrote: > This commit makes the changes to the hypervisor, the build system as > well as libxc necessary in order to facilitate tracing of program counters. There are no libxc changes here afaics. > A discussion of the design can be found in

[Xen-devel] [PATCH v9 3/3] tools/libxc: use superpages during restore of HVM guest

During creating of a HVM domU meminit_hvm() tries to map superpages. After save/restore or migration this mapping is lost, everything is allocated in single pages. This causes a performance degradition after migration. Add neccessary code to preallocate a superpage for the chunk of pfns that is

[Xen-devel] [PATCH v9 1/3] tools/libxc: move SUPERPAGE macros to common header

The macros SUPERPAGE_2MB_SHIFT and SUPERPAGE_1GB_SHIFT will be used by other code in libxc. Move the macros to a header file. Signed-off-by: Olaf Hering Acked-by: Wei Liu --- tools/libxc/xc_dom_x86.c | 5 - tools/libxc/xc_private.h | 5 + 2 files

[Xen-devel] [PATCH v9 0/3] tools/libxc: use superpages

Using superpages on the receiving dom0 will avoid performance regressions. Olaf v9: update hole checking in x86_hvm_populate_pfns add out of bounds check to xc_sr_test_and_set/clear_bit v8: remove double check of 1G/2M idx in x86_hvm_populate_pfns v7: cover holes that span multiple

[Xen-devel] [PATCH v9 2/3] tools/libxc: add API for bitmap access for restore

Extend API for managing bitmaps. Each bitmap is now represented by a generic struct xc_sr_bitmap. Switch the existing populated_pfns to this API. Signed-off-by: Olaf Hering Acked-by: Wei Liu --- tools/libxc/xc_sr_common.c | 41 ++

Re: [Xen-devel] [PATCH v2 3/6] libxl:xl: add parsing code to parse "libxl_static_sshm" from xl config files

On Sun, Aug 27, 2017 at 04:36:12PM +0800, Zhongze Liu wrote: > Add the parsing utils for the newly introduced libxl_static_sshm struct > to the libxl/libxlu_* family. And add realated parsing code in xl to > parse the struct from xl config files. This is for the proposal "Allow > setting up shared

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

Dario, I didn't include your Reviewed-by tag because I made one small change. On Fri, Sep 1, 2017 at 11:58 AM, Meng Xu wrote: > > Modify libxl_vcpu_sched_params_get/set and sched_rtds_vcpu_get/set > functions to support per-VCPU extratime flag > > Signed-off-by: Meng Xu

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

Revise xl tool use case by adding -e option Remove work-conserving from TODO list Signed-off-by: Meng Xu --- Changes from v1 Revise rtds docs --- docs/features/sched_rtds.pandoc | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[Xen-devel] [PATCH v2 0/5] Towards work-conserving RTDS

This series of patches make RTDS scheduler work-conserving without breaking real-time guarantees. VCPUs with extratime flag set can get extra time from the unreserved system resource. System administrators can decide which VCPUs have extratime flag set. Example: Set the extratime bit of all VCPUs

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

Modify libxl_vcpu_sched_params_get/set and sched_rtds_vcpu_get/set functions to support per-VCPU extratime flag Signed-off-by: Meng Xu --- Changes from v1 1) Add LIBXL_HAVE_SCHED_RTDS_VCPU_EXTRA to indicate if extratime flag is supported 2) Change flag name in domctl.h

[Xen-devel] [PATCH v2 3/5] xl: enable per-VCPU extratime flag for RTDS

Change main_sched_rtds and related output functions to support per-VCPU extratime flag. Signed-off-by: Meng Xu --- Changes from v1 No change because we agree on using -e 0/1 option to set if a vcpu will get extra time or not Changes from RFC v1 Changes work_conserving

[Xen-devel] [PATCH v2 4/5] xentrace: enable per-VCPU extratime flag for RTDS

Change repl_budget event output for xentrace formats and xenalyze Signed-off-by: Meng Xu --- Changes from v1 Add this changes from v1 --- tools/xentrace/formats| 2 +- tools/xentrace/xenalyze.c | 8 +--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git

[Xen-devel] [PATCH v2 1/5] xen:rtds: towards work conserving RTDS

Make RTDS scheduler work conserving without breaking the real-time guarantees. VCPU model: Each real-time VCPU is extended to have an extratime flag and a priority_level field. When a VCPU's budget is depleted in the current period, if it has extratime flag set, its priority_level will increase

Re: [Xen-devel] [PATCH v2] tools: change the type of '*nr' in 'libxl_psr_cat_get_info'

On Fri, Sep 01, 2017 at 08:15:05PM +0800, Yi Sun wrote: > Due to historical reason, type of parameter '*nr' in 'libxl_psr_cat_get_info' > is 'int'. But this is not right. It should be 'unsigned int'. This patch fixes > this and does related changes. > > Suggested-by: Roger Pau Monné

[Xen-devel] [OSSTEST PATCH 1/2] mg-hosts: Fix another couple of prepared statements

Use our $blah_q //= ... since these are used only here. This is a cheap kind of memoisation of a global constant. Reported-by: George Dunlap Signed-off-by: Ian Jackson --- mg-hosts | 9 + 1 file changed, 5 insertions(+), 4

[Xen-devel] [OSSTEST PATCH 2/2] PERLLIB, @INC: Use BEGIN { }

unshifting @INC is only effective if done at compile time, so that it affects `use' directives. So "Perl: Put . on @INC" f261b07cbe96844dba5d8594c38149c0b179068a is simply broken This patch was autogenerated using the following rune: git-grep -l 'unshift @INC' | xargs perl -i~ -pe

Re: [Xen-devel] [PATCH] tools: change the type of '*nr' in 'libxl_psr_cat_get_info'

On Fri, Sep 01, 2017 at 02:59:23PM +0800, Yi Sun wrote: > Due to historical reason, type of parameter '*nr' in 'libxl_psr_cat_get_info' > is 'int'. But this is not right. It should be 'unsigned int'. This patch fixes > this and does related changes. > > Suggested-by: Roger Pau Monné

Re: [Xen-devel] [PATCH 2/2] tools: remove unnecessary PSR macros

On Thu, Aug 31, 2017 at 04:08:06PM +0800, Yi Sun wrote: > Per Roger Pau's coment, the libxl interfaces and related functions are not > necessary to be included by 'LIBXL_HAVE_PSR_CMT' and 'LIBXL_HAVE_PSR_CAT'. > So remove them. > > Signed-off-by: Yi Sun Acked-by: Wei

Re: [Xen-devel] [RFC PATCH V2 4/4] xl/libacpi: extend lapic_id() to uint32_t

On Thu, Aug 31, 2017 at 01:01:49AM -0400, Lan Tianyu wrote: > From: Chao Gao > > This patch is to extend lapic_id() to support more vcpus. > > Signed-off-by: Chao Gao > Signed-off-by: Lan Tianyu Acked-by: Wei Liu

Re: [Xen-devel] [PATCH v3 2/2] x86/mm: merge ptwr and mmio_ro page fault handlers

>>> On 01.09.17 at 16:35, wrote: > Provide a unified entry to avoid going through pte look-up, decode and > emulation cycle more than necessary. The path taken is determined by > the faulting address. > > Note that the order of checks is changed in the new function, but the

Re: [Xen-devel] [PATCH v4 37/39] altp2m: Allow activating altp2m on ARM domains

On Wed, Aug 30, 2017 at 08:32:56PM +0200, Sergej Proskurin wrote: > The previous libxl implemention limited the use of altp2m to x86 HVM domains. > This commit extends libxl by introducing the altp2m switch to ARM domains. > > Additionally, we introduce the macro LIBXL_HAVE_ARM_ALTP2M in parallel

Re: [Xen-devel] [PATCH] libxl: fix incremental parallel build

>>> On 01.09.17 at 17:28, wrote: > On Wed, Aug 30, 2017 at 01:06:10AM -0600, Jan Beulich wrote: >> Short-circuit absolute paths of generated headers to their pathless >> equivalents, to avoid two racing invocations of the same rule producing >> them. >> >> Signed-off-by: Jan

Re: [Xen-devel] [PATCH v4 36/39] altp2m: Document external-only use on ARM

On Wed, Aug 30, 2017 at 08:32:55PM +0200, Sergej Proskurin wrote: > From: Tamas K Lengyel > > Currently, the altp2m feature has been used and thus documented for the > x86 architecture. As we aim to introduce altp2m to ARM, in this commit, > we adjust the

Re: [Xen-devel] [PATCH v2 2/2] x86/mm: merge ptwr and mmio_ro page fault handlers

>>> On 01.09.17 at 15:50, wrote: > On Fri, Sep 01, 2017 at 02:46:27PM +0100, Wei Liu wrote: >> On Fri, Sep 01, 2017 at 03:38:49AM -0600, Jan Beulich wrote: >> > > + struct cpu_user_regs *regs) >> > > +{ >> > > +l1_pgentry_t pte; >> > > +struct

[Xen-devel] [PATCH] x86/shadow: Clarify the safety of guest-linear mapping construction

sh_install_xen_entries_in_l4() looks unsafe, as it creates a guest-linear mapping with gl4mfn. However, it is correct because of the way monitor tables are constructed for translated domains. Leave a comment and some clarifying assertions. Also, there is no longer support for translate !=

Re: [Xen-devel] [PATCH] libxl: fix incremental parallel build

On Wed, Aug 30, 2017 at 01:06:10AM -0600, Jan Beulich wrote: > Short-circuit absolute paths of generated headers to their pathless > equivalents, to avoid two racing invocations of the same rule producing > them. > > Signed-off-by: Jan Beulich > --- > This wants to be

[Xen-devel] [OSSTEST PATCH] README: document need for libnet-snmp-perl

Signed-off-by: Ian Jackson --- README | 1 + 1 file changed, 1 insertion(+) diff --git a/README b/README index b45058d..ffe0018 100644 --- a/README +++ b/README @@ -254,6 +254,7 @@ To run osstest in standalone mode: chiark-utils-bin libxml-libxml-perl

[Xen-devel] [OSSTEST PATCH] mg-host: Prepare $checkhostq only if needed

This is helpful because in standalone mode there is no resources table and this fails. But we want people to use `mg-hosts power' in standalone mode, if they can, rather than running ./msu-pdu ad-hoc (or whatever). Reported-by: George Dunlap Signed-off-by: Ian Jackson

[Xen-devel] [OSSTEST PATCH] PERLLIB, @INC: Add . in some places we missed

* cri-getconfig (invoked by many cr* scripts, standalone-reset, etc.) * mgi-common (invoked by many mg*) * Two literal perl -e includes in Tcl code. I think this leaves no scripts which do not set PERLLIB or @INC, either by themselves or by something they source with `.'. Reported-by: George

Re: [Xen-devel] [PATCH RFC] Add SUPPORT.md\

On Thu, Aug 31, 2017 at 11:27:19AM +0100, George Dunlap wrote: > +### Direct-boot kernel image format > + > +Supported, x86: bzImage Do you mean booting a PV guest? If so there are a few more formats. > +Supported, ARM32: zImage > +Supported, ARM64: Image [XXX - Not sure if this is

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

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 v2 0/1] netif: staging grants for I/O requests

Hey, This is v2 taking into consideration all comments received from RFC/v1. The approach has significantly changed and we use control ring messages to the manage the permanent gref mappings - as also presented at XDDS17. The specification is right after the diffstat. I kept the same name

Re: [Xen-devel] [PATCH 00/13] mmu_notifier kill invalidate_page callback

On Wed, Aug 30, 2017, at 10:57 AM, Adam Borowski wrote: > On Tue, Aug 29, 2017 at 08:56:15PM -0400, Jerome Glisse wrote: > > I will wait for people to test and for result of my own test before > > reposting if need be, otherwise i will post as separate patch. > > > > > But from a _very_ quick

[Xen-devel] [linux-linus test] 112993: tolerable trouble: blocked/broken/fail/pass - PUSHED

flight 112993 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/112993/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemut-win7-amd64 17 guest-stop fail REGR. vs. 112983 Tests which did not

Re: [Xen-devel] [PATCH] x86/public: Further correctsion to vcpu context comments

There's a typo in the subject line s/correctsion/correction/. On Fri, Sep 01, 2017 at 02:14:41PM +0100, Andrew Cooper wrote: > VCPUOP_initialise and DOMCTL_setvcpucontext are not symetric. > > Signed-off-by: Andrew Cooper Reviewed-by: Roger Pau Monné

[Xen-devel] [PATCH v3 2/2] x86/mm: merge ptwr and mmio_ro page fault handlers

Provide a unified entry to avoid going through pte look-up, decode and emulation cycle more than necessary. The path taken is determined by the faulting address. Note that the order of checks is changed in the new function, but the order of the checks is performed shouldn't matter. The sole

[Xen-devel] [PATCH v3 1/2] x86/mm: don't wrap x86_emulate_ctxt in ptwr_emulate_ctxt

Rewrite the code so that it has the same structure as mmio_ro_emualte_ctxt. x86_emulate_ctxt now points to ptwr_emulate_ctxt via its data pointer. This patch will help unify mmio_ro and ptwr code paths later. Signed-off-by: Wei Liu Reviewed-by: Andrew Cooper

[Xen-devel] [PATCH v3 0/2] x86/mm: merge ptwr and mmio_ro page fault handlers

Can be found at my xenbits/xen.git wip.merge-mmio-ptwr-emul-v3 Wei Liu (2): x86/mm: don't wrap x86_emulate_ctxt in ptwr_emulate_ctxt x86/mm: merge ptwr and mmio_ro page fault handlers xen/arch/x86/mm.c| 222 --- xen/arch/x86/traps.c |

Re: [Xen-devel] [PATCH v2] tools: change the type of '*nr' in 'libxl_psr_cat_get_info'

On Fri, Sep 01, 2017 at 08:15:05PM +0800, Yi Sun wrote: > Due to historical reason, type of parameter '*nr' in 'libxl_psr_cat_get_info' > is 'int'. But this is not right. It should be 'unsigned int'. This patch fixes > this and does related changes. > > Suggested-by: Roger Pau Monné

Re: [Xen-devel] [PATCH v2 2/2] x86/mm: merge ptwr and mmio_ro page fault handlers

On Fri, Sep 01, 2017 at 02:46:27PM +0100, Wei Liu wrote: > On Fri, Sep 01, 2017 at 03:38:49AM -0600, Jan Beulich wrote: > > > /* > > > * fault handling for read-only MMIO pages > > > */ > > > > Note how you move the remains of the function above below this > >

Re: [Xen-devel] [PATCH v2 2/2] x86/mm: merge ptwr and mmio_ro page fault handlers

On Fri, Sep 01, 2017 at 03:38:49AM -0600, Jan Beulich wrote: > > /* > > * fault handling for read-only MMIO pages > > */ > > Note how you move the remains of the function above below this > comment, which isn't really correct. I can place the new function where the

Re: [Xen-devel] [PATCH] x86/public: Further correctsion to vcpu context comments

>>> On 01.09.17 at 15:14, wrote: > VCPUOP_initialise and DOMCTL_setvcpucontext are not symetric. > > Signed-off-by: Andrew Cooper Acked-by: Jan Beulich ___ Xen-devel mailing

[Xen-devel] [PATCH] x86/public: Further correctsion to vcpu context comments

VCPUOP_initialise and DOMCTL_setvcpucontext are not symetric. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Roger Pau Monné CC: Wei Liu --- xen/include/public/arch-x86/xen.h | 10 ++ 1

[Xen-devel] [libvirt test] 112994: tolerable trouble: blocked/broken/pass - PUSHED

flight 112994 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/112994/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-libvirt-xsm 1 build-check(1) blocked n/a build-arm64-libvirt 1

[Xen-devel] [PATCH v2] tools: change the type of '*nr' in 'libxl_psr_cat_get_info'

Due to historical reason, type of parameter '*nr' in 'libxl_psr_cat_get_info' is 'int'. But this is not right. It should be 'unsigned int'. This patch fixes this and does related changes. Suggested-by: Roger Pau Monné Signed-off-by: Yi Sun ---

[Xen-devel] [xen-unstable-smoke test] 112997: tolerable trouble: broken/pass - PUSHED

flight 112997 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/112997/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a build-arm64-pvops 2

[Xen-devel] [xen-unstable test] 112992: tolerable trouble: blocked/broken/fail/pass - PUSHED

flight 112992 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/112992/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-libvirt-xsm 1 build-check(1) blocked n/a test-arm64-arm64-xl 1

Re: [Xen-devel] [PATCH v1] x86/hvm: Expose MSR_SHADOW_GS_BASE

On 01.09.2017 13:55, Andrew Cooper wrote: On 01/09/17 11:44, Alexandru Isaila wrote: diff --git a/xen/include/public/arch-x86/hvm/save.h b/xen/include/public/arch-x86/hvm/save.h index fd7bf3f..e6e8e87 100644 --- a/xen/include/public/arch-x86/hvm/save.h +++

Re: [Xen-devel] [PATCH] MAINTAINERS: add arch specific public headers to arch file groups

On 01/09/17 11:45, Jan Beulich wrote: > I've recently got sufficiently annoyed by people not applying enough > common sense to get_maintainer.pl output, Cc-ing all REST maintainers > on ARM-only public interface changes. > > Sort ARM's xen/ groups of path specifications at the same time. > >

Re: [Xen-devel] [PATCH v1] x86/hvm: Expose MSR_SHADOW_GS_BASE

>>> On 01.09.17 at 12:44, wrote: > This patch is adding an new param in the hvm_hw_cpu structure > so it can be exposed to user space. But you don't say why. > --- a/xen/include/public/arch-x86/hvm/save.h > +++ b/xen/include/public/arch-x86/hvm/save.h > @@ -134,6 +134,8

Re: [Xen-devel] [PATCH v1] x86/hvm: Expose MSR_SHADOW_GS_BASE

On 01/09/17 11:44, Alexandru Isaila wrote: > diff --git a/xen/include/public/arch-x86/hvm/save.h > b/xen/include/public/arch-x86/hvm/save.h > index fd7bf3f..e6e8e87 100644 > --- a/xen/include/public/arch-x86/hvm/save.h > +++ b/xen/include/public/arch-x86/hvm/save.h > @@ -134,6 +134,8 @@ struct

Re: [Xen-devel] [PATCH] xen: reset creation_finished flag on soft reset

> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: 01 September 2017 11:43 > To: Paul Durrant > Cc: Andrew Cooper ; xen- > de...@lists.xen.org; George Dunlap ; Ian > Jackson

[Xen-devel] [PATCH] MAINTAINERS: add arch specific public headers to arch file groups

I've recently got sufficiently annoyed by people not applying enough common sense to get_maintainer.pl output, Cc-ing all REST maintainers on ARM-only public interface changes. Sort ARM's xen/ groups of path specifications at the same time. Signed-off-by: Jan Beulich ---

[Xen-devel] [PATCH v1] x86/hvm: Expose MSR_SHADOW_GS_BASE

This patch is adding an new param in the hvm_hw_cpu structure so it can be exposed to user space. Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/hvm.c | 4 xen/include/public/arch-x86/hvm/save.h | 2 ++ 2 files changed, 6 insertions(+) diff

Re: [Xen-devel] [PATCH] xen: reset creation_finished flag on soft reset

Paul Durrant writes: >> -Original Message- >> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] >> Sent: 01 September 2017 10:27 >> To: Andrew Cooper ; Paul Durrant >> >> Cc: xen-devel@lists.xen.org; George

Re: [Xen-devel] [PATCH v9 1/2] x86emul: New return code for unimplemented instruction

>>> On 30.08.17 at 20:57, wrote: > --- a/xen/arch/x86/hvm/emulate.c > +++ b/xen/arch/x86/hvm/emulate.c > @@ -2044,6 +2044,7 @@ int hvm_emulate_one_mmio(unsigned long mfn, unsigned > long gla) Way earlier in this file there are consumers of X86EMUL_UNHANDLEABLE in

Re: [Xen-devel] [RFC PATCH V2 2/4] Tool/ACPI: DSDT extension to support more vcpus

On Fri, Sep 01, 2017 at 04:10:26AM -0600, Jan Beulich wrote: > >>> On 01.09.17 at 11:41, wrote: > > I'm not sure why lapic_id needs to be a hook in any case, both it's > > callers use the same exact formula (cpu_id * 2). > > You're aware that this is meant to be improved

[Xen-devel] [ovmf baseline-only test] 72050: all pass

This run is configured for baseline tests only. flight 72050 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72050/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 17cbf7359f04a4eb762b2b4bc26c2df635ef2812 baseline

[Xen-devel] [OSSTEST PATCH] production-config: Publish per-flight osstest git refs in separate tree

Those refs need to be somewhere public so that the relevant git objects are accessible. I think they probably have to be in refs/tags/ or refs/heads/ of some repo. But it doesn't have to be the main one on xenbits. Most people won't want these refs. Most git commits they are interested in (eg

Re: [Xen-devel] [RFC PATCH V2 2/4] Tool/ACPI: DSDT extension to support more vcpus

>>> On 01.09.17 at 11:41, wrote: > I'm not sure why lapic_id needs to be a hook in any case, both it's > callers use the same exact formula (cpu_id * 2). You're aware that this is meant to be improved rather sooner than later. At that point I'm then not sure a suitable

Re: [Xen-devel] [PATCH v5 1/4] VT-d PI: track the number of vcpus on pi blocking list

>>> On 01.09.17 at 11:55, wrote: On 01.09.17 at 10:37, wrote: >> it seems add_sized() won't be a LOCKed instruction. >> #define build_add_sized(name, size, type, reg) \ >> static inline void name(volatile type *addr, type val) \ >>

Re: [Xen-devel] [RFC PATCH V2 3/4] hvmload: Add x2apic entry support in the MADT build

On Thu, Aug 31, 2017 at 01:01:48AM -0400, Lan Tianyu wrote: > This patch is to add x2apic entry support for ACPI MADT table > according to ACPI spec 5.2.12.12 Processor Local x2APIC Structure > > Signed-off-by: Chao Gao > Signed-off-by: Lan Tianyu > ---

Re: [Xen-devel] [PATCH v5 1/4] VT-d PI: track the number of vcpus on pi blocking list

>>> On 01.09.17 at 10:37, wrote: > it seems add_sized() won't be a LOCKed instruction. > #define build_add_sized(name, size, type, reg) \ > static inline void name(volatile type *addr, type val) \ > {

Re: [Xen-devel] [RFC PATCH V2 2/4] Tool/ACPI: DSDT extension to support more vcpus

On Fri, Sep 01, 2017 at 10:54:02AM +0800, Lan Tianyu wrote: > On 2017年08月31日 23:38, Roger Pau Monné wrote: > > On Thu, Aug 31, 2017 at 01:01:47AM -0400, Lan Tianyu wrote: > >> This patch is to change DSDT table for processor object to support >128 > >> vcpus > >> accroding to ACPI spec 8.4

Re: [Xen-devel] [PATCH v5 03/10] public: xen.h: add definitions for UUID handling

Volodymyr Babchuk writes ("[PATCH v5 03/10] public: xen.h: add definitions for UUID handling"): > Added type xen_uuid_t. This type represents UUID as an array of 16 > bytes in big endian format. > > Added macro XEN_DEFINE_UUID that constructs UUID in the usual way: > >

Re: [Xen-devel] [PATCH v2 2/2] x86/mm: merge ptwr and mmio_ro page fault handlers

>>> On 31.08.17 at 13:22, wrote: > --- a/xen/arch/x86/mm.c > +++ b/xen/arch/x86/mm.c > @@ -5176,91 +5176,6 @@ static const struct x86_emulate_ops ptwr_emulate_ops = > { > .cpuid = pv_emul_cpuid, > }; > > -/* Write page fault handler: check if guest is trying to

Re: [Xen-devel] [PATCH v5 1/4] VT-d PI: track the number of vcpus on pi blocking list

On Fri, Sep 01, 2017 at 03:13:17AM -0600, Jan Beulich wrote: On 01.09.17 at 09:55, wrote: >> On Fri, Sep 01, 2017 at 02:24:08AM -0600, Jan Beulich wrote: >> On 01.09.17 at 03:39, wrote: After thinking it again, I want to define the counter as

Re: [Xen-devel] [PATCH] xen: reset creation_finished flag on soft reset

> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: 01 September 2017 10:27 > To: Andrew Cooper ; Paul Durrant > > Cc: xen-devel@lists.xen.org; George Dunlap ; > Ian Jackson

Re: [Xen-devel] [PATCH] xen: reset creation_finished flag on soft reset

Andrew Cooper writes: > On 01/09/2017 10:11, Vitaly Kuznetsov wrote: >> C/s e7dabe5 ("x86/hvm: don't unconditionally create a default ioreq >> server") broke soft reset when QEMU traditional is being used. During >> soft reset QEMU is relaunched and default ioreq

Re: [Xen-devel] [PATCH v2 1/2] x86/mm: don't wrap x86_emulate_ctxt in ptwr_emulate_ctxt

>>> On 31.08.17 at 13:22, wrote: > @@ -5221,10 +5217,12 @@ int ptwr_do_page_fault(struct vcpu *v, unsigned long > addr, > goto bail; > } > > -ptwr_ctxt.cr2 = addr; > -ptwr_ctxt.pte = pte; > +ptwr_ctxt = (struct ptwr_emulate_ctxt) { > +.cr2

  1   2   >