coreboot

coreboot@coreboot.org

6 participants
38360 discussions

by Jonathan Zhang (Infra)

Hi, FW reserved regions are needed for OS to access FW data such as ACPI APEI (ACPI Platform Error Interface) data. The ACPI APEI data may include previous boot error record (BERT), or error record reported through firmware runtime component (such as SMI handler). Such FW reserved region needs to be: a. Updateable by firmware bootloader or firmware runtime component. b. Updateable by OS drivers (for instance, the Linux ACPI BERT driver needs to be able to grab and clear the record. c. Reserved from being used as normal memory by OS. If I just add BERT memory region through cbmem_add(), I get this OS error message: [ 12.899311] APEI: Can not request [mem 0x75319000-0x7531911b] for APEI BERT registers If I also make this change: diff --git a/src/lib/bootmem.c b/src/lib/bootmem.c index 1fe23c2828..721d74ac75 100644 --- a/src/lib/bootmem.c +++ b/src/lib/bootmem.c @@ -49,7 +49,7 @@ static uint32_t bootmem_to_lb_tag(const enum bootmem_type tag) case BM_MEM_BL31: return LB_MEM_RESERVED; case BM_MEM_TABLE: - return LB_MEM_TABLE; + return LB_MEM_RESERVED; Then it works fine, I get this: [ 12.902253] BERT: Error records from previous boot: [ 12.912270] [Hardware Error]: event severity: fatal [ 12.922288] [Hardware Error]: precise tstamp: 2020-08-13 21:56:18 [ 12.934980] [Hardware Error]: Error 0, type: fatal [ 12.944995] [Hardware Error]: section_type: general processor error [ 12.958227] [Hardware Error]: processor_type: 0, IA32/X64 [ 12.969674] [Hardware Error]: error_type: 0x02 [ 12.979157] [Hardware Error]: TLB error [ 12.987390] [Hardware Error]: version_info: 0x000000000005065a [ 12.999733] [Hardware Error]: processor_id: 0x0000000000000000 So the question is why we need to make memory attribute of cbmem as LB_MEM_TABLE, instead of LB_MEM_RESERVED? Thank you, Jonathan

1 day, 6 hours

[ANNOUNCE] SeaBIOS 1.14.0

by Kevin O'Connor

The 1.14.0 version of SeaBIOS has now been released. For more information on the release, please see: http://seabios.org/Releases New in this release: * New virtio MMIO support. Support for finding virtio MMIO devices via an ACPI DSDT parser. Support for handling a large number of virtio devices. * Improved handling of USB keyboards with non-standard packet size. * Improved KVM CPU frequency detection. * Support for PCI mmconfig support on QEMU. * Several bug fixes and code cleanups. For information on obtaining SeaBIOS, please see: http://seabios.org/Download ===== git shortlog -n rel-1.13.0..rel-1.14.0 ===== Gerd Hoffmann (25): boot: cache HALT priority virtio-scsi: skip initializing non-bootable devices nvme: skip initializing non-bootable devices timer: add tsctimer_setfreq() kvm: detect unconditionally kvm: add support for reading tsc frequency via cpuid. kvm: add support for reading tsc frequency from kvmclock sercon: vbe modeset is int 10h function 4f02 not 4f00 pci: factor out ioconfig_cmd() pci: add mmconfig support qemu: factor out qemu_cfg_detect() qemu: rework e820 detection qemu: check rtc presence before reading cpu count from cmos virtio-mmio: device probing and initialization. virtio-mmio: add support to vp_*() functions virtio-mmio: add support for scsi devices. virtio-mmio: add support for block devices. virtio-mmio: print device type acpi: add xsdt support acpi: add dsdt parser acpi: skip kbd init if not present acpi: find and register virtio-mmio devices rewrap Makefile lines. pci: fix mmconfig support vga: fix cirrus bios Kevin O'Connor (6): usb-hid: Improve max packet size checking Revert "ps2port: adjust init routine to fix PS/2 keyboard issues" boot: Fixup check for only one item in boot list vgabios: Fix preserve memory flag in handle_1000 ldnoexec: Add script to remove ET_EXEC flag from intermediate build objects docs: Note v1.14.0 release Paul Menzel (5): std/tcg: Replace zero-length array with flexible-array member boot: Extend `etc/show-boot-menu` to configure skipping boot menu with only one device boot: Log, if boot menu is skipped cdrom: Demote `scsi_is_ready` return print to debug level nvme: Increase `nvme_cmd_readwrite()` message log level from 3 to 5 Matt DeVillier (4): hw/usb-hid: Don't abort if setting key repeat rate fails Skip boot menu and timeout with only one boot device ps2port: adjust init routine to fix PS/2 keyboard issues boot: Fix logic for boot menu display Stefan Berger (3): tcgbios: Only write logs for PCRs that are in active PCR banks tcgbios: Fix the vendorInfoSize to be of type u8 tcgbios: Add support for SHA3 type of algorithms Alexey Kirillov (2): boot: Detect strict boot order (HALT record) in function virtio: Do not init non-bootable devices Christian Ehrhardt (1): build: use -fcf-protection=none when available Jason Andryuk (1): serialio: Preserve Xen DebugOutputPort Roman Bolshakov (1): timer: Handle decrements of PIT counter Stefan Reiter (1): virtio-scsi: fix boot prio detection by using correct lun

4 days, 14 hours

[GSoC]: Request for testing of "Address Sanitizer" feature

by Harshit Sharma

Hello everyone, I hope you are staying safe and healthy through these unusual times! Many of you may already know about the project I am working on and the progress we made through my blog post <https://blogs.coreboot.org/blog/2020/07/16/gsoc-address-sanitizer-part-1/>. We had added support for ASan in ramstage for all x86 platforms. In the next phase, to check the boards on which ASan in romstage can be enabled, I ran 'make what-jenkins-does' and build failed only for the boards having either Braswell or i440bx chipsets. So, I am pretty hopeful we can add support for ASan in romstage on all x86 platforms except these two. We have already tested and added support for ASan in romstage on Haswell <https://review.coreboot.org/c/coreboot/+/44160> and Apollolake <https://review.coreboot.org/c/coreboot/+/44159> platforms based on the hardware that Werner and I had access to. I am currently in the final phase of GSoC and before signing off, I want this debugging tool to be available on as many platforms as possible. So, if you have access to a hardware based on an x86 platform (other than Haswell and Apollolake), I would be grateful if you volunteer to test this feature. Please follow these steps if you are willing to test ASan: 1. Pull this change: https://review.coreboot.org/c/coreboot/+/43938 2. Rebuild the toolchain 3. Select 'Address sanitizer support' from the 'General setup' menu 3. Build coreboot 4. Flash the image and run it on your hardware It should boot cleanly and you must see statements prefixed with 'ASan' in your console log in case of both romstage and ramstage. If ASan in romstage works on your hardware, please reply to this thread with the x86 platform name and the name of the board you used, so that I can push a change enabling ASan on that platform. If it gets stuck while booting or you get a compile error while building, please share the log along with the hardware details. We might be able to resolve those errors and add support for ASan on that platform too. Thanking you in advance, Best, Harshit (hst on IRC)

5 days, 2 hours

New Defects reported by Coverity Scan for coreboot

by scan-admin＠coverity.com

Hi, Please find the latest report on new defect(s) introduced to coreboot found with Coverity Scan. 2 new defect(s) introduced to coreboot found with Coverity Scan. 1 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 2 of 2 defect(s) ** CID 1431154: Control flow issues (DEADCODE) /src/ec/kontron/kempld/kempld_i2c.c: 254 in kempld_i2c_device_init() ________________________________________________________________________________________________________ *** CID 1431154: Control flow issues (DEADCODE) /src/ec/kontron/kempld/kempld_i2c.c: 254 in kempld_i2c_device_init() 248 if (spec_major == 1) 249 prescale = KEMPLD_CLK / (KEMPLD_I2C_FREQ_STD * 5) - 1000; 250 else 251 prescale = KEMPLD_CLK / (KEMPLD_I2C_FREQ_STD * 4) - 3000; 252 253 if (prescale < 0) >>> CID 1431154: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "prescale = 0L;". 254 prescale = 0; 255 256 /* Round to the best matching value */ 257 prescale_corr = prescale / 1000; 258 if (prescale % 1000 >= 500) 259 prescale_corr++; ** CID 1353302: (OVERRUN) /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 74 in keyboard_raw_drive_column() /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 71 in keyboard_raw_drive_column() /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 66 in keyboard_raw_drive_column() /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 83 in keyboard_raw_drive_column() ________________________________________________________________________________________________________ *** CID 1353302: (OVERRUN) /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 74 in keyboard_raw_drive_column() 68 69 if (out == KEYBOARD_COLUMN_ALL) { 70 /* drive low (clear bit) */ 71 bsrr |= gpio_list[j].mask << 16; 72 } else if (out == KEYBOARD_COLUMN_NONE) { 73 /* put output in hi-Z state (set bit) */ >>> CID 1353302: (OVERRUN) >>> Overrunning array "gpio_list" of 41 16-byte elements at element index 47 (byte offset 767) using index "j" (which evaluates to 47). 74 bsrr |= gpio_list[j].mask; 75 } else if (j - GPIO_KB_OUT00 == out) { 76 /* 77 * Drive specified output low, others => hi-Z. 78 * 79 * To avoid conflict, tri-state all outputs /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 71 in keyboard_raw_drive_column() 65 for (j = GPIO_KB_OUT00; j <= GPIO_KB_OUT12; j++) { 66 if (gpio_list[j].port != kb_out_ports[i]) 67 continue; 68 69 if (out == KEYBOARD_COLUMN_ALL) { 70 /* drive low (clear bit) */ >>> CID 1353302: (OVERRUN) >>> Overrunning array "gpio_list" of 41 16-byte elements at element index 47 (byte offset 767) using index "j" (which evaluates to 47). 71 bsrr |= gpio_list[j].mask << 16; 72 } else if (out == KEYBOARD_COLUMN_NONE) { 73 /* put output in hi-Z state (set bit) */ 74 bsrr |= gpio_list[j].mask; 75 } else if (j - GPIO_KB_OUT00 == out) { 76 /* /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 66 in keyboard_raw_drive_column() 60 61 for (i = 0; i < ARRAY_SIZE(kb_out_ports); i++) { 62 uint32_t bsrr = 0; 63 int j; 64 65 for (j = GPIO_KB_OUT00; j <= GPIO_KB_OUT12; j++) { >>> CID 1353302: (OVERRUN) >>> Overrunning array "gpio_list" of 41 16-byte elements at element index 47 (byte offset 767) using index "j" (which evaluates to 47). 66 if (gpio_list[j].port != kb_out_ports[i]) 67 continue; 68 69 if (out == KEYBOARD_COLUMN_ALL) { 70 /* drive low (clear bit) */ 71 bsrr |= gpio_list[j].mask << 16; /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 83 in keyboard_raw_drive_column() 77 * Drive specified output low, others => hi-Z. 78 * 79 * To avoid conflict, tri-state all outputs 80 * first, then assert specified output. 81 */ 82 keyboard_raw_drive_column(KEYBOARD_COLUMN_NONE); >>> CID 1353302: (OVERRUN) >>> Overrunning array "gpio_list" of 41 16-byte elements at element index 47 (byte offset 767) using index "j" (which evaluates to 47). 83 bsrr |= gpio_list[j].mask << 16; 84 done = 1; 85 break; 86 } 87 } 88 ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P…

