Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-24 Thread Jan Beulich
>>> On 23.09.15 at 17:46, wrote: > At 16:18 +0100 on 23 Sep (1443025126), Wei Liu wrote: >> With the discussion still not finalised I'm a bit worried that this >> issue will block the release. >> >> I think we have a few options here. I will list them in order of my >> preference. Please correct

Re: [Xen-devel] [PATCH] Remove a set operation for VCPU_KICK_SOFTIRQ when post interrupt to vm.

2015-09-24 Thread Liuqiming (John)
On 2015/9/24 11:25, Zhang, Yang Z wrote: Liuqiming (John) wrote on 2015-09-24: On 2015/9/23 21:41, Zhang, Yang Z wrote: Hanweidong (Randy) wrote on 2015-09-23: Zhang, Yang Z wrote on ent: 2015年9月23日 11:51: VCPU_KICK_SOFTIRQ when post interrupt to vm. Zhang, Yang Z wrote on 2015-09-08: Liu

Re: [Xen-devel] Regression in RMRRs identity mapping for PVH Dom0

2015-09-24 Thread Chen, Tiejun
On 9/23/2015 11:56 PM, Elena Ufimtseva wrote: Hi There is a regression in RMRR patch 5ae03990c120a7b3067a52d9784c9aa72c0705a6 in new set_identity_p2m_entry. RMRRs are not being mapped in IOMMU for PVH Dom0. This causes pages faults and some long 'hang-like' delays during boot and device assignme

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-24 Thread Ian Campbell
On Wed, 2015-09-23 at 18:36 +0100, Stefano Stabellini wrote: > On Wed, 23 Sep 2015, Ian Campbell wrote: > > On Tue, 2015-09-22 at 22:31 +0100, Stefano Stabellini wrote: > > > The oldest Xen version I build-test for every pull request is Xen 4.0.0, I setup a build trees for 4.0 thru 4.6 yesterday

