Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 06:09, wrote: > It's always good to have a clear definition to which extend a performance > issue would become a security risk. I saw 200us/500us used as example > in this thread, however no one can give an accrual criteria. In that case, > how do we call it a problem even when

[Xen-devel] [distros-debian-wheezy test] 44236: trouble: blocked/broken

2016-03-10 Thread Platform Team regression test user
flight 44236 distros-debian-wheezy real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44236/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 3 host-install(3) broken REG

[Xen-devel] [V4] x86/xsaves: fix overwriting between non-lazy/lazy xsaves

2016-03-10 Thread Shuai Ruan
The offset at which components xsaved by xsave[sc] are not fixed. So when when a save with v->fpu_dirtied set is followed by one with v->fpu_dirtied clear, non-lazy xsave[sc] may overwriting data written by the lazy one. The solution is when using_xsave_compact is enabled and taking xcr0_accum int

Re: [Xen-devel] [PATCH v2 1/2] tools/foreign: Avoid using alignment directives when not appropriate

2016-03-10 Thread Jan Beulich
>>> On 09.03.16 at 20:03, wrote: > The foreign header generation blindly replaces 'uint64_t' with '__align8__ > uint64_t', to get correct alignment when built as 32bit. This is correct in > most circumstances, but Clang objects to two specific uses. > > * Inside a sizeof() expression > * As pa

Re: [Xen-devel] [PATCH v2 1/2] tools/foreign: Avoid using alignment directives when not appropriate

2016-03-10 Thread Jan Beulich
>>> On 09.03.16 at 20:56, wrote: > On Wed, Mar 09, 2016 at 07:03:15PM +, Andrew Cooper wrote: >> The foreign header generation blindly replaces 'uint64_t' with '__align8__ >> uint64_t', to get correct alignment when built as 32bit. This is correct in >> most circumstances, but Clang objects t

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, March 10, 2016 4:07 PM > > >>> On 10.03.16 at 06:09, wrote: > > It's always good to have a clear definition to which extend a performance > > issue would become a security risk. I saw 200us/500us used as example > > in this thread,

Re: [Xen-devel] arm: Missing memset in setup_frametable_mappings

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 01:08, wrote: > I believe there is a bug in setup_frametable_mappings() where the function > allocates pages for pagetables via alloc_boot_pages() but does not zero them > out. This results in a crash on Qualcomm systems when prefetching is enabled > since the processor is free

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 09:43, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Thursday, March 10, 2016 4:07 PM >> >> theoretical basis for some sort of measurement would be to >> determine how long a worst case list traversal would take. With >> "worst case" being derived from the th

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

2016-03-10 Thread osstest service owner
flight 85820 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/85820/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-multivcpu 3 host-install(3)broken REGR. vs. 853

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, March 10, 2016 5:06 PM > > > > There are many linked list usages today in Xen hypervisor, which > > have different theoretical maximum possible number. The closest > > one to PI might be the usage in tmem (pool->share_list) which is

Re: [Xen-devel] [V4] x86/xsaves: fix overwriting between non-lazy/lazy xsaves

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 09:22, wrote: > --- a/xen/arch/x86/domain.c > +++ b/xen/arch/x86/domain.c > @@ -948,7 +948,7 @@ int arch_set_info_guest( > fpu_sse->fcw = FCW_DEFAULT; > fpu_sse->mxcsr = MXCSR_DEFAULT; > } > -if ( cpu_has_xsaves ) > +if ( using_xsave_compact ) >

Re: [Xen-devel] On setting clear criteria for declaring a feature acceptable (was "vmx: VT-d posted-interrupt core logic handling")

2016-03-10 Thread George Dunlap
On Wed, Mar 9, 2016 at 6:02 PM, David Vrabel wrote: > On 09/03/16 16:23, George Dunlap wrote: >> >> I don't know why this is controversial -- this seems obvious to me. >> What do other committers / maintainers think? > > I started on a reply to this but then I went back and read the original > thr

[Xen-devel] [PATCH v2 0/3] x86: accommodate 32-bit PV guests with SMEP/SMAP handling

2016-03-10 Thread Jan Beulich
As was previously explained[1], SMAP (and with less relevance also SMEP) is not compatible with 32-bit PV guests which aren't aware/ prepared to be run with that feature enabled. Andrew's original approach either sacrificed architectural correctness for making 32-bit guests work again or by disabli

Re: [Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-10 Thread Jan Beulich
>>> On 09.03.16 at 14:17, wrote: > Signed-off-by: Quan Xu > Acked-by: Kevin Tian The patch itself looks mostly fine now (see below for the minor left issues), but the complete lack of a description (which should state why this change is being done) makes this not ready to go in anyway. > --- a

[Xen-devel] [PATCH v2 1/3] x86: suppress SMEP and SMAP while running 32-bit PV guest code

2016-03-10 Thread Jan Beulich
Since such guests' kernel code runs in ring 1, their memory accesses, at the paging layer, are supervisor mode ones, and hence subject to SMAP/SMEP checks. Such guests cannot be expected to be aware of those two features though (and so far we also don't expose the respective feature flags), and hen