1 week

intel leak hits torrents

by Simon Newton

https://www.tomshardware.com/news/massive-20gb-intel-data-breach-floods-the… n fact, the title of many of the documents do correlate to the list of purported information posted by the leaker: - Intel ME Bringup guides + (flash) tooling + samples for various platforms - Kabylake (Purley Platform) BIOS Reference Code and Sample Code + Initialization code (some of it as exported git repos with full history) - Intel CEFDK (Consumer Electronics Firmware Development Kit (Bootloader stuff)) SOURCES - Silicon / FSP source code packages for various platforms - Various Intel Development and Debugging Tools - Simics Simulation for Rocket Lake S and potentially other platforms - Various roadmaps and other documents - Binaries for Camera drivers Intel made for SpaceX - Schematics, Docs, Tools + Firmware for the unreleased Tiger Lake platform - (very horrible) Kabylake FDK training videos - Intel Trace Hub + decoder files for various Intel ME versions - Elkhart Lake Silicon Reference and Platform Sample Code - Some Verilog stuff for various Xeon Platforms, unsure what it is exactly. - Debug BIOS/TXE builds for various Platforms - Bootguard SDK (encrypted zip) - Intel Snowridge / Snowfish Process Simulator ADK - Various schematics - Intel Marketing Material Templates (InDesign) - Lots of other things -- Kind Regards, Simon Newton E: Simon.newton(a)gmail.com

