[SeaBIOS] [PATCHv3] acpi: make default DSDT optional
paulepanter at users.sourceforge.net
Mon Mar 18 15:53:29 CET 2013
Am Montag, den 18.03.2013, 16:17 +0200 schrieb Michael S. Tsirkin:
> On Mon, Mar 18, 2013 at 02:22:53PM +0100, Paul Menzel wrote:
> > Am Montag, den 18.03.2013, 08:57 -0400 schrieb Kevin O'Connor:
> > > On Mon, Mar 18, 2013 at 01:00:42PM +0200, Michael S. Tsirkin wrote:
> > > > Since commit f7e4dd6c18ccfbaf6cd2f5eaaed2b77cabc8a406 QEMU loads its
> > Michael, thank you for putting this in. As commit hashes are hard to
> > memorize adding the summary to is useful in my opinion. For example for
> > this commit like this.
> > Since commit »acpi: autoload dsdt« (f7e4dd6c)  …
> > […]
> >  http://git.qemu.org/?p=qemu.git;a=commit;h=f7e4dd6c18ccfbaf6cd2f5eaaed2b77cabc8a406
> > […]
> URLs are best avoided as servers come and go and git history is
> immutable. Not sure what does the summary add - I think
> the main thing you would want to do is check whether your
> history includes a specific hash.
Ok. In this case the version is good enough. But if you backport,
cherry-pick or do something different the hash changes too. So in my
opinion it is best to have both.
> You do it like this:
> git log |grep f7e4dd6c18ccfbaf6cd2f5eaaed2b77cabc8a406
$ git branch --contains f7e4dd6c18ccfbaf6cd2f5eaaed2b77cabc8a406
> > > > --- a/src/Kconfig
> > > > +++ b/src/Kconfig
> > > > @@ -387,6 +387,12 @@ menu "BIOS Tables"
> > > > default y
> > > > help
> > > > Support generation of ACPI tables.
> > > > + config ACPI_DSDT
> > > > + bool "Include default ACPI DSDT"
> > > > + default y
> > > > + depends on ACPI
> > > > + help
> > > > + Include default DSDT ACPI table in BIOS.
> > >
> > > As Paul points out, it would really help if the help stated which
> > > released version of QEMU is needed to turn this off.
> > I just meant the commit message, but it is useful in Kconfig too. Kevin,
> > good suggestion.
> OK I put in the QEMU release version in Kconfig, that should be best I think?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 198 bytes
Desc: This is a digitally signed message part
More information about the SeaBIOS