June 2013 - coreboot-gerrit - mail.coreboot.org

Patch merged into filo/master: 48add62 Fix a warning in fsys_squashfs.c
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit 48add62a6a5210fc46de9edfc01c8451f3fefea1 Author: Nico Huber <nico.huber(a)secunet.com> Date: Sun May 26 15:14:57 2013 +0200 Fix a warning in fsys_squashfs.c Fix the following warning: fs/fsys_squashfs.c: In function 'read_bytes': fs/fsys_squashfs.c:74:30: warning: variable 'offset' set but not used [-Wunused-but-set-variable] by removing the unused code to assign 'offset'. Change-Id: If85647f98ea8e89eabb7d043632e39865cf8da7b Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-on: http://review.coreboot.org/3300 Tested-by: build bot (Jenkins) Reviewed-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> See http://review.coreboot.org/3300 for details. -gerrit

1 0

Patch merged into filo/master: 617a159 Fix two warnings by removing unused code
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit 617a159ece01279721f74ed3f88927d346650ff9 Author: Nico Huber <nico.huber(a)secunet.com> Date: Sun May 26 15:10:01 2013 +0200 Fix two warnings by removing unused code Fix these warnings: fs/fsys_cramfs.c: In function 'cramfs_read': fs/fsys_cramfs.c:206:7: warning: variable 'devread_ret' set but not used [-Wunused-but-set-variable] fs/fsys_cramfs.c: In function 'cramfs_dir': fs/fsys_cramfs.c:295:6: warning: variable 'namelen' set but not used [-Wunused-but-set-variable] by removing the assignments and one unused call to strlen(). Change-Id: Ia2552b24263675bdef44a8c5fde8f67edf107912 Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-on: http://review.coreboot.org/3299 Tested-by: build bot (Jenkins) Reviewed-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> See http://review.coreboot.org/3299 for details. -gerrit

1 0

Patch merged into filo/master: 8100727 Always use endianness functions from libpayload
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit 8100727bc19038a9f1245c2278d33c3d727025bb Author: Nico Huber <nico.huber(a)secunet.com> Date: Sun May 26 14:55:44 2013 +0200 Always use endianness functions from libpayload We have endianness functions in libpayload (in <endian.h>) now. Those make the filo versions of them obsolete. So always use libpayload's. Change-Id: I41829c2128b355ccb5096d88df8b669b4e920dfc Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-on: http://review.coreboot.org/3298 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> See http://review.coreboot.org/3298 for details. -gerrit

1 0

Patch merged into filo/master: ac010e3 Fix compilation of fsys_ext2 with debugging
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit ac010e321fe2f3f1b790c3241da0f64749d8edd3 Author: Nico Huber <nico.huber(a)secunet.com> Date: Sun May 26 14:26:52 2013 +0200 Fix compilation of fsys_ext2 with debugging With time some changes to fsys_ext2 didn't make it into the debugging code. So track those changes. Change-Id: I576eaa2d481d45c178f3b1f51298dfa9a9061bbb Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-on: http://review.coreboot.org/3297 Tested-by: build bot (Jenkins) Reviewed-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> See http://review.coreboot.org/3297 for details. -gerrit

1 0

Patch merged into filo/master: c584f09 Correct some format strings
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit c584f09abb45356f846960af9dcbb335e3606523 Author: Nico Huber <nico.huber(a)secunet.com> Date: Sun May 26 14:23:45 2013 +0200 Correct some format strings Correct format strings to get rid of some warnings. Change-Id: Iaf30b5b95159c71daded95e6a603cdebd40d81a6 Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-on: http://review.coreboot.org/3296 Tested-by: build bot (Jenkins) Reviewed-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> See http://review.coreboot.org/3296 for details. -gerrit

1 0