1 week, 1 day

New Defects reported by Coverity Scan for coreboot

by scan-admin＠coverity.com

Hi, Please find the latest report on new defect(s) introduced to coreboot found with Coverity Scan. 7 new defect(s) introduced to coreboot found with Coverity Scan. 2 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 7 of 7 defect(s) ** CID 1431127: Memory - corruptions (ARRAY_VS_SINGLETON) /src/lib/gcov-glue.c: 128 in coverage_init() ________________________________________________________________________________________________________ *** CID 1431127: Memory - corruptions (ARRAY_VS_SINGLETON) /src/lib/gcov-glue.c: 128 in coverage_init() 122 extern long __CTOR_LIST__; 123 typedef void (*func_ptr)(void); 124 func_ptr *ctor = (func_ptr *) &__CTOR_LIST__; 125 if (ctor == NULL) 126 return; 127 >>> CID 1431127: Memory - corruptions (ARRAY_VS_SINGLETON) >>> Using "ctor" as an array. This might corrupt or misinterpret adjacent memory locations. 128 for (; *ctor != (func_ptr) 0; ctor++) 129 (*ctor)(); 130 } 131 132 void __gcov_flush(void); 133 static void coverage_exit(void *unused) ** CID 1431126: (DEADCODE) /src/lib/libgcov.c: 425 in gcov_exit() /src/lib/libgcov.c: 391 in gcov_exit() ________________________________________________________________________________________________________ *** CID 1431126: (DEADCODE) /src/lib/libgcov.c: 425 in gcov_exit() 419 fname += 2; 420 421 /* Build relocated filename, stripping off leading 422 * directories from the initial filename if requested. 423 */ 424 if (gcov_prefix_strip > 0) { >>> CID 1431126: (DEADCODE) >>> Execution cannot reach this statement: "level = 0;". 425 int level = 0; 426 427 s = fname; 428 if (IS_DIR_SEPARATOR(*s)) 429 ++s; 430 /src/lib/libgcov.c: 391 in gcov_exit() 385 } else 386 #endif 387 prefix_length = 0; 388 389 /* If no prefix was specified and a prefix strip, then we assume 390 relative. */ >>> CID 1431126: (DEADCODE) >>> Execution cannot reach the expression "prefix_length == 0UL" inside this statement: "if (gcov_prefix_strip != 0 ...". 391 if (gcov_prefix_strip != 0 && prefix_length == 0) { 392 gcov_prefix = "."; 393 prefix_length = 1; 394 } 395 /* Allocate and initialize the filename scratch space plus one. */ 396 gi_filename = (char *) alloca(prefix_length + gcov_max_filename + 2); ** CID 1431125: Control flow issues (DEADCODE) /src/lib/gcov-glue.c: 126 in coverage_init() ________________________________________________________________________________________________________ *** CID 1431125: Control flow issues (DEADCODE) /src/lib/gcov-glue.c: 126 in coverage_init() 120 static void coverage_init(void *unused) 121 { 122 extern long __CTOR_LIST__; 123 typedef void (*func_ptr)(void); 124 func_ptr *ctor = (func_ptr *) &__CTOR_LIST__; 125 if (ctor == NULL) >>> CID 1431125: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "return;". 126 return; 127 128 for (; *ctor != (func_ptr) 0; ctor++) 129 (*ctor)(); 130 } 131 ** CID 1431124: Integer handling issues (BAD_SHIFT) /src/security/intel/txt/common.c: 166 in validate_acm() ________________________________________________________________________________________________________ *** CID 1431124: Integer handling issues (BAD_SHIFT) /src/security/intel/txt/common.c: 166 in validate_acm() 160 /* 161 * Causes #GP if acm_header->size > processor internal authenticated 162 * code area capacity. 163 * SAFER MODE EXTENSIONS REFERENCE. 164 * Intel 64 and IA-32 Architectures Software Developer Manuals Vol 2D 165 */ >>> CID 1431124: Integer handling issues (BAD_SHIFT) >>> In expression "1UL << log2_ceil((acm_header->size & 0xffffffU) << 2)", shifting by a negative amount has undefined behavior. The shift amount, "log2_ceil((acm_header->size & 0xffffffU) << 2)", is -1. 166 const size_t acm_len = 1UL << log2_ceil((acm_header->size & 0xffffff) << 2); 167 if (max_size_acm_area < acm_len) { 168 printk(BIOS_ERR, "TEE-TXT: BIOS ACM doesn't fit into AC execution region\n"); 169 return ACM_E_NOT_FIT_INTO_CPU_ACM_MEM; 170 } 171 ** CID 1431123: Control flow issues (DEADCODE) /src/lib/libgcov.c: 398 in gcov_exit() ________________________________________________________________________________________________________ *** CID 1431123: Control flow issues (DEADCODE) /src/lib/libgcov.c: 398 in gcov_exit() 392 gcov_prefix = "."; 393 prefix_length = 1; 394 } 395 /* Allocate and initialize the filename scratch space plus one. */ 396 gi_filename = (char *) alloca(prefix_length + gcov_max_filename + 2); 397 if (prefix_length) >>> CID 1431123: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "memcpy(gi_filename, gcov_pr...". 398 memcpy(gi_filename, gcov_prefix, prefix_length); 399 gi_filename_up = gi_filename + prefix_length; 400 401 /* Now merge each file. */ 402 for (gi_ptr = gcov_list; gi_ptr; gi_ptr = gi_ptr->next) { 403 unsigned int n_counts; ** CID 1255942: Code maintainability issues (UNUSED_VALUE) /src/drivers/usb/ehci_debug.c: 572 in usbdebug_init_() ________________________________________________________________________________________________________ *** CID 1255942: Code maintainability issues (UNUSED_VALUE) /src/drivers/usb/ehci_debug.c: 572 in usbdebug_init_() 566 dbgp_mdelay(100); 567 568 struct ehci_dbg_port *port = (void *)(uintptr_t)info->ehci_debug; 569 ret = dbgp_probe_gadget(port, &info->ep_pipe[0]); 570 if (ret < 0) { 571 dprintk(BIOS_INFO, "Could not probe gadget on debug port.\n"); >>> CID 1255942: Code maintainability issues (UNUSED_VALUE) >>> Assigning value "-6" to "ret" here, but that stored value is overwritten before it can be used. 572 ret = -6; 573 goto err; 574 } 575 576 info->ep_pipe[0].status &= ~DBGP_EP_NOT_PRESENT; 577 ** CID 1241836: Code maintainability issues (UNUSED_VALUE) /src/drivers/usb/ehci_debug.c: 561 in usbdebug_init_() ________________________________________________________________________________________________________ *** CID 1241836: Code maintainability issues (UNUSED_VALUE) /src/drivers/usb/ehci_debug.c: 561 in usbdebug_init_() 555 ctrl |= DBGP_CLAIM; 556 write32(&ehci_debug->control, ctrl); 557 ctrl = read32(&ehci_debug->control); 558 if ((ctrl & DBGP_CLAIM) != DBGP_CLAIM) { 559 dprintk(BIOS_INFO, "No device in EHCI debug port.\n"); 560 write32(&ehci_debug->control, ctrl & ~DBGP_CLAIM); >>> CID 1241836: Code maintainability issues (UNUSED_VALUE) >>> Assigning value "-4" to "ret" here, but that stored value is overwritten before it can be used. 561 ret = -4; 562 goto err; 563 } 564 dprintk(BIOS_INFO, "EHCI debug port enabled.\n"); 565 566 dbgp_mdelay(100); ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P…

