historical-speck.lore.kernel.org archive mirror
 help / color / mirror / Atom feed
* The Annnoying Affair has No eXcuses - Bulletin #2
@ 2019-11-04 12:09 Thomas Gleixner
  2019-11-04 18:34 ` [MODERATED] " Luck, Tony
  2019-11-04 18:52 ` [MODERATED] " Ben Hutchings
  0 siblings, 2 replies; 10+ messages in thread
From: Thomas Gleixner @ 2019-11-04 12:09 UTC (permalink / raw)
  To: speck

[-- Attachment #1: Type: text/plain, Size: 729 bytes --]

Folks,

I've merged Linus tree into master to pick up the KVM fix and applied
Paolo's V9 series. The commits are considered stable from now on. Any
further tweaks need to go on top.

The stable branches linux-5.3.y, linux-4.19.y and linix-4.14.y have been
updated to the latest stable and the fallout in the backports of TAA has
been fixed.

The non KVM parts of the NX series has been backported to them. Paolo will
provide the KVM backports including the commits which need to be
additionally backported as prerequisites for 5.3, 4.19 and 4.14.

The changes are available as usual from:

  cvs.ou.linutronix.de:linux/speck/linux  

Git bundle for the master branch is attached. Applies on top of 5.4-rc6.

Thanks,

        tglx

[-- Attachment #2: Type: application/octet-stream, Size: 41777 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [MODERATED] Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-04 12:09 The Annnoying Affair has No eXcuses - Bulletin #2 Thomas Gleixner
@ 2019-11-04 18:34 ` Luck, Tony
  2019-11-04 19:20   ` Thomas Gleixner
  2019-11-04 18:52 ` [MODERATED] " Ben Hutchings
  1 sibling, 1 reply; 10+ messages in thread
From: Luck, Tony @ 2019-11-04 18:34 UTC (permalink / raw)
  To: speck

On Mon, Nov 04, 2019 at 01:09:21PM +0100, speck for Thomas Gleixner wrote:
> Folks,
> 
> I've merged Linus tree into master to pick up the KVM fix and applied
> Paolo's V9 series. The commits are considered stable from now on. Any
> further tweaks need to go on top.

Small typo causes build error:

arch/x86/kvm/../../../virt/kvm/kvm_main.c: In function ‘kvm_create_vm’:
arch/x86/kvm/../../../virt/kvm/kvm_main.c:703:31: error: expected ‘;’ before ‘:’ token
   goto out_err_no_mmu_notifier:
                               ^
                               ;

diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
index d32a50aa104c..4aab3547a165 100644
--- a/virt/kvm/kvm_main.c
+++ b/virt/kvm/kvm_main.c
@@ -700,7 +700,7 @@ static struct kvm *kvm_create_vm(unsigned long type)
 
 	r = kvm_init_mmu_notifier(kvm);
 	if (r)
-		goto out_err_no_mmu_notifier:
+		goto out_err_no_mmu_notifier;
 
 	r = kvm_arch_post_init_vm(kvm);
 	if (r)

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [MODERATED] Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-04 12:09 The Annnoying Affair has No eXcuses - Bulletin #2 Thomas Gleixner
  2019-11-04 18:34 ` [MODERATED] " Luck, Tony
@ 2019-11-04 18:52 ` Ben Hutchings
  2019-11-04 19:30   ` Thomas Gleixner
  1 sibling, 1 reply; 10+ messages in thread
From: Ben Hutchings @ 2019-11-04 18:52 UTC (permalink / raw)
  To: speck

[-- Attachment #1: Type: text/plain, Size: 1572 bytes --]

On Mon, 2019-11-04 at 13:09 +0100, speck for Thomas Gleixner wrote:
> Folks,
> 
> I've merged Linus tree into master to pick up the KVM fix and applied
> Paolo's V9 series. The commits are considered stable from now on. Any
> further tweaks need to go on top.
> 
> The stable branches linux-5.3.y, linux-4.19.y and linix-4.14.y have been
> updated to the latest stable and the fallout in the backports of TAA has
> been fixed.
> 
> The non KVM parts of the NX series has been backported to them. Paolo will
> provide the KVM backports including the commits which need to be
> additionally backported as prerequisites for 5.3, 4.19 and 4.14.

Based on a quick comparison between your 4.19 branch and mine:

* You've included "x86/cpu: Add Atom Tremont (Jacobsville)" and
"x86/cpu: Add Tremont to the cpu vulnerability whitelist", but Paolo
seemed to say these weren't worth adding so I've dropped them.

* Your backport of "KVM: x86: use Intel speculation bugs and features
as derived in generic x86 code" is messed up; at least the indentation
is wrong.

* I backported commit 3ff519f29d98 "KVM: x86: adjust kvm_mmu_page
member to save 8 bytes" as this made it easier to apply the other
changes in arch/x86/include/asm/kvm_host.h.

Ben.

> The changes are available as usual from:
> 
>   cvs.ou.linutronix.de:linux/speck/linux  
> 
> Git bundle for the master branch is attached. Applies on top of 5.4-rc6.
> 
> Thanks,
> 
>         tglx
-- 
Ben Hutchings
Design a system any fool can use, and only a fool will want to use it.


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-04 18:34 ` [MODERATED] " Luck, Tony
@ 2019-11-04 19:20   ` Thomas Gleixner
  2019-11-04 19:28     ` Thomas Gleixner
  0 siblings, 1 reply; 10+ messages in thread
From: Thomas Gleixner @ 2019-11-04 19:20 UTC (permalink / raw)
  To: speck

[-- Attachment #1: Type: text/plain, Size: 668 bytes --]

On Mon, 4 Nov 2019, speck for Luck, Tony wrote:

> On Mon, Nov 04, 2019 at 01:09:21PM +0100, speck for Thomas Gleixner wrote:
> > Folks,
> > 
> > I've merged Linus tree into master to pick up the KVM fix and applied
> > Paolo's V9 series. The commits are considered stable from now on. Any
> > further tweaks need to go on top.
> 
> Small typo causes build error:
> 
> arch/x86/kvm/../../../virt/kvm/kvm_main.c: In function ‘kvm_create_vm’:
> arch/x86/kvm/../../../virt/kvm/kvm_main.c:703:31: error: expected ‘;’ before ‘:’ token
>    goto out_err_no_mmu_notifier:
                                 ^
Uuurgh. /me looks for a brown paperbag and goes to fix.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-04 19:20   ` Thomas Gleixner
@ 2019-11-04 19:28     ` Thomas Gleixner
  0 siblings, 0 replies; 10+ messages in thread
From: Thomas Gleixner @ 2019-11-04 19:28 UTC (permalink / raw)
  To: speck

[-- Attachment #1: Type: text/plain, Size: 853 bytes --]

On Mon, 4 Nov 2019, speck for Thomas Gleixner wrote:
> On Mon, 4 Nov 2019, speck for Luck, Tony wrote:
> 
> > On Mon, Nov 04, 2019 at 01:09:21PM +0100, speck for Thomas Gleixner wrote:
> > > Folks,
> > > 
> > > I've merged Linus tree into master to pick up the KVM fix and applied
> > > Paolo's V9 series. The commits are considered stable from now on. Any
> > > further tweaks need to go on top.
> > 
> > Small typo causes build error:
> > 
> > arch/x86/kvm/../../../virt/kvm/kvm_main.c: In function ‘kvm_create_vm’:
> > arch/x86/kvm/../../../virt/kvm/kvm_main.c:703:31: error: expected ‘;’ before ‘:’ token
> >    goto out_err_no_mmu_notifier:
>                                  ^
> Uuurgh. /me looks for a brown paperbag and goes to fix.

Fixed in the master branch. New bundle attached.

Sorry for the noise!

	tglx

[-- Attachment #2: Type: application/octet-stream, Size: 41776 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-04 18:52 ` [MODERATED] " Ben Hutchings
@ 2019-11-04 19:30   ` Thomas Gleixner
  2019-11-04 23:44     ` [MODERATED] " Ben Hutchings
  2019-11-05  0:28     ` Thomas Gleixner
  0 siblings, 2 replies; 10+ messages in thread
From: Thomas Gleixner @ 2019-11-04 19:30 UTC (permalink / raw)
  To: speck

On Mon, 4 Nov 2019, speck for Ben Hutchings wrote:
> On Mon, 2019-11-04 at 13:09 +0100, speck for Thomas Gleixner wrote:
> > Folks,
> > 
> > I've merged Linus tree into master to pick up the KVM fix and applied
> > Paolo's V9 series. The commits are considered stable from now on. Any
> > further tweaks need to go on top.
> > 
> > The stable branches linux-5.3.y, linux-4.19.y and linix-4.14.y have been
> > updated to the latest stable and the fallout in the backports of TAA has
> > been fixed.
> > 
> > The non KVM parts of the NX series has been backported to them. Paolo will
> > provide the KVM backports including the commits which need to be
> > additionally backported as prerequisites for 5.3, 4.19 and 4.14.
> 
> Based on a quick comparison between your 4.19 branch and mine:
> 
> * You've included "x86/cpu: Add Atom Tremont (Jacobsville)" and
> "x86/cpu: Add Tremont to the cpu vulnerability whitelist", but Paolo
> seemed to say these weren't worth adding so I've dropped them.

Hmm. Not sure about that, but it doesn't really matter.

> * Your backport of "KVM: x86: use Intel speculation bugs and features
> as derived in generic x86 code" is messed up; at least the indentation
> is wrong.

Yes indeed.
 
> * I backported commit 3ff519f29d98 "KVM: x86: adjust kvm_mmu_page
> member to save 8 bytes" as this made it easier to apply the other
> changes in arch/x86/include/asm/kvm_host.h.

For which part TAA or NX?

Thanks,

	tglx

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [MODERATED] Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-04 19:30   ` Thomas Gleixner
@ 2019-11-04 23:44     ` Ben Hutchings
  2019-11-05  0:28       ` Thomas Gleixner
  2019-11-05  0:28     ` Thomas Gleixner
  1 sibling, 1 reply; 10+ messages in thread
From: Ben Hutchings @ 2019-11-04 23:44 UTC (permalink / raw)
  To: speck

[-- Attachment #1: Type: text/plain, Size: 471 bytes --]

On Mon, 2019-11-04 at 20:30 +0100, speck for Thomas Gleixner wrote:
> On Mon, 4 Nov 2019, speck for Ben Hutchings wrote:
[...]
> > * I backported commit 3ff519f29d98 "KVM: x86: adjust kvm_mmu_page
> > member to save 8 bytes" as this made it easier to apply the other
> > changes in arch/x86/include/asm/kvm_host.h.
> 
> For which part TAA or NX?

For NX.

Ben.

-- 
Ben Hutchings
Design a system any fool can use, and only a fool will want to use it.



[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-04 23:44     ` [MODERATED] " Ben Hutchings
@ 2019-11-05  0:28       ` Thomas Gleixner
  2019-11-05 22:46         ` [MODERATED] " Paolo Bonzini
  0 siblings, 1 reply; 10+ messages in thread
From: Thomas Gleixner @ 2019-11-05  0:28 UTC (permalink / raw)
  To: speck

On Mon, 4 Nov 2019, speck for Ben Hutchings wrote:
> On Mon, 2019-11-04 at 20:30 +0100, speck for Thomas Gleixner wrote:
> > On Mon, 4 Nov 2019, speck for Ben Hutchings wrote:
> [...]
> > > * I backported commit 3ff519f29d98 "KVM: x86: adjust kvm_mmu_page
> > > member to save 8 bytes" as this made it easier to apply the other
> > > changes in arch/x86/include/asm/kvm_host.h.
> > 
> > For which part TAA or NX?
> 
> For NX.

Assumed that. Thanks for confirming. I still let Paolo sort this out as he
already offered to do so.

Thanks,

	tglx

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-04 19:30   ` Thomas Gleixner
  2019-11-04 23:44     ` [MODERATED] " Ben Hutchings
@ 2019-11-05  0:28     ` Thomas Gleixner
  1 sibling, 0 replies; 10+ messages in thread
From: Thomas Gleixner @ 2019-11-05  0:28 UTC (permalink / raw)
  To: speck

On Mon, 4 Nov 2019, speck for Thomas Gleixner wrote:
> On Mon, 4 Nov 2019, speck for Ben Hutchings wrote:
> > * Your backport of "KVM: x86: use Intel speculation bugs and features
> > as derived in generic x86 code" is messed up; at least the indentation
> > is wrong.
> 
> Yes indeed.

Fixed that up and pushed out updated 4.19/4.14 branches.

Thanks,

	tglx

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [MODERATED] Re: The Annnoying Affair has No eXcuses - Bulletin #2
  2019-11-05  0:28       ` Thomas Gleixner
@ 2019-11-05 22:46         ` Paolo Bonzini
  0 siblings, 0 replies; 10+ messages in thread
From: Paolo Bonzini @ 2019-11-05 22:46 UTC (permalink / raw)
  To: speck

[-- Attachment #1: Type: text/plain, Size: 668 bytes --]

On 05/11/19 01:28, speck for Thomas Gleixner wrote:
> On Mon, 4 Nov 2019, speck for Ben Hutchings wrote:
>> On Mon, 2019-11-04 at 20:30 +0100, speck for Thomas Gleixner wrote:
>>> On Mon, 4 Nov 2019, speck for Ben Hutchings wrote:
>> [...]
>>>> * I backported commit 3ff519f29d98 "KVM: x86: adjust kvm_mmu_page
>>>> member to save 8 bytes" as this made it easier to apply the other
>>>> changes in arch/x86/include/asm/kvm_host.h.
>>>
>>> For which part TAA or NX?
>>
>> For NX.
> 
> Assumed that. Thanks for confirming. I still let Paolo sort this out as he
> already offered to do so.

Yes, I should get everything sorted out tomorrow.

Paolo



^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2019-11-05 22:46 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-04 12:09 The Annnoying Affair has No eXcuses - Bulletin #2 Thomas Gleixner
2019-11-04 18:34 ` [MODERATED] " Luck, Tony
2019-11-04 19:20   ` Thomas Gleixner
2019-11-04 19:28     ` Thomas Gleixner
2019-11-04 18:52 ` [MODERATED] " Ben Hutchings
2019-11-04 19:30   ` Thomas Gleixner
2019-11-04 23:44     ` [MODERATED] " Ben Hutchings
2019-11-05  0:28       ` Thomas Gleixner
2019-11-05 22:46         ` [MODERATED] " Paolo Bonzini
2019-11-05  0:28     ` Thomas Gleixner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).