Change in coreboot[master]: 3rdparty/intel-microcode: Update submodule to 20200616 release

List overview All Threads
Download

newer

older

Change in coreboot[master]:...

Angel Pons (Code Review)

23 Jul 2020 23 Jul '20

5:03 p.m.

Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

3rdparty/intel-microcode: Update submodule to 20200616 release

Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Signed-off-by: Angel Pons th3fanbus@gmail.com --- M 3rdparty/intel-microcode 1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/58/43758/1

diff --git a/3rdparty/intel-microcode b/3rdparty/intel-microcode index 33b7b2f..0e4288f 160000 --- a/3rdparty/intel-microcode +++ b/3rdparty/intel-microcode @@ -1 +1 @@ -Subproject commit 33b7b2f3817e362111cd91910026ab8907f21710 +Subproject commit 0e4288f81f806620c65f70ee2bcf94b69d574096

Attachments:

attachment.htm (text/html — 2.1 KB)

Show replies by date

Angel Pons (Code Review)

23 Jul 23 Jul

5:35 p.m.

Hello build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/43758

to look at the new patch set (#2).

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

3rdparty/intel-microcode: Update submodule to 20200616 release

Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Signed-off-by: Angel Pons th3fanbus@gmail.com --- M 3rdparty/intel-microcode 1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/58/43758/2

Angel Pons (Code Review)

5:46 p.m.

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG@7 PS2, Line 7: 3rdparty/intel-microcode: Update submodule to 20200616 release Known changes: CPUID 306C3 (06-3c-03) ucode revision bumped from 27 to 28.

On Asrock B85M Pro4, Windows 10: - revision 28: Hangs while loading. - revision 27 and earlier: Hangs with a BSOD about MACHINE_CHECK_EXCEPTION.

https://ticket.coreboot.org/issues/259 has some info about T440p with a similar? problem.

Angel Pons (Code Review)

24 Jul 24 Jul

11:09 p.m.

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG@7 PS2, Line 7: 3rdparty/intel-microcode: Update submodule to 20200616 release

...

Known changes: CPUID 306C3 (06-3c-03) ucode revision bumped from 27 to 28. […]

Unrelated

Michael Niewöhner (Code Review)

26 Jul 26 Jul

11:34 a.m.

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2: Code-Review+1

Arthur Heymans (Code Review)

11:39 a.m.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2: Code-Review+2

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 2 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-Comment-Date: Sun, 26 Jul 2020 11:39:38 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

Benjamin Doron (Code Review)

27 Jul 27 Jul

7:58 p.m.

Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG@7 PS2, Line 7: 20200616 Why not 20200609? The 20200616 release reverts the Skylake microcodes due to an issue with OS microcode update that apparently does not affect updates loaded from the firmware[1].

1. https://github.com/intel/Intel-Linux-Processor-Microcode-Data-Files/issues/3...

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 2 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Benjamin Doron benjamin.doron00@gmail.com Gerrit-Comment-Date: Mon, 27 Jul 2020 19:58:01 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

Angel Pons (Code Review)

8:10 p.m.

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG@7 PS2, Line 7: 20200616

...

Why not 20200609? The 20200616 release reverts the Skylake microcodes due to an issue with OS microc […]

I just picked the latest master.

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 2 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Benjamin Doron benjamin.doron00@gmail.com Gerrit-Comment-Date: Mon, 27 Jul 2020 20:10:52 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Benjamin Doron benjamin.doron00@gmail.com Gerrit-MessageType: comment

Angel Pons (Code Review)

8:12 p.m.

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG@7 PS2, Line 7: 20200616

...

I just picked the latest master.

Also see https://github.com/intel/Intel-Linux-Processor-Microcode-Data-Files/issues/3...

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 2 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Benjamin Doron benjamin.doron00@gmail.com Gerrit-Comment-Date: Mon, 27 Jul 2020 20:12:14 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Benjamin Doron benjamin.doron00@gmail.com Comment-In-Reply-To: Angel Pons th3fanbus@gmail.com Gerrit-MessageType: comment

Benjamin Doron (Code Review)

4 Aug 4 Aug

3:31 a.m.

Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/43758/2//COMMIT_MSG@7 PS2, Line 7: 20200616

...

Also see https://github. […]

I can see, but 20200609 contains later microcode.

In any event, while I haven't had any issues with the 0xdc microcode on CPUID 406E3 (06-4e-03), if others have encountered stability issues, 20200616 may be a safer choice until Intel has resolved the issue.

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 2 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Benjamin Doron benjamin.doron00@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Tue, 04 Aug 2020 03:31:54 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Benjamin Doron benjamin.doron00@gmail.com Comment-In-Reply-To: Angel Pons th3fanbus@gmail.com Gerrit-MessageType: comment

Angel Pons (Code Review)

6 Aug 6 Aug

7:27 a.m.

Angel Pons has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Abandoned

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 2 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Benjamin Doron benjamin.doron00@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: abandon

Angel Pons (Code Review)

10:33 p.m.

Angel Pons has restored this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Restored

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 2 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Benjamin Doron benjamin.doron00@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: restore

Paul Menzel (Code Review)

7 Aug 7 Aug

7:24 a.m.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

Patch Set 2: Code-Review+1

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 2 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Benjamin Doron benjamin.doron00@gmail.com Gerrit-Comment-Date: Fri, 07 Aug 2020 07:24:02 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

Angel Pons (Code Review)

8 Aug 8 Aug

8:32 p.m.

Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/43758 )

Change subject: 3rdparty/intel-microcode: Update submodule to 20200616 release ......................................................................

3rdparty/intel-microcode: Update submodule to 20200616 release

Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Signed-off-by: Angel Pons th3fanbus@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/43758 Reviewed-by: Michael Niewöhner Reviewed-by: Arthur Heymans arthur@aheymans.xyz Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M 3rdparty/intel-microcode 1 file changed, 1 insertion(+), 1 deletion(-)

Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Arthur Heymans: Looks good to me, approved Michael Niewöhner: Looks good to me, but someone else must approve

-- To view, visit https://review.coreboot.org/c/coreboot/+/43758 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia250765e2cb81d6a39ad00ebbab20e7b87fa42d1 Gerrit-Change-Number: 43758 Gerrit-PatchSet: 3 Gerrit-Owner: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Angel Pons th3fanbus@gmail.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Benjamin Doron benjamin.doron00@gmail.com Gerrit-MessageType: merged

1598

days inactive

1614

days old

coreboot-gerrit@coreboot.org

13 comments

5 participants

tags (0)

participants (5)

Angel Pons (Code Review)
Arthur Heymans (Code Review)
Benjamin Doron (Code Review)
Michael Niewöhner (Code Review)
Paul Menzel (Code Review)