[Xen-devel] [libvirt test] 100404: regressions - FAIL

flight 100404 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/100404/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-xsm 5 xen-buildfail REGR. vs. 100381 Tests which did not suc

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

flight 100395 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100395/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-raw 6 xen-boot fail REGR. vs. 100377 test-amd64-amd64-x

Re: [Xen-devel] [BUG] kernel BUG at drivers/block/xen-blkfront.c:1711

On 11.08.2016 05:10, Bob Liu wrote: On 08/10/2016 10:54 PM, Evgenii Shatokhin wrote: On 10.08.2016 15:49, Bob Liu wrote: On 08/10/2016 08:33 PM, Evgenii Shatokhin wrote: On 14.07.2016 15:04, Bob Liu wrote: On 07/14/2016 07:49 PM, Evgenii Shatokhin wrote: On 11.07.2016 15:04, Bob Liu wrote

Re: [Xen-devel] Livepatch, symbol resolutions between two livepatchs (new_symbol=0)

On 08/11/2016 02:28 AM, Konrad Rzeszutek Wilk wrote: Hey Ross, I am running in a symbol dependency issue that I am not exactly sure how to solve. I have an payload that introduces a new function (xen_foobar) which will patch over xen_extra_version(). snip As livepatch_symbols_lookup_by_name

Re: [Xen-devel] [PATCH v2 01/25] arm/altp2m: Add first altp2m HVMOP stubs.

Hello Tamas, On 10/08/2016 16:49, Tamas K Lengyel wrote: On Aug 10, 2016 03:52, "Julien Grall" mailto:julien.gr...@arm.com>> wrote: On 09/08/2016 21:16, Tamas K Lengyel wrote: On Wed, Aug 3, 2016 at 10:54 AM, Julien Grall > wrote: There is a rcu_lock_domain_by_an

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

On Wed, Aug 10, 2016 at 04:07:30PM +0100, Wei Liu wrote: [...] > > +def run_test_logfile(opts, test): > +""" Run a specific test via grepping log file""" > + > +fn = opts.logfile_dir + (opts.logfile_pattern % test) > +local_time = time.strftime('%Y-%m-%d %H:%M:%S', time.localtime()) >

Re: [Xen-devel] [PATCH v2 14/25] arm/altp2m: Make get_page_from_gva ready for altp2m.

On 06/08/2016 18:58, Sergej Proskurin wrote: Hi Julien, Hello Sergej, On 08/06/2016 03:45 PM, Julien Grall wrote: On 06/08/2016 11:38, Sergej Proskurin wrote: Hi Julien, Hello Serge, On 08/04/2016 01:59 PM, Julien Grall wrote: Hello Sergej, On 01/08/16 18:10, Sergej Proskurin wrot

Re: [Xen-devel] [PATCH 3/3] x86/microcode: Avoid undefined behaviour from signed integer overflow

> From: Andrew Cooper [mailto:andrew.coop...@citrix.com] > Sent: Friday, August 05, 2016 9:50 PM > To: Xen-devel > Cc: Andrew Cooper; Jan Beulich; Tian, Kevin; Nakajima, Jun > Subject: [PATCH 3/3] x86/microcode: Avoid undefined behaviour from signed > integer > overflow > > The checksum should be

Re: [Xen-devel] [PATCH 1/2] x86/vmx: dump MSR load area

>>> On 10.08.16 at 16:25, wrote: > On Wed, Aug 10, 2016 at 04:44:21AM -0600, Jan Beulich wrote: >> >>> On 10.08.16 at 08:59, wrote: >> > @@ -1879,6 +1893,13 @@ void vmcs_dump_vcpu(struct vcpu *v) >> > (SECONDARY_EXEC_ENABLE_VPID | >> > SECONDARY_EXEC_ENABLE_VM_FUNCTIONS) > >> > ) >>

Re: [Xen-devel] [PATCH v2 20/25] arm/altp2m: Add altp2m paging mechanism.