1 week, 3 days

How to choose my ‘‘mainboard vendor’’ for Sony laptops?

by hd-scania＠users.sf.net

Laptop model: Sony VPCCB17FG Cores: Intel i7-2620M Coreboot release to be flashed: 4.12 RAM’s: 7.45GiB Graphics: AMD Radeon Old BIOS: By AMI, versioned R0242v2 (no longer receiving Sony’s BIOS upgrades since 2012) Mainboard vendor 1. 51NB (VENDOR_51NB) (NEW) 2. ADLINK (VENDOR_ADLINK) (NEW) 3. AMD (VENDOR_AMD) (NEW) 4. AOpen (VENDOR_AOPEN) (NEW) 5. Apple (VENDOR_APPLE) (NEW) 6. ASROCK (VENDOR_ASROCK) (NEW) 7. ASUS (VENDOR_ASUS) (NEW) 8. BAP (VENDOR_BAP) (NEW) 9. Biostar (VENDOR_BIOSTAR) (NEW) 10. Cavium (VENDOR_CAVIUM) (NEW) 11. CompuLab (VENDOR_COMPULAB) (NEW) 12. ELMEX (VENDOR_ELMEX) (NEW) > 13. Emulation (VENDOR_EMULATION) (NEW) 14. Facebook (VENDOR_FACEBOOK) (NEW) 15. Foxconn (VENDOR_FOXCONN) (NEW) 16. Getac (VENDOR_GETAC) (NEW) 17. GIGABYTE (VENDOR_GIGABYTE) (NEW) 18. GizmoSphere (VENDOR_GIZMOSPHERE) (NEW) 19. Google (VENDOR_GOOGLE) (NEW) 20. HP (VENDOR_HP) (NEW) 21. iBase (VENDOR_IBASE) (NEW) 22. Intel (VENDOR_INTEL) (NEW) 23. Jetway (VENDOR_JETWAY) (NEW) 24. Kontron (VENDOR_KONTRON) (NEW) 25. Lenovo (VENDOR_LENOVO) (NEW) 26. Libretrend (VENDOR_LIBRETREND) (NEW) 27. LiPPERT (VENDOR_LIPPERT) (NEW) 28. MSI (VENDOR_MSI) (NEW) 29. Open Compute Project (VENDOR_OCP) (NEW) 30. OpenCellular (VENDOR_OPENCELLULAR) (NEW) 31. Packard Bell (VENDOR_PACKARDBELL) (NEW) 32. PC Engines (VENDOR_PCENGINES) (NEW) 33. Portwell (VENDOR_PORTWELL) (NEW) 34. Protectli (VENDOR_PROTECTLI) (NEW) 35. Purism (VENDOR_PURISM) (NEW) 36. RAZER (VENDOR_RAZER) (NEW) 37. Roda (VENDOR_RODA) (NEW) 38. SAMSUNG (VENDOR_SAMSUNG) (NEW) 39. Sapphire (VENDOR_SAPPHIRE) (NEW) 40. Scaleway (VENDOR_SCALEWAY) (NEW) 41. Siemens (VENDOR_SIEMENS) (NEW) 42. SiFive (VENDOR_SIFIVE) (NEW) 43. Supermicro (VENDOR_SUPERMICRO) (NEW) 44. System76 (VENDOR_SYSTEM76) (NEW) 45. TI (VENDOR_TI) (NEW) 46. UP (VENDOR_UP) (NEW) choice[1-46]:

