[coreboot] Gerrit config (was: Re: Patch merged into coreboot/master: 7f8e685 Load an IDT with NULL limit)
Patrick Georgi
patrick at georgi-clan.de
Fri Oct 14 19:26:03 CEST 2011
Am Freitag, den 14.10.2011, 01:32 +0200 schrieb Uwe Hermann:
> On Thu, Oct 13, 2011 at 08:04:27PM +0200, gerrit at coreboot.org wrote:
> > the following patch was just integrated into master:
> > commit 7f8e685996f65f2c67d1113fcfaab33ccc385da7
> > Author: Stefan Reinauer <reinauer at chromium.org>
> > Date: Wed Jun 1 14:01:46 2011 -0700
> >
> > Load an IDT with NULL limit
> >
> > Load an IDT with NULL limit to prevent the 16bit IDT being used
> > in protected mode before c_start.S sets up a 32bit IDT when entering
> > ram stage.
> >
> > Signed-off-by: Stefan Reinauer <reinauer at google.com>
> > Change-Id: I8d048c894c863ac4971fcef8f065be6b899e1d3e
> >
> > See http://review.coreboot.org/259 for details.
>
> Can gerrit be configured to show
>
> - who approved a patch and when
I implemented this (but didn't add it to the hook yet):
> - the comment(s) (if any) left by the reviewer?
>
> Having to open the URL in a browser instead of that info being in the
> "Patch merged into coreboot/master" mail is really annoying, IMHO.
>
>
> Uwe.
> --
> http://hermann-uwe.de | http://sigrok.org
> http://randomprojects.org | http://unmaintained-free-software.org
>
More information about the coreboot
mailing list