The xen store preserves this information across module invocations.
If you insmod netfront with two queues and later insmod again with one
queue, the backend will still believe you asked for two queues.
Signed-off-by: Chas Williams <3ch...@gmail.com>
---
drivers/net/xen-netfront.c | 12 +++---
El 07/09/15 a les 12.05, Jan Beulich ha escrit:
On 07.09.15 at 11:34, wrote:
>> So AFAICS we have 3 options:
>>
>> 1. Overload VCPUOP_initialise like it's done in the current series (v6).
>> For PV guests the hypercall parameter is of type vcpu_guest_context,
>> while for HVM guests the param
... where possible. This is in response to
http://lists.xenproject.org/archives/html/xen-devel/2015-09/msg01502.html,
albeit only the first three patches are really needed for that (the
fourth one just leverages what the first two do in another way).
The third (main) patch is RFC for a number of re
>>> On 15.09.15 at 09:08, wrote:
> El 07/09/15 a les 12.05, Jan Beulich ha escrit:
> On 07.09.15 at 11:34, wrote:
>>> So AFAICS we have 3 options:
>>>
>>> 1. Overload VCPUOP_initialise like it's done in the current series (v6).
>>> For PV guests the hypercall parameter is of type vcpu_guest_c
>>> On 15.09.15 at 09:13, wrote:
> ... where possible. This is in response to
> http://lists.xenproject.org/archives/html/xen-devel/2015-09/msg01502.html,
> albeit only the first three patches are really needed for that (the
> fourth one just leverages what the first two do in another way).
> The
This is so that callers can determine what range of address space would
get altered by a corresponding "set".
Signed-off-by: Jan Beulich
--- a/xen/arch/x86/mm/p2m-ept.c
+++ b/xen/arch/x86/mm/p2m-ept.c
@@ -879,7 +879,13 @@ static mfn_t ept_get_entry(struct p2m_do
/* This pfn is higher than
This is so that callers can determine what range of address space would
get altered by a corresponding "set".
Signed-off-by: Jan Beulich
--- a/xen/arch/x86/mm/p2m-pt.c
+++ b/xen/arch/x86/mm/p2m-pt.c
@@ -715,14 +715,26 @@ p2m_pt_get_entry(struct p2m_domain *p2m,
*a = p2m_access_rwx;
When mapping large BARs (e.g. the frame buffer of a graphics card) the
overhead or establishing such mappings using onle 4k pages has,
particularly after the XSA-125 fix, become unacceptable. Alter the
XEN_DOMCTL_memory_mapping semantics once again, so that there's no
longer a fixed amount of guest
Please disregard this - hit "send" too early.
>>> On 15.09.15 at 09:31, wrote:
On 15.09.15 at 09:13, wrote:
>> ... where possible. This is in response to
>> http://lists.xenproject.org/archives/html/xen-devel/2015-09/msg01502.html,
>> albeit only the first three patches are really needed fo
Now that p2m->get_entry() always returns a valid order, utilize this
to accelerate some of the operations in PoD code. (There are two uses
of p2m->get_entry() left which don't easily lend themselves to this
optimization.)
Also adjust a few types as needed and remove stale comments from
p2m_pod_cac
flight 61883 qemu-mainline real [real]
http://logs.test-lab.xenproject.org/osstest/logs/61883/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-armhf-armhf-xl-multivcpu 16 guest-start/debian.repeat fail REGR. vs. 61666
test-amd64-amd64-x
flight 61890 libvirt real [real]
http://logs.test-lab.xenproject.org/osstest/logs/61890/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 16
guest-start/debianhvm.repeat fail REGR. vs. 61770
>>> On 9/11/2015 at 09:26 PM, in message <1441978018.3549.33.ca...@citrix.com>,
>>> Ian
Campbell wrote:
> On Thu, 2015-09-10 at 23:42 -0600, Chun Yan Liu wrote:
> >
> > > Do these fields have any particular size requirements arising from e.g.
> the
> > > USB spec or from possible dom0 i
Remove trailing spaces in xen-mceinj.c.
Signed-off-by: Haozhong Zhang
---
tools/tests/mce-test/tools/xen-mceinj.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/tools/tests/mce-test/tools/xen-mceinj.c
b/tools/tests/mce-test/tools/xen-mceinj.c
index e2e49cb.
The existing xen-mceinj can not inject MCE through MSR_MCI_ADDR to a
domain w/ more than 4GB memory, e.g. if domain 0 has more than 4GB
memory, the execution of the command
xen-mceinj -d 0 -t 0 -p 0x2721a900
will fail w/ a message "Failed to get pfn list : Operation not
supported".
This patch removes the address translation in xen-mceinj which
translates the guest physical address passed-in through the argument
of '-p' to the host machine address.
Signed-off-by: Haozhong Zhang
---
tools/tests/mce-test/tools/xen-mceinj.c | 135 +---
1 file change
Remove trailing spaces and fix indentations in mce.c and xen-mca.h.
Signed-off-by: Haozhong Zhang
---
xen/arch/x86/cpu/mcheck/mce.c | 10 +-
xen/include/public/arch-x86/xen-mca.h | 28 ++--
2 files changed, 19 insertions(+), 19 deletions(-)
diff --git a/x
This patch adds a new flag MC_MSRINJ_F_GPADDR to
xen_mc_msrinject.mcinj_flags, and makes do_mca() to translate the
guest physical address passed-in through
xen_mc_msrinject.mcinj_msr[i].value to the host machine address if
this flag is present.
Signed-off-by: Haozhong Zhang
---
xen/arch/x86/cpu/
On Mon, 2015-09-14 at 17:15 +0100, Wei Liu wrote:
> FYI all other patches of this series were applied by Jan. You only need
> to resend this one.
Jan,
I appreciate you do not want to do so routinely for all commits you make
but when you are partially applying a series it would be very useful if y
On Tue, 2015-09-15 at 00:08 +0100, Julien Grall wrote:
> On 14/09/2015 16:58, Will Deacon wrote:
> > > The Xen ctxt switch code[0] has DACR_EL2 in the midst of it all, and
> > > certainly followed by some sysregs, which I've got my fingers crossed
> > > happens to be sufficient (other than maybe ad
On Thu, 2015-09-10 at 10:32 +0100, Ian Campbell wrote:
> The Chardonnay case suggests that either something has been backported into
> 3.4.x which has broken things (current real flights, which reliably fail,
> are running on 3.4.108) or that it is simply unreliable (or both). I think
> I need to r
On 2015/09/15 10:29, Haozhong Zhang wrote:
> Remove trailing spaces and fix indentations in mce.c and xen-mca.h.
>
> Signed-off-by: Haozhong Zhang
Acked-by: Christoph Egger
> ---
> xen/arch/x86/cpu/mcheck/mce.c | 10 +-
> xen/include/public/arch-x86/xen-mca.h | 28
flight 37928 distros-debian-squeeze real [real]
http://osstest.xs.citrite.net/~osstest/testlogs/logs/37928/
Perfect :-)
All tests in this flight passed
baseline version:
flight 37917
jobs:
build-amd64 pass
build-armhf
On 2015/09/15 10:29, Haozhong Zhang wrote:
> Remove trailing spaces in xen-mceinj.c.
>
> Signed-off-by: Haozhong Zhang
Acked-by: Christoph Egger
> ---
> tools/tests/mce-test/tools/xen-mceinj.c | 16
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/tools/tes
On Mon, Sep 14, 2015 at 7:33 PM, Olaf Hering wrote:
> On Mon, Sep 14, George Dunlap wrote:
>
>> Well if you "know nothing about SELinux", and you don't use it, and
>> don't have any test systems that use it, then why did you assert
>> "The proper place to specify [an SELinux mount context] is /etc
When constructing an adhoc test it may be useful to copy an existing
job's configuration but run it with a custom recipe.
Signed-off-by: Ian Campbell
---
cs-adjust-flight | 16
1 file changed, 16 insertions(+)
diff --git a/cs-adjust-flight b/cs-adjust-flight
index 9e011c6..662b
By reading sg-run-job-adhoc if it exists.
Signed-off-by: Ian Campbell
---
sg-run-job | 4
1 file changed, 4 insertions(+)
diff --git a/sg-run-job b/sg-run-job
index 0b0449b..c51a508 100755
--- a/sg-run-job
+++ b/sg-run-job
@@ -370,6 +370,10 @@ proc run-job/test-rumpuserxen {} {
While I was doing some adhoc testing on chardonnay I needed to define some
adhoc recipes and ended up writing the following two patches.
In the end I didn't end up using the first (I just created the job out of
whole cloth rather than copying an existing template and modifying).
For the second I
On Mon, 2015-09-14 at 07:53 -0600, Jan Beulich wrote:
> While "ioport" list element parsing already validates that the entire
> input string got consumed, its two siblings so far didn't.
>
> Signed-off-by: Jan Beulich
>
Reviewed-by: Dario Faggioli
Regards,
Dario
--
<> (Raistlin Majere)
--
On Tue, Sep 15, 2015 at 09:17:07AM +0800, Chen, Tiejun wrote:
> >>But looks its not better, so any idea?
> >
> >Did you at least make an attempt to find other examples of where
> >we dynamically determine the log level to be used for a message?
> >I would assume that if you did, you'd have come to
On 2015/09/15 10:29, Haozhong Zhang wrote:
> This patch adds a new flag MC_MSRINJ_F_GPADDR to
> xen_mc_msrinject.mcinj_flags, and makes do_mca() to translate the
> guest physical address passed-in through
> xen_mc_msrinject.mcinj_msr[i].value to the host machine address if
> this flag is present.
>
Hello,
This series adds two minor patches. The first one allows finer-grained
control over the emulation behaviour of REP instructions. Previously,
once vm_event-based emulation was enabled, no optimizations were allowed.
However, this has a performance impact on the monitored guest, so I've
added
Previously, if vm_event emulation support was enabled, then REP
optimizations were disabled when emulating REP-compatible
instructions. This patch allows fine-tuning of this behaviour by
providing a dedicated libxc helper function.
Signed-off-by: Razvan Cojocaru
---
tools/libxc/include/xenctrl.h
A previous version of this patch dealing with support for skipping
the current instruction when a vm_event response requested it
computed the instruction length in the hypervisor, adding non-trivial
code dependencies. This patch allows a userspace vm_event client to
simply request that the guest's
On Mon, 2015-09-14 at 10:50 +0100, George Dunlap wrote:
> On Thu, Sep 10, 2015 at 1:36 PM, Jan Beulich wrote:
> > While it appears to be intentional for "xl pci-assignable-remove" to
> > not re-bind the original driver by default (requires the -r option),
> > permanently losing the information whi
On Mon, 2015-09-14 at 07:53 -0600, Jan Beulich wrote:
> While "ioport" list element parsing already validates that the entire
> input string got consumed, its two siblings so far didn't.
>
> Signed-off-by: Jan Beulich
Acked-by: Ian Campbell
___
Xen-
On Mon, 2015-09-14 at 16:30 +0100, Julien Grall wrote:
> Signed-off-by: Julien Grall
Acked-by: Ian Campbell
___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
On Mon, 2015-09-14 at 16:32 +0100, Julien Grall wrote:
> GICR_PIDR2 and GICD_PIDR2 use the same register layout. Rather than
> define twice, one of which is an alias to the other, introduce
> GIC_PIDR2_*
> defines.
>
> Also:
> * Use the same prefix for the mask and the value
> * Integrate
The spatch provided in previous patch didn't handle all sites that need
transformation.
Signed-off-by: Wei Liu
---
tools/libxl/libxl.c| 17 -
tools/libxl/libxl_create.c | 4 ++--
tools/libxl/libxl_pci.c| 39 ++-
tools/libxl/libxl_x
There is code in libxl using XTL_VERBOSE. We should provide a libxl
mapping for it.
Signed-off-by: Wei Liu
---
tools/libxl/libxl_internal.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h
index 5fa55a7..070b7d3 100644
--- a/tools/li
There are mixed usage of different logging macros. Ideally we only use one
style to avoid confusion.
Wei Liu (4):
libxl: convert to use LOG() macro
libxl: fix overly lines and delete extraneous quotes
libxl: map LIBXL__LOG_VERBOSE to XTL_VERBOSE
libxl: fix places missed by spatch
tools/l
Signed-off-by: Wei Liu
---
tools/libxl/libxl.c | 21 +
tools/libxl/libxl_utils.c | 8 ++--
2 files changed, 19 insertions(+), 10 deletions(-)
diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c
index ce8965d..9b2047f 100644
--- a/tools/libxl/libxl.c
+++ b/tools/
This patch converts most LIBXL__LOG* macros to LOG macro. It's done with
spatch plus some hand coding.
Using spatch rune:
spatch --in-place --no-includes --include-headers \
--sp-file libxl.spatch \
tools/libxl/libxl*.c
with some exceptions.
libxl_json.c is untouched because
On Tue, Sep 15, 2015 at 11:14:26AM +0200, Egger, Christoph wrote:
> On 2015/09/15 10:29, Haozhong Zhang wrote:
> > This patch adds a new flag MC_MSRINJ_F_GPADDR to
> > xen_mc_msrinject.mcinj_flags, and makes do_mca() to translate the
> > guest physical address passed-in through
> > xen_mc_msrinject
On Mon, 2015-09-14 at 16:32 +0100, Julien Grall wrote:
> It seems that there is some hardware which report start to report GICv4
s/report start to reports/reports/ ?
Also, this is an odd way to express it, what you mean is that some hardware
is now shipping with GICv4. Unless you are trying to im
On Tue, 2015-09-15 at 12:19 +0300, Razvan Cojocaru wrote:
> Previously, if vm_event emulation support was enabled, then REP
> optimizations were disabled when emulating REP-compatible
> instructions. This patch allows fine-tuning of this behaviour by
> providing a dedicated libxc helper function.
>
The current libxl code doesn't deal with read-only drives at all.
Upstream QEMU and qemu-xen only support read-only cdrom drives: make
sure to specify "readonly=on" for cdrom drives and return error in case
the user requested a non-cdrom read-only drive.
This is XSA-142, discovered by Lin Liu
(ht
On Mon, 14 Sep 2015, Paolo Bonzini wrote:
> On 10/09/2015 12:29, Stefano Stabellini wrote:
> > +if (lseek(config_fd, pos, SEEK_SET) != pos) {
> > +return -errno;
> > +}
> > do {
> > -rc = pread(config_fd, (uint8_t *)&val, len, pos);
> > +rc = read(config_fd, (ui
I don't know this piece of code so my comments might be stupid.
On Tue, Sep 15, 2015 at 04:29:40PM +0800, Haozhong Zhang wrote:
> This patch removes the address translation in xen-mceinj which
> translates the guest physical address passed-in through the argument
> of '-p' to the host machine addr
CC Konrad
On Mon, 14 Sep 2015, Paolo Bonzini wrote:
> On 10/09/2015 19:15, Stefano Stabellini wrote:
> > +
> > +switch (reg->size) {
> > +case 1: rc = xen_host_pci_get_byte(&s->real_device, offset,
> > (uint8_t *)&val);
>
> A bit ugly, and it relies on the host being little endia
On 2015/09/15 12:02, Wei Liu wrote:
> I don't know this piece of code so my comments might be stupid.
>
> On Tue, Sep 15, 2015 at 04:29:40PM +0800, Haozhong Zhang wrote:
>> This patch removes the address translation in xen-mceinj which
>> translates the guest physical address passed-in through the
On Tue, Sep 15, 2015 at 12:08:32PM +0200, Egger, Christoph wrote:
> On 2015/09/15 12:02, Wei Liu wrote:
> > I don't know this piece of code so my comments might be stupid.
> >
> > On Tue, Sep 15, 2015 at 04:29:40PM +0800, Haozhong Zhang wrote:
> >> This patch removes the address translation in xen
On Tue, 2015-09-15 at 10:48 +0200, Egger, Christoph wrote:
> On 2015/09/15 10:29, Haozhong Zhang wrote:
> > Remove trailing spaces in xen-mceinj.c.
> >
> > Signed-off-by: Haozhong Zhang
>
> Acked-by: Christoph Egger
Acked-by: Ian Campbell
___
Xen-
On 2015/09/15 10:29, Haozhong Zhang wrote:
> This patch removes the address translation in xen-mceinj which
> translates the guest physical address passed-in through the argument
> of '-p' to the host machine address.
>
> Signed-off-by: Haozhong Zhang
Comments inline.
> ---
> tools/tests/mce-t
On Tue, 2015-09-15 at 16:29 +0800, Haozhong Zhang wrote:
> This patch removes the address translation in xen-mceinj which
> translates the guest physical address passed-in through the argument
> of '-p' to the host machine address.
I think there is a missing/implied "... and replaces it with $some
On Mon, 2015-08-03 at 12:29 +0100, Ian Campbell wrote:
> From: David Vrabel
>
> Instead of cpu_relax() while spinning and observing the ticket head,
> introduce arch_lock_relax() which executes a WFE instruction. After
> the ticket head is changed call arch_lock_signal() to execute an SEV
> inst
On Tue, Sep 15, 2015 at 12:17:01PM +0200, Egger, Christoph wrote:
> On 2015/09/15 10:29, Haozhong Zhang wrote:
> > This patch removes the address translation in xen-mceinj which
> > translates the guest physical address passed-in through the argument
> > of '-p' to the host machine address.
> >
> >
On 09/09/15 20:34, David Miller wrote:
> From: David Vrabel
> Date: Tue, 8 Sep 2015 14:25:14 +0100
>
>> Commit f48da8b14d04ca87ffcffe68829afd45f926ec6a (xen-netback: fix
>> unlimited guest Rx internal queue and carrier flapping) introduced a
>> regression.
>>
>> The PV frontend in IPXE only place
Using sh to exec xenstored breaks on selinux systems (at least, on
CentOS 7). The only purpose of doing that was to be able to expand
the $XENSTORED variable; this can be done with /usr/bin/env instead,
which still works on systemd systems (at least on CentOS 7).
Suggested-by: Olaf Hering
Signed
Hi,
On 15/09/15 10:49, Ian Campbell wrote:
> On Mon, 2015-09-14 at 16:32 +0100, Julien Grall wrote:
>> It seems that there is some hardware which report start to report GICv4
>
> s/report start to reports/reports/ ?
>
> Also, this is an odd way to express it, what you mean is that some hardware
On 2015/09/15 12:28, Haozhong Zhang wrote:
> On Tue, Sep 15, 2015 at 12:17:01PM +0200, Egger, Christoph wrote:
>> On 2015/09/15 10:29, Haozhong Zhang wrote:
>>> This patch removes the address translation in xen-mceinj which
>>> translates the guest physical address passed-in through the argument
>>
On Tue, Sep 15, 2015 at 11:30:00AM +0100, George Dunlap wrote:
> Using sh to exec xenstored breaks on selinux systems (at least, on
> CentOS 7). The only purpose of doing that was to be able to expand
> the $XENSTORED variable; this can be done with /usr/bin/env instead,
> which still works on sys
On Tue, Sep 15, 2015 at 11:20:20AM +0100, Ian Campbell wrote:
> On Tue, 2015-09-15 at 16:29 +0800, Haozhong Zhang wrote:
> > This patch removes the address translation in xen-mceinj which
> > translates the guest physical address passed-in through the argument
> > of '-p' to the host machine addres
On Tue, 15 Sep 2015, Ian Campbell wrote:
> On Mon, 2015-08-03 at 12:29 +0100, Ian Campbell wrote:
> > From: David Vrabel
> >
> > Instead of cpu_relax() while spinning and observing the ticket head,
> > introduce arch_lock_relax() which executes a WFE instruction. After
> > the ticket head is cha
On 15/09/2015 11:55, Stefano Stabellini wrote:
> On Mon, 14 Sep 2015, Paolo Bonzini wrote:
>> > On 10/09/2015 12:29, Stefano Stabellini wrote:
>>> > > +if (lseek(config_fd, pos, SEEK_SET) != pos) {
>>> > > +return -errno;
>>> > > +}
>>> > > do {
>>> > > -rc = pread(co
On 14/09/15 22:28, Charles (Chas) Williams wrote:
> The xen store preserves this information across module invocations.
> If you insmod netfront with two queues and later insmod again with one
> queue, the backend will still believe you asked for two queues.
Can you rewrite the commit message to b
On 15/09/15 10:49, Ian Campbell wrote:
> On Mon, 2015-09-14 at 16:32 +0100, Julien Grall wrote:
>> It seems that there is some hardware which report start to report GICv4
>
> s/report start to reports/reports/ ?
Yes, reports.
> Also, this is an odd way to express it, what you mean is that some h
On Tue, 2015-09-15 at 09:47 +0100, Ian Campbell wrote:
> On Thu, 2015-09-10 at 10:32 +0100, Ian Campbell wrote:
> > The Chardonnay case suggests that either something has been backported
> > into
> > 3.4.x which has broken things (current real flights, which reliably
> > fail,
> > are running on 3.
On 09/15/2015 11:30 AM, George Dunlap wrote:
> Using sh to exec xenstored breaks on selinux systems (at least, on
> CentOS 7). The only purpose of doing that was to be able to expand
> the $XENSTORED variable; this can be done with /usr/bin/env instead,
> which still works on systemd systems (at l
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 normally on a GICv4 hardware.
>
> Though, the GICv4-only feature won't be used."
I've a few grammatical quibbles, pl
On Tue, 2015-09-15 at 10:31 +0100, Ian Campbell wrote:
> On Mon, 2015-09-14 at 10:50 +0100, George Dunlap wrote:
> > On Thu, Sep 10, 2015 at 1:36 PM, Jan Beulich wrote:
> > > While it appears to be intentional for "xl pci-assignable-remove" to
> > > not re-bind the original driver by default (requ
On Tue, 2015-09-15 at 11:09 +0200, Dario Faggioli wrote:
> On Mon, 2015-09-14 at 07:53 -0600, Jan Beulich wrote:
> > While "ioport" list element parsing already validates that the entire
> > input string got consumed, its two siblings so far didn't.
> >
> > Signed-off-by: Jan Beulich
> >
> Revie
On Tue, 2015-09-15 at 10:32 +0100, Ian Campbell wrote:
> On Mon, 2015-09-14 at 16:30 +0100, Julien Grall wrote:
> > Signed-off-by: Julien Grall
>
> Acked-by: Ian Campbell
Applied. I wish I'd spotted the typo in the title first though :-/
Ian.
___
Xe
On Tue, 2015-08-11 at 09:07 -0600, Jan Beulich wrote:
> > > > On 03.08.15 at 13:29, wrote:
> > From: David Vrabel
> >
> > Instead of cpu_relax() while spinning and observing the ticket head,
> > introduce arch_lock_relax() which executes a WFE instruction. After
> > the ticket head is changed c
On Tue, 2015-09-15 at 11:39 +0100, Stefano Stabellini wrote:
> On Tue, 15 Sep 2015, Ian Campbell wrote:
> > On Mon, 2015-08-03 at 12:29 +0100, Ian Campbell wrote:
> > > From: David Vrabel
> > >
> > > Instead of cpu_relax() while spinning and observing the ticket head,
> > > introduce arch_lock_re
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 normally on a GICv4 hardware.
>>
>> Though, the GICv4-only feature won't b
On Tue, Sep 15, George Dunlap wrote:
> It's very reasonable for you to expect it to be fixed on non-SELinux
> systems. But what you did is fix it for non-SELinux systems by simply
> breaking it on SELinux systems -- that's not at all reasonable.
Konrad did some testing at that time and said 4.5
On Tue, Sep 15, 2015 at 1:48 PM, Olaf Hering wrote:
> On Tue, Sep 15, George Dunlap wrote:
>
>> It's very reasonable for you to expect it to be fixed on non-SELinux
>> systems. But what you did is fix it for non-SELinux systems by simply
>> breaking it on SELinux systems -- that's not at all reas
>>> On 15.09.15 at 10:38, wrote:
> On Mon, 2015-09-14 at 17:15 +0100, Wei Liu wrote:
>> FYI all other patches of this series were applied by Jan. You only need
>> to resend this one.
>
> I appreciate you do not want to do so routinely for all commits you make
> but when you are partially applying
This is to guard against buggy callers (luckily Dom0 only) invoking
the respective hypercall for a device not being MSI-capable.
Signed-off-by: Jan Beulich
---
While for a well behaved Dom0 this is a benign change, I still think
this should go into 4.6.
--- a/xen/arch/x86/msi.c
+++ b/xen/arch/x8
On Tue, Sep 15, 2015 at 04:29:38PM +0800, Haozhong Zhang wrote:
> Remove trailing spaces in xen-mceinj.c.
>
> Signed-off-by: Haozhong Zhang
Acked-by: Wei Liu
___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
On 15/09/15 14:07, Jan Beulich wrote:
> This is to guard against buggy callers (luckily Dom0 only) invoking
> the respective hypercall for a device not being MSI-capable.
>
> Signed-off-by: Jan Beulich
Reviewed-by: Andrew Cooper
___
Xen-devel mailing
On Tue, Sep 15, 2015 at 07:07:37AM -0600, Jan Beulich wrote:
> This is to guard against buggy callers (luckily Dom0 only) invoking
> the respective hypercall for a device not being MSI-capable.
>
> Signed-off-by: Jan Beulich
> ---
> While for a well behaved Dom0 this is a benign change, I still t
>>> On 15.09.15 at 10:29, wrote:
> --- a/xen/arch/x86/cpu/mcheck/mce.c
> +++ b/xen/arch/x86/cpu/mcheck/mce.c
> @@ -105,7 +105,7 @@ void x86_mce_callback_register(x86_mce_callback_t cbfunc)
> mc_callback_bank_extended = cbfunc;
> }
>
> -/* Machine check recoverable judgement callback handle
On 14/09/15 12:32, Arnd Bergmann wrote:
> On Monday 14 September 2015 13:04:59 Roger Pau Monné wrote:
>>> TBH, I'm expecting a small impact to the performance. It would be hard
>>> to get the exactly the same performance as today if we keep the helpers
>>> to avoid the backend dealing himself with
On Tue, Sep 15, 2015 at 07:13:16AM -0600, Jan Beulich wrote:
> >>> On 15.09.15 at 10:29, wrote:
> > --- a/xen/arch/x86/cpu/mcheck/mce.c
> > +++ b/xen/arch/x86/cpu/mcheck/mce.c
> > @@ -105,7 +105,7 @@ void x86_mce_callback_register(x86_mce_callback_t
> > cbfunc)
> > mc_callback_bank_extended
On Tuesday 15 September 2015 14:14:09 David Vrabel wrote:
> On 14/09/15 12:32, Arnd Bergmann wrote:
> > On Monday 14 September 2015 13:04:59 Roger Pau Monné wrote:
> >>> TBH, I'm expecting a small impact to the performance. It would be hard
> >>> to get the exactly the same performance as today if
>>> On 15.09.15 at 10:29, wrote:
> @@ -1422,6 +1423,38 @@ long do_mca(XEN_GUEST_HANDLE_PARAM(xen_mc_t) u_xen_mc)
> if (mc_msrinject->mcinj_count == 0)
> return 0;
>
> +if (mc_msrinject->mcinj_flags & MC_MSRINJ_F_GPADDR) {
Perhaps you earlier patch fixing coding sty
On Tue, Sep 15, 2015 at 11:07:02AM +0100, Stefano Stabellini wrote:
> CC Konrad
>
> On Mon, 14 Sep 2015, Paolo Bonzini wrote:
> > On 10/09/2015 19:15, Stefano Stabellini wrote:
> > > +
> > > +switch (reg->size) {
> > > +case 1: rc = xen_host_pci_get_byte(&s->real_device, offset,
>
On Tue, 15 Sep 2015, Konrad Rzeszutek Wilk wrote:
> On Tue, Sep 15, 2015 at 11:07:02AM +0100, Stefano Stabellini wrote:
> > CC Konrad
> >
> > On Mon, 14 Sep 2015, Paolo Bonzini wrote:
> > > On 10/09/2015 19:15, Stefano Stabellini wrote:
> > > > +
> > > > +switch (reg->size) {
> > > > +
On 15/09/2015 15:28, Stefano Stabellini wrote:
>
>> > 2). Send an follow up patch to change this to abort()? (and wherever else
>> > I used
>> > assert(..)?
> Yes, that would be good.
>
>
>> > 3). Wait till Paolo is done going through the patchset and then revisit
>> > 1) or 2)?
> I d
On Tue, Sep 15, 2015 at 07:24:34AM -0600, Jan Beulich wrote:
> >>> On 15.09.15 at 10:29, wrote:
> > @@ -1422,6 +1423,38 @@ long do_mca(XEN_GUEST_HANDLE_PARAM(xen_mc_t)
> > u_xen_mc)
> > if (mc_msrinject->mcinj_count == 0)
> > return 0;
> >
> > +if (mc_msrinject->mcin
flight 61892 linux-3.4 real [real]
http://logs.test-lab.xenproject.org/osstest/logs/61892/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-xl-qemuu-debianhvm-amd64 6 xen-boot fail REGR. vs. 30511
test-amd64-i386-xl-qemu
>>> On 15.09.15 at 15:42, wrote:
> On Tue, Sep 15, 2015 at 07:24:34AM -0600, Jan Beulich wrote:
>> >>> On 15.09.15 at 10:29, wrote:
>> > +mfn = mfn_x(get_gfn(d, gpfn, &t));
>> > +
>> > +if (mfn == INVALID_MFN) {
>> > +put_domain(d);
>> > +
On Tue, Sep 15, 2015 at 09:42:40AM -0400, Konrad Rzeszutek Wilk wrote:
> Hey,
CC-ing Xen-devel which I forgot.
>
> We are running into issues where our guests immediately exhaust
> the amount of PIRQs. Using the 'extra_guest_irqs' of course fixes
> it but it would be nice to have this be dynamic.
On Tue, Sep 15, 2015 at 07:47:49AM -0600, Jan Beulich wrote:
> >>> On 15.09.15 at 15:42, wrote:
> > On Tue, Sep 15, 2015 at 07:24:34AM -0600, Jan Beulich wrote:
> >> >>> On 15.09.15 at 10:29, wrote:
> >> > +mfn = mfn_x(get_gfn(d, gpfn, &t));
> >> > +
> >> > +if (mf
Signed-off-by: Andrew Cooper
---
v3:
* Corrections, and include LIBXL_HAVE_*,
* #Testing and #Reference, following template review
v2:
* %Revision and #History, following template review
* Grammar fixes
---
docs/features/migration.pandoc | 123
1 file
On Tue, Sep 15, 2015 at 02:28:51PM +0100, Stefano Stabellini wrote:
> On Tue, 15 Sep 2015, Konrad Rzeszutek Wilk wrote:
> > On Tue, Sep 15, 2015 at 11:07:02AM +0100, Stefano Stabellini wrote:
> > > CC Konrad
> > >
> > > On Mon, 14 Sep 2015, Paolo Bonzini wrote:
> > > > On 10/09/2015 19:15, Stefano
On 15/09/15 14:42, Haozhong Zhang wrote:
>
>>> +mfn = mfn_x(get_gfn(d, gpfn, &t));
>>> +
>>> +if (mfn == INVALID_MFN) {
>>> +put_domain(d);
>>> +return x86_mcerr("do_mca inject: illegal MSR value",
>>> +
On Tue, Sep 15, 2015 at 02:48:57PM +0200, Olaf Hering wrote:
> On Tue, Sep 15, George Dunlap wrote:
>
> > It's very reasonable for you to expect it to be fixed on non-SELinux
> > systems. But what you did is fix it for non-SELinux systems by simply
> > breaking it on SELinux systems -- that's not
1 - 100 of 187 matches
Mail list logo