1 week, 5 days

New Defects reported by Coverity Scan for coreboot

by scan-admin＠coverity.com

Hi, Please find the latest report on new defect(s) introduced to coreboot found with Coverity Scan. 1 new defect(s) introduced to coreboot found with Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 1 of 1 defect(s) ** CID 1353302: (OVERRUN) /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 74 in keyboard_raw_drive_column() /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 83 in keyboard_raw_drive_column() /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 71 in keyboard_raw_drive_column() /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 66 in keyboard_raw_drive_column() ________________________________________________________________________________________________________ *** CID 1353302: (OVERRUN) /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 74 in keyboard_raw_drive_column() 68 69 if (out == KEYBOARD_COLUMN_ALL) { 70 /* drive low (clear bit) */ 71 bsrr |= gpio_list[j].mask << 16; 72 } else if (out == KEYBOARD_COLUMN_NONE) { 73 /* put output in hi-Z state (set bit) */ >>> CID 1353302: (OVERRUN) >>> Overrunning array "gpio_list" of 41 16-byte elements at element index 47 (byte offset 767) using index "j" (which evaluates to 47). 74 bsrr |= gpio_list[j].mask; 75 } else if (j - GPIO_KB_OUT00 == out) { 76 /* 77 * Drive specified output low, others => hi-Z. 78 * 79 * To avoid conflict, tri-state all outputs /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 83 in keyboard_raw_drive_column() 77 * Drive specified output low, others => hi-Z. 78 * 79 * To avoid conflict, tri-state all outputs 80 * first, then assert specified output. 81 */ 82 keyboard_raw_drive_column(KEYBOARD_COLUMN_NONE); >>> CID 1353302: (OVERRUN) >>> Overrunning array "gpio_list" of 41 16-byte elements at element index 47 (byte offset 767) using index "j" (which evaluates to 47). 83 bsrr |= gpio_list[j].mask << 16; 84 done = 1; 85 break; 86 } 87 } 88 /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 71 in keyboard_raw_drive_column() 65 for (j = GPIO_KB_OUT00; j <= GPIO_KB_OUT12; j++) { 66 if (gpio_list[j].port != kb_out_ports[i]) 67 continue; 68 69 if (out == KEYBOARD_COLUMN_ALL) { 70 /* drive low (clear bit) */ >>> CID 1353302: (OVERRUN) >>> Overrunning array "gpio_list" of 41 16-byte elements at element index 47 (byte offset 767) using index "j" (which evaluates to 47). 71 bsrr |= gpio_list[j].mask << 16; 72 } else if (out == KEYBOARD_COLUMN_NONE) { 73 /* put output in hi-Z state (set bit) */ 74 bsrr |= gpio_list[j].mask; 75 } else if (j - GPIO_KB_OUT00 == out) { 76 /* /3rdparty/chromeec/chip/stm32/keyboard_raw.c: 66 in keyboard_raw_drive_column() 60 61 for (i = 0; i < ARRAY_SIZE(kb_out_ports); i++) { 62 uint32_t bsrr = 0; 63 int j; 64 65 for (j = GPIO_KB_OUT00; j <= GPIO_KB_OUT12; j++) { >>> CID 1353302: (OVERRUN) >>> Overrunning array "gpio_list" of 41 16-byte elements at element index 47 (byte offset 767) using index "j" (which evaluates to 47). 66 if (gpio_list[j].port != kb_out_ports[i]) 67 continue; 68 69 if (out == KEYBOARD_COLUMN_ALL) { 70 /* drive low (clear bit) */ 71 bsrr |= gpio_list[j].mask << 16; ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P…

2 weeks

will my machine support coreboot ?

by Nihar Khuntia

Hi Developers, my laptop has hardwares details as attached in this email. will it support coreboot ? regards nihar

2 weeks, 1 day

Externally flashing Coreboot using microSD as external programmer

by HD Scania

As my title suggests, what to be prepared for _‘‘externally’’_ flashing Coreboot on a microSD? If external flash is done, *what* to be prepared to port Coreboot for my Sony VPCCB17FG laptop, with known supported Intel *HM65* chipsets, but even the vendor, Sony, is *unknown* to be supported or not? Are there any up-to-date wikies or docs on committing like this? > (Reference: https://mail.coreboot.org/hyperkitty/list/coreboot@coreboot.org/thread/62E6…)

2 weeks, 1 day

Jump to page:

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

coreboot