Agree with Chao. This is more accurate statement.
We have spent lots of effort on adding vIOMMU support with irq remapping and
should continue after George wraps up vNVDIMM design discussions with us in
this short period.
Best Regards
John Ji
-Original Message-
From: Gao, Chao
Se
>>> On 11.04.18 at 22:21, wrote:
> @@ -243,17 +244,21 @@ static int enter_state(u32 state)
> if ( (state == ACPI_STATE_S3) && error )
> tboot_s3_error(error);
>
> +console_end_sync();
> +
> +error = microcode_resume_cpu(0);
> +if (error && error != -ENOENT)
Missing bla
>>> On 11.04.18 at 23:31, wrote:
> Il Mer 11 Apr 2018, 22:48 Olaf Hering ha scritto:
>
>> On Wed, Apr 11, Dario Faggioli wrote:
>>
>> > It will crash, again, possibly with the same stack trace, but I think
>> > it's worth a try.
>>
>> BUG_ON(__vcpu_on_runq(CSCHED_VCPU(vc)));
>>
>> (XEN) Xen
On 04/11/2018 11:17 PM, Tamas K Lengyel wrote:
> On Wed, Apr 11, 2018 at 12:39 AM, Razvan Cojocaru
> wrote:
>> Also related to this part of the altp2m design, I've also noticed that
>> creating a new EPT view with libxc involves this function:
>>
>> int xc_altp2m_create_view(xc_interface *handle,
>>> On 11.04.18 at 22:32, wrote:
> I was testing 'virsh migrate domU host' and did some libvirtd debugging
> on 'host'. This means the migration was attempted a few times, but did
> not actually start because libvirtd was in gdb. Not sure if libvirt on
> the sender does anything with the domU befo
* Jan Beulich wrote:
> >>> On 11.04.18 at 13:53, wrote:
> > * Jan Beulich wrote:
> >
> >> Additionally, x86 maintainers: is there a particular reason this (or
> >> any functionally equivalent patch) isn't upstream yet? As indicated
> >> before, I had not been able to find any discussion, and
> On Apr 12, 2018, at 2:18 AM, Chao Gao wrote:
>
> On Wed, Apr 11, 2018 at 05:10:40PM +, Lars Kurth wrote:
>> ## Longer Term - Agreed to Pause
>>
>> ### [PATCH v4 00/28] add vIOMMU support with irq remapping function of
>> ### virtual VT-d
>>
>> Sent in for meeting agenda by George
>> v3
Hi Stefano,
v->processor has to be updated before the sched_move_irqs is called. Since
updating v->processor is done after the 'if (v->affinity_broken)' check I
had to put sched_move_irqs below, where it is.
Thanks,
Mirela
On Thu, Apr 12, 2018 at 2:20 AM, Stefano Stabellini
wrote:
> On Wed, 11
>>> On 12.04.18 at 09:32, wrote:
> * Jan Beulich wrote:
>
>> >>> On 11.04.18 at 13:53, wrote:
>> > * Jan Beulich wrote:
>> >
>> >> Additionally, x86 maintainers: is there a particular reason this (or
>> >> any functionally equivalent patch) isn't upstream yet? As indicated
>> >> before, I ha
flight 122170 xen-unstable real [real]
http://logs.test-lab.xenproject.org/osstest/logs/122170/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-armhf-armhf-libvirt 14 saverestore-support-checkfail like 122155
test-armhf-armhf-libvirt-xsm 14 save
On 11/04/18 22:32, Olaf Hering wrote:
> I was testing 'virsh migrate domU host' and did some libvirtd debugging
> on 'host'. This means the migration was attempted a few times, but did
> not actually start because libvirtd was in gdb. Not sure if libvirt on
> the sender does anything with the domU
Jürgen, Boris,
looks like commit 47b02f4c62 ("x86/xen: add tty0 and hvc0 as
preferred consoles for dom0") doesn't get us quite there yet - non-
kernel boot output (and a console prompt) still doesn't appear on
the screen. For now I'm using
--- a/arch/x86/xen/enlighten_pv.c
+++ b/arch/x86/xen/enli
* Jan Beulich wrote:
> >>> On 12.04.18 at 09:32, wrote:
>
> > * Jan Beulich wrote:
> >
> >> >>> On 11.04.18 at 13:53, wrote:
> >> > * Jan Beulich wrote:
> >> >
> >> >> Additionally, x86 maintainers: is there a particular reason this (or
> >> >> any functionally equivalent patch) isn't ups
> On 12 Apr 2018, at 08:32, George Dunlap wrote:
>
>
>
>> On Apr 12, 2018, at 2:18 AM, Chao Gao wrote:
>>
>> On Wed, Apr 11, 2018 at 05:10:40PM +, Lars Kurth wrote:
>>> ## Longer Term - Agreed to Pause
>>>
>>> ### [PATCH v4 00/28] add vIOMMU support with irq remapping function of
>>> #
On 11/04/18 17:37, Mirela Simonovic wrote:
Hi Julien,
Hi,
May I ask you to configure your mail client to use > for quoting and use
plain text? Otherwise, this is going to be really difficult to follow
the discussion after few round (see already below).
On Wed, Apr 11, 2018 at 6:02 PM, Ju
On 12/04/18 00:46, Stefano Stabellini wrote:
On Wed, 11 Apr 2018, Julien Grall wrote:
On 11/04/18 14:19, Mirela Simonovic wrote:
Freeing percpu area is done when a non-boot CPU is disabled upon suspend.
This use to be scheduled for execution after a period of time, what caused
the following r
On 12/04/18 10:01, Juergen Gross wrote:
> On 11/04/18 22:32, Olaf Hering wrote:
>> I was testing 'virsh migrate domU host' and did some libvirtd debugging
>> on 'host'. This means the migration was attempted a few times, but did
>> not actually start because libvirtd was in gdb. Not sure if libvirt
Hi,
On 12/04/18 01:07, Stefano Stabellini wrote:
On Wed, 11 Apr 2018, Mirela Simonovic wrote:
diff --git a/xen/arch/arm/smpboot.c b/xen/arch/arm/smpboot.c
index 5666efcd3a..d15ea8df5e 100644
--- a/xen/arch/arm/smpboot.c
+++ b/xen/arch/arm/smpboot.c
@@ -52,8 +52,8 @@ nodemask_t __read_mostly nod
> On Apr 11, 2018, at 10:31 PM, Dario Faggioli wrote:
>
> Il Mer 11 Apr 2018, 22:48 Olaf Hering ha scritto:
> On Wed, Apr 11, Dario Faggioli wrote:
>
> > It will crash, again, possibly with the same stack trace, but I think
> > it's worth a try.
>
> BUG_ON(__vcpu_on_runq(CSCHED_VCPU(vc))
On Thu, 2018-04-12 at 09:38 +, George Dunlap wrote:
> > On Apr 11, 2018, at 10:31 PM, Dario Faggioli
> > wrote:
> > (XEN) Xen BUG at sched_credit.c:876
> > (XEN) [ Xen-4.11.20180410T125709.50f8ba84a5-
> > 7.bug1087289_411 x86_64 debug=y Not tainted ]
> > (XEN) CPU:108
> > (XEN)
flight 122172 linux-next real [real]
http://logs.test-lab.xenproject.org/osstest/logs/122172/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-xl-shadow 21 guest-start.2fail REGR. vs. 122143
test-armhf-armhf-xl-
Hi,
On Thu, Apr 12, 2018 at 1:31 AM, Stefano Stabellini
wrote:
> On Wed, 11 Apr 2018, Julien Grall wrote:
>> On 11/04/18 14:19, Mirela Simonovic wrote:
>> > Guests attempt to write into these registers on resume (for example Linux).
>> > Without this patch a data abort exception will be raised t
Hi Julien,
On Wed, Apr 11, 2018 at 4:46 PM, Julien Grall wrote:
> Hi,
>
> On 11/04/18 14:19, Mirela Simonovic wrote:
>>
>> This patch adds the PSCI CPU_OFF call to the EL3 in order to
>> trigger powering down of the calling CPU when the CPU is stopped.
>> If CPU_OFF call fails for some reason, e.
Older gcc doesn't like "#pragma GCC diagnostic" inside functions.
Signed-off-by: Jan Beulich
--- a/tools/debugger/kdd/kdd.c
+++ b/tools/debugger/kdd/kdd.c
@@ -695,10 +695,10 @@ static void kdd_handle_read_ctrl(kdd_sta
KDD_LOG(s, "Request outside of known control space\n");
We are going to want to format SUPPORT.md which does not match the
filename patterns in docs/. So provide a way to make an ad-hoc rule
using pandoc with the standard options.
No functional change in this patch.
Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Acked-by: Lars Kurth
--
This series provides code to generate a feature support matrix, to
replace the one on the wiki. You can see an example of the output
here:
https://xenbits.xen.org/people/iwj/2018/support-matrix-example-v3a/t.html
https://xenbits.xen.org/people/iwj/2018/support-matrix-example-v3b/t.html
There i
Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Acked-by: George Dunlap
Acked-by: Lars Kurth
---
SUPPORT.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/SUPPORT.md b/SUPPORT.md
index 1c5220b..e447069 100644
--- a/SUPPORT.md
+++ b/SUPPORT.md
@@ -360,7 +360,7 @@
This archaeology script:
- figures out what the current and previous Xen versions were
- looks for appropriate git branches for them
- finds SUPPORT.md for each one
- feeds its findings to parse-support-md
We do not intend to integrate this into docs/Makefile, because it
relies on the git hist
Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Acked-by: Lars Kurth
---
docs/Makefile | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/docs/Makefile b/docs/Makefile
index d82463f..b300bb6 100644
--- a/docs/Makefile
+++ b/docs/Makefile
@@ -28,7 +28,8 @@ DOC_MAN
Continuations of bullet list items must be indented by exactly 4
spaces (according to pandoc_markdown(5) on Debian jessie).
This is most easily achieved by making the bullet list items have two
spaces before the `*'.
Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Acked-by: George Du
This commits (more or less) this file to be processed with pandoc,
rather than other markdown processors. There is, unfortunately, no
widely-accepted way to declare a title for the document.
I tested feeding the document to markdown(1) on Debian jessie and it
reproduced the % line as if it were s
A section in the SUPPORT.md may mention multiple
Status, something: Supported
and then have some text. The text is linked to from [*] footnotes
in the table. But, this means that each bit of text needs to
apply to multiple rows.
Before this commit this was a separate [*] after each applicable
Otherwise paste [*] right onto the end.
I'm not sure if this is desirable.
Signed-off-by: Ian Jackson
---
v3: New patch
---
docs/parse-support-md | 21 +++--
1 file changed, 15 insertions(+), 6 deletions(-)
diff --git a/docs/parse-support-md b/docs/parse-support-md
index 38c832
This utility reads json format pandoc output, from parsing one or more
SUPPORT.md files, and generates an HTML table element containing the
principal version and feature information.
This is rather hairier than I anticipated when I started out; hence
the 400-odd-line Perl script.
Machinery to ass
Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Acked-by: Lars Kurth
---
docs/gen-html-index | 13 +
1 file changed, 13 insertions(+)
diff --git a/docs/gen-html-index b/docs/gen-html-index
index e9792bf..5b43b42 100644
--- a/docs/gen-html-index
+++ b/docs/gen-html-index
There are none yet.
Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Acked-by: Lars Kurth
---
docs/gen-html-index | 4
1 file changed, 4 insertions(+)
diff --git a/docs/gen-html-index b/docs/gen-html-index
index 5b43b42..8258e2b 100644
--- a/docs/gen-html-index
+++ b/docs/gen-h
Am Thu, 12 Apr 2018 12:16:34 +0200
schrieb Dario Faggioli :
> Olaf, new patch. Please, remove _everything_ and apply _only_ this one.
dies after the first iteration.
BUG_ON(!test_bit(_VPF_migrating, &prev->pause_flags));
(XEN) Xen BUG at schedule.c:1570
(XEN) [ Xen-4.11.20180411T100
Hi,
On Thu, Apr 12, 2018 at 11:03 AM, Julien Grall wrote:
> Hi,
>
> On 12/04/18 01:07, Stefano Stabellini wrote:
>>
>> On Wed, 11 Apr 2018, Mirela Simonovic wrote:
>>>
>>> diff --git a/xen/arch/arm/smpboot.c b/xen/arch/arm/smpboot.c
>>> index 5666efcd3a..d15ea8df5e 100644
>>> --- a/xen/arch/arm/s
Hi,
On 12/04/18 13:50, Mirela Simonovic wrote:
Hi,
On Thu, Apr 12, 2018 at 11:03 AM, Julien Grall wrote:
Hi,
On 12/04/18 01:07, Stefano Stabellini wrote:
On Wed, 11 Apr 2018, Mirela Simonovic wrote:
diff --git a/xen/arch/arm/smpboot.c b/xen/arch/arm/smpboot.c
index 5666efcd3a..d15ea8df5e
On Thu, 2018-04-12 at 14:45 +0200, Olaf Hering wrote:
> Am Thu, 12 Apr 2018 12:16:34 +0200
> schrieb Dario Faggioli :
>
> > Olaf, new patch. Please, remove _everything_ and apply _only_ this
> > one.
>
> dies after the first iteration.
>
> BUG_ON(!test_bit(_VPF_migrating, &prev->pause_fl
On 12/04/18 14:28, Ian Jackson wrote:
> A section in the SUPPORT.md may mention multiple
>Status, something: Supported
> and then have some text. The text is linked to from [*] footnotes
> in the table. But, this means that each bit of text needs to
> apply to multiple rows.
>
> Before this
On 12/04/18 14:28, Ian Jackson wrote:
> Otherwise paste [*] right onto the end.
>
> I'm not sure if this is desirable.
>
> Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Juergen
___
Xen-devel mailing list
Xen-devel@lists.xenproject.or
On 12/04/18 12:33, Mirela Simonovic wrote:
On Wed, Apr 11, 2018 at 4:46 PM, Julien Grall wrote:
On 11/04/18 14:19, Mirela Simonovic wrote:
local_irq_disable();
cpu_is_dead = true;
/* Make sure the write happens before we sleep forever */
dsb(sy);
isb();
+
Hi Julien,
On Thu, Apr 12, 2018 at 2:56 PM, Julien Grall wrote:
> Hi,
>
> On 12/04/18 13:50, Mirela Simonovic wrote:
>>
>> Hi,
>>
>> On Thu, Apr 12, 2018 at 11:03 AM, Julien Grall
>> wrote:
>>>
>>> Hi,
>>>
>>> On 12/04/18 01:07, Stefano Stabellini wrote:
On Wed, 11 Apr 2018, Mirel
flight 122178 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/122178/
Perfect :-)
All tests in this flight passed as required
version targeted for testing:
ovmf 153f5c7a93be09403891404c06e5b0e24eb019a3
baseline version:
ovmf 8b0e67821bd66af70433e
Hi Peng,
Sorry for late response, this email got buried and I accidentally saw it now.
On Thu, Apr 12, 2018 at 4:26 AM, Peng Fan wrote:
> Hi Edgar,
>
>> -Original Message-
>> From: Edgar E. Iglesias [mailto:edgar.igles...@xilinx.com]
>> Sent: 2018年3月26日 19:43
>> To: Peng Fan
>> Cc: Mire
>>> On 28.03.18 at 18:35, wrote:
> Its one of the many items on the TODO list, along with maintaining a
> proper virtual TLB to avoid rewalks during a single emulation.
Having thought about this some more I agree that for correctness
a virtual TLB would be sufficient. Also caching values read mig
On Wed, Mar 21, 2018 at 09:15:36AM +0200, Oleksandr Andrushchenko wrote:
> On 03/20/2018 10:22 PM, Takashi Iwai wrote:
> > On Mon, 19 Mar 2018 08:22:19 +0100,
> > Oleksandr Andrushchenko wrote:
> > > From: Oleksandr Andrushchenko
> > >
> > > Hello, all!
> > >
> > > In order to provide explicit s
On 04/12/2018 05:31 PM, Konrad Rzeszutek Wilk wrote:
On Wed, Mar 21, 2018 at 09:15:36AM +0200, Oleksandr Andrushchenko wrote:
On 03/20/2018 10:22 PM, Takashi Iwai wrote:
On Mon, 19 Mar 2018 08:22:19 +0100,
Oleksandr Andrushchenko wrote:
From: Oleksandr Andrushchenko
Hello, all!
In order to
Ian Jackson writes ("[PATCH for-4.11 v3 0/11] Provide support matrix
generator"):
> This series provides code to generate a feature support matrix, to
> replace the one on the wiki. You can see an example of the output
> here:
I spoke to Wei IRL and he expressed a disinclination to review my
mat
>>> On 29.03.18 at 17:36, wrote:
> @@ -967,6 +968,54 @@ static long xatp_permission_check(struct domain *d,
> unsigned int space)
> return xsm_add_to_physmap(XSM_TARGET, current->domain, d);
> }
>
> +static int acquire_grant_table(struct domain *d, unsigned int id,
> +
> -Original Message-
> From: Jan Beulich [mailto:jbeul...@suse.com]
> Sent: 12 April 2018 16:28
> To: Paul Durrant
> Cc: Andrew Cooper ; Wei Liu
> ; George Dunlap ; Ian
> Jackson ; Stefano Stabellini
> ; xen-devel ;
> Konrad Rzeszutek Wilk ; Tim (Xen.org)
>
> Subject: Re: [PATCH v19 10/11
On Thu, 2018-04-12 at 15:15 +0200, Dario Faggioli wrote:
> On Thu, 2018-04-12 at 14:45 +0200, Olaf Hering wrote:
> >
> > dies after the first iteration.
> >
> > BUG_ON(!test_bit(_VPF_migrating, &prev->pause_flags));
> >
>
Update. I replaced this:
+BUG_ON(vcpu_runnable(prev));
+
On 12/04/2018, 13:28, "Ian Jackson" wrote:
This series provides code to generate a feature support matrix, to
replace the one on the wiki. You can see an example of the output
here:
https://xenbits.xen.org/people/iwj/2018/support-matrix-example-v3a/t.html
I prefer this one.
In order to provide explicit stream parameter negotiation between
backend and frontend the following changes are introduced in the protocol:
add XENSND_OP_HW_PARAM_QUERY request to read/update
configuration space for the parameters given: request passes
desired parameter's intervals/masks and the r
In order to provide explicit synchronization between backend and
frontend the following changes are introduced in the protocol:
- add new ring buffer for sending asynchronous events from
backend to frontend to report number of bytes played by the
frontend (XENSND_EVT_CUR_POS)
- introduce tr
Protocol version was referenced in the protocol description,
but missed its definition. Fix this by adding a constant
for current protocol version.
Signed-off-by: Oleksandr Andrushchenko
---
include/xen/interface/io/sndif.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/include/xen/i
Hello, all!
This is the syncup version of the sound protocol changes for
Linux Kernel with the only difference from the corresponding Xen
version being spaces to tabs conversion. Regradless of this only
change I have dropped R-b tags received for Xen version.
In order to provide explicit synchron
Extend the size of the requests and responses to 64 octets.
Bump protocol version to 2.
Signed-off-by: Oleksandr Andrushchenko
---
include/xen/interface/io/sndif.h | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/include/xen/interface/io/sndif.h b/incl
Some of the request descriptions have "reserved" fields
missed: fix this by adding corresponidng entries.
Signed-off-by: Oleksandr Andrushchenko
---
include/xen/interface/io/sndif.h | 4
1 file changed, 4 insertions(+)
diff --git a/include/xen/interface/io/sndif.h b/include/xen/interface/i
Hello, Konrad, Takashi!
Could you please review the *Linux Kernel* version of the changes?
As I said in the cover letter below there is no functional changes
comparing to the corresponding Xen version, but spaces to tabs.
Still, formally, I have to drop the R-b tags and request for the new review
Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Acked-by: George Dunlap
Acked-by: Lars Kurth
---
SUPPORT.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/SUPPORT.md b/SUPPORT.md
index 1c5220b..e447069 100644
--- a/SUPPORT.md
+++ b/SUPPORT.md
@@ -360,7 +360,7 @@
Continuations of bullet list items must be indented by exactly 4
spaces (according to pandoc_markdown(5) on Debian jessie).
This is most easily achieved by making the bullet list items have two
spaces before the `*'.
Signed-off-by: Ian Jackson
Release-acked-by: Juergen Gross
Acked-by: George Du
Ian Jackson writes ("[PATCH 01/11] SUPPORT.md: Syntax: Fix some bullet lists"):
> Continuations of bullet list items must be indented by exactly 4
> spaces (according to pandoc_markdown(5) on Debian jessie).
Please disregard this and the next mail, which were sent by mistake.
I think I managed to
On 04/11/2018 11:04 AM, Razvan Cojocaru wrote:
>> After much debugging, it turns out that the
>> "p2m_is_ram(p2mt)" test in hvm_hap_nested_page_fault() fails if I switch
>> to the new altp2m view fast enough, and that in turn disables the
>> logdirty processing gated on it
>
> Actually as it turns
Ian Jackson writes ("Re: [PATCH for-4.11 v3 0/11] Provide support matrix
generator"):
> For 8-11 I'm awaiting any opinions about the output and in particular
> whether to include
> > + 11/11] docs/parse-support-md: Identical [*]: only use extra
> > table cell if necessary
AFter IRL
Hi all,
I had an action to set up a call on discussing the future direction of PCI
Emulation. I CC’ed everyone who raised an interest. I propose to use
Gotomeeting unless there are objections.
As far as I can tell, we have people in the following time-zones: PST to EST
and BST. Not sure where
On Thu, Apr 12, 2018 at 05:32:57PM +0100, Lars Kurth wrote:
>Hi all,
>
>
>
>I had an action to set up a call on discussing the future direction of PCI
>Emulation. I CC’ed everyone who raised an interest. I propose to use
>Gotomeeting unless there are objections.
>
>
>
flight 122191 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/122191/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
build-arm64-xsm 6 xen-buildfail REGR. vs. 122174
build-amd64
On Wed, 2018-04-11 at 15:19 +0200, Mirela Simonovic wrote:
> Secondary pCPUs will be offlined on system suspend and hotplugged
> on resume. When offlining secondary CPUs all interrupts targeted
> to those CPUs will be routed to the boot CPU. The boot CPU
> is responsible for finalizing suspend proc
On 12/04/2018, 17:41, "Roger Pau Monne" wrote:
On Thu, Apr 12, 2018 at 05:32:57PM +0100, Lars Kurth wrote:
>may work. For me Mon, Wed and Fri’s generally work at those time-slots.
>Next week is a little busy for me, so I would prefer the following
week.
>If you c
On 04/12/2018 12:11 PM, Oleksandr Andrushchenko wrote:
> Hello, Konrad, Takashi!
>
> Could you please review the *Linux Kernel* version of the changes?
> As I said in the cover letter below there is no functional changes
> comparing to the corresponding Xen version, but spaces to tabs.
> Still, for
do_get_debugreg() has several bugs:
* The %cr4.de condition is inverted. %dr4/5 should be accessible only when
%cr4.de is disabled.
* When %cr4.de is disabled, emulation should yield #UD rather than complete
with zero.
* Using -EINVAL for errors is a broken ABI, as it overlaps with valid
On 04/12/2018 07:55 PM, Boris Ostrovsky wrote:
On 04/12/2018 12:11 PM, Oleksandr Andrushchenko wrote:
Hello, Konrad, Takashi!
Could you please review the *Linux Kernel* version of the changes?
As I said in the cover letter below there is no functional changes
comparing to the corresponding Xen
set_debugreg() has several bugs:
* %dr4/5 should function correctly as aliases of %dr6/7 when CR4.DE is clear.
* Attempting to set the upper 32 bits of %dr6/7 should fail with #GP[0]
rather than be silently corrected and complete.
* For emulation, the #UD and #GP[0] cases need properly disti
* Change 'int i' to being unsigned, and move it into its most narrow scope.
* Fold the access_ok() checks for %dr{0..3}. This halves the compiled size
of the function.
* Additional newlines in appropriate places.
Signed-off-by: Andrew Cooper
---
CC: Jan Beulich
CC: Wei Liu
CC: Roger Pau
At this point, I think these patches are plausible candidates for inclusion
into 4.11. Whether they want backporting is a slightly harder matter, as
these patches necesserily alter some error values for the get/set_debug_reg()
hypercalls.
If however there is objection to these going into 4.11, th
On 04/12/2018 12:55 PM, Oleksandr Andrushchenko wrote:
> On 04/12/2018 07:55 PM, Boris Ostrovsky wrote:
>> On 04/12/2018 12:11 PM, Oleksandr Andrushchenko wrote:
>>> Hello, Konrad, Takashi!
>>>
>>> Could you please review the *Linux Kernel* version of the changes?
>>> As I said in the cover letter
On 04/12/2018 08:13 PM, Boris Ostrovsky wrote:
On 04/12/2018 12:55 PM, Oleksandr Andrushchenko wrote:
On 04/12/2018 07:55 PM, Boris Ostrovsky wrote:
On 04/12/2018 12:11 PM, Oleksandr Andrushchenko wrote:
Hello, Konrad, Takashi!
Could you please review the *Linux Kernel* version of the changes
On Thu, 2018-04-12 at 17:38 +0200, Dario Faggioli wrote:
> On Thu, 2018-04-12 at 15:15 +0200, Dario Faggioli wrote:
> > On Thu, 2018-04-12 at 14:45 +0200, Olaf Hering wrote:
> > >
> > > dies after the first iteration.
> > >
> > > BUG_ON(!test_bit(_VPF_migrating, &prev->pause_flags));
> >
This is the sync up with the canonical definition of the sound
protocol in Xen:
1. Protocol version was referenced in the protocol description,
but missed its definition. Fixed by adding a constant
for current protocol version.
2. Some of the request descriptions have "reserved" fields
m
On 04/12/2018 01:26 PM, Oleksandr Andrushchenko wrote:
> This is the sync up with the canonical definition of the sound
> protocol in Xen:
>
> 1. Protocol version was referenced in the protocol description,
>but missed its definition. Fixed by adding a constant
>for current protocol version
On 04/12/2018 04:06 AM, Jan Beulich wrote:
> Jürgen, Boris,
>
> looks like commit 47b02f4c62 ("x86/xen: add tty0 and hvc0 as
> preferred consoles for dom0") doesn't get us quite there yet - non-
> kernel boot output (and a console prompt) still doesn't appear on
> the screen.
Hmm.. I get both ke
Linus,
Please git pull the following tag:
git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git
for-linus-4.17-rc1-tag
xen: fixes for 4.17-rc1
It contains only a few fixes of Xen related core code and drivers.
Thanks.
Juergen
arch/x86/xen/enlighten_pv.c | 8
arc
Instead of flushing the TLB from global pages when switching address
spaces with XPTI being active just disable global pages via %cr4
completely when a domain subject to XPTI is active. This avoids the
need for extra TLB flushes as loading %cr3 will remove all TLB
entries.
In order to avoid states
Instead of switching XPTI globally on or off add a per-domain flag for
that purpose. This allows to modify the xpti boot parameter to support
running dom0 without Meltdown mitigations. Using "xpti=nodom0" as boot
parameter will achieve that.
Move the xpti boot parameter handling to xen/arch/x86/pv
Instead of having multiple places with more or less identical asm
statements just have one function doing a write to cr3.
As this function should be named write_cr3() rename the current
write_cr3() function to switch_cr3().
Suggested-by: Andrew Copper
Signed-off-by: Juergen Gross
Reviewed-by: J
If possible use the INVPCID instruction for flushing the TLB instead of
toggling cr4.pge for that purpose.
While at it remove the dependency on cr4.pge being required for mtrr
loading, as this will be required later anyway.
Add a command line option "invpcid" for controlling the use of
INVPCID (d
Today cpu_info->xen_cr3 is either 0 to indicate %cr3 doesn't need to
be switched on entry to Xen, or negative for keeping the value while
indicating not to restore %cr3, or positive in case %cr3 is to be
restored.
Switch to use a flag byte instead of a negative xen_cr3 value in order
to allow %cr3
Add some helper macros to access the address and pcid parts of cr3.
Use those helpers where appropriate.
Signed-off-by: Juergen Gross
Reviewed-by: Jan Beulich
---
V6:
- new patch (Andrew Cooper)
---
xen/arch/x86/debug.c| 2 +-
xen/arch/x86/domain_page.c | 2 +-
xen/include/a
This patch series aims at reducing the overhead of the XPTI Meltdown
mitigation.
Patch 1 had been posted before, the main changes in this patch are due
to addressing Jan's comments on my first version. The main objective of
that patch is to avoid copying the L4 page table each time the guest is
be
For mitigation of Meltdown the current L4 page table is copied to the
cpu local root page table each time a 64 bit pv guest is entered.
Copying can be avoided in cases where the guest L4 page table hasn't
been modified while running the hypervisor, e.g. when handling
interrupts or any hypercall no
pv_guest_cr4_to_real_cr4() is becoming more and more complex. Convert
it from a macro to an ordinary function.
Signed-off-by: Juergen Gross
Reviewed-by: Jan Beulich
---
V6:
- new patch, split off from (old) patch 7 (Andrew Cooper)
---
xen/arch/x86/mm.c| 14 ++
xen/includ
Avoid flushing the complete TLB when switching %cr3 for mitigation of
Meltdown by using the PCID feature if available.
We are using 4 PCID values for a 64 bit pv domain subject to XPTI and
2 values for the non-XPTI case:
- guest active and in kernel mode
- guest active and in user mode
- hypervis
Running text in feature sections in the markdown document currently
might be (i) a caveat, qualifying or clarifying the support statement
(ii) a plain description of the feature.
Caveats can be version-specific and deserve the [*] annotation in the
relevant feature matrix cell. They must link to
The new support matrix output puts a [*] after each entry in the
support matrix in many cases where the linked-to text is simply a
longer description of the feature.
Remedy this by distinguishing text which expands on a feature
description from text which qualifies its support status.
There are 3
No functional change.
Signed-off-by: Ian Jackson
---
docs/parse-support-md | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/docs/parse-support-md b/docs/parse-support-md
index 5bf8405..6953930 100755
--- a/docs/parse-support-md
+++ b/docs/parse-support-md
@@ -34,7 +
No functional change.
Signed-off-by: Ian Jackson
---
docs/parse-support-md | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/docs/parse-support-md b/docs/parse-support-md
index decda33..5bf8405 100755
--- a/docs/parse-support-md
+++ b/docs/parse-support-md
@@ -318,6
Signed-off-by: Ian Jackson
---
SUPPORT.md | 5 +
1 file changed, 5 insertions(+)
diff --git a/SUPPORT.md b/SUPPORT.md
index 5ae84cf..098262b 100644
--- a/SUPPORT.md
+++ b/SUPPORT.md
@@ -725,6 +725,11 @@ The file is in markdown format.
The machine-readable fragments are markdown literals
co
This turns all the things which were treated as caveats, but which
don't need to be footnoted in the matrix, into descriptions.
For the benefit of the support matrix generator, this patch (or a
version of it) should be backported to 4.10.
Signed-off-by: Ian Jackson
---
SUPPORT.md | 213
1 - 100 of 130 matches
Mail list logo