Patch merged into filo/master: 2afbd63 Fix a warning in elfload.c
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit 2afbd6381c2e5e6bf0be8809ba450c350633ad26 Author: Nico Huber <nico.huber(a)secunet.com> Date: Sun May 26 14:06:41 2013 +0200 Fix a warning in elfload.c Fix the warning: main/elfload.c: In function 'load_segments': main/elfload.c:138:21: warning: variable 'time' set but not used [-Wunused-but-set-variable] by setting 'time' (and 'start_time') only when DEBUG is enabled. Change-Id: I19975963f180f7380dcae8d07f088505203fdbc9 Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-on: http://review.coreboot.org/3295 Tested-by: build bot (Jenkins) Reviewed-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> See http://review.coreboot.org/3295 for details. -gerrit

1 0

Patch merged into filo/master: fcb1a04 Fix random warnings in char_io.c
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit fcb1a046733a7e4270860183cf5402883f636434 Author: Nico Huber <nico.huber(a)secunet.com> Date: Sun May 26 12:00:16 2013 +0200 Fix random warnings in char_io.c Fix two warnings: char_io.c: In function 'grub_printf': char_io.c:86:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable] char_io.c: In function 'init_page': char_io.c:104:4: warning: implicit declaration of function 'get_cb_version' [-Wimplicit-function-declaration] by removing the variable 'ret' and including sys_info.h for the 'get_cb_version' prototype. Change-Id: I596b17b3594f8909ff5d6eccb9cb0be475bf052e Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-on: http://review.coreboot.org/3294 Tested-by: build bot (Jenkins) Reviewed-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> See http://review.coreboot.org/3294 for details. -gerrit

1 0

Patch merged into coreboot/master: 038aa29 console: add support for QEMU's debugcon
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit 038aa29dc283205a993578f0ff3881df12ebdb0a Author: Gerd Hoffmann <kraxel(a)redhat.com> Date: Wed May 29 13:06:22 2013 +0200 console: add support for QEMU's debugcon Add support for sending debug output to an I/O port. It can be used together with QEMU's isa-debugcon driver to log the coreboot output to a file. The port is configurable and defaults to 0x402 which has established as the de facto standard. For example, SeaBIOS+OVMF [1] use that one too. [1] http://www.linux-kvm.org/page/OVMF Open Virtual Machine Firmware Change-Id: I0803f7fc70030242f80003e25c9449c37d71975e Signed-off-by: Gerd Hoffmann <kraxel(a)redhat.com> Reviewed-on: http://review.coreboot.org/3331 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com> See http://review.coreboot.org/3331 for details. -gerrit

1 0

Patch merged into coreboot/master: 66da043 haswell: allow for disabled hyperthreading
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit 66da043e48c65e6d89b385d840e5f7c53e482db9 Author: Aaron Durbin <adurbin(a)chromium.org> Date: Tue May 28 14:26:29 2013 -0500 haswell: allow for disabled hyperthreading There were assumptions being made in the haswell MP and SMM code which assumed the APIC id space was 1:1 w.r.t. cpu number. When hyperthreading is disabled the APIC ids of the logical processors are all even. That means the APIC id space is sparse. Handle this situation. Change-Id: Ibe79ab156c0a171208a77db8a252aa5b73205d6c Signed-off-by: Aaron Durbin <adurbin(a)chromium.org> Reviewed-on: http://review.coreboot.org/3353 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com> See http://review.coreboot.org/3353 for details. -gerrit

1 0

Patch merged into coreboot/master: 27435d3 haswell: fix overflow handling TOUUD
by gerrit＠coreboot.org June 3, 2013

June 3, 2013

the following patch was just integrated into master: commit 27435d3bcdd4c7bccb326f77ca64a54d3fb1170a Author: Aaron Durbin <adurbin(a)chromium.org> Date: Mon Jun 3 09:46:56 2013 -0500 haswell: fix overflow handling TOUUD It's possible that the TOUUD can be set to less than 4GiB. When that is the case the size_k variable is an extremely large value. Instead ensure TOUUD is greater than 4GiB before adding said resources. Change-Id: I456633d6210824e60665281538300fd15656b86d Signed-off-by: Aaron Durbin <adurbin(a)chromium.org> Reviewed-on: http://review.coreboot.org/3352 Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com> See http://review.coreboot.org/3352 for details. -gerrit

1 0