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] [PATCH][RFC] gfx_passthru: warning when vgabios rom has inva

To: weidong.han@xxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxx, Ian.Jackson@xxxxxxxxxxxxx
Subject: [Xen-devel] [PATCH][RFC] gfx_passthru: warning when vgabios rom has invalid checksum
From: Noboru Iwamatsu <n_iwamatsu@xxxxxxxxxxxxxx>
Date: Mon, 22 Feb 2010 14:45:21 +0900
Cc:
Delivery-date: Sun, 21 Feb 2010 21:46:29 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; ja; rv:1.9.1.7) Gecko/20100111 Thunderbird/3.0.1
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 do you think, Weidong?
Is just warning enough? Or, should we stop loading the rom?

Regards,
Noboru.

Attachment: vgabios-checksum-warn.patch
Description: Text document

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