Re: [Xen-devel] [PATCH v7 15/17] vmx: VT-d posted-interrupt core logic handling

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 03:50, wrote: > One issue is the number of vmexits is far far bigger than the number of > context switch. I test it for a quite short time and it shows there are > 2910043 vmexits and 71733 context switch (only count the number in > __context_switch() since we only change the PI

Re: [Xen-devel] [PATCH v6 12/29] xen/x86: allow disabling the emulated local apic

2015-09-24 Thread Jan Beulich
>>> On 23.09.15 at 17:45, wrote: > El 16/09/15 a les 12.05, Jan Beulich ha escrit: > On 04.09.15 at 14:08, wrote: >> Also - aren't all the changes to this file (and perhaps othersfurther >> down) bug fixes in their own right? > > Whether they should be considered bugs or not it's hard to tel

Re: [Xen-devel] [PATCH] x86emul: support clzero

2015-09-24 Thread Jan Beulich
>>> On 23.09.15 at 19:37, wrote: > On 22/09/15 14:06, Jan Beulich wrote: >> ... in anticipation of this possibly going to get used by guests for >> basic thinks like memset() or clearing or pages. >> >> Since the emulation doesn't use clzero itself, checking the guest's >> CPUID for the feature to

Re: [Xen-devel] [PATCH v7 15/17] vmx: VT-d posted-interrupt core logic handling

2015-09-24 Thread Wu, Feng
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, September 24, 2015 3:52 PM > To: Wu, Feng > Cc: Andrew Cooper; Dario Faggioli; George Dunlap; George Dunlap; Tian, Kevin; > xen-devel@lists.xen.org; Keir Fraser > Subject: RE: [Xen-devel] [PATCH v7 15/17

Re: [Xen-devel] [PATCH] Remove a set operation for VCPU_KICK_SOFTIRQ when post interrupt to vm.

2015-09-24 Thread Zhang, Yang Z
Liuqiming (John) wrote on 2015-09-24: > > > On 2015/9/24 11:25, Zhang, Yang Z wrote: >> Liuqiming (John) wrote on 2015-09-24: >>> On 2015/9/23 21:41, Zhang, Yang Z wrote: Hanweidong (Randy) wrote on 2015-09-23: > Zhang, Yang Z wrote on ent: 2015年9月23日 11:51: >> VCPU_KICK_SOFTIRQ when

Re: [Xen-devel] [PATCH] x86: detect CMOS aliasing on ports other then 0x70/0x71

2015-09-24 Thread Jan Beulich
>>> On 23.09.15 at 20:34, wrote: > On 22/09/15 14:10, Jan Beulich wrote: >> +for ( offs = 2; offs < 8; offs <<= 1 ) >> +{ >> +bool_t read = 1; >> + >> +for ( i = RTC_REG_D + 1; i < 0x80; ++i ) >> +{ >> +uint8_t normal, alt; >> +unsigned long

[Xen-devel] [PATCH] x86/boot: Move/copy sections more efficiently

2015-09-24 Thread Andrew Cooper
Both the trampoline copy and BSS initialise can be performed more efficiently by using 4-byte variants of the string operations. The ALIGN(STACK_SIZE) actually belongs with .bss.stack_aligned, but __init_end still needs page alignment because of the init sections being freed and returned to the do

Re: [Xen-devel] [PATCH RFC v2] x86/p2m: use large pages for MMIO mappings

2015-09-24 Thread Jan Beulich
>>> On 23.09.15 at 18:22, wrote: > On 09/22/2015 01:56 PM, Jan Beulich wrote: >> When mapping large BARs (e.g. the frame buffer of a graphics card) the >> overhead of establishing such mappings using only 4k pages has, >> particularly after the XSA-125 fix, become unacceptable. Alter the >> XEN_DO

Re: [Xen-devel] [PATCH v7 18/28] xen/arm: ITS: Export ITS info to Virtual ITS

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 10:56 +0530, Vijay Kilari wrote: > > > + > > > plpi = its_get_plpi(pdev, i); > > > /* TODO: Route lpi */ > > > } > > > @@ -1366,6 +1374,8 @@ static int its_probe(struct dt_device_node > > > *node) > > > its->phys_size = its_size; > > > typer =

[Xen-devel] [BUG] 16 vcpus + 2 vif bridge = issue ?

2015-09-24 Thread Roman Shubovich
hi i have physical server with 40 cpu cores and i need to create a hvm domu with at least 16 vcpus and 2 network bridges when i start that domu i have some not understable issue - the second bridge has no traffic from network (works only first interface - first declared in config file). i can see

[Xen-devel] [linux-3.14 test] 62270: regressions - trouble: broken/fail/pass

2015-09-24 Thread osstest service owner
flight 62270 linux-3.14 real [real] http://logs.test-lab.xenproject.org/osstest/logs/62270/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-stubdom-debianhvm-amd64-xsm 13 guest-localmigrate fail REGR. vs. 60666 tes

Re: [Xen-devel] [PATCH 4/4] x86/PoD: shorten certain operations on higher order ranges

2015-09-24 Thread Jan Beulich
>>> On 23.09.15 at 19:10, wrote: > On 09/15/2015 08:37 AM, Jan Beulich wrote: >> @@ -574,41 +576,46 @@ recount: >> * + There are PoD entries to handle, or >> * + There is ram left, and we want to steal it >> */ >> -for ( i=0; >> - i<(1<0 || (steal_for_cache && ram >

Re: [Xen-devel] [BUG] 16 vcpus + 2 vif bridge = issue ?

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 03:16 +0300, Roman Shubovich wrote: > hi > > i have physical server with 40 cpu cores > and i need to create a hvm domu with at least 16 vcpus and 2 network > bridges > when i start that domu i have some not understable issue - the second > bridge has no traffic from network

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-24 Thread Fabio Fantoni
Il 24/09/2015 09:15, Ian Campbell ha scritto: On Wed, 2015-09-23 at 18:36 +0100, Stefano Stabellini wrote: On Wed, 23 Sep 2015, Ian Campbell wrote: On Tue, 2015-09-22 at 22:31 +0100, Stefano Stabellini wrote: The oldest Xen version I build-test for every pull request is Xen 4.0.0, I setup a b

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-24 Thread Tim Deegan
At 01:02 -0600 on 24 Sep (1443056566), Jan Beulich wrote: > >>> On 23.09.15 at 17:46, wrote: > > At 16:18 +0100 on 23 Sep (1443025126), Wei Liu wrote: > >> With the discussion still not finalised I'm a bit worried that this > >> issue will block the release. > >> > >> I think we have a few option

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-24 Thread Andrew Cooper
etc along with adjusting the existing gating of PML on AD being available (perhaps by simply stripping the respective bit from what we read from MSR_IA32_VMX_EPT_VPID_CAP). Of course this then ignores the fact that the erratum only affects the A bit, but I think we can live with that. I also th

Re: [Xen-devel] Regression in RMRRs identity mapping for PVH Dom0

2015-09-24 Thread Tim Deegan
At 15:17 +0800 on 24 Sep (1443107852), Chen, Tiejun wrote: > On 9/23/2015 11:56 PM, Elena Ufimtseva wrote: > > Hi > > > > There is a regression in RMRR patch > > 5ae03990c120a7b3067a52d9784c9aa72c0705a6 in > > new set_identity_p2m_entry. RMRRs are not being mapped in IOMMU for PVH > > Dom0. > > T

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-24 Thread Tim Deegan
At 10:13 +0100 on 24 Sep (1443089607), Andrew Cooper wrote: > > >> etc along with adjusting the existing gating of PML on AD being > >> available (perhaps by simply stripping the respective bit from what > >> we read from MSR_IA32_VMX_EPT_VPID_CAP). Of course this > >> then ignores the fact that t

Re: [Xen-devel] [PATCH] xen: credit1: fix tickling when it happens from a remote pCPU

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 06:31, wrote: > --- a/xen/common/sched_credit.c > +++ b/xen/common/sched_credit.c > @@ -168,14 +168,16 @@ struct csched_pcpu { > }; > > /* > - * Convenience macro for accessing the per-PCPU cpumask we need for > + * Convenience macros for accessing the per-PCPU cpumask we ne

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 11:10, wrote: > At 01:02 -0600 on 24 Sep (1443056566), Jan Beulich wrote: >> --- a/xen/arch/x86/mm/p2m-ept.c >> +++ b/xen/arch/x86/mm/p2m-ept.c >> @@ -130,14 +130,18 @@ static void ept_p2m_type_to_flags(struct p2m_domain >> *p2m, ept_entry_t *entry, >> break; >>

Re: [Xen-devel] [BUG] 16 vcpus + 2 vif bridge = issue ?

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 09:56 +0100, Ian Campbell wrote: > On Thu, 2015-09-24 at 03:16 +0300, Roman Shubovich wrote: > > hi > > > > i have physical server with 40 cpu cores > > and i need to create a hvm domu with at least 16 vcpus and 2 network > > bridges > > when i start that domu i have some not

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 11:13, wrote: >>> etc along with adjusting the existing gating of PML on AD being >>> available (perhaps by simply stripping the respective bit from what >>> we read from MSR_IA32_VMX_EPT_VPID_CAP). Of course this >>> then ignores the fact that the erratum only affects the A bi

Re: [Xen-devel] [PATCH] xen: credit1: fix tickling when it happens from a remote pCPU

2015-09-24 Thread Dario Faggioli
On Thu, 2015-09-24 at 03:29 -0600, Jan Beulich wrote: > > > > On 24.09.15 at 06:31, wrote: > > --- a/xen/common/sched_credit.c > > +++ b/xen/common/sched_credit.c > > #define csched_balance_mask (CSCHED_PCPU(smp_processor_id()) > > ->balance_mask) > > > > +#define csched_balance_mask_cpu(c) (C

Re: [Xen-devel] [PATCH] x86/boot: Move/copy sections more efficiently

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 10:14, wrote: > The ALIGN(STACK_SIZE) actually belongs with .bss.stack_aligned, but > __init_end still needs page alignment because of the init sections being > freed and returned to the domheap after boot. Logically that change makes sense, but ... > --- a/xen/arch/x86/xen.ld

[Xen-devel] [xen-4.0-testing baseline-only test] 38026: tolerable FAIL

2015-09-24 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38026 xen-4.0-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38026/ Failures :-/ but no regressions. Tests which did not succeed, including tests which could not be run: build-i386-libvirt1 build-check(

Re: [Xen-devel] [PATCH] xen: credit1: fix tickling when it happens from a remote pCPU

2015-09-24 Thread Dario Faggioli
On Thu, 2015-09-24 at 08:44 +0200, Juergen Gross wrote: > On 09/24/2015 06:31 AM, Dario Faggioli wrote: > > Signed-off-by: Dario Faggioli > > Reviewed-by: Juergen Gross > > regardless whether you address my suggestion below or not. > > --- a/xen/common/sched_credit.c > > +++ b/xen/common/sched

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

2015-09-24 Thread Jan Beulich
>>> On 23.09.15 at 17:55, wrote: > +#define KEYHANDLER(k, f, desc, diag)\ > +[k] = { .fn = (f), (desc), 0, (diag) } > + > +#define IRQ_KEYHANDLER(k, f, desc, diag)\ > +[k] = { .irq_fn = (f), (desc), 1, (diag) } Parenthesizing desc and diag is really unnecessary. > -vo

[Xen-devel] [distros-debian-wheezy test] 38027: all pass

2015-09-24 Thread Platform Team regression test user
flight 38027 distros-debian-wheezy real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38027/ Perfect :-) All tests in this flight passed baseline version: flight 37941 jobs: build-amd64 pass build-armhf

Re: [Xen-devel] Regression in RMRRs identity mapping for PVH Dom0

2015-09-24 Thread Wei Liu
Hi Elena On Wed, Sep 23, 2015 at 11:56:12AM -0400, Elena Ufimtseva wrote: > Hi > > > > There is a regression in RMRR patch 5ae03990c120a7

Re: [Xen-devel] Regression in RMRRs identity mapping for PVH Dom0

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 11:18, wrote: > AIUI the problem is that before the call to set_identity_p2m_entry(), > PVH dom0 has a p2m entry covering this range but no IOMMU entry. Is > that right? So the fix will be to make PVH dom0 construction set up > the IOMMU correctly when it sets up the p2m. Rig

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

2015-09-24 Thread Andrew Cooper
+void register_irq_keyhandler(unsigned char key, + irq_keyhandler_fn_t *fn, + const char *desc, + bool_t diagnostic); I wonder whether the last parameter is really useful here. Yes. There are diagnostic irq_k

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

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 12:36, wrote: >>> +void register_irq_keyhandler(unsigned char key, >>> + irq_keyhandler_fn_t *fn, >>> + const char *desc, >>> + bool_t diagnostic); >> I wonder whether the last parameter is real

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

2015-09-24 Thread Andrew Cooper
On 24/09/15 11:43, Jan Beulich wrote: On 24.09.15 at 12:36, wrote: +void register_irq_keyhandler(unsigned char key, + irq_keyhandler_fn_t *fn, + const char *desc, + bool_t diagnostic); I wonder whether the last

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

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-17 at 11:38 +0100, Andrew Cooper wrote: > On 17/09/15 10:35, He Chen wrote: > > @@ -2798,7 +2798,9 @@ enum xc_psr_cmt_type { > > typedef enum xc_psr_cmt_type xc_psr_cmt_type; > > > > enum xc_psr_cat_type { > > -XC_PSR_CAT_L3_CBM = 1, > > +XC_PSR_CAT_L3_CBM = 1, > > +

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

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-17 at 11:38 +0100, Andrew Cooper wrote: > @@ -320,6 +333,8 @@ int xc_psr_cat_get_l3_info(xc_interface *xch, > > uint32_t socket, > > { > > *cos_max = sysctl.u.psr_cat_op.u.l3_info.cos_max; > > *cbm_len = sysctl.u.psr_cat_op.u.l3_info.cbm_len; > > +*cdp

Re: [Xen-devel] [PATCH] Add missing license and copyright statements to public interface headers.

2015-09-24 Thread Ian Campbell
On Tue, 2015-09-22 at 16:02 +0200, Mike Belopuhov wrote: > The copyright line indicates a person, a group of people and/or a company > granting rights stated in the license text and is a required part of the > license. > > The year of the copyright is chosen to be the same as when the license has

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-24 Thread Wei Liu
On Thu, Sep 24, 2015 at 11:45:00AM +0100, Wei Liu wrote: > On Thu, Sep 24, 2015 at 03:33:18AM -0600, Jan Beulich wrote: > > >>> On 24.09.15 at 11:10, wrote: > > > At 01:02 -0600 on 24 Sep (1443056566), Jan Beulich wrote: > > >> --- a/xen/arch/x86/mm/p2m-ept.c > > >> +++ b/xen/arch/x86/mm/p2m-ept.c

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

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: > diff --git a/tools/libxl/libxl_psr.c b/tools/libxl/libxl_psr.c > index 3378239..62963cf 100644 > --- a/tools/libxl/libxl_psr.c > +++ b/tools/libxl/libxl_psr.c > @@ -87,6 +87,9 @@ static void libxl__psr_cat_log_err_msg(libxl__gc *gc, > int err) >

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-24 Thread Wei Liu
On Thu, Sep 24, 2015 at 03:33:18AM -0600, Jan Beulich wrote: > >>> On 24.09.15 at 11:10, wrote: > > At 01:02 -0600 on 24 Sep (1443056566), Jan Beulich wrote: > >> --- a/xen/arch/x86/mm/p2m-ept.c > >> +++ b/xen/arch/x86/mm/p2m-ept.c > >> @@ -130,14 +130,18 @@ static void ept_p2m_type_to_flags(struc

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

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

Re: [Xen-devel] [PATCH v7 17/28] xen/arm: ITS: Add GITS registers emulation

2015-09-24 Thread Julien Grall
On 24/09/15 06:07, Vijay Kilari wrote: > On Tue, Sep 22, 2015 at 8:00 PM, Julien Grall wrote: >> Hi Vijay, >> >> On 18/09/15 14:09, vijay.kil...@gmail.com wrote: > [...] >>> +case 0x0010 ... 0x007c: >>> +case 0xc000 ... 0xffcc: >>> +/* Implementation defined -- write ignored */ >>>

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

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: > @@ -8410,20 +8415,29 @@ static void psr_cat_print_one_domain_cbm(uint32_t > domid, uint32_t socketid) > printf("%5d%25s", domid, domain_name); > free(domain_name); > > -if (!libxl_psr_cat_get_cbm(ctx, domid, LIBXL_PSR_CBM_TYPE_L3_C

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

2015-09-24 Thread Andrew Cooper
On 24/09/15 11:57, Ian Campbell wrote: On Thu, 2015-09-17 at 11:38 +0100, Andrew Cooper wrote: @@ -320,6 +333,8 @@ int xc_psr_cat_get_l3_info(xc_interface *xch, uint32_t socket, { *cos_max = sysctl.u.psr_cat_op.u.l3_info.cos_max; *cbm_len = sysctl.u.psr_cat_op.u.l3_inf

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

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: > Add new CDP options with CAT commands in xl interface man page. > Add description of CDP in xl-psr.markdown. It would have been fine to include this in the previous patch by the way. > > Signed-off-by: He Chen > --- > docs/man/xl.pod.1

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

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 12:07 +0100, Ian Campbell wrote: > @@ -8517,8 +8535,19 @@ int main_psr_cat_cbm_set(int argc, char **argv) > > libxl_string_list_dispose(&socket_list); > > free(value); > > break; > > +case 'd': > > +type = LIBXL_PSR_CBM_TYPE_L3_DATA; > >

Re: [Xen-devel] Regression in RMRRs identity mapping for PVH Dom0

2015-09-24 Thread Elena Ufimtseva
On Thu, Sep 24, 2015 at 04:31:09AM -0600, Jan Beulich wrote: > >>> On 24.09.15 at 11:18, wrote: > > AIUI the problem is that before the call to set_identity_p2m_entry(), > > PVH dom0 has a p2m entry covering this range but no IOMMU entry. Is > > that right? So the fix will be to make PVH dom0 co

Re: [Xen-devel] [PATCH v7 17/28] xen/arm: ITS: Add GITS registers emulation

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 12:05 +0100, Julien Grall wrote: > On 24/09/15 06:07, Vijay Kilari wrote: > > On Tue, Sep 22, 2015 at 8:00 PM, Julien Grall > > wrote: > > > Hi Vijay, > > > > > > On 18/09/15 14:09, vijay.kil...@gmail.com wrote: > > [...] > > > > +case 0x0010 ... 0x007c: > > > > +cas

Re: [Xen-devel] [PATCH v2 for-4.6 0/2] libxl: devd fixes

2015-09-24 Thread Ian Campbell
On Wed, 2015-09-23 at 16:21 +0100, Wei Liu wrote: > Release-acked-by: Wei Liu Both patches applied to staging and staging-4.6. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v12 1/1] (lib)xl: soft reset support

2015-09-24 Thread Ian Campbell
On Wed, 2015-09-23 at 14:21 +0100, Wei Liu wrote: > On Mon, Sep 21, 2015 at 11:57:34AM +0200, Vitaly Kuznetsov wrote: > > Use existing create/restore path to perform 'soft reset' for HVM > > domains. Tear everything down, e.g. destroy domain's device model, > > remove the domain from xenstore, save

Re: [Xen-devel] [PATCH for 4.6] xen/arm: vgic: Correctly emulate write when byte is used

2015-09-24 Thread Ian Campbell
On Wed, 2015-09-23 at 14:23 +0100, Wei Liu wrote: > On Tue, Sep 22, 2015 at 09:18:48PM +0100, Julien Grall wrote: > > When a guest is writing a byte, the value will be located in bits[7:0] > > of the register. > > > > Although the current implementation is expecting the byte at the Nth > > byte of

Re: [Xen-devel] [PATCH v7 18/28] xen/arm: ITS: Export ITS info to Virtual ITS

2015-09-24 Thread Julien Grall
Hi Ian, On 24/09/15 09:27, Ian Campbell wrote: > On Thu, 2015-09-24 at 10:56 +0530, Vijay Kilari wrote: > + plpi = its_get_plpi(pdev, i); /* TODO: Route lpi */ } @@ -1366,6 +1374,8 @@ static int its_probe(struct dt_device_node *node)

Re: [Xen-devel] [PATCH 2/2] xen/arm: gic-v3: Allow Xen to run on hardware reporting GICv4

2015-09-24 Thread Ian Campbell
On Tue, 2015-09-15 at 12:19 +0100, Julien Grall wrote: > On 15/09/15 12:15, Ian Campbell wrote: > > On Tue, 2015-09-15 at 12:02 +0100, Julien Grall wrote: > > > What about: > > > > > > "The GICv4 is an extension of GICv3 (see 1.1 in ARM IHI 0069A) which > > > means that the GICv3 driver can run no

Re: [Xen-devel] Regression in RMRRs identity mapping for PVH Dom0

2015-09-24 Thread Elena Ufimtseva
On Thu, Sep 24, 2015 at 10:18:54AM +0100, Tim Deegan wrote: > At 15:17 +0800 on 24 Sep (1443107852), Chen, Tiejun wrote: > > On 9/23/2015 11:56 PM, Elena Ufimtseva wrote: > > > Hi > > > > > > There is a regression in RMRR patch > > > 5ae03990c120a7b3067a52d9784c9aa72c0705a6 in > > > new set_identi

Re: [Xen-devel] [PATCH v7 18/28] xen/arm: ITS: Export ITS info to Virtual ITS

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 12:31 +0100, Julien Grall wrote: > Hi Ian, > > On 24/09/15 09:27, Ian Campbell wrote: > > On Thu, 2015-09-24 at 10:56 +0530, Vijay Kilari wrote: > > > > > > > + > > > > > plpi = its_get_plpi(pdev, i); > > > > > /* TODO: Route lpi */ > > > > > } > > > >

Re: [Xen-devel] Regression in RMRRs identity mapping for PVH Dom0

2015-09-24 Thread Elena Ufimtseva
On Thu, Sep 24, 2015 at 11:29:54AM +0100, Wei Liu wrote: > Hi Elena > > On Wed, Sep 23, 2015 at 11:56:12AM -0400, Elena Ufimtseva wrote: > > Hi > > > >

Re: [Xen-devel] [PATCH v7 17/28] xen/arm: ITS: Add GITS registers emulation

2015-09-24 Thread Julien Grall
On 24/09/15 12:29, Ian Campbell wrote: > On Thu, 2015-09-24 at 12:05 +0100, Julien Grall wrote: >> On 24/09/15 06:07, Vijay Kilari wrote: >>> On Tue, Sep 22, 2015 at 8:00 PM, Julien Grall >>> wrote: Hi Vijay, On 18/09/15 14:09, vijay.kil...@gmail.com wrote: >>> [...] > +case

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

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 13:05, wrote: > +static struct keyhandler { > +union { > +keyhandler_fn_t *fn; > +irq_keyhandler_fn_t *irq_fn; > +}; > + > +const char *desc;/* Description for help message. */ > +bool_t irq_callback, /* Call in irq context? if

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

2015-09-24 Thread Andrew Cooper
On 24/09/15 12:46, Jan Beulich wrote: On 24.09.15 at 13:05, wrote: >> +static struct keyhandler { >> +union { >> +keyhandler_fn_t *fn; >> +irq_keyhandler_fn_t *irq_fn; >> +}; >> + >> +const char *desc;/* Description for help message. */ >> +

Re: [Xen-devel] [PATCH v7 20/28] xen/arm: ITS: Add virtual ITS availability check helper

2015-09-24 Thread Julien Grall
On 24/09/15 07:44, Vijay Kilari wrote: > On Wed, Sep 23, 2015 at 2:22 PM, Julien Grall wrote: >> Hi Vijay, >> >> On 18/09/15 14:09, vijay.kil...@gmail.com wrote: >>> From: Vijaya Kumar K >>> >>> Introduce vgic_is_lpi_supported() helper function >>> to know virtual ITS availability for a domain >>

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

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 13:00, wrote: > On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: >> diff --git a/tools/libxl/libxl_psr.c b/tools/libxl/libxl_psr.c >> index 3378239..62963cf 100644 >> --- a/tools/libxl/libxl_psr.c >> +++ b/tools/libxl/libxl_psr.c >> @@ -87,6 +87,9 @@ static void libxl__psr_cat_

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

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 13:22, wrote: > On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: >> Add new CDP options with CAT commands in xl interface man page. >> --- a/docs/man/xl.pod.1 >> +++ b/docs/man/xl.pod.1 >> @@ -1530,6 +1530,12 @@ applications. In the Xen implementation, CAT is >> used to control

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

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 13:47, wrote: > On 24/09/15 12:46, Jan Beulich wrote: > On 24.09.15 at 13:05, wrote: >>> +static struct keyhandler { >>> +union { >>> +keyhandler_fn_t *fn; >>> +irq_keyhandler_fn_t *irq_fn; >>> +}; >>> + >>> +const char *desc;/* Description f

Re: [Xen-devel] [PATCH] x86emul: support clzero

2015-09-24 Thread Andrew Cooper
On 24/09/15 09:02, Jan Beulich wrote: On 23.09.15 at 19:37, wrote: >> On 22/09/15 14:06, Jan Beulich wrote: >>> ... in anticipation of this possibly going to get used by guests for >>> basic thinks like memset() or clearing or pages. >>> >>> Since the emulation doesn't use clzero itself, chec

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

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 05:50 -0600, Jan Beulich wrote: > > > > On 24.09.15 at 13:00, wrote: > > On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: > > > diff --git a/tools/libxl/libxl_psr.c b/tools/libxl/libxl_psr.c > > > index 3378239..62963cf 100644 > > > --- a/tools/libxl/libxl_psr.c > > > +++ b/

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

2015-09-24 Thread Andrew Cooper
On 24/09/15 12:58, Jan Beulich wrote: On 24.09.15 at 13:47, wrote: >> On 24/09/15 12:46, Jan Beulich wrote: >> On 24.09.15 at 13:05, wrote: +static struct keyhandler { +union { +keyhandler_fn_t *fn; +irq_keyhandler_fn_t *irq_fn; +}; +

Re: [Xen-devel] [PATCH] x86emul: support clzero

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 13:59, wrote: > On 24/09/15 09:02, Jan Beulich wrote: > On 23.09.15 at 19:37, wrote: >>> On 22/09/15 14:06, Jan Beulich wrote: ... in anticipation of this possibly going to get used by guests for basic thinks like memset() or clearing or pages. Since the

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

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 14:09, wrote: > This appears to work for me. Looks okay with ... > --- a/xen/common/keyhandler.c > +++ b/xen/common/keyhandler.c > @@ -43,10 +43,10 @@ > } key_table[128] __read_mostly = > { > #define KEYHANDLER(k, f, desc, diag)\ > -[k] = { .fn = (f), desc,

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

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 14:07, wrote: > On Thu, 2015-09-24 at 05:50 -0600, Jan Beulich wrote: >> a specific error type >> to me seems to go too far. Surely out of the several dozen >> possibilities a handful of not-so-common ones can be picked? > > I was thinking in particular EINVAL was not in the n

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

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

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

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 06:20 -0600, Jan Beulich wrote: > > > > On 24.09.15 at 14:07, wrote: > > On Thu, 2015-09-24 at 05:50 -0600, Jan Beulich wrote: > > > a specific error type > > > to me seems to go too far. Surely out of the several dozen > > > possibilities a handful of not-so-common ones can

[Xen-devel] [xen-4.6-testing test] 62273: trouble: broken/fail/pass

2015-09-24 Thread osstest service owner
flight 62273 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/62273/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-arndale 3 host-install(3) broken REGR. vs. 620

Re: [Xen-devel] [PATCH v7 23/28] xen/arm: ITS: Allocate pending_lpi descriptors for LPIs

2015-09-24 Thread Julien Grall
On 18/09/15 14:09, vijay.kil...@gmail.com wrote: > diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h > index 6a5c6a0..49630a3 100644 > --- a/xen/include/asm-arm/domain.h > +++ b/xen/include/asm-arm/domain.h > @@ -101,6 +101,9 @@ struct arch_domain > * struct arch_vc

Re: [Xen-devel] [PATCH] Add missing license and copyright statements to public interface headers.

2015-09-24 Thread Boris Ostrovsky
On 09/24/2015 06:45 AM, Ian Campbell wrote: On Tue, 2015-09-22 at 16:02 +0200, Mike Belopuhov wrote: The copyright line indicates a person, a group of people and/or a company granting rights stated in the license text and is a required part of the license. The year of the copyright is chosen to

[Xen-devel] EFI GetNextVariableName hang/crash

2015-09-24 Thread Marek Marczykowski-Górecki
Hi, I have well known problem of buggy firmware, where GetNextVariableName doesn't work from dom0 running on Xen. I've read both threads about this: "EFI GetNextVariableName crashes when running under Xen, but not under Linux. efi-rs=0 works. No memmap issues" http://lists.xenproject.org/archives/

Re: [Xen-devel] [PATCH] Add missing license and copyright statements to public interface headers.

2015-09-24 Thread Konrad Rzeszutek Wilk
On September 24, 2015 9:17:08 AM EDT, Boris Ostrovsky wrote: >On 09/24/2015 06:45 AM, Ian Campbell wrote: >> On Tue, 2015-09-22 at 16:02 +0200, Mike Belopuhov wrote: >>> The copyright line indicates a person, a group of people and/or a >company >>> granting rights stated in the license text and i

Re: [Xen-devel] EFI GetNextVariableName hang/crash

2015-09-24 Thread Ross Lagerwall
On 09/24/2015 03:53 PM, Marek Marczykowski-Górecki wrote: Hi, I have well known problem of buggy firmware, where GetNextVariableName doesn't work from dom0 running on Xen. I've read both threads about this: "EFI GetNextVariableName crashes when running under Xen, but not under Linux. efi-rs=0 wo

Re: [Xen-devel] [PATCH v2] x86/bigmem: eliminate struct domain address width restriction

2015-09-24 Thread Jan Beulich
>>> On 22.09.15 at 14:51, wrote: > PDX-es are 64 bits wide in that case, and hence no limit needs to be > enforced. > > Reported-by: Andrew Cooper > Signed-off-by: Jan Beulich > Reviewed-by: Andrew Cooper Wei - another one I should have Cc-ed you on. Relevant to 4.6 as that bigmem thing is ne

[Xen-devel] Commit moratorium for possible rc4

2015-09-24 Thread Ian Campbell
Wei, It seems like rc4 is about due, but you are away. So in anticipation of you getting back and wanting to cut rc4 ASAP we discussed on IRC and decided to institute a commit moratorium from ~now so that on Monday when you get back stable-4.6 == staging-4.6 (hopefully!) and you can either call i

[Xen-devel] [PATCH] xen: sched: rename vcpu_destroy perf counter to vcpu_remove

2015-09-24 Thread Dario Faggioli
It seems this have had to be done as part of 7e6b926a ("cpupools: Make interface more consistent"), which renamed the function but not the counter. In fact, because of cpupools, vcpus are not only removed from a scheduler when they are destroyed, but also when domains move between pools. Make the

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

2015-09-24 Thread Jan Beulich
>>> On 17.09.15 at 11:35, wrote: > Add boot parameter `psr=cdp` to enable CDP at boot time. > Intel Code/Data Prioritization(CDP) feature is based on CAT. Note that > cos_max would be half when CDP is on. struct psr_cat_cbm is extended to > support CDP operation. Extend psr_get_cat_l3_info sysctl

Re: [Xen-devel] Commit moratorium for possible rc4

2015-09-24 Thread Wei Liu
On Thu, Sep 24, 2015 at 04:37:14PM +0100, Ian Campbell wrote: > Wei, > > It seems like rc4 is about due, but you are away. > > So in anticipation of you getting back and wanting to cut rc4 ASAP we > discussed on IRC and decided to institute a commit moratorium from ~now so > that on Monday when y

Re: [Xen-devel] [PATCH] Add missing license and copyright statements to public interface headers.

2015-09-24 Thread Wei Liu
On Thu, Sep 24, 2015 at 11:45:56AM +0100, Ian Campbell wrote: > On Tue, 2015-09-22 at 16:02 +0200, Mike Belopuhov wrote: > > The copyright line indicates a person, a group of people and/or a company > > granting rights stated in the license text and is a required part of the > > license. > > > > T

Re: [Xen-devel] [PATCH v2] x86/bigmem: eliminate struct domain address width restriction

2015-09-24 Thread Wei Liu
On Thu, Sep 24, 2015 at 09:35:50AM -0600, Jan Beulich wrote: > >>> On 22.09.15 at 14:51, wrote: > > PDX-es are 64 bits wide in that case, and hence no limit needs to be > > enforced. > > > > Reported-by: Andrew Cooper > > Signed-off-by: Jan Beulich > > Reviewed-by: Andrew Cooper > > Wei - ano

Re: [Xen-devel] [PATCH] x86/PV: properly populate descriptor tables

2015-09-24 Thread Wei Liu
On Wed, Sep 23, 2015 at 09:34:16AM -0600, Jan Beulich wrote: > Us extending the GDT limit past the Xen descriptors so far meant that > guests (including user mode programs) accessing any descriptor table > slot above the original OS'es limit but below the first Xen descriptor > caused a #PF, conver

[Xen-devel] List etiquette [Was: Re: EFI GetNextVariableName hang/crash]

2015-09-24 Thread Dario Faggioli
[Unrelated to anything technical, dropping some Cc-s] On Thu, 2015-09-24 at 16:26 +0100, Ross Lagerwall wrote: > On 09/24/2015 03:53 PM, Marek Marczykowski-Górecki wrote: > > (XEN) Xen call trace: > > (XEN)[] ca9e6f98 > > (XEN)[] csched_vcpu_wake+0x1b7/0x4b0 > >

[Xen-devel] [linux-3.4 test] 62277: tolerable FAIL - PUSHED

2015-09-24 Thread osstest service owner
flight 62277 linux-3.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/62277/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemut-stubdom-debianhvm-amd64-xsm 13 guest-localmigrate fail baseline untested test-amd64-a

Re: [Xen-devel] [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries

2015-09-24 Thread Ian Campbell
(dropping mini-os-devel and some cc's, adding David) Hi David, On Wed, 2015-07-15 at 16:46 +0100, Ian Campbell wrote: > (this is clearly not 4.6 material, aiming for 4.7) > > In <1431963008.4944.80.ca...@citrix.com> I proposed stabilising some > parts of the libxenctrl API/ABI by disaggregating

[Xen-devel] [OSSTEST PATCH 2/3] Shell fixup: Make all invocations of `.' (`source') use ./

2015-09-24 Thread Ian Jackson
In POSIX, `.' (the shell builtin) respects PATH, and does not search `.' (the current directory). Change all the invocations which refer to files which are part of osstest to say `. ./foo' instead of simply `. foo'. I have checked the results of git-grep '^[ \t]*\. [^./]' after this patch and t

[Xen-devel] [OSSTEST PATCH 3/3] Shell fixup: Use bash in posix mode

2015-09-24 Thread Ian Jackson
When bash is started as /bin/sh it run in posix compatibility mode. But when invoked as /bin/bash it does some things ... differently. Most notably: Subshells spawned to execute command substitutions inherit the value of the -e option from the parent shell. When not in posix mode, bash

[Xen-devel] [OSSTEST PATCH 1/3] Remove obsolete file "test.sched"

2015-09-24 Thread Ian Jackson
Signed-off-by: Ian Jackson --- test.sched |5 - 1 file changed, 5 deletions(-) delete mode 100755 test.sched diff --git a/test.sched b/test.sched deleted file mode 100755 index ca309fb..000 --- a/test.sched +++ /dev/null @@ -1,5 +0,0 @@ -#!/bin/sh -set -e -./ts-install-debian -./ts-

Re: [Xen-devel] [BUG] XEN domU crash when PV grub chainloads 32-bit domU grub

2015-09-24 Thread Andreas Sundstrom
On 2015-09-23 16:18, Ian Campbell wrote: > On Wed, 2015-09-23 at 12:47 +, Andreas Sundstrom wrote: >> Citerar Ian Campbell : >> >>> Along those lines, if the _host_ has buckets of RAM then might it be >>> worth >>> restricting it in case the issue is with getting MFNs which are not >>> represen

[Xen-devel] Make ready for 4.6! Xen Project Document Day is Wednesday, Sept 30

2015-09-24 Thread Russ Pavlicek
Our next Xen Project Document Day is this Wednesday, September 30! OUR THEME OF THE MONTH: "Ready for 4.6" This month, we prepare for the release of Xen Project 4.6 early next month. We need to make sure that users of the new release can find the documentation they need to make it all work. So, t

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-24 Thread Stefano Stabellini
On Thu, 24 Sep 2015, Ian Campbell wrote: > On Wed, 2015-09-23 at 18:36 +0100, Stefano Stabellini wrote: > > On Wed, 23 Sep 2015, Ian Campbell wrote: > > > On Tue, 2015-09-22 at 22:31 +0100, Stefano Stabellini wrote: > > > > The oldest Xen version I build-test for every pull request is Xen 4.0.0, >

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-24 at 20:33 +0100, Stefano Stabellini wrote: > On Thu, 24 Sep 2015, Ian Campbell wrote: > > On Wed, 2015-09-23 at 18:36 +0100, Stefano Stabellini wrote: > > > On Wed, 23 Sep 2015, Ian Campbell wrote: > > > > On Tue, 2015-09-22 at 22:31 +0100, Stefano Stabellini wrote: > > > > > The

Re: [Xen-devel] Xen, ACPI and Linux

2015-09-24 Thread Stefano Stabellini
On Wed, 23 Sep 2015, Stefano Stabellini wrote: > On Wed, 23 Sep 2015, Ian Campbell wrote: > > On Wed, 2015-09-23 at 01:18 -0700, Ard Biesheuvel wrote: > > > On 23 September 2015 at 01:12, Jan Beulich wrote: > > > > > > > On 23.09.15 at 02:49, wrote: > > > > > Regarding Runtime Services, the EFI s

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-24 Thread Stefano Stabellini
On Thu, 24 Sep 2015, Ian Campbell wrote: > On Thu, 2015-09-24 at 20:33 +0100, Stefano Stabellini wrote: > > On Thu, 24 Sep 2015, Ian Campbell wrote: > > > On Wed, 2015-09-23 at 18:36 +0100, Stefano Stabellini wrote: > > > > On Wed, 23 Sep 2015, Ian Campbell wrote: > > > > > On Tue, 2015-09-22 at 22

  1   2   >