On 10/08/2016 11:32, Sergej Proskurin wrote: Hi Julien, Hello Sergej, [...] switch ( fsc ) { +case FSC_FLT_TRANS: +{ +if ( altp2m_active(d) ) +{ +const struct npfec npfec = { +.insn_fetch = 1, +.gla_valid = 1, +

Re: [Xen-devel] [PATCH 2/2] x86/vmx: conditionally disable LBR support due to TSX format quirk

>>> On 10.08.16 at 17:47, wrote: > On Wed, Aug 10, 2016 at 06:34:10AM -0600, Jan Beulich wrote: >> >>> On 10.08.16 at 08:59, wrote: >> > --- a/xen/arch/x86/hvm/vmx/vmx.c >> > +++ b/xen/arch/x86/hvm/vmx/vmx.c >> > @@ -2576,8 +2576,22 @@ static const struct lbr_info >> > *last_branch_msr_get(void)

Re: [Xen-devel] [PATCH v5 3/4] x86/ioreq server: Add HVMOP to map guest ram with p2m_ioreq_server to an ioreq server.

On 8/10/2016 6:43 PM, Yu Zhang wrote: On 8/10/2016 6:33 PM, Jan Beulich wrote: On 10.08.16 at 10:09, wrote: On 8/8/2016 11:40 PM, Jan Beulich wrote: On 12.07.16 at 11:02, wrote: @@ -178,8 +179,34 @@ static int hvmemul_do_io( break; case X86EMUL_UNHANDLEABLE: { -

Re: [Xen-devel] [PATCH v2 1/3] livepach: Add .livepatch.hooks functions and test-case

>>> On 10.08.16 at 11:46, wrote: > Odd. I've tried this simple example: > > typedef int fn_t(void); > > struct s { > unsigned n; > fn_t**fn; > fn_t*const*fnc; > const fn_t**cfn; > }; > > int test1(const struct s*ps) { > unsigned i; > int rc = 0; > > fo

[Xen-devel] [qemu-mainline test] 100397: tolerable FAIL - PUSHED

flight 100397 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100397/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop fail like 100379 test-amd64-amd64-xl-qemuu-

Re: [Xen-devel] [PATCH v5 3/4] x86/ioreq server: Add HVMOP to map guest ram with p2m_ioreq_server to an ioreq server.

>>> On 11.08.16 at 10:47, wrote: > On 8/10/2016 6:43 PM, Yu Zhang wrote: >> For " && p2mt != p2m_ioreq_server" condition, it is just to guarantee >> that if a write >> operation is trapped, and at the same period, device model changed the >> status of >> ioreq server, it should be discarded. >

Re: [Xen-devel] [PATCH v2 08/25] arm/altp2m: Add HVMOP_altp2m_set_domain_state.

Hello Sergej, On 06/08/2016 11:36, Sergej Proskurin wrote: + +/* Initialize the new altp2m view. */ +rc = p2m_init_one(d, p2m); +if ( rc ) +goto err; + +/* Allocate a root table for the altp2m view. */ +rc = p2m_alloc_table(p2m); +if ( rc ) +goto err; + +

[Xen-devel] [PATCH v3 06/19] mini-os: let memory allocation fail if no free page available

Instead of panicing when no page can be allocated try to fail the memory allocation by returning NULL instead. Signed-off-by: Juergen Gross Reviewed-by: Wei Liu Acked-by: Samuel Thibault --- V2: fixed minor style issue --- mm.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/mm.c b/

[Xen-devel] [PATCH v3 03/19] mini-os: remove MM_DEBUG code

mm.c contains unused code inside #ifdef MM_DEBUG areas. Its usability is rather questionable and some parts are even wrong (e.g. print_chunks() called with nr_pages > 1000 will clobber an arbitrary stack content with a 0 byte). Remove this code. Signed-off-by: Juergen Gross Reviewed-by: Wei Liu

[Xen-devel] [PATCH v3 07/19] mini-os: add ballooning config item

Add CONFIG_BALLOON defaulting to 'n' as a config item to Mini-OS. Add balloon.c, balloon.h and arch/*/balloon.c for future use. Signed-off-by: Juergen Gross Acked-by: Samuel Thibault --- V2: Added dummy sources and header --- Makefile | 3 +++ arch/arm/balloon.c | 28 +++

[Xen-devel] [PATCH v3 04/19] mini-os: add description of x86 memory usage

Add a brief description how the physical and virtual address usage looks like on x86 to include/x86/arch_mm.h Signed-off-by: Juergen Gross Reviewed-by: Wei Liu Acked-by: Samuel Thibault --- include/x86/arch_mm.h | 20 1 file changed, 20 insertions(+) diff --git a/include/

[Xen-devel] [PATCH v3 00/19] mini-os: support of auto-ballooning

Support ballooning Mini-OS automatically up in case of memory shortage. Do some cleanups, a small correction and add some basic features to lay groundwork for support of ballooning in Mini-OS (patches 1-14). The main visible change is the virtual memory layout: to be able to add memory to the run

[Xen-devel] [PATCH v3 02/19] mini-os: remove unused alloc_contig_pages() function

alloc_contig_pages() is never used anywhere in mini-os. Remove it. Signed-off-by: Juergen Gross Reviewed-by: Wei Liu Acked-by: Samuel Thibault --- arch/x86/mm.c | 142 -- include/mm.h | 1 - 2 files changed, 143 deletions(-) diff --gi

[Xen-devel] [PATCH v3 05/19] mini-os: add nr_free_pages counter

Add a variable holding the number of available memory pages. This will aid auto-ballooning later. Signed-off-by: Juergen Gross Reviewed-by: Wei Liu Acked-by: Samuel Thibault --- include/mm.h | 1 + mm.c | 6 ++ 2 files changed, 7 insertions(+) diff --git a/include/mm.h b/include/m

[Xen-devel] [PATCH v3 08/19] mini-os: get maximum memory size from hypervisor

Add support for obtaining the maximum memory size from the hypervisor. This will make it possible to support ballooning. Signed-off-by: Juergen Gross Acked-by: Samuel Thibault --- V2: Moved new stuff to balloon.c --- balloon.c | 22 ++ include/balloon.h | 6 ++

[Xen-devel] [PATCH v3 10/19] mini-os: remove unused mem_test() function

mem_test() isn't used anywhere and its value is rather questionable with mini-os being in a mature state. Remove the function. Signed-off-by: Juergen Gross Reviewed-by: Samuel Thibault --- arch/x86/mm.c | 55 --- 1 file changed, 55 deletions(-

[Xen-devel] [PATCH v3 09/19] mini-os: modify virtual memory layout for support of ballooning

In order to be able to support ballooning the virtual memory layout of Mini-OS has to be modified: instead of a (nearly) consecutive area used for physical memory mapping, on demand mappings, and heap we need enough spare place for adding new memory. So instead of dynamically place the different r

[Xen-devel] [PATCH v3 15/19] mini-os: remap p2m list in case of ballooning

In case of enabled ballooning we must be prepared for a growing p2m list. If the maximum memory size of the domain can't be covered by the actual p2m list remap it to the kernel virtual mapping area and leave enough space at the end. Signed-off-by: Juergen Gross Reviewed-by: Samuel Thibault ---

[Xen-devel] [PATCH v3 11/19] mini-os: add checks for out of memory

There are several core functions in Mini-OS not checking for failed memory allocations. Add such checks. Add do_map_frames() dummy function to arm architecture as it will be needed in future for compilations to succeed. Signed-off-by: Juergen Gross Reviewed-by: Samuel Thibault --- arch/arm/mm.

[Xen-devel] [PATCH v3 18/19] mini-os: balloon up in case of oom

If a memory shortage is detected balloon up. Be careful to always leave some pages free as ballooning up might need some memory, too: - new p2m frames - page tables for addressing new p2m frame - new frame for page allocation bitmap - page table for addressing new page allocation bitmap frame - p

[Xen-devel] [PATCH v3 14/19] mini-os: move p2m related macros to header file

In order to be able to use p2m related macros for ballooning move their definitions to arch/x86/mm.h. There is no need to define different macros regarding index masks and number of entries for the different levels, as all levels share the same entry format (a plain mfn). So reduce the number of m

[Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

In case of CONFIG_BALLOON the page allocator's bitmap needs some space to be able to grow. Remap it to kernel virtual area if the preallocated area isn't large enough. Signed-off-by: Juergen Gross --- V3: - add assertion as requested by Samuel Thibault - rename functions to have mm_ prefix as

[Xen-devel] [PATCH v3 19/19] mini-os: repair build system

The build system of Mini-OS is using different settings for arch/*/* than for the rest of the tree. The main reasons are that Config.mk is included only conditionally in the top level Makefile, while minios.mk isn't included by the arch Makefiles. Repairing this mess enables us to move the CONFIG_

[Xen-devel] [PATCH v3 01/19] mini-os: correct first free pfn

The first free pfn available for allocation is calculated by adding the number of page table frames to the pfn of the first page table and then the magic number 3 to account for start info page et al. As the start info page, xenstore page and console page are allocated _before_ the page tables lea

[Xen-devel] [PATCH v3 13/19] mini-os: add function to map one frame

Add a function to map one physical frame to a specified virtual address as read/write. This will be used later multiple times. Signed-off-by: Juergen Gross Reviewed-by: Samuel Thibault --- include/arm/arch_mm.h | 2 ++ include/mm.h | 1 + mm.c | 5 + 3 files change

[Xen-devel] [PATCH v3 17/19] mini-os: add support for ballooning up

Add support for ballooning the domain up by a specified amount of pages. Following steps are performed: - extending the p2m map - extending the page allocator's bitmap - getting new memory pages from the hypervisor - adding the memory at the current end of guest memory Signed-off-by: Juergen Gros

[Xen-devel] [PATCH v3 12/19] mini-os: don't allocate new pages for level 1 p2m tree

When constructing the 3 level p2m tree there is no need to allocate new pages for the level 1 containing the p2m info for all pages. The pages from the linear p2m list constructed by the hypervisor can be used for that purpose. Signed-off-by: Juergen Gross Reviewed-by: Samuel Thibault --- arch/

Re: [Xen-devel] [PATCH v2 21/25] arm/altp2m: Add HVMOP_altp2m_change_gfn.

On 06/08/2016 19:42, Sergej Proskurin wrote: Hi Julien, Hello Sergej, On 08/06/2016 04:34 PM, Julien Grall wrote: On 06/08/2016 14:45, Sergej Proskurin wrote: Hi Julien, Hello Sergej, On 08/04/2016 04:04 PM, Julien Grall wrote: On 01/08/16 18:10, Sergej Proskurin wrote: +r

[Xen-devel] [PATCH V2] xen: support enabling SMEP/SMAP for HVM only

Enhance "skaj...@intel.com>mep" and "smap" command line options to support enabling SMEP or SMAP for HVM only with allowing "hvm" as a value. Signed-off-by: He Chen --- Changes in V2: * Allow "hvm" as a value to "smep" and "smap" command line options. * Clear SMEP/SMAP CPUID bits for pv guests

Re: [Xen-devel] [PATCH v5 3/4] x86/ioreq server: Add HVMOP to map guest ram with p2m_ioreq_server to an ioreq server.

On 8/11/2016 4:58 PM, Jan Beulich wrote: On 11.08.16 at 10:47, wrote: On 8/10/2016 6:43 PM, Yu Zhang wrote: For " && p2mt != p2m_ioreq_server" condition, it is just to guarantee that if a write operation is trapped, and at the same period, device model changed the status of ioreq server, it

[Xen-devel] [PATCH V3] common/vm_event: synchronize vCPU state in vm_event_resume()

Vm_event_vcpu_pause() needs to use vcpu_pause_nosync() in order for the current vCPU to not get stuck. A consequence of this is that the custom vm_event response handlers will not always see the real vCPU state in v->arch.user_regs. This patch makes sure that the state is always synchronized in vm_

[Xen-devel] [linux-3.14 test] 100400: tolerable FAIL - PUSHED

flight 100400 linux-3.14 real [real] http://logs.test-lab.xenproject.org/osstest/logs/100400/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): build-i386-rumpuserxen6 xen-buildfail like 99747 build-amd64-rumpuserxen

Re: [Xen-devel] xen does not support the 8G large bar

>>> On 11.08.16 at 05:30, wrote: > According to the analysis hvmloader code, find a problem: > > if (is_64bar) { > bar_data_upper = pci_readl(devfn, bar_reg + 4); > pci_writel(devfn, bar_reg + 4, ~0); > bar_sz_upper = pci_readl(devfn, bar_reg + 4);

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: > +extern unsigned long *mm_bitmap; > +extern unsigned long mm_bitmap_size; Ah I was thinking to have these use mm_malloc_ too. "mm_bitmap" seems short for namespace pollution. Samuel ___ Xen-dev

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

On Thu, Aug 11, 2016 at 09:33:57AM +0100, Wei Liu wrote: > On Wed, Aug 10, 2016 at 04:07:30PM +0100, Wei Liu wrote: > [...] > > > > +def run_test_logfile(opts, test): > > +""" Run a specific test via grepping log file""" > > + > > +fn = opts.logfile_dir + (opts.logfile_pattern % test) > >

Re: [Xen-devel] [PATCH] Remove ambiguities in the COPYING file; add CONTRIBUTING file

On 11/08/16 01:51, Stefano Stabellini wrote: > On Wed, 10 Aug 2016, Lars Kurth wrote: >> COPYING file: >> The motivation of this change is to make it easier for new >> contributors to conduct a license and patent review, WITHOUT >> changing any licenses. >> - Remove references to BSD-style licenses

Re: [Xen-devel] [PATCH 1/2] page-alloc/x86: don't restrict DMA heap to node 0

Hi Jan, On 10/08/2016 11:23, Jan Beulich wrote: --- a/xen/include/asm-arm/numa.h +++ b/xen/include/asm-arm/numa.h @@ -17,6 +17,11 @@ static inline __attribute__((pure)) node #define node_start_pfn(nid) (pdx_to_pfn(frametable_base_pdx)) #define __node_distance(a, b) (20) +static inline unsigne

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

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

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

On 11/08/16 10:44, Wei Liu wrote: > On Thu, Aug 11, 2016 at 09:33:57AM +0100, Wei Liu wrote: >> On Wed, Aug 10, 2016 at 04:07:30PM +0100, Wei Liu wrote: >> [...] >>> >>> +def run_test_logfile(opts, test): >>> +""" Run a specific test via grepping log file""" >>> + >>> +fn = opts.logfile_d

Re: [Xen-devel] [PATCH] Remove ambiguities in the COPYING file; add CONTRIBUTING file

On 10/08/16 12:30, Lars Kurth wrote: > COPYING file: > The motivation of this change is to make it easier for new > contributors to conduct a license and patent review, WITHOUT > changing any licenses. > - Remove references to BSD-style licenses as we have more > common license exceptions and rep

Re: [Xen-devel] [PATCH 1/2] page-alloc/x86: don't restrict DMA heap to node 0

>>> On 11.08.16 at 11:53, wrote: > Hi Jan, > > On 10/08/2016 11:23, Jan Beulich wrote: >> --- a/xen/include/asm-arm/numa.h >> +++ b/xen/include/asm-arm/numa.h >> @@ -17,6 +17,11 @@ static inline __attribute__((pure)) node >> #define node_start_pfn(nid) (pdx_to_pfn(frametable_base_pdx)) >> #defi

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

On Thu, Aug 11, 2016 at 10:56:23AM +0100, Andrew Cooper wrote: > On 11/08/16 10:44, Wei Liu wrote: > > On Thu, Aug 11, 2016 at 09:33:57AM +0100, Wei Liu wrote: > >> On Wed, Aug 10, 2016 at 04:07:30PM +0100, Wei Liu wrote: > >> [...] > >>> > >>> +def run_test_logfile(opts, test): > >>> +""" Ru

Re: [Xen-devel] [XTF PATCH 1/3] xtf-runner: introduce get_xen_version

On 10/08/16 16:07, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > xtf-runner | 13 + > 1 file changed, 13 insertions(+) > > diff --git a/xtf-runner b/xtf-runner > index c063699..7b69c45 100755 > --- a/xtf-runner > +++ b/xtf-runner > @@ -151,6 +151,19 @@ def __repr__(self): >

[Xen-devel] [PATCH] libxc: use DPRINTF in xc_domain_dumpcore_via_callback

That line doesn't reveal much information to ordinary users. Change that to debug output. Signed-off-by: Wei Liu --- tools/libxc/xc_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxc/xc_core.c b/tools/libxc/xc_core.c index d792566..e581905 100644 --- a/tools/

Re: [Xen-devel] [XTF PATCH 1/3] xtf-runner: introduce get_xen_version

On Thu, Aug 11, 2016 at 11:10:41AM +0100, Andrew Cooper wrote: > On 10/08/16 16:07, Wei Liu wrote: > > Signed-off-by: Wei Liu > > --- > > xtf-runner | 13 + > > 1 file changed, 13 insertions(+) > > > > diff --git a/xtf-runner b/xtf-runner > > index c063699..7b69c45 100755 > > --- a/xt

Re: [Xen-devel] [XTF PATCH 1/3] xtf-runner: introduce get_xen_version

On 11/08/16 11:12, Wei Liu wrote: > On Thu, Aug 11, 2016 at 11:10:41AM +0100, Andrew Cooper wrote: >> On 10/08/16 16:07, Wei Liu wrote: >>> Signed-off-by: Wei Liu >>> --- >>> xtf-runner | 13 + >>> 1 file changed, 13 insertions(+) >>> >>> diff --git a/xtf-runner b/xtf-runner >>> index

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

On 11/08/16 11:40, Samuel Thibault wrote: > Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: >> +extern unsigned long *mm_bitmap; >> +extern unsigned long mm_bitmap_size; > > Ah I was thinking to have these use mm_malloc_ too. "mm_bitmap" seems > short for namespace pollution. Sorry, you

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

On Thu, Aug 11, 2016 at 12:19:20PM +0200, Juergen Gross wrote: > On 11/08/16 11:40, Samuel Thibault wrote: > > Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: > >> +extern unsigned long *mm_bitmap; > >> +extern unsigned long mm_bitmap_size; > > > > Ah I was thinking to have these use mm_m

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

Juergen Gross, on Thu 11 Aug 2016 12:19:20 +0200, wrote: > On 11/08/16 11:40, Samuel Thibault wrote: > > Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: > >> +extern unsigned long *mm_bitmap; > >> +extern unsigned long mm_bitmap_size; > > > > Ah I was thinking to have these use mm_malloc_

Re: [Xen-devel] [PATCH v3 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

On 11/08/16 12:21, Wei Liu wrote: > On Thu, Aug 11, 2016 at 12:19:20PM +0200, Juergen Gross wrote: >> On 11/08/16 11:40, Samuel Thibault wrote: >>> Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote: +extern unsigned long *mm_bitmap; +extern unsigned long mm_bitmap_size; >>> >>> Ah I

Re: [Xen-devel] [PATCH] Remove ambiguities in the COPYING file; add CONTRIBUTING file

On 11/08/2016 01:51, "Stefano Stabellini" wrote: >> +Developer's Certificate of Origin >> +- >> + >> +All patches to the Xen Project code base must include the the line > ^ double "the" Thanks: will fix. A

Re: [Xen-devel] [PATCH] common/vm_event: Fix comment

>>> On 09.08.16 at 10:32, wrote: > --- a/xen/common/vm_event.c > +++ b/xen/common/vm_event.c > @@ -255,7 +255,7 @@ static inline void vm_event_release_slot(struct domain *d, > > /* > * vm_event_mark_and_pause() tags vcpu and put it to sleep. > - * The vcpu will resume execution in vm_event_wa

[Xen-devel] [PATCH v2 1/2] page-alloc/x86: don't restrict DMA heap to node 0

When node zero has no memory, the DMA bit width will end up getting set to 9, which is obviously not helpful to hold back a reasonable amount of low enough memory for Dom0 to use for DMA purposes. Find the lowest node with memory below 4Gb instead. Introduce arch_get_dma_bitsize() to keep this arc

Re: [Xen-devel] [PATCH v2 1/2] page-alloc/x86: don't restrict DMA heap to node 0

On 11/08/16 11:44, Jan Beulich wrote: > When node zero has no memory, the DMA bit width will end up getting set > to 9, which is obviously not helpful to hold back a reasonable amount > of low enough memory for Dom0 to use for DMA purposes. Find the lowest > node with memory below 4Gb instead. > >

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

Wei Liu writes ("[XTF PATCH 3/3] xtf-runner: support two modes for getting output"): > We need two modes for getting output: ... > +# Use time to generate unique stamps > +start_stamp = "= XTF TEST START %s =" % local_time > +end_stamp = "= XTF TEST END %s =" % local_ti

Re: [Xen-devel] [PATCH] Remove ambiguities in the COPYING file; add CONTRIBUTING file

On 11/08/2016 10:49, "George Dunlap" wrote: >On 11/08/16 01:51, Stefano Stabellini wrote: >> On Wed, 10 Aug 2016, Lars Kurth wrote: >>> COPYING file: >>> The motivation of this change is to make it easier for new >>> contributors to conduct a license and patent review, WITHOUT >>> changing any

Re: [Xen-devel] [PATCH v2] domctl: relax getdomaininfo permissions

On 08/08/16 07:12, Jan Beulich wrote: On 05.08.16 at 19:07, wrote: >> On 05/08/16 14:54, Jan Beulich wrote: >> On 05.08.16 at 15:10, wrote: On 05/08/16 12:20, Jan Beulich wrote: > I wonder what good the duplication of the returned domain ID does: I'm > tempted to remove the

Re: [Xen-devel] [PATCH v2 1/3] livepach: Add .livepatch.hooks functions and test-case

Jan Beulich writes ("Re: [Xen-devel] [PATCH v2 1/3] livepach: Add .livepatch.hooks functions and test-case"): > On 10.08.16 at 11:46, wrote: > > Odd. I've tried this simple example: > > > > typedef int fn_t(void); ... > > const fn_t**cfn; Ie, const int **cfn(void); > > for(i =

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

On Thu, Aug 11, 2016 at 11:49:29AM +0100, Ian Jackson wrote: > Wei Liu writes ("[XTF PATCH 3/3] xtf-runner: support two modes for getting > output"): > > We need two modes for getting output: > ... > > +# Use time to generate unique stamps > > +start_stamp = "= XTF TEST START %s ="

Re: [Xen-devel] [PATCH] Remove ambiguities in the COPYING file; add CONTRIBUTING file

On 11/08/2016 10:56, "George Dunlap" wrote: >> +GPL v2 License >> +-- >> + >> +/* >> + * >> + * >> + * >> + * >> + * Copyright (C) >> + * >> + * This program is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU General Public >> + * Lice

[Xen-devel] [PATCH v4 17/19] mini-os: add support for ballooning up

Add support for ballooning the domain up by a specified amount of pages. Following steps are performed: - extending the p2m map - extending the page allocator's bitmap - getting new memory pages from the hypervisor - adding the memory at the current end of guest memory Signed-off-by: Juergen Gros

[Xen-devel] [PATCH v4 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

In case of CONFIG_BALLOON the page allocator's bitmap needs some space to be able to grow. Remap it to kernel virtual area if the preallocated area isn't large enough. Signed-off-by: Juergen Gross --- V4: - mm_bitmap* -> mm_alloc_bitmap* as requested by Samuel Thibault V3: - add assertion as req

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

Andrew Cooper writes ("Re: [XTF PATCH 3/3] xtf-runner: support two modes for getting output"): > Can't you just open the log file as read, seek to the end, run the test > and read again from the same FD? It would be rather more simple than > marking the logs. This is a good suggestion, I think.

Re: [Xen-devel] [PATCH] libxc: use DPRINTF in xc_domain_dumpcore_via_callback

Wei Liu writes ("[PATCH] libxc: use DPRINTF in xc_domain_dumpcore_via_callback"): > That line doesn't reveal much information to ordinary users. > > Change that to debug output. I think you are right. Acked-by: Ian Jackson ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v2 1/3] livepach: Add .livepatch.hooks functions and test-case

>>> On 11.08.16 at 12:56, wrote: > Jan Beulich writes ("Re: [Xen-devel] [PATCH v2 1/3] livepach: Add > .livepatch.hooks functions and test-case"): >> On 10.08.16 at 11:46, wrote: >> > Odd. I've tried this simple example: >> > >> > typedef int fn_t(void); > ... >> >const fn_t**cfn; > > Ie,

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

On Thu, Aug 11, 2016 at 12:08:03PM +0100, Ian Jackson wrote: > Andrew Cooper writes ("Re: [XTF PATCH 3/3] xtf-runner: support two modes for > getting output"): > > Can't you just open the log file as read, seek to the end, run the test > > and read again from the same FD? It would be rather more

Re: [Xen-devel] xen does not support the 8G large bar

On August 10, 2016 11:30:37 PM EDT, "Gaofeng (GaoFeng, Euler)" wrote: >Hi George, > >I found that you have submitted a patch about "libxl,hvmloader: Don't >relocate memory for MMIO hole". So, I have a "8G large bar PCI >device(NVIDIA Tesla M60)" passthrough question to ask you. > > > >Host passth

Re: [Xen-devel] [PATCH v2] domctl: relax getdomaininfo permissions

>>> On 05.08.16 at 13:20, wrote: Daniel, I've only now realized that I forgot to Cc you on this v2. Jan > Qemu needs access to this for the domain it controls, both due to it > being used by xc_domain_memory_mapping() (which qemu calls) and the > explicit use in hw/xenpv/xen_domainbuild.c:xen_

[Xen-devel] Scheduler regression in 4.7

Hello, XenServer testing has discovered a regression from recent changes in staging-4.7. The actual cause is _csched_cpu_pick() falling over LIST_POISON, which happened to occur at the same time as a domain was shutting down. The instruction in question is `mov 0x10(%rax),%rax` which looks like

Re: [Xen-devel] [XTF PATCH 3/3] xtf-runner: support two modes for getting output

On 11/08/16 12:19, Wei Liu wrote: > On Thu, Aug 11, 2016 at 12:08:03PM +0100, Ian Jackson wrote: >> Andrew Cooper writes ("Re: [XTF PATCH 3/3] xtf-runner: support two modes for >> getting output"): >>> Can't you just open the log file as read, seek to the end, run the test >>> and read again from

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

flight 100412 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/100412/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl 1

[Xen-devel] [PATCH 0/7] x86emul: misc small adjustments

This is mainly to reduce the amount of fetching of insn bytes past the initial phase of decoding, as a little bit of preparation for the intended splitting of decoding and execution stages. Having fully read all insn bytes by that point would be specifically required if the first stage alone should

Re: [Xen-devel] [PATCH v4 16/19] mini-os: map page allocator's bitmap to virtual kernel area for ballooning

Juergen Gross, on Thu 11 Aug 2016 13:06:36 +0200, wrote: > In case of CONFIG_BALLOON the page allocator's bitmap needs some space > to be able to grow. Remap it to kernel virtual area if the preallocated > area isn't large enough. > > Signed-off-by: Juergen Gross Reviewed-by: Samuel Thibault >

Re: [Xen-devel] [PATCH v2 1/2] x86/altp2m: use __get_gfn_type_access to avoid lock conflicts

>>> On 10.08.16 at 17:00, wrote: > From: Tamas K Lengyel > > Use __get_gfn_type_access instead of get_gfn_type_access when checking > the hostp2m entries during altp2m mem_access setting and gfn remapping > to avoid a lock conflict which can make dom0 freeze. During mem_access > setting the hp2m

Re: [Xen-devel] [PATCH 4/9] x86/pv: Implement pv_hypercall() in C

On 02/08/16 14:12, Jan Beulich wrote: On 18.07.16 at 11:51, wrote: >> +long pv_hypercall(struct cpu_user_regs *regs) >> +{ >> +struct vcpu *curr = current; >> +#ifndef NDEBUG >> +unsigned long old_rip = regs->rip; >> +#endif >> +long ret; >> +uint32_t eax = regs->eax; >> + >>

Re: [Xen-devel] [PATCH 5/9] x86/hypercall: Move the hypercall tables into C

On 02/08/16 14:40, Jan Beulich wrote: On 02.08.16 at 15:30, wrote: >> On 02/08/16 14:23, Jan Beulich wrote: >> On 18.07.16 at 11:51, wrote: +hypercall_fn_t *const hypercall_table[NR_hypercalls] = { +HYPERCALL(set_trap_table), +HYPERCALL(mmu_update), +HYPER

Re: [Xen-devel] [PATCH v2 2/2] x86/altp2m: allow specifying external-only use-case

>>> On 10.08.16 at 17:00, wrote: > @@ -5238,18 +5238,19 @@ static int do_altp2m_op( > goto out; > } > > -if ( (rc = xsm_hvm_altp2mhvm_op(XSM_TARGET, d)) ) > +if ( !d->arch.hvm_domain.params[HVM_PARAM_ALTP2M] ) > +{ > +rc = -EINVAL; > +goto out; > +}

[Xen-devel] [PATCH 1/7] x86emul: don't special case fetching the immediate of PUSH

These immediates follow the standard patterns in all modes, so they're better fetched by the generic source operand handling code. To facilitate testing, instead of adding yet another of these pretty convoluted individual test cases, simply introduce another blowfish run with -mno-accumulate-outgo

[Xen-devel] [PATCH 2/7] x86emul: don't special case fetching immediates of near and short branches

These immediates follow the standard patterns in all modes, so they're better fetched by the generic source operand handling code. Signed-off-by: Jan Beulich --- a/xen/arch/x86/x86_emulate/x86_emulate.c +++ b/xen/arch/x86/x86_emulate/x86_emulate.c @@ -98,11 +98,15 @@ static uint8_t opcode_table[

[Xen-devel] [PATCH 3/7] x86emul: all push flavors are data moves

Make all paths leading to the "push" label have the Mov flag set, and ASSERT() that to be the case. For the opcode FF group the adjustment is benign for the paths not leading to "push", as they all set dst.type to OP_NONE Signed-off-by: Jan Beulich --- a/xen/arch/x86/x86_emulate/x86_emulate.c ++

[Xen-devel] [PATCH 4/7] x86emul: fold SrcImmByte fetching

There's no need for having identical code spelled out twice. Signed-off-by: Jan Beulich --- a/xen/arch/x86/x86_emulate/x86_emulate.c +++ b/xen/arch/x86/x86_emulate/x86_emulate.c @@ -1979,9 +1979,12 @@ x86_emulate( goto done; break; case SrcImm: +if ( !(d & Byte

[Xen-devel] [PATCH 5/7] x86emul: don't special case fetching unsigned 8-bit immediates

These can be made work using SrcImmByte, making sure the low 8 bits of src.val get suitably zero extended upon consumption. SHLD and SHRD require a little more adjustment: Their source operands get changed away from SrcReg, handling the register access "manually" instead of the insn byte fetching.

[Xen-devel] [PATCH 6/7] x86emul: use DstEax where possible

While it avoids just a few instructions, we should nevertheless make use of generic code as much as possible. Signed-off-by: Jan Beulich --- a/xen/arch/x86/x86_emulate/x86_emulate.c +++ b/xen/arch/x86/x86_emulate/x86_emulate.c @@ -124,7 +124,7 @@ static uint8_t opcode_table[256] = { Implici

[Xen-devel] [PATCH 7/7] x86emul: introduce SrcImm16

... and use it for RET, LRET, and ENTER processing to limit the amount of "manual" insn bytes fetching. Signed-off-by: Jan Beulich --- a/xen/arch/x86/x86_emulate/x86_emulate.c +++ b/xen/arch/x86/x86_emulate/x86_emulate.c @@ -39,6 +39,7 @@ #define SrcMem16(4<<3) /* Memory operand (16-bit). *

Re: [Xen-devel] [PATCH] common/vm_event: Fix comment

On 08/11/2016 01:38 PM, Jan Beulich wrote: On 09.08.16 at 10:32, wrote: >> --- a/xen/common/vm_event.c >> +++ b/xen/common/vm_event.c >> @@ -255,7 +255,7 @@ static inline void vm_event_release_slot(struct domain >> *d, >> >> /* >> * vm_event_mark_and_pause() tags vcpu and put it to slee

Re: [Xen-devel] [Minios-devel] [PATCH v3 00/19] mini-os: support of auto-ballooning

Series pushed, except for the last patch that changed the build system because it is not yet acked or reviewed. Note that patch #16 and #17 are resent in another thread. Thank you both. Wei. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://l

Re: [Xen-devel] [PATCH 4/9] x86/pv: Implement pv_hypercall() in C

>>> On 11.08.16 at 13:57, wrote: > On 02/08/16 14:12, Jan Beulich wrote: > On 18.07.16 at 11:51, wrote: >>> +mov %rsp, %rdi >>> +call pv_hypercall >>> movl %eax,UREGS_rax(%rsp) # save the return value >> To follow the HVM model, this should also move into C.

Re: [Xen-devel] [PATCH] common/vm_event: Fix comment

>>> On 11.08.16 at 14:10, wrote: > On 08/11/2016 01:38 PM, Jan Beulich wrote: > On 09.08.16 at 10:32, wrote: >>> --- a/xen/common/vm_event.c >>> +++ b/xen/common/vm_event.c >>> @@ -255,7 +255,7 @@ static inline void vm_event_release_slot(struct domain >>> *d, >>> >>> /* >>> * vm_event_m

Re: [Xen-devel] [PATCH 7/9] x86/pv: Merge the pv hypercall tables

On 03/08/16 16:07, Jan Beulich wrote: On 18.07.16 at 11:51, wrote: >> For the same reason as c/s 33a231e3f "x86/HVM: fold hypercall tables", this >> removes the risk of accidentally updating only one of the tables. >> >> Signed-off-by: Andrew Cooper > Reviewed-by: Jan Beulich > > But having

[Xen-devel] [PATCH] Xen: remove -fshort-wchar gcc flag

A previous patch added the --no-wchar-size-warning to the Makefile to avoid this harmless warning: arm-linux-gnueabi-ld: warning: drivers/xen/efi.o uses 2-byte wchar_t yet the output is to use 4-byte wchar_t; use of wchar_t values across objects may fail Changing kbuild to use thin archives inst

Re: [Xen-devel] [PATCH] Xen: remove -fshort-wchar gcc flag

>>> On 11.08.16 at 14:39, wrote: > A previous patch added the --no-wchar-size-warning to the Makefile to > avoid this harmless warning: > > arm-linux-gnueabi-ld: warning: drivers/xen/efi.o uses 2-byte wchar_t yet the > output is to use 4-byte wchar_t; use of wchar_t values across objects may >

  1   2   3   >