From: Gerd Hoffmann <kraxel@suse.de>
To: "Petersson, Mats" <Mats.Petersson@amd.com>
Cc: Xen devel list <xen-devel@lists.xensource.com>
Subject: Re: VT/ioemu: vga memory access?
Date: Tue, 16 May 2006 18:49:48 +0200 [thread overview]
Message-ID: <446A02AC.9010601@suse.de> (raw)
In-Reply-To: <907625E08839C4409CE5768403633E0BA7FC75@sefsexmb1.amd.com>
[-- Attachment #1: Type: text/plain, Size: 355 bytes --]
Hi,
> I'll write some simple test code to check it out, and let you know...
Fix attached, seems to work ok on a quick test.
cheers,
Gerd
--
Gerd Hoffmann <kraxel@suse.de>
Erst mal heiraten, ein, zwei Kinder, und wenn alles läuft
geh' ich nach drei Jahren mit der Familie an die Börse.
http://www.suse.de/~kraxel/julika-dora.jpeg
[-- Attachment #2: fix --]
[-- Type: text/plain, Size: 968 bytes --]
diff -r be5a6188cd9f xen/arch/x86/hvm/platform.c
--- a/xen/arch/x86/hvm/platform.c Thu May 11 10:34:34 2006
+++ b/xen/arch/x86/hvm/platform.c Tue May 16 18:11:57 2006
@@ -865,7 +865,7 @@
* copy ourself. After this copy succeeds, "rep movs" is executed
* again.
*/
- if ((addr & PAGE_MASK) != ((addr + size - 1) & PAGE_MASK)) {
+ if ((addr & PAGE_MASK) != ((addr + sign * (size - 1)) & PAGE_MASK)) {
unsigned long value = 0;
mmio_opp->flags |= OVERLAP;
@@ -876,7 +876,7 @@
hvm_copy(&value, addr, size, HVM_COPY_IN);
send_mmio_req(IOREQ_TYPE_COPY, gpa, 1, size, value, dir, 0);
} else {
- if ((addr & PAGE_MASK) != ((addr + count * size - 1) & PAGE_MASK)) {
+ if ((addr & PAGE_MASK) != ((addr + sign * (count * size - 1)) & PAGE_MASK)) {
regs->eip -= inst_len; /* do not advance %eip */
if (sign > 0)
[-- Attachment #3: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
next prev parent reply other threads:[~2006-05-16 16:49 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-05-16 16:20 VT/ioemu: vga memory access? Petersson, Mats
2006-05-16 16:49 ` Gerd Hoffmann [this message]
2006-05-16 16:57 ` Keir Fraser
-- strict thread matches above, loose matches on Subject: below --
2006-05-16 17:24 Petersson, Mats
2006-05-16 17:03 Petersson, Mats
2006-05-16 15:26 Petersson, Mats
2006-05-16 15:57 ` Gerd Hoffmann
2006-05-16 14:44 Gerd Hoffmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=446A02AC.9010601@suse.de \
--to=kraxel@suse.de \
--cc=Mats.Petersson@amd.com \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.