coreboot September 2011

coreboot@coreboot.org

40 participants
224 discussions

New patch to review: 0966f88 Add multiboot header
by Patrick Georgi 05 Sep '11

05 Sep '11

Patrick Georgi (patrick(a)georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/197 -gerrit commit 0966f88267d3723b22d8a5c088cb7daeab21e953 Author: Mathias Krause <mathias.krause(a)secunet.com> Date: Tue Jun 7 13:53:31 2011 +0200 Add multiboot header Change-Id: Ibe8e879a3a6f533f97aa8381697afb1f14d1f87d Signed-off-by: Mathias Krause <mathias.krause(a)secunet.com> Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> --- Config.in | 7 +++++++ i386/ldscript | 2 +- main/Makefile.inc | 1 + main/mb_hdr.c | 32 ++++++++++++++++++++++++++++++++ 4 files changed, 41 insertions(+), 1 deletions(-) diff --git a/Config.in b/Config.in index b1da531..251bd2a 100644 --- a/Config.in +++ b/Config.in @@ -9,6 +9,13 @@ config TARGET_I386 bool default y +config MULTIBOOT_IMAGE + bool "Include a MultiBoot header" + default y + help + Including a MultiBoot header makes FILO chainloadable by MultiBoot + compliant boot loaders like GRUB. + menu "Interface Options" config USE_GRUB diff --git a/i386/ldscript b/i386/ldscript index 7fba856..b260c51 100644 --- a/i386/ldscript +++ b/i386/ldscript @@ -40,7 +40,7 @@ SECTIONS /* Start of the program. * Now the version string is in the note, we must include it * in the program. Otherwise we lose the string after relocation. */ - . = ALIGN(16); + . = ALIGN(4096); _start = .; /* Putting ELF notes near beginning of file might help bootloaders. diff --git a/main/Makefile.inc b/main/Makefile.inc index c7290cf..e2fab04 100644 --- a/main/Makefile.inc +++ b/main/Makefile.inc @@ -16,6 +16,7 @@ # Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. # +TARGETS-$(CONFIG_MULTIBOOT_IMAGE) += main/mb_hdr.o TARGETS-y += main/filo.o main/strtox.o TARGETS-y += main/elfload.o main/ipchecksum.o TARGETS-$(CONFIG_SUPPORT_SOUND) += main/sound.o diff --git a/main/mb_hdr.c b/main/mb_hdr.c new file mode 100644 index 0000000..bb01427 --- /dev/null +++ b/main/mb_hdr.c @@ -0,0 +1,32 @@ +/* + * This file is part of FILO. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + */ + +/* Support for MultiBoot */ +#define MB_HEADER_MAGIC 0x1BADB002 +#define MB_RQ_FLAGS_4K 1 +#define MB_REQUESTED_FLAGS (MB_RQ_FLAGS_4K) + +/* this header is parsed by the boot loader that loads FILO, e.g. GRUB */ +struct mb_header { + long magic; /* multiboot magic */ + long features; /* requested features */ + long chksum; /* chksum for whole structure */ +} mb_header __attribute__((section(".hdr.mb"))) = { + .magic = MB_HEADER_MAGIC, + .features = MB_REQUESTED_FLAGS, + .chksum = -(MB_HEADER_MAGIC + MB_REQUESTED_FLAGS), +};

1 0

New patch to review: 0ca6ae9 Show which file cannot be openend in file_open(), eases debuging.
by Patrick Georgi 05 Sep '11

05 Sep '11

Patrick Georgi (patrick(a)georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/193 -gerrit commit 0ca6ae94bb6be913aef972b93e8e1a05f98cdfad Author: Mathias Krause <mathias.krause(a)secunet.com> Date: Thu Jun 3 15:39:46 2010 +0200 Show which file cannot be openend in file_open(), eases debuging. Change-Id: I06fd3576aa3443588e67234bce748f533c36a8ff Signed-off-by: Mathias Krause <mathias.krause(a)secunet.com> Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> --- fs/vfs.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/vfs.c b/fs/vfs.c index 18dbcbc..18db6ec 100644 --- a/fs/vfs.c +++ b/fs/vfs.c @@ -191,7 +191,7 @@ int file_open(const char *filename) filepos = 0; errnum = 0; if (!fsys->dir_func((char *) path)) { - printf("File not found.\n"); + printf("File not found '%s'.\n", filename); goto out; } retval = 1;

1 0

New patch to review: 207611a Make 'configfile' and 'root' commands be available from scripting mode.
by Patrick Georgi 05 Sep '11

05 Sep '11

Patrick Georgi (patrick(a)georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/194 -gerrit commit 207611a5910a4604c730658e5b919e93271b1ebc Author: Mathias Krause <mathias.krause(a)secunet.com> Date: Mon Mar 8 11:46:29 2010 +0100 Make 'configfile' and 'root' commands be available from scripting mode. When a 'configfile' command is encountered, restart with that new config file. Change-Id: I042506e236b0fb7f10e20ea3f0f3764414167dce Signed-off-by: Mathias Krause <mathias.krause(a)secunet.com> Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> --- main/grub/builtins.c | 9 +++++++-- main/grub/grub.c | 16 +++++++++++++++- 2 files changed, 22 insertions(+), 3 deletions(-) diff --git a/main/grub/builtins.c b/main/grub/builtins.c index b09076a..79d237e 100644 --- a/main/grub/builtins.c +++ b/main/grub/builtins.c @@ -49,6 +49,9 @@ unsigned long install_partition = 0x20000; unsigned long boot_drive = 0; char config_file[128] = "\0"; +/* indicator if we encountered a 'configfile' command and have to restart */ +int reload_configfile = 0; + kernel_t kernel_type; /* The fallback entry. */ @@ -91,6 +94,7 @@ void init_config(void) fallback_entryno = -1; fallback_entries[0] = -1; grub_timeout = -1; + reload_configfile = 0; } int check_password(char *entered, char *expected, password_t type) @@ -318,6 +322,7 @@ static int configfile_func(char *arg, int flags) /* Force to load the configuration file. */ is_opened = 0; keep_cmdline_running = 0; + reload_configfile = 1; /* Make sure that the user will not be authoritative. */ auth = 0; @@ -328,7 +333,7 @@ static int configfile_func(char *arg, int flags) static struct builtin builtin_configfile = { "configfile", configfile_func, - BUILTIN_CMDLINE | BUILTIN_HELP_LIST, + BUILTIN_CMDLINE | BUILTIN_MENU | BUILTIN_HELP_LIST, "configfile FILE", "Load FILE as the configuration file." }; @@ -1269,7 +1274,7 @@ static int root_func(char *arg, int flags) static struct builtin builtin_root = { "root", root_func, - BUILTIN_CMDLINE | BUILTIN_HELP_LIST, + BUILTIN_CMDLINE | BUILTIN_MENU | BUILTIN_HELP_LIST, "root [DEVICE]", "Set the current \"root device\" to the device DEVICE." }; diff --git a/main/grub/grub.c b/main/grub/grub.c index 15f0ece..7881a98 100644 --- a/main/grub/grub.c +++ b/main/grub/grub.c @@ -28,6 +28,7 @@ #include <arch/timer.h> extern char config_file[]; +extern int reload_configfile; char PASSWORD_BUF[PASSWORD_BUFLEN]; /* The buffer for the password. */ char DEFAULT_FILE_BUF[DEFAULT_FILE_BUFLEN]; /* THe buffer for the filename of "/boot/grub/default". */ @@ -813,6 +814,11 @@ static void run_menu(char *menu_entries, char *config_entries, int num_entries, break; } + /* return to reload the config file */ + if (reload_configfile) { + return; + } + show_menu = 1; goto restart; } @@ -1004,9 +1010,12 @@ void grub_main(void) /* Never return. */ for (;;) { char buf[10]; /* This is good enough. */ - char *default_file = (char *) DEFAULT_FILE_BUF; + char *default_file; int i; +restart: + default_file = (char *) DEFAULT_FILE_BUF; + reset(); /* Here load the configuration file. */ @@ -1124,6 +1133,11 @@ void grub_main(void) cmdline); (builtin->func) (arg, BUILTIN_MENU); errnum = 0; + + /* if the command was configfile, restart */ + if (reload_configfile) { + goto restart; + } } else /* Ignored. */ continue;

1 0

New patch to review: a7ae3fb Wait longer on ATAPI devices
by Patrick Georgi 05 Sep '11

05 Sep '11

Patrick Georgi (patrick(a)georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/191 -gerrit commit a7ae3fb9d6257f80036b0db090fbe021d04494b4 Author: Patrick Georgi <patrick.georgi(a)secunet.com> Date: Tue Mar 8 12:43:29 2011 +0100 Wait longer on ATAPI devices Some devices require more time for themselves to get up and running. Change-Id: I347617c97bd02920e49b3cc45aa6c6b37573e0ff Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> --- drivers/ide_new.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/ide_new.c b/drivers/ide_new.c index 4f612be..ddcc027 100644 --- a/drivers/ide_new.c +++ b/drivers/ide_new.c @@ -633,7 +633,7 @@ ob_ide_atapi_drive_ready(struct ide_drive *drive) if (!ob_ide_atapi_packet(drive,cmd)) break; /* Give the drive some time to breathe */ - mdelay(500); + mdelay(5000); } memset(cmd, 0, sizeof(*cmd));

1 0

New patch to review: a8b71bd Unlock ATAPI devices on boot
by Patrick Georgi 05 Sep '11

05 Sep '11

Patrick Georgi (patrick(a)georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/192 -gerrit commit a8b71bd56bf205fd2add685ba8e0f0412a123080 Author: Patrick Georgi <patrick.georgi(a)secunet.com> Date: Tue Aug 24 11:07:21 2010 +0200 Unlock ATAPI devices on boot This makes CD/DVD drives accessible that were locked prior to a reboot. Change-Id: I4f2a0ddafac2debe0a1564904f37025850cc519e Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> --- drivers/ide_new.c | 13 +++++++++++++ drivers/ide_new.h | 1 + 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/drivers/ide_new.c b/drivers/ide_new.c index ddcc027..a2870b8 100644 --- a/drivers/ide_new.c +++ b/drivers/ide_new.c @@ -658,6 +658,19 @@ ob_ide_atapi_drive_ready(struct ide_drive *drive) return 1; } + /* clear Persistent Prevent State */ + memset (cmd, 0, sizeof(*cmd)); + cmd->cdb[0] = ATAPI_PREVENT_ALLOW_MEDIUM_REMOVAL; + cmd->cdb[2] = 2; + if (ob_ide_atapi_packet(drive, cmd)) + printf("could not persistently unlock device\n"); + + /* clear Prevent State */ + memset (cmd, 0, sizeof(*cmd)); + cmd->cdb[0] = ATAPI_PREVENT_ALLOW_MEDIUM_REMOVAL; + if (ob_ide_atapi_packet(drive, cmd)) + printf("could not unlock device\n"); + /* * finally, get capacity and block size */ diff --git a/drivers/ide_new.h b/drivers/ide_new.h index 8d3979a..f667e41 100644 --- a/drivers/ide_new.h +++ b/drivers/ide_new.h @@ -70,6 +70,7 @@ #define ATAPI_READ_10 0x28 #define ATAPI_REQ_SENSE 0x03 #define ATAPI_START_STOP_UNIT 0x1b +#define ATAPI_PREVENT_ALLOW_MEDIUM_REMOVAL 0x1e #define ATAPI_READ_CAPACITY 0x25 /*

1 0

New patch to review: e6e5a03 More compatible use of mktemp
by Patrick Georgi 05 Sep '11

05 Sep '11

Patrick Georgi (patrick(a)georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/190 -gerrit commit e6e5a03bca182471fa00d937c3235ac00ef0234e Author: Patrick Georgi <patrick.georgi(a)secunet.com> Date: Thu Sep 1 09:16:37 2011 +0200 More compatible use of mktemp Change-Id: I961a7ddcd39657c9463806d7b82757eff0a4ac57 Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> --- util/xcompile/xcompile | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile index 5e9ce83..7c1adf3 100644 --- a/util/xcompile/xcompile +++ b/util/xcompile/xcompile @@ -35,7 +35,7 @@ for make in make gmake gnumake; do done GCCPREFIX=invalid -TMPFILE=`mktemp /tmp/temp.XXXX 2>/dev/null || echo /tmp/temp.78gOIUGz` +TMPFILE=`mktemp /tmp/temp.XXXXXX 2>/dev/null || echo /tmp/temp.78gOIUGz` touch $TMPFILE # This should be a loop over all supported architectures

1 0

Patch merged into master: 7f65f07 Add POSIXy readdir/scandir support
by gerrit＠coreboot.org 05 Sep '11

05 Sep '11

the following patch was just integrated into master: commit 7f65f079c80bf52ff9e2f12b3c118ad3dcb5647b Author: Patrick Georgi <patrick.georgi(a)secunet.com> Date: Tue Jul 19 16:38:23 2011 +0200 Add POSIXy readdir/scandir support This abuses the print_a_completion mechanism to build the right data structures to support opendir/readdir/closedir and scandir on top of them. It's BSD-licensed so it can eventually be moved over to libpayload. Change-Id: Iadc1daed3cab4cf2ead778c23ad756d36cbbb99a Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> See http://review.coreboot.org/119 for details. -gerrit

1 0

Patch merged into master: b431050 Remove useless print_a_completion() stub
by gerrit＠coreboot.org 05 Sep '11

05 Sep '11

the following patch was just integrated into master: commit b4310501fa616a4e5267b57adcb06fbc226a5a14 Author: Patrick Georgi <patrick.georgi(a)secunet.com> Date: Tue Jul 19 16:37:13 2011 +0200 Remove useless print_a_completion() stub There's a real implementation elsewhere and this one only confuses things. Change-Id: I841d9baecbaf298114e508feec89f1b2ea864bb9 Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> See http://review.coreboot.org/118 for details. -gerrit

1 0

Patch merged into master: f916d99 Update defconfig to silence build
by gerrit＠coreboot.org 05 Sep '11

05 Sep '11

the following patch was just integrated into master: commit f916d994f716444edc72ec8af0c9020e16f486ea Author: Patrick Georgi <patrick.georgi(a)secunet.com> Date: Tue Jul 19 16:35:34 2011 +0200 Update defconfig to silence build Change-Id: Iecc5fef8df78e66bb80b00d786aa4925f13c823a Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> See http://review.coreboot.org/117 for details. -gerrit

1 0

Re: [coreboot] question
by Corey Osgood 04 Sep '11

04 Sep '11

On Sun, Sep 4, 2011 at 10:58 AM, Lyubomir Minchev <ljbomir(a)gmail.com> wrote: > Hi All, > > I have Laptop HP Pavilion dv4 with Chipset Intel 82801 I (ICH9) > > Shall it be compatible with coreboot ? > > The simple answer is no, at this time your chipset is not supported. -Corey

1 0

← Newer
1
...
16
17
18
19
20
21
22
23
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

coreboot September 2011