[Xen-devel] [PATCH v2 2/3] x86: use optimal NOPs to fill the SMEP/SMAP placeholders

2016-03-10 Thread Jan Beulich
Alternatives patching code picks the most suitable NOPs for the running system, so simply use it to replace the pre-populated ones. Use an arbitrary, always available feature to key off from, but hide this behind the new X86_FEATURE_ALWAYS. Signed-off-by: Jan Beulich --- v2: Introduce and use X8

[Xen-devel] [PATCH v2 3/3] x86: use 32-bit loads for 32-bit PV guest state reload

2016-03-10 Thread Jan Beulich
This is slightly more efficient than loading 64-bit quantities. Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper --- a/xen/include/asm-x86/asm_defns.h +++ b/xen/include/asm-x86/asm_defns.h @@ -313,6 +313,13 @@ static always_inline void stac(void) 987: .endm +#define LOAD_ONE_REG(reg, c

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Tian, Kevin
> From: Tian, Kevin > Sent: Thursday, March 10, 2016 5:20 PM > > > From: Jan Beulich [mailto:jbeul...@suse.com] > > Sent: Thursday, March 10, 2016 5:06 PM > > > > > > > There are many linked list usages today in Xen hypervisor, which > > > have different theoretical maximum possible number. The cl

[Xen-devel] [PATCH 0/2] x86/shadow: further code size reduction

2016-03-10 Thread Jan Beulich
Following some earlier work in this regard, the first patch here constitutes what I recognized as further potential back then, but which needed a little more time to consider the options of how to reasonably split/move things. The second patch is just another small step recognized as being possible

[Xen-devel] [PATCH] xen: Don't disable the REP emulation optimizations for regular IO

2016-03-10 Thread Razvan Cojocaru
Currently REP emulations optimizations remain disabled even if the emulation does not happen as a result of a vm_event reply requestion emulation (i.e. even for regular IO). This patch takes emulate_each_rep into account only if emulation has been requested by a vm_event-capable application, and is

[Xen-devel] [PATCH 1/2] x86/shadow: compile most write emulation code just once

2016-03-10 Thread Jan Beulich
No need to compile all of this code three times, as most of it really is guest mode independent. The savings are between 3k and 4k of binary code in my builds. No functional change (i.e. only formatting and naming changes) except for - sh_emulate_map_dest()'s user mode check corrected for the PV c

[Xen-devel] [PATCH 2/2] x86/shadow: avoid extra local array variable

2016-03-10 Thread Jan Beulich
mfns[2] was there just because struct sh_emulate_ctxt's two MFN values can't be used to hand to vmap(). Making the structure fields an array avoids the extra copying. Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm/shadow/common.c +++ b/xen/arch/x86/mm/shadow/common.c @@ -1746,11 +1746,11 @@ vo

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 11:05, wrote: >> From: Tian, Kevin >> Sent: Thursday, March 10, 2016 5:20 PM >> >> > From: Jan Beulich [mailto:jbeul...@suse.com] >> > Sent: Thursday, March 10, 2016 5:06 PM >> > >> > >> > > There are many linked list usages today in Xen hypervisor, which >> > > have different

Re: [Xen-devel] [PATCH] xen: Don't disable the REP emulation optimizations for regular IO

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 11:07, wrote: > Currently REP emulations optimizations remain disabled even if > the emulation does not happen as a result of a vm_event reply > requestion emulation (i.e. even for regular IO). This patch takes > emulate_each_rep into account only if emulation has been requested

Re: [Xen-devel] [PATCH v4 3/3] libxl: add force option for xl vcpu-pin

2016-03-10 Thread Dario Faggioli
On Thu, 2016-03-10 at 06:27 +0100, Juergen Gross wrote: > In order to be able to undo a vcpu pin override in case of a kernel > driver error add a flag "-f" to the "xl vcpu-pin" command forcing the > hypervisor to undo the override. > > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Wei Liu > S

Re: [Xen-devel] [PATCH v4 1/3] libxc: do some retries in xc_cpupool_removecpu() for EBUSY case

2016-03-10 Thread Dario Faggioli
On Thu, 2016-03-10 at 06:27 +0100, Juergen Gross wrote: > The hypervisor might return EBUSY when trying to remove a cpu from a > cpupool when a domain running in this cpupool has pinned a vcpu > temporarily. Do some retries in this case, perhaps the situation > cleans up. > > Cc: Ian Jackson > Cc

Re: [Xen-devel] Status of multiboot2 support on EFI?

