WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Re: [PATCH][RFC] gfx_passthru: warning when vgabios rom has

To: Noboru Iwamatsu <n_iwamatsu@xxxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH][RFC] gfx_passthru: warning when vgabios rom has invalid checksum
From: Weidong Han <weidong.han@xxxxxxxxx>
Date: Mon, 22 Feb 2010 16:56:37 +0800
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, "Ian.Jackson@xxxxxxxxxxxxx" <Ian.Jackson@xxxxxxxxxxxxx>
Delivery-date: Mon, 22 Feb 2010 00:57:37 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4B82398B.6020109@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <4B8219F1.5040301@xxxxxxxxxxxxxx> <4B823088.8090706@xxxxxxxxx> <4B82398B.6020109@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Thunderbird 2.0.0.21 (Windows/20090302)
Noboru Iwamatsu wrote:
Hi,

 >> Intel DX58SO + GeForce GTS250:        checksum is OK.
 >> Intel DX58SO + GeForce 9600GT:        checksum is OK.
 >> Fujitsu Q35 M/B + IGD:                checksum is bad.
 >>
 > What does "checksum is bad" mean here?

It means shadowed VGABIOS has invalid checksum
and requires recalculating.

 >> What do you think, Weidong?
 >> Is just warning enough? Or, should we stop loading the rom?
 >>
 >>
 > Does it still work or not when checksum is bad? If it works, we should
 > not stop loading the rom.

When the checksum is bad,
if it is not recalculated, guest has some trouble about handling the
VGA (unable to show BIOS screen, unable to load the gfx driver
properly, ...).
For this case, it's reasonable to warning and stop loading rom. We didn't encounter this issue.
If it is recalculated, it seems to work fine.
How did you recalculate it? is it possible to recalculate it when detect invalid checksum?

Regards,
Weidong

Regards,
Noboru.

Noboru Iwamatsu wrote:
Hi,

According to the "PCI Firmware Spec Rev 3.0",
the system firmware have to write the new checksum after resizing
the expansion ROM area.

So, when re-using the shadowed VGABIOS, the checksum must
be valid, and if it is invalid, we should consider the BIOS has
a bug or memory area is corrupted.

This patch just add the warning message when checksum requires
recalculation.

I tried this following environments.

Intel DX58SO + GeForce GTS250:  checksum is OK.
Intel DX58SO + GeForce 9600GT:  checksum is OK.
Fujitsu Q35 M/B + IGD:          checksum is bad.

What does "checksum is bad" mean here?
What do you think, Weidong?
Is just warning enough? Or, should we stop loading the rom?


Does it still work or not when checksum is bad? If it works, we should
not stop loading the rom.

Regards,
Weidong
Regards,
Noboru.





_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel