Re: [Qemu-devel] [PATCH] i6300esb: fix reading config registers and accept writes of all length

[Richard W.M. Jones]

On Tue, Dec 02, 2014 at 07:25:51PM +0100, Adam Hoka wrote:
> On 11/30/2014 11:31 PM, Richard W.M. Jones wrote:
> >http://git.annexia.org/?p=watchdog-test-framework.git;a=summary
> >
> >Rich.
> 
> Thanks for the feedback. I guess I could test it on Linux. Should I
> just run this utility?

The idea of the watchdog test framework is to make it easy to test the
watchdog, compared to configuring and using the watchdog daemon.  Read
the 'README' file first.

> What's wrong with the Intel/Win driver?

Hah hah, where to start?  Completely broken would be a good
executive level description of it.  Some of the problems we found:

 - It assumes that the card is located in a particular PCI slot (slot
   0 IIRC), and if it's not then it randomly writes to physical memory
   addresses >= 0x1_0000_0000.  In other words, it ignores boring old
   stuff like PCI config.

 - It's 32 bit / old Windows only.

 - No source.

 - No watchdog framework, so it's not actually useful in the real
   world.  (Windows itself does have a watchdog framework, but not in
   consumer/server versions.)

> BTW, could you ever find some documentation on where the hell is the
> IRQ line is connected on real HW?

The only docs I'm aware of are the Intel ones, section 16 of:

http://www.intel.co.uk/content/dam/doc/datasheet/6300esb-io-controller-hub-datasheet.pdf

plus of course the Linux driver.

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-df lists disk usage of guests without needing to install any
software inside the virtual machine.  Supports Linux and Windows.
http://people.redhat.com/~rjones/virt-df/