2016-03-10 Thread Daniel Kiper
On Fri, Feb 12, 2016 at 08:08:58PM +0100, Daniel Kiper wrote: > On Wed, Feb 03, 2016 at 06:03:50PM +0100, Daniel Kiper wrote: > > On Wed, Feb 03, 2016 at 05:08:44PM +0100, Daniel Kiper wrote: > > > On Tue, Feb 02, 2016 at 03:50:17PM -0600, Doug Goldstein wrote: > > > > On 2/2/16 3:20 PM, Daniel Kip

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread David Vrabel
On 10/03/16 10:18, Jan Beulich wrote: On 10.03.16 at 11:05, wrote: >>> From: Tian, Kevin >>> Sent: Thursday, March 10, 2016 5:20 PM >>> From: Jan Beulich [mailto:jbeul...@suse.com] Sent: Thursday, March 10, 2016 5:06 PM > There are many linked list usages today in Xen

Re: [Xen-devel] [PATCH v7]xen: sched: convert RTDS from time to event driven model

2016-03-10 Thread Dario Faggioli
On Wed, 2016-03-09 at 23:00 -0500, Meng Xu wrote: > On Wed, Mar 9, 2016 at 10:46 AM, Dario Faggioli > wrote: > >  > > Basically, by doing all the replenishments (which includes updating > > all > > the deadlines) upfront, we should be able to prevent the above > > situation. > > > > So, again, th

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread George Dunlap
On 10/03/16 08:07, Jan Beulich wrote: On 10.03.16 at 06:09, wrote: >> It's always good to have a clear definition to which extend a performance >> issue would become a security risk. I saw 200us/500us used as example >> in this thread, however no one can give an accrual criteria. In that case

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread George Dunlap
On 10/03/16 10:35, David Vrabel wrote: > On 10/03/16 10:18, Jan Beulich wrote: > On 10.03.16 at 11:05, wrote: From: Tian, Kevin Sent: Thursday, March 10, 2016 5:20 PM > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, March 10, 2016 5:06 PM > > >>>

Re: [Xen-devel] [PATCH v2 1/2] tools/foreign: Avoid using alignment directives when not appropriate

2016-03-10 Thread Wei Liu
On Thu, Mar 10, 2016 at 01:33:16AM -0700, Jan Beulich wrote: > >>> On 09.03.16 at 20:56, wrote: > > On Wed, Mar 09, 2016 at 07:03:15PM +, Andrew Cooper wrote: > >> The foreign header generation blindly replaces 'uint64_t' with '__align8__ > >> uint64_t', to get correct alignment when built as

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread George Dunlap
On 10/03/16 10:18, Jan Beulich wrote: On 10.03.16 at 11:05, wrote: >>> From: Tian, Kevin >>> Sent: Thursday, March 10, 2016 5:20 PM >>> From: Jan Beulich [mailto:jbeul...@suse.com] Sent: Thursday, March 10, 2016 5:06 PM > There are many linked list usages today in Xen

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread David Vrabel
On 10/03/16 10:46, George Dunlap wrote: > On 10/03/16 10:35, David Vrabel wrote: >> On 10/03/16 10:18, Jan Beulich wrote: >> On 10.03.16 at 11:05, wrote: > From: Tian, Kevin > Sent: Thursday, March 10, 2016 5:20 PM > >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent:

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Dario Faggioli
On Thu, 2016-03-10 at 11:00 +, George Dunlap wrote: > > > > > On 10.03.16 at 11:05, wrote: > > > >  From: Tian, Kevin > > > > Sent: Thursday, March 10, 2016 5:20 PM > > > >  > > > Actually after more thinking, there is no hard requirement that > > > the vcpu must block on the pcpu which is con

Re: [Xen-devel] [PATCH] xen: Don't disable the REP emulation optimizations for regular IO

2016-03-10 Thread Andrew Cooper
On 10/03/16 10:07, Razvan Cojocaru wrote: > Currently REP emulations optimizations remain disabled even if > the emulation does not happen as a result of a vm_event reply > requestion emulation (i.e. even for regular IO). This patch takes > emulate_each_rep into account only if emulation has been r

[Xen-devel] Call For Participation for the Xen Project Developer Summit in Toronto (Deadline May 6th)

2016-03-10 Thread Lars Kurth
Dear Community Members! We’re excited to announce the call for speaking proposals for Xen Project Developer Summit 2016, which will be held in Toronto, Canada, August 25-26, 2016. The Xen Project Developer Summit brings together the Xen Project’s community of developers and power users for thei

