coreboot-gerrit September 2017

coreboot-gerrit@coreboot.org

1 participants
1960 discussions

Change in coreboot[master]: amd/stoneyridge: Put AGESA heap into cbmem
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21594 ) Change subject: amd/stoneyridge: Put AGESA heap into cbmem ...................................................................... Patch Set 5: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60896/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16082/ : SUCCESS -- To view, visit https://review.coreboot.org/21594 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9ff35eefb2a68879ff44c6e29f58635831b19848 Gerrit-Change-Number: 21594 Gerrit-PatchSet: 5 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Jonathan Neuschäfer <j.neuschaefer(a)gmx.net> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:51:49 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: amd/stoneyridge: Clean up include files in northbridge.h
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21615 ) Change subject: amd/stoneyridge: Clean up include files in northbridge.h ...................................................................... Patch Set 2: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60894/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16080/ : SUCCESS -- To view, visit https://review.coreboot.org/21615 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3e5ab39fd0640d2983fc5b7b202fb65d42c5ce3d Gerrit-Change-Number: 21615 Gerrit-PatchSet: 2 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:51:28 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/amd/common: Clean up GetHeapBase function
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21593 ) Change subject: soc/amd/common: Clean up GetHeapBase function ...................................................................... Patch Set 5: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60895/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16081/ : SUCCESS -- To view, visit https://review.coreboot.org/21593 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id4a024d000a514ea9a44f9dfc2caffae9ff01789 Gerrit-Change-Number: 21593 Gerrit-PatchSet: 5 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Jonathan Neuschäfer <j.neuschaefer(a)gmx.net> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:50:58 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: amd/stoneyridge: Enable SMM in TSEG
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21501 ) Change subject: amd/stoneyridge: Enable SMM in TSEG ...................................................................... Patch Set 8: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60893/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16079/ : SUCCESS -- To view, visit https://review.coreboot.org/21501 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4dc03ed27d0d109ab919a4f0861de9c7420d03ce Gerrit-Change-Number: 21501 Gerrit-PatchSet: 8 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:46:54 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: src/lib/ubsan: Indent macros to improve readability
by Marc Jones (Code Review) 21 Sep '17

21 Sep '17

Marc Jones has submitted this change and it was merged. ( https://review.coreboot.org/21600 ) Change subject: src/lib/ubsan: Indent macros to improve readability ...................................................................... src/lib/ubsan: Indent macros to improve readability Change-Id: Ide4e58e584a1a2bbc1b861e2c4dd943a1aeb35ab Signed-off-by: Jonathan Neuschäfer <j.neuschaefer(a)gmx.net> Reviewed-on: https://review.coreboot.org/21600 Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org> Reviewed-by: Ryan Salsamendi <rsalsamendi(a)hotmail.com> Reviewed-by: Marc Jones <marc(a)marcjonesconsulting.com> --- M src/lib/ubsan.c 1 file changed, 9 insertions(+), 11 deletions(-) Approvals: build bot (Jenkins): Verified Ryan Salsamendi: Looks good to me, but someone else must approve Marc Jones: Looks good to me, approved diff --git a/src/lib/ubsan.c b/src/lib/ubsan.c index 1cd0612..ead0e18 100644 --- a/src/lib/ubsan.c +++ b/src/lib/ubsan.c @@ -61,22 +61,20 @@ } #define ABORT_VARIANT(name, params, call) \ -__attribute__((noreturn)) \ -void __ubsan_handle_##name##_abort params; \ -__attribute__((noreturn)) \ -void __ubsan_handle_##name##_abort params { \ - __ubsan_handle_##name call; \ - __builtin_unreachable(); \ -} + __attribute__((noreturn)) void __ubsan_handle_##name##_abort params; \ + __attribute__((noreturn)) void __ubsan_handle_##name##_abort params { \ + __ubsan_handle_##name call; \ + __builtin_unreachable(); \ + } #define ABORT_VARIANT_VP(name) \ -ABORT_VARIANT(name, (void *a), (a)) + ABORT_VARIANT(name, (void *a), (a)) #define ABORT_VARIANT_VP_VP(name) \ -ABORT_VARIANT(name, (void *a, void *b), (a, b)) + ABORT_VARIANT(name, (void *a, void *b), (a, b)) #define ABORT_VARIANT_VP_IP(name) \ -ABORT_VARIANT(name, (void *a, intptr_t b), (a, b)) + ABORT_VARIANT(name, (void *a, intptr_t b), (a, b)) #define ABORT_VARIANT_VP_VP_VP(name) \ -ABORT_VARIANT(name, (void *a, void *b, void *c), (a, b, c)) + ABORT_VARIANT(name, (void *a, void *b, void *c), (a, b, c)) struct ubsan_type_mismatch_data { struct ubsan_source_location location; -- To view, visit https://review.coreboot.org/21600 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ide4e58e584a1a2bbc1b861e2c4dd943a1aeb35ab Gerrit-Change-Number: 21600 Gerrit-PatchSet: 2 Gerrit-Owner: Jonathan Neuschäfer <j.neuschaefer(a)gmx.net> Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com> Gerrit-Reviewer: Ryan Salsamendi <rsalsamendi(a)hotmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: cpu/amd/amdfam15: Add misc. SMM definitions
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21500 ) Change subject: cpu/amd/amdfam15: Add misc. SMM definitions ...................................................................... Patch Set 7: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60892/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16078/ : SUCCESS -- To view, visit https://review.coreboot.org/21500 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iad702bbdb459a09f9fef60d8280bb2684e365f4b Gerrit-Change-Number: 21500 Gerrit-PatchSet: 7 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:45:48 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: amd/stoneyridge: Remove EXT_CONF_SUPPORT check
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21497 ) Change subject: amd/stoneyridge: Remove EXT_CONF_SUPPORT check ...................................................................... Patch Set 7: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60887/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16073/ : SUCCESS -- To view, visit https://review.coreboot.org/21497 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic58c47fc5f568d17f5027c96d4152b0e5b3e1d14 Gerrit-Change-Number: 21497 Gerrit-PatchSet: 7 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:41:32 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: cpu/x86/smm: Add define for AMD64 save area
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21499 ) Change subject: cpu/x86/smm: Add define for AMD64 save area ...................................................................... Patch Set 7: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60891/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16077/ : SUCCESS -- To view, visit https://review.coreboot.org/21499 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0a051066b142cccae3d2c7df33be11994bafaae0 Gerrit-Change-Number: 21499 Gerrit-PatchSet: 7 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:41:32 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: amd/stoneyridge: Convert MP init to mp_init_with_smm
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21498 ) Change subject: amd/stoneyridge: Convert MP init to mp_init_with_smm ...................................................................... Patch Set 7: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60890/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16076/ : SUCCESS -- To view, visit https://review.coreboot.org/21498 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie54295cb00c6835947456e8818a289b7eb260914 Gerrit-Change-Number: 21498 Gerrit-PatchSet: 7 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:41:16 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: amd/stoneyridge: Simplify memory hole calculation
by build bot (Jenkins) (Code Review) 21 Sep '17

21 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21496 ) Change subject: amd/stoneyridge: Simplify memory hole calculation ...................................................................... Patch Set 6: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/60888/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/16074/ : SUCCESS -- To view, visit https://review.coreboot.org/21496 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id15a88897e29bff28ab7c498dc4d3818834f08b2 Gerrit-Change-Number: 21496 Gerrit-PatchSet: 6 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 21 Sep 2017 19:35:20 +0000 Gerrit-HasComments: No

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit September 2017