All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Lamprecht <thomas@lamprecht.org>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] vga: add sr_vbe register set
Date: Tue, 17 May 2016 13:16:13 +0200	[thread overview]
Message-ID: <573AFD7D.5000603@lamprecht.org> (raw)
In-Reply-To: <1463482205.30045.6.camel@redhat.com>

Hi,

On 05/17/2016 12:50 PM, Gerd Hoffmann wrote:
>   Hi,
>
>>> This way we can allow guests update sr[] registers as they want, without
>>> allowing them disrupt vbe video modes that way.
>> Just documenting my test with the patch here:
>>
>> This fixes the issue with QEMU 2.5.1.1 but only if I'm using SeaBIOS.
>>
>> OVMF leads to a almost similar result as without the patch, after
>> "windows is loading files" the "Starting Windows" appears short then
>>  hangs then the screen remains blank, so the blank screen is new with OVMF.
> Doesn't reproduce here.
>
> Details please:  Which ovmf version?  With/without csm?  32/64 bit
> windows version?

I tested with:

edk2.git-ovmf-x64-0-20151117.b1317.g386cdfb.noarch.rpm
$ git show 386cdfb
commit 386cdfbecbbacb600ffc8e2ffa8c7af1b3855a61
Author: Mark Rutland <mark.rutland@arm.com>
Date:   Tue Nov 17 13:58:19 2015 +0000

which I used for a little while now and to see if its just my "old" OVMF
I also tested the newest version from your jenkins repo (much thanks for them!!):

git show 05b2f9c
commit 05b2f9c94e0c0b663ff2d2fb55397d8215eeb3f5
Author: Dandan Bi <dandan.bi@intel.com>
Date:   Tue May 10 18:51:44 2016 +0800

So without csm.

The OS is 64bit windows 7 professional SP1

The QEMU command for my test:

./x86_64-softmmu/qemu-system-x86_64 -boot d -cdrom W7SP1_PROFESSIONAL.iso -m 1024 -smp 2 -enable-kvm -cpu host -drive if=pflash,format=raw,unit=0,readonly,file=OVMF_CODE-pure-efi.fd -drive if=pflash,format=raw,unit=1,file=/tmp/OVMF_VARS.fd


Happy to provide more details/test if needed.

best regards,
Thomas

  reply	other threads:[~2016-05-17 11:16 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-17  8:54 [Qemu-devel] [PATCH] vga: add sr_vbe register set Gerd Hoffmann
2016-05-17  9:48 ` Thomas Lamprecht
2016-05-17 10:50   ` Gerd Hoffmann
2016-05-17 11:16     ` Thomas Lamprecht [this message]
2016-05-20 10:06       ` Gerd Hoffmann
2016-05-23 21:39         ` Thomas Lamprecht
2016-05-24 10:14           ` Thomas Lamprecht

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=573AFD7D.5000603@lamprecht.org \
    --to=thomas@lamprecht.org \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /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.