[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH] spice: add qxl device
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] Re: [PATCH] spice: add qxl device |
Date: |
Thu, 18 Nov 2010 13:33:08 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Nov 18, 2010 at 11:57:51AM +0200, Gleb Natapov wrote:
> On Thu, Nov 18, 2010 at 11:30:27AM +0200, Michael S. Tsirkin wrote:
> > On Thu, Nov 18, 2010 at 11:11:49AM +0200, Gleb Natapov wrote:
> > > On Thu, Nov 18, 2010 at 11:03:21AM +0200, Michael S. Tsirkin wrote:
> > > > On Thu, Nov 18, 2010 at 10:09:35AM +0200, Gleb Natapov wrote:
> > > > > On Wed, Nov 17, 2010 at 08:00:08PM +0200, Michael S. Tsirkin wrote:
> > > > > > > >> If so: does qemu
> > > > > > > >>emulate this correctly?
> > > > > > > >
> > > > > > > >It mostly does.
> > > > > > >
> > > > > > > I doubt it actually enables/disables the legacy vga ports.
> > > > > >
> > > > > > I'll check when I have the time. We can fix it if it doesn't,
> > > > > >
> > > > > So many guests (all of them?) just assume that vga ports and
> > > > > framebuffer is there.
> > > >
> > > > Why do you think they disable io memory then?
> > > >
> > > Who and how and when disables io memory?
> >
> > I think guest will do this if you disable the device through the device
> > manager. This might need a reboot to become effective.
> >
> Try to do it with primary VGA adapter and tell us what happens :)
>
> > > Some guests are designed to run
> > > even on old ISA machines that have no way to disable anything. The
> > > device is just there.
> > >
> > > This is the same with IDE ports. BIOS "knows" legacy ISA ports and just
> > > program them into PCI IO bars to be nice.
> >
> > HAven't checked IDE, for VGA AFAIK BIOS does not program legacy ports in
> > the card, they are hardwired there. However, the card must not claim any
> > io transactions if IO memory is disabled in command register.
> >
> Is this correct also for legacy ports?
Yes. The spec is quite explicit on this point:
A function that supports a PC legacy function (IDE, VGA, etc.) is
allowed to claim those addresses associated with the specific function
when the I/O Space (see Figure 6-2) enable bit is set. These addresses
are not requested using a Base Address register but are assigned by
initialization software. If a device identifies itself as a legacy
function (class code), the initialization software grants the device
permission to claim the I/O legacy addresses by setting the device’s I/O
Space enable bit.
> This wouldn't be backwards
> compatible to ISA machines, so old software my not run properly back in
> the days when transaction from ISA to PCI happened.
initialization software could be the BIOS.
So maybe BIOS update was needed in the transition.
> So my guess is that
> old ISA ports works in backwards compatible way.
The spec seems to contradict this.
> > When qemu is started, it works correctly: the io memory is disabled and
> > card does
> > not claim any io. Then BIOS comes along and enables io. At this point
> > map callback is invoked and maps io memory, card starts claiming io.
> Looking at the code I see that cirrus claims all IO ports and
> framebuffer memory during init function unconditionally.
So that may be OK for ISA, but not for PCI.
> >
> > What is broken is that if BIOS/guest then disables IO memory,
> > (I think - even if guest is rebooted!) we will keep claiming IO
> > transactions.
> > That our emulation does this seems to be a clear spec violation, we are
> > just lucky that BIOS/guest does not do this at the moment.
> >
> >
> >
> > > > > So what "fixing" this will buy us?
> > > >
> > > > Besides spec compliancy, you mean? Ability to support multiple VGA
> > > > cards. That's how it works I think: BIOS enables IO on the primary
> > > > VGA device only.
> > > >
> > > What spec defines hot-plug for primary VGA adapter?
> >
> > No idea about hotplug. I am talking about multiple VGA cards,
> > enabling/disabling them dynamically should be possible.
> Of course. With properly designed VGA card you should be able to have
> more then one,
And, for that to have a chance to work when all cards are identical, you
don't claim IO when IO is disabled.
> but one of them will provide legacy functionality
> and is not removable.
The guest might not support hotplug. But there's no way
it can prevent surprise removal. qemu should not crash
when this happens.
> >
> > > Our BIOS should support
> > > -M isa machine too.
> >
> > > There is no way to disable VGA or even check if it
> > > is present there.
> >
> > So I guess with isa you can't have multiple VGA cards work. With PCI
> > I think you can.
> >
> With isa you just need custom designed HW and software and you can have
> as many VGA cards as you want :)
>
> --
> Gleb.
- [Qemu-devel] Re: [PATCH] spice: add qxl device, (continued)
- [Qemu-devel] Re: [PATCH] spice: add qxl device, Gerd Hoffmann, 2010/11/17
- [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/17
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gleb Natapov, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gerd Hoffmann, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gerd Hoffmann, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gleb Natapov, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gleb Natapov, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device,
Michael S. Tsirkin <=
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gleb Natapov, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gleb Natapov, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gleb Natapov, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gleb Natapov, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Gleb Natapov, 2010/11/18
- Re: [Qemu-devel] Re: [PATCH] spice: add qxl device, Michael S. Tsirkin, 2010/11/18