From: Paul Brook <paul@codesourcery.com>
To: Gleb Natapov <gleb@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] Register usb-uhci reset function.
Date: Tue, 16 Jun 2009 20:38:24 +0100 [thread overview]
Message-ID: <200906162038.25324.paul@codesourcery.com> (raw)
In-Reply-To: <20090616191148.GI782@redhat.com>
>What about the mantra that we should be as close to HW as possible?
>Reseting state only on a bus level will not work for hot-unplug anyway
Real hardware consists of electrical signals. Qemu emulates the logical
effects of a device/subsystem, often at a much higher level. Thus "close to HW
as possible" is no longer a straightforward concept.
Apart from anything else, I'm pretty sure than on all real hotplug PCI systems
each socket is on its own PCI bus to allow electrical isolation of the device
before physical hotplug occurs.
> > However having the device explicit set its IRQ line during a full system
> > reset is a different matter. This is probably harmless most of the time,
> > and may paper over other bugs (e.g. the PCI bus not being reset
> > properly). However I do not believe it is the correct justification for
> > these changes.
>
> The IRQ state inside piix3 code is QEMU implementation detail. There is
> now such thing on real HW.
Yes there is. It's a long strip of copper wire with a pullup resistor on the
end.
Paul
next prev parent reply other threads:[~2009-06-16 19:38 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-16 12:47 [Qemu-devel] [PATCH] Register usb-uhci reset function Gleb Natapov
2009-06-16 17:14 ` Paul Brook
2009-06-16 17:37 ` Gleb Natapov
2009-06-16 18:41 ` Paul Brook
2009-06-16 19:11 ` Gleb Natapov
2009-06-16 19:38 ` Paul Brook [this message]
2009-06-16 22:57 ` Zachary Amsden
2009-06-17 8:12 ` Gleb Natapov
2009-06-16 18:02 ` Blue Swirl
2009-06-16 19:19 ` Anthony Liguori
2009-06-16 19:26 ` Gleb Natapov
2009-06-17 9:07 ` Filip Navara
2009-06-17 9:43 ` Gleb Natapov
2009-06-17 10:17 ` Filip Navara
2009-06-17 11:06 ` Gleb Natapov
2009-06-17 11:25 ` Dor Laor
2009-06-17 11:39 ` Gleb Natapov
2009-06-17 11:50 ` Filip Navara
2009-06-17 11:36 ` Filip Navara
2009-06-17 12:12 ` Gleb Natapov
2009-06-17 13:03 ` Filip Navara
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=200906162038.25324.paul@codesourcery.com \
--to=paul@codesourcery.com \
--cc=gleb@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).