Re: [Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-10 Thread Xu, Quan
On March 10, 2016 5:53pm, wrote: > >>> On 09.03.16 at 14:17, wrote: > > Signed-off-by: Quan Xu > > Acked-by: Kevin Tian > > The patch itself looks mostly fine now (see below for the minor left issues), > but > the complete lack of a description (which should state why this change is > being

Re: [Xen-devel] [PATCH 1/2] x86/shadow: compile most write emulation code just once

2016-03-10 Thread Andrew Cooper
On 10/03/16 10:13, Jan Beulich wrote: > No need to compile all of this code three times, as most of it really > is guest mode independent. The savings are between 3k and 4k of binary > code in my builds. > > No functional change (i.e. only formatting and naming changes) except > for > - sh_emulate_

Re: [Xen-devel] [PATCH 2/2] x86/shadow: avoid extra local array variable

2016-03-10 Thread Andrew Cooper
On 10/03/16 10:13, Jan Beulich wrote: > mfns[2] was there just because struct sh_emulate_ctxt's two MFN values > can't be used to hand to vmap(). Making the structure fields an array > avoids the extra copying. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper __

[Xen-devel] [xen-4.5-testing test] 85821: tolerable FAIL - PUSHED

2016-03-10 Thread osstest service owner
flight 85821 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/85821/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop fail like 85360 test-amd64-amd64-xl-rtds

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread George Dunlap
On 10/03/16 11:16, David Vrabel wrote: > On 10/03/16 10:46, George Dunlap wrote: >> On 10/03/16 10:35, David Vrabel wrote: >>> On 10/03/16 10:18, Jan Beulich wrote: >>> On 10.03.16 at 11:05, wrote: >> From: Tian, Kevin >> Sent: Thursday, March 10, 2016 5:20 PM >> >>> From: Jan

Re: [Xen-devel] [PATCH] x86/xstate: undo bogus adjustment to xsave()

2016-03-10 Thread Andrew Cooper
On 10/03/16 07:39, Jan Beulich wrote: > This reverts an unintended change in commit 879b44b041 ("x86/fpu: add > a per-domain field to set the width of FIP/FDP"), which I had done > intermediately while fixing the build issue: After having reverted that > adjustment I must have forgotten to "git add

Re: [Xen-devel] [PATCH] x86/xstate: undo bogus adjustment to xsave()

2016-03-10 Thread David Vrabel
On 10/03/16 07:39, Jan Beulich wrote: > This reverts an unintended change in commit 879b44b041 ("x86/fpu: add > a per-domain field to set the width of FIP/FDP"), which I had done > intermediately while fixing the build issue: After having reverted that > adjustment I must have forgotten to "git add

[Xen-devel] [PATCH net-next 2/3] xen-netback: support multiple extra info fragments passed from frontend

2016-03-10 Thread Paul Durrant
The code does not currently support a frontend passing multiple extra info fragments to the backend in a tx request. The xenvif_get_extras() function handles multiple extra_info fragments but make_tx_response() assumes there is only ever a single extra info fragment. This patch modifies xenvif_get

[Xen-devel] [PATCH net-next 3/3] xen-netback: reduce log spam

2016-03-10 Thread Paul Durrant
Remove the "prepare for reconnect" pr_info in xenbus.c. It's largely uninteresting and the states of the frontend and backend can easily be observed by watching the (o)xenstored log. Signed-off-by: Paul Durrant Cc: Wei Liu --- drivers/net/xen-netback/xenbus.c | 2 -- 1 file changed, 2 deletions

[Xen-devel] [PATCH net-next 0/3] xen-netback: fix multiple extra info handling

2016-03-10 Thread Paul Durrant
If a frontend passes multiple extra info fragments to netback on the guest transmit side, because xen-netback does not account for this properly, only a single ack response will be sent. This will eventually cause processing of the shared ring to wedge. This series re-imports the canonical netif.h

[Xen-devel] [PATCH net-next 1/3] xen-netback: re-import canonical netif header

2016-03-10 Thread Paul Durrant
The canonical netif header (in the Xen source repo) and the Linux variant have diverged significantly. Recently much documentation has been added to the canonical header which is highly useful for developers making modifications to either xen-netfront or xen-netback. This patch therefore re-imports

[Xen-devel] [PATCHv3] x86: don't flush the whole cache when changing cachability

2016-03-10 Thread David Vrabel
Introduce the FLUSH_VA_VALID flag to flush_area_mask() and friends to say that it is safe to use CLFLUSH (i.e., the virtual address is still valid). Use this when changing the cachability of the Xen direct mappings (in response to the guest changing the cachability of its mappings). This significa

Re: [Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 12:27, wrote: > On March 10, 2016 5:53pm, wrote: >> >>> On 09.03.16 at 14:17, wrote: >> > Signed-off-by: Quan Xu >> > Acked-by: Kevin Tian >> >> The patch itself looks mostly fine now (see below for the minor left > issues), but >> the complete lack of a description (which

Re: [Xen-devel] [PATCH] x86/xstate: undo bogus adjustment to xsave()

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 13:27, wrote: > On 10/03/16 07:39, Jan Beulich wrote: >> This reverts an unintended change in commit 879b44b041 ("x86/fpu: add >> a per-domain field to set the width of FIP/FDP"), which I had done >> intermediately while fixing the build issue: After having reverted that >> adju

Re: [Xen-devel] [PATCH 1/2] x86/shadow: compile most write emulation code just once

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 13:17, wrote: > Hi, > > At 03:13 -0700 on 10 Mar (1457579586), Jan Beulich wrote: >> No need to compile all of this code three times, as most of it really >> is guest mode independent. The savings are between 3k and 4k of binary >> code in my builds. > > Thanks for this. This

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 12:16, wrote: > On 10/03/16 10:46, George Dunlap wrote: >> On 10/03/16 10:35, David Vrabel wrote: >>> You could also consider only waking the first N VCPUs and just making >>> the rest runnable. If you wake more VCPUs than PCPUs at the same time >>> most of them won't actually

Re: [Xen-devel] vmx: VT-d posted-interrupt core logic handling

2016-03-10 Thread Wu, Feng
> -Original Message- > From: Tian, Kevin > Sent: Thursday, March 10, 2016 6:06 PM > To: Jan Beulich > Cc: Andrew Cooper ; Dario Faggioli > ; David Vrabel ; > GeorgeDunlap ; Lars Kurth ; > George Dunlap ; Ian Jackson > ; Wu, Feng ; xen- > de...@lists.xen.org; Konrad Rzeszutek Wilk > Subj

[Xen-devel] [PATCH v4 0/2] Make the pcidevs_lock a recursive one

2016-03-10 Thread Quan Xu
This patch set makes the pcidevs_lock a recursive one, as pcidevs_lock is going to be recursively taken for hiding ATS device, when VT-d Device-TLB flush timed out. It is a prereq patch set for Patch:'VT-d Device-TLB flush issue'. In detail: 1. Fix a bug found in AMD IOMMU initialization pcide

[Xen-devel] [PATCH v4 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-10 Thread Quan Xu
pcidevs_lock doesn't require interrupts to be disabled while being acquired. However there remains an exception in AMD IOMMU code, where the lock is acquired with interrupt disabled. This inconsistency might lead to deadlock. The fix is straightforward to use spin_lock instead. Also interrupt has

[Xen-devel] [PATCH v4 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one.

2016-03-10 Thread Quan Xu
The pcidevs_lock is going to be recursively taken for hiding ATS device, when VT-d Device-TLB flush timed out. Signed-off-by: Quan Xu Acked-by: Kevin Tian CC: Keir Fraser CC: Jan Beulich CC: Andrew Cooper CC: Suravee Suthikulpanit CC: Feng Wu CC: Kevin Tian CC: Dario Faggioli --- xen/arc

Re: [Xen-devel] [PATCH] xen: Don't disable the REP emulation optimizations for regular IO

2016-03-10 Thread Tamas K Lengyel
On Mar 10, 2016 11:09, "Razvan Cojocaru" wrote: > > Currently REP emulations optimizations remain disabled even if > the emulation does not happen as a result of a vm_event reply > requestion emulation (i.e. even for regular IO). This patch takes > emulate_each_rep into account only if emulation h

Re: [Xen-devel] [PATCH v4 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one.

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 15:10, wrote: > The pcidevs_lock is going to be recursively taken for hiding ATS > device, when VT-d Device-TLB flush timed out. > > Signed-off-by: Quan Xu > Acked-by: Kevin Tian Acked-by: Jan Beulich ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v4 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one.

2016-03-10 Thread Dario Faggioli
On Thu, 2016-03-10 at 07:32 -0700, Jan Beulich wrote: > > > > On 10.03.16 at 15:10, wrote: > > The pcidevs_lock is going to be recursively taken for hiding ATS > > device, when VT-d Device-TLB flush timed out. > > > > Signed-off-by: Quan Xu > > Acked-by: Kevin Tian > Acked-by: Jan Beulich > I

[Xen-devel] [PATCH] xen: sched: Add Meng as RTDS maintainer.

2016-03-10 Thread Dario Faggioli
Meng Xu is one of the maintainers of the RT-Xen project, which is from where the RTDS scheduler comes. He also is the main author of the version of RTDS that we currently have here upstream. Since the upstreaming effort, he's continued looking after the code, engaging with the community, coming an

[Xen-devel] [PATCH 2/3] libxl: add domain config parameter to force start of qemu

2016-03-10 Thread Juergen Gross
Today the device model (qemu) is started for a pv domain only in case a device requiring qemu is specified in the domain configuration (qdisk, vfb, channel). If there is no such device the device model isn't started and hence it is possible to add such a device to the domain later. Add a domain co

[Xen-devel] [PATCH 0/3] libxl: add support for qemu base pvusb backend

2016-03-10 Thread Juergen Gross
This patch series is meant to be applied on top of Chunyan's series to support pvusb in libxl. It is adding support for an alternative pvusb backend "qusb" via qemu. Juergen Gross (3): libxl: make libxl__need_xenpv_qemu() operate on domain config libxl: add domain config parameter to force s

[Xen-devel] [PATCH 1/3] libxl: make libxl__need_xenpv_qemu() operate on domain config

2016-03-10 Thread Juergen Gross
libxl__need_xenpv_qemu() is called with configuration data for console, vfbs, disks and channels today in order to evaluate the need for starting a device model for a pv domain. The console data is local to the caller and setup in a way to never require a device model. All other data is taken from

[Xen-devel] [PATCH 3/3] libxl: add new pvusb backend "qusb" provided by qemu

2016-03-10 Thread Juergen Gross
Add a new pvusb backend type "qusb" which is provided by qemu. It can be selected either by specifying the type directly in the configuration or it is selected automatically by libxl in case there is no "usbback" driver loaded. Signed-off-by: Juergen Gross --- docs/man/xl.cfg.pod.5

Re: [Xen-devel] [PATCH] xen: sched: Add Meng as RTDS maintainer.

2016-03-10 Thread George Dunlap
On 10/03/16 14:42, Dario Faggioli wrote: > Meng Xu is one of the maintainers of the RT-Xen project, > which is from where the RTDS scheduler comes. He also > is the main author of the version of RTDS that we currently > have here upstream. > > Since the upstreaming effort, he's continued looking a

Re: [Xen-devel] [PATCH 1/2] libxl: ensure var is inited in libxl__domain_firmware

2016-03-10 Thread Doug Goldstein
On 3/8/16 9:38 AM, Wei Liu wrote: > On Mon, Mar 07, 2016 at 08:23:39PM -0600, Doug Goldstein wrote: >> Some versions of GCC complain that the 'firmware' variable can be used >> uninitialized. It looks like the switch inside of the else case is just >> confusing GCC. >> >> Signed-off-by: Doug Goldst

Re: [Xen-devel] [PATCH] xen: sched: Add Meng as RTDS maintainer.

2016-03-10 Thread Meng Xu
On Thu, Mar 10, 2016 at 9:42 AM, Dario Faggioli wrote: > Meng Xu is one of the maintainers of the RT-Xen project, > which is from where the RTDS scheduler comes. He also > is the main author of the version of RTDS that we currently > have here upstream. > > Since the upstreaming effort, he's conti

[Xen-devel] [PATCH v2 0/2] usb, xen: add pvUSB backend

2016-03-10 Thread Juergen Gross
This series adds a Xen pvUSB backend driver to qemu. USB devices connected to the host can be passed through to a Xen guest. The devices are specified via Xenstore. Access to the devices is done via host-libusb.c I've tested the backend with various USB devices (memory sticks, keyboard, ...). Cha

[Xen-devel] [PATCH v2 1/2] xen: introduce dummy system device

2016-03-10 Thread Juergen Gross
Introduce a new dummy system device serving as parent for virtual buses. This will enable new pv backends to introduce virtual buses which are removable again opposed to system buses which are meant to stay once added. Signed-off-by: Juergen Gross --- V2: NOT changed, even if requested by Stefano

[Xen-devel] [PATCH v2 2/2] xen: add pvUSB backend

2016-03-10 Thread Juergen Gross
Add a backend for para-virtualized USB devices for xen domains. The backend is using host-libusb to forward USB requests from a domain via libusb to the real device(s) passed through. Signed-off-by: Juergen Gross --- V2: use xen_be_printf() instead of fprintf() for diagnostic prints as reque

Re: [Xen-devel] [PATCH v7]xen: sched: convert RTDS from time to event driven model

2016-03-10 Thread Meng Xu
On Thu, Mar 10, 2016 at 5:38 AM, Dario Faggioli wrote: > On Wed, 2016-03-09 at 23:00 -0500, Meng Xu wrote: >> On Wed, Mar 9, 2016 at 10:46 AM, Dario Faggioli >> wrote: >> > >> > Basically, by doing all the replenishments (which includes updating >> > all >> > the deadlines) upfront, we should be

Re: [Xen-devel] [PATCH] xen: sched: Add Meng as RTDS maintainer.

2016-03-10 Thread Wei Liu
On Thu, Mar 10, 2016 at 10:08:04AM -0500, Meng Xu wrote: > On Thu, Mar 10, 2016 at 9:42 AM, Dario Faggioli > wrote: > > Meng Xu is one of the maintainers of the RT-Xen project, > > which is from where the RTDS scheduler comes. He also > > is the main author of the version of RTDS that we currently

Re: [Xen-devel] [PATCH v7]xen: sched: convert RTDS from time to event driven model

2016-03-10 Thread Dario Faggioli
On Thu, 2016-03-10 at 10:28 -0500, Meng Xu wrote: > On Thu, Mar 10, 2016 at 5:38 AM, Dario Faggioli > wrote: > > > > I don't think we really need to count anything. In fact, what I had > > in > > mind and tried to put down in pseudocode is that we traverse the > > list > > of replenishment events

[Xen-devel] [RFC PATCH v1] Splitting off stubdom to a different tree

2016-03-10 Thread Wei Liu
Hi all This is a very RFC seris for splitting off stubdom from xen.git. It's RFC because: 1. There are some open questions -- see individual commits. 2. I haven't tested it, everything should compile though. 3. I'm not entirely sure about the arrangement between toolstack and stubdom is corre

[Xen-devel] [PATCH RFC v1 0/2] mini-os: split stubdom from xen.git

2016-03-10 Thread Wei Liu
git://xenbits.xen.org/people/liuw/mini-os.git wip.split-stubdom-v1 Wei Liu (2): Config.mk: use MINI_OS if specified Makefile: introduce STUBDOM_ROOT Config.mk | 4 Makefile | 13 +++-- 2 files changed, 11 insertions(+), 6 deletions(-) -- 2.1.4 _

[Xen-devel] [PATCH RFC v1 04/14] Makefile: require XEN_ROOT and MINI_OS to be defined

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- Makefile | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/Makefile b/Makefile index cfbaa99..f3f5c02 100644 --- a/Makefile +++ b/Makefile @@ -1,5 +1,10 @@ -XEN_ROOT = $(CURDIR)/.. -MINI_OS = $(XEN_ROOT)/extras/mini-os +ifeq (,$(XEN_ROOT))

[Xen-devel] [PATCH RFC v1 3/6] libxc: use $XEN_ROOT to reference libelf in xen

2016-03-10 Thread Wei Liu
Stubdom needs to compile and link libelf in xen. Using relative path breaks that. Signed-off-by: Wei Liu --- tools/libxc/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/libxc/Makefile b/tools/libxc/Makefile index 1619c06..3a0a839 100644 --- a/tools/libxc/Ma

[Xen-devel] [PATCH RFC v1 07/14] grub: don't use hard-coded XEN_ROOT

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- grub/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/grub/Makefile b/grub/Makefile index 26dff45..9d91779 100644 --- a/grub/Makefile +++ b/grub/Makefile @@ -1,4 +1,3 @@ -XEN_ROOT = $(CURDIR)/../.. include $(XEN_ROOT)/Config.mk vpath %.c ../grub-upstream

[Xen-devel] [PATCH RFC v1 02/14] Makefile: introduce dist target

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- Makefile | 4 1 file changed, 4 insertions(+) diff --git a/Makefile b/Makefile index c8984d4..cfbaa99 100644 --- a/Makefile +++ b/Makefile @@ -642,6 +642,10 @@ downloadclean: patchclean distclean: downloadclean rm -rf config.log config.status config.cache

[Xen-devel] [PATCH RFC v1 01/14] Import more files from xen.git

2016-03-10 Thread Wei Liu
Imported from top level build system in xen.git. Modify them to work with stubdom build environment. Signed-off-by: Wei Liu --- I've committed configure changes in this patch, feel free to drop that and regenerate while committing. Not sure what the version number of stubdom.git should be. Previ

[Xen-devel] [PATCH RFC v1 00/14] stubdom: split stubdom from xen.git

2016-03-10 Thread Wei Liu
The base tree is prepared by: #!/bin/sh set -ex git filter-branch --tag-name-filter cat \ --subdirectory-filter stubdom/ -- --all # There is already a tag name 4.3.0-rc2 which points to the same commit. git tag -d xen-4.3.0-rc2 # Add xen- prefix to

[Xen-devel] [PATCH RFC v1 1/6] config: don't hard-code mini-os path

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- config/MiniOS.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/MiniOS.mk b/config/MiniOS.mk index 32260ad..191226e 100644 --- a/config/MiniOS.mk +++ b/config/MiniOS.mk @@ -1,5 +1,5 @@ include $(XEN_ROOT)/config/StdGNU.mk -include $(XEN_ROOT

[Xen-devel] [PATCH RFC v1 4/6] libxl: generate and install stubdompath.sh

2016-03-10 Thread Wei Liu
It's a script used by libxl to launch stubdom. This is split off from stubdom build system. Signed-off-by: Wei Liu --- .gitignore | 1 + tools/libxl/Makefile | 9 - 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index 91f690c..00811e7 1006

[Xen-devel] [PATCH RFC v1 10/14] Makefile: require QEMU_ROOT to be defined

2016-03-10 Thread Wei Liu
Don't tread into xen.git and clone one there. Signed-off-by: Wei Liu --- Makefile | 27 +-- 1 file changed, 5 insertions(+), 22 deletions(-) diff --git a/Makefile b/Makefile index 19b80a0..ff15d16 100644 --- a/Makefile +++ b/Makefile @@ -6,6 +6,10 @@ ifeq (,$(MINIOS_ROOT

[Xen-devel] [PATCH RFC v1 14/14] Add README file

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- Not sure what I should put in here other than building. And we also need COPYING file. --- README | 16 1 file changed, 16 insertions(+) create mode 100644 README diff --git a/README b/README new file mode 100644 index 000..76def8b --- /dev/null

[Xen-devel] [PATCH RFC v1 11/14] Makefile: always include Config.mk from xen source tree

2016-03-10 Thread Wei Liu
It contains various definitions needed in make clean. Signed-off-by: Wei Liu --- Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Makefile b/Makefile index ff15d16..84c5023 100644 --- a/Makefile +++ b/Makefile @@ -20,9 +20,7 @@ export DISTDIR=$(STUBDOM_ROOT)/dist

[Xen-devel] [PATCH RFC v1 13/14] Makefile: fix removing *.mk

2016-03-10 Thread Wei Liu
Now we have at least two mk files generated, fix the path and use glob. Signed-off-by: Wei Liu --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 004d096..9de7db5 100644 --- a/Makefile +++ b/Makefile @@ -615,7 +615,7 @@ downloadclean: patch

[Xen-devel] [PATCH RFC v1 12/14] Makefile: delete ioemu directory in clean target

2016-03-10 Thread Wei Liu
After we mandate an external QEMU tree there will only be object files in ioemu directory. Just delete the whole directory in make clean. Signed-off-by: Wei Liu --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 84c5023..004d096 100644 ---

[Xen-devel] [PATCH RFC v1 03/14] Add a gitignore

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- .gitignore | 74 ++ 1 file changed, 74 insertions(+) create mode 100644 .gitignore diff --git a/.gitignore b/.gitignore new file mode 100644 index 000..241601e --- /dev/null +++ b/.gitignore @@ -0,0 +1,74

[Xen-devel] [PATCH RFC v1 06/14] pciutils: don't use hard-coded XEN_ROOT

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- pciutils.patch | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pciutils.patch b/pciutils.patch index 5ab84d6..46af50a 100644 --- a/pciutils.patch +++ b/pciutils.patch @@ -33,12 +33,11 @@ diff -urN pciutils-2.2.9.orig/lib/access.c pciutils-2.2.9/li

[Xen-devel] [PATCH RFC v1 1/2] Config.mk: use MINI_OS if specified

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- Config.mk | 4 1 file changed, 4 insertions(+) diff --git a/Config.mk b/Config.mk index e5d8ade..788b0cb 100644 --- a/Config.mk +++ b/Config.mk @@ -31,6 +31,10 @@ MINI-OS_ROOT=$(XEN_ROOT)/extras/mini-os else MINI-OS_ROOT=$(TOPLEVEL_DIR) endif +# It is also poss

[Xen-devel] [PATCH RFC v1 6/6] Remove stubdom directory

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu [...] ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH RFC v1 08/14] Makefile: export STUBDOM_ROOT

2016-03-10 Thread Wei Liu
Mini-os needs to know where the libraries are when linking. Export a new environment variable called STBUDOM_ROOT so that mini-os knows the location of libraries. Signed-off-by: Wei Liu --- This needs to be used in conjunction with the patch to mini-os.git. --- Makefile | 1 + 1 file changed, 1

[Xen-devel] [PATCH RFC v1 2/2] Makefile: introduce STUBDOM_ROOT

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- Makefile | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/Makefile b/Makefile index cfe015a..db671e8 100644 --- a/Makefile +++ b/Makefile @@ -163,14 +163,15 @@ endif OBJS := $(filter-out $(OBJ_DIR)/lwip%.o $(LWO), $(OBJS)) +STUBDOM_

[Xen-devel] [PATCH RFC v1 5/6] Unhook stubdom from build system

2016-03-10 Thread Wei Liu
Signed-off-by: Wei Liu --- I've run autogen.sh and commit configure change as well. --- Makefile | 55 ++ autogen.sh | 3 -- config/Stubdom.mk.in | 39 - configure| 75 conf

[Xen-devel] [PATCH RFC v1 2/6] tools: use "?=" to set XEN_ROOT in Makefile

2016-03-10 Thread Wei Liu
This is because when doing a stubdom build using $(CURDIR) to refer to XEN_ROOT is wrong. Instead, when make enters directories of interest, XEN_ROOT is already. So use "?=" is the right choice. While in theory I can only fix up the places needed (a few libraries), do this for all Makefiles under

[Xen-devel] [PATCH RFC v1 05/14] Makefile: replace MINI_OS with MINIOS_ROOT

2016-03-10 Thread Wei Liu
This is more in line with how other repositories are named. Also export MINI_OS with the same value to be backward compatible. Signed-off-by: Wei Liu --- Makefile | 29 +++-- 1 file changed, 15 insertions(+), 14 deletions(-) diff --git a/Makefile b/Makefile index f3f5c0

[Xen-devel] [PATCH RFC v1 0/6] xen: split stubdom from xen.git

2016-03-10 Thread Wei Liu
git://xenbits.xen.org/people/liuw/xen.git wip.split-stubdom-v1 Wei Liu (6): config: don't hard-code mini-os path tools: use "?=" to set XEN_ROOT in Makefile libxc: use $XEN_ROOT to reference libelf in xen libxl: generate and install stubdompath.sh Unhook stubdom from build system Remov

[Xen-devel] [PATCH RFC v1 09/14] Makefile: delete STUBDOMPATH target

2016-03-10 Thread Wei Liu
It is used to generate a script used by toolstack to launch stubdom. It relies paths generated when building Xen, so it should be part of xen.git. Signed-off-by: Wei Liu --- Makefile | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/Makefile b/Makefile index 875

  1   2   >