Re: [SeaBIOS] [Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.0

2012-04-17 Thread Michael Tokarev
On 17.04.2012 11:37, Gerd Hoffmann wrote:
[]
>>> You'll need to build multiple times with different configurations to get
>>> the pci ids right for the various vga variants emulated by qemu.  I'll
>>> have some build scripts to do that here, just need to rebase and polish
>>> them, I'll post the bits ASAP.
>>
>> Do you think you'll get this out in the next couple days?
> 
> This is my plan.

Um, this should be done by replacing a "choice" in the config
with multiple booleans, so it builds several vgabios binaries.

I thought about implementing this the very moment I tried to
build my first seabios with vgabios enabled, but was distracted
by something else and now you say you already have something
almost ready... ;)

Anyway, I can confirm that seaVGAbios works quite well so
far, except the usual issue with Linux VESA driver (which
present with bochs vgabios too).  So using seabios for all
this stuff appears to be a good idea.

Thank you all for the great work!

/mjt

___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios


Re: [SeaBIOS] [Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.0

2012-04-17 Thread Gerd Hoffmann
On 04/16/12 22:11, Anthony Liguori wrote:
> On 04/16/2012 06:35 AM, Gerd Hoffmann wrote:
>>Hi,
>>
>>> WinXP guest is unable to find drivers for the video adaptor
>>> with -vga std and this vgabios.bin, not even its "standard VGA"
>>> fallback driver.  The device in question is detected as "Other
>>> devices / Video controller (VGA compat.)", and its identification
>>> looks like:
>>>
>>>   PCI\VEN_1234&DEV_&SUBSYS_11001AF4&REV_00\3&13C0B0C5&0&10
>>
>> You'll need to build multiple times with different configurations to get
>> the pci ids right for the various vga variants emulated by qemu.  I'll
>> have some build scripts to do that here, just need to rebase and polish
>> them, I'll post the bits ASAP.
> 
> Do you think you'll get this out in the next couple days?

This is my plan.

>  I'd like to
> move to 1.7.0 and SeaVGABIOS for the 1.1 release if at all possible.

/me too.

cheers,
  Gerd


___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios


Re: [SeaBIOS] [Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.0

2012-04-16 Thread Eric Blake
On 04/15/2012 01:59 PM, Michael Tokarev wrote:
> On 15.04.2012 06:48, Kevin O'Connor wrote:
>> The 1.7.0 version of SeaBIOS has now been released.  For more
>> information on the release, please see:
>>
>> http://seabios.org/Releases
> 
> Um.  I can't compile it here, due to an error in ./tools/buildversion.sh.
> 
> $ sh -x ./tools/buildversion.sh out/vgaversion.c VAR16
> + OUTFILE=out/vgaversion.c
> + VAR16MODE=VAR16
> + [ -d .git ]
> + git describe --tags --long --dirty
> + VERSION=rel-1.7.0-0-ga026308
> + date +%Y%m%d_%H%M%S
> + hostname
> + VERSION=rel-1.7.0-0-ga026308-20120415_235702-gandalf
> + echo Version: rel-1.7.0-0-ga026308-20120415_235702-gandalf
> Version: rel-1.7.0-0-ga026308-20120415_235702-gandalf
> + [ VAR16 == VAR16 ]
> [: 1: VAR16: unexpected operator
> + cat
> 
> AFIAK, "==" operator is a bash(?) extension, standard
> POSIX shell only mentions "=" operator.

Not just a bash extension, as ksh and zsh have it too, but you are right
that dash doesn't implement it because POSIX 2008 (Issue 7) doesn't
require '==', just '='.  [There is a good chance that the next POSIX
revision (Issue 8) will require '==' as a synonym for '=', but as that
version of POSIX is still several years away, you're still better off
sticking to '=' for now.]

-- 
Eric Blake   ebl...@redhat.com+1-919-301-3266
Libvirt virtualization library http://libvirt.org



signature.asc
Description: OpenPGP digital signature
___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios


Re: [SeaBIOS] [Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.0

2012-04-16 Thread Kevin O'Connor
On Mon, Apr 16, 2012 at 02:28:20PM -0600, Eric Blake wrote:
> On 04/15/2012 01:59 PM, Michael Tokarev wrote:
> > On 15.04.2012 06:48, Kevin O'Connor wrote:
> >> The 1.7.0 version of SeaBIOS has now been released.  For more
> >> information on the release, please see:
> >>
> >> http://seabios.org/Releases
> > 
> > Um.  I can't compile it here, due to an error in ./tools/buildversion.sh.
[...]
> > AFIAK, "==" operator is a bash(?) extension, standard
> > POSIX shell only mentions "=" operator.
> 
> Not just a bash extension, as ksh and zsh have it too, but you are right
> that dash doesn't implement it because POSIX 2008 (Issue 7) doesn't
> require '==', just '='.  [There is a good chance that the next POSIX
> revision (Issue 8) will require '==' as a synonym for '=', but as that
> version of POSIX is still several years away, you're still better off
> sticking to '=' for now.]

It's easy enough to fix in SeaBIOS (for the next release).

-Kevin


>From 385a7d0dec28841a05531cba96c62138c3959fef Mon Sep 17 00:00:00 2001
From: Kevin O'Connor 
Date: Sun, 15 Apr 2012 22:02:15 -0400
Subject: [PATCH] buildversion: Don't rely on bash syntax.
To: seabios@seabios.org

Use '=' instead of '=='.

Signed-off-by: Kevin O'Connor 
---
 tools/buildversion.sh |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/tools/buildversion.sh b/tools/buildversion.sh
index 9b301e6..c8c1725 100755
--- a/tools/buildversion.sh
+++ b/tools/buildversion.sh
@@ -15,7 +15,7 @@ VERSION="${VERSION}-`date +"%Y%m%d_%H%M%S"`-`hostname`"
 echo "Version: ${VERSION}"
 
 # Build header file
-if [ "$VAR16MODE" == "VAR16" ]; then
+if [ "$VAR16MODE" = "VAR16" ]; then
 cat > ${OUTFILE} 

Re: [SeaBIOS] [Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.0

2012-04-16 Thread Anthony Liguori

On 04/16/2012 06:35 AM, Gerd Hoffmann wrote:

   Hi,


WinXP guest is unable to find drivers for the video adaptor
with -vga std and this vgabios.bin, not even its "standard VGA"
fallback driver.  The device in question is detected as "Other
devices / Video controller (VGA compat.)", and its identification
looks like:

  PCI\VEN_1234&DEV_&SUBSYS_11001AF4&REV_00\3&13C0B0C5&0&10


You'll need to build multiple times with different configurations to get
the pci ids right for the various vga variants emulated by qemu.  I'll
have some build scripts to do that here, just need to rebase and polish
them, I'll post the bits ASAP.


Do you think you'll get this out in the next couple days?  I'd like to move to 
1.7.0 and SeaVGABIOS for the 1.1 release if at all possible.


Regards,

Anthony LIguori



cheers,
   Gerd




___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios


Re: [SeaBIOS] [Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.0

2012-04-16 Thread Gerd Hoffmann
  Hi,

> WinXP guest is unable to find drivers for the video adaptor
> with -vga std and this vgabios.bin, not even its "standard VGA"
> fallback driver.  The device in question is detected as "Other
> devices / Video controller (VGA compat.)", and its identification
> looks like:
> 
>  PCI\VEN_1234&DEV_&SUBSYS_11001AF4&REV_00\3&13C0B0C5&0&10

You'll need to build multiple times with different configurations to get
the pci ids right for the various vga variants emulated by qemu.  I'll
have some build scripts to do that here, just need to rebase and polish
them, I'll post the bits ASAP.

cheers,
  Gerd

___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios


Re: [SeaBIOS] [Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.0

2012-04-15 Thread Michael Tokarev
On 15.04.2012 06:48, Kevin O'Connor wrote:
> The 1.7.0 version of SeaBIOS has now been released.  For more
> information on the release, please see:
> 
> http://seabios.org/Releases
> 
> 
> New in this release:
> 
> * Many enhancements to VGA BIOS code - it should now be feature complete with 
> LGPL vgabios.

This is something which prompted me to test the new release,
but it gives... interesting results.

WinXP guest is unable to find drivers for the video adaptor
with -vga std and this vgabios.bin, not even its "standard VGA"
fallback driver.  The device in question is detected as "Other
devices / Video controller (VGA compat.)", and its identification
looks like:

 PCI\VEN_1234&DEV_&SUBSYS_11001AF4&REV_00\3&13C0B0C5&0&10

This appears to be bogus somehow, no?

Thanks!

/mjt

___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios


Re: [SeaBIOS] [Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.0

2012-04-15 Thread Michael Tokarev
On 15.04.2012 06:48, Kevin O'Connor wrote:
> The 1.7.0 version of SeaBIOS has now been released.  For more
> information on the release, please see:
> 
> http://seabios.org/Releases

Um.  I can't compile it here, due to an error in ./tools/buildversion.sh.

$ sh -x ./tools/buildversion.sh out/vgaversion.c VAR16
+ OUTFILE=out/vgaversion.c
+ VAR16MODE=VAR16
+ [ -d .git ]
+ git describe --tags --long --dirty
+ VERSION=rel-1.7.0-0-ga026308
+ date +%Y%m%d_%H%M%S
+ hostname
+ VERSION=rel-1.7.0-0-ga026308-20120415_235702-gandalf
+ echo Version: rel-1.7.0-0-ga026308-20120415_235702-gandalf
Version: rel-1.7.0-0-ga026308-20120415_235702-gandalf
+ [ VAR16 == VAR16 ]
[: 1: VAR16: unexpected operator
+ cat

AFIAK, "==" operator is a bash(?) extension, standard
POSIX shell only mentions "=" operator.  Changing this
"==" to "=" makes it buildable.

Thanks,

/mjt

___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios