Re: [Qemu-devel] [PATCH 1/4] eth: add speed and duplex definitions

2018-03-07 Thread Jason Baron via Qemu-devel
On 03/06/2018 01:15 PM, Michael S. Tsirkin wrote: > On Tue, Mar 06, 2018 at 12:53:14PM -0500, Jason Baron wrote: >> >> >> On 03/02/2018 12:54 PM, Michael S. Tsirkin wrote: >>> On Thu, Mar 01, 2018 at 10:46:33PM -0500, Jason Baron wrote: >>>> Pull in def

[Qemu-devel] [PATCH v2 0/3] virtio-net: allow linkspeed and duplex setting

2018-03-07 Thread Jason Baron via Qemu-devel
/msg07072.html Thanks, -Jason v2 -pull in include/linux/ethtool.h (Michael Tsirkin) Jason Baron (3): scripts/update-linux-headers: add ethtool.h and update to 4.16.0-rc4 virtio-net: use 64-bit values for feature flags virtio-net: add linkspeed and duplex settings to virtio-net hw/net

[Qemu-devel] [PATCH v2 1/3] scripts/update-linux-headers: add ethtool.h and update to 4.16.0-rc4

2018-03-07 Thread Jason Baron via Qemu-devel
A subsequent patch to add support for setting linkspeed/duplex in virtio-net, requires a few definitions from ethtool.h, which ends up pulling in kernel.h and sysinfo.h as well. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org --

[Qemu-devel] [PATCH v2 3/3] virtio-net: add linkspeed and duplex settings to virtio-net

2018-03-07 Thread Jason Baron via Qemu-devel
bsequently overwrite it later if desired via: 'ethtool -s'. Linkspeed and duplex settings can be set as: '-device virtio-net,speed=1,duplex=full' where speed is [0...INT_MAX], and duplex is ["half"|"full"]. Signed-off-by: Jason Baron Cc: "Michael

[Qemu-devel] [PATCH v2 2/3] virtio-net: use 64-bit values for feature flags

2018-03-07 Thread Jason Baron via Qemu-devel
In prepartion for using some of the high order feature bits, make sure that virtio-net uses 64-bit values everywhere. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org --- hw/net/virtio-net.c

Re: [Qemu-devel] [virtio-dev] [PATCH 4/4] virtio-net: add linkspeed and duplex settings to virtio-net

2018-03-06 Thread Jason Baron via Qemu-devel
On 03/04/2018 08:05 AM, Yan Vugenfirer wrote: > > >> On 2 Mar 2018, at 22:19, Michael S. Tsirkin > <mailto:m...@redhat.com>> wrote: >> >> On Fri, Mar 02, 2018 at 03:14:01PM +0800, Jason Wang wrote: >>> >>> >>> On 2018年03月02日 11:4

Re: [Qemu-devel] [PATCH 4/4] virtio-net: add linkspeed and duplex settings to virtio-net

2018-03-06 Thread Jason Baron via Qemu-devel
On 03/02/2018 03:22 PM, Michael S. Tsirkin wrote: > On Fri, Mar 02, 2018 at 11:59:00AM -0500, Jason Baron wrote: >> On 03/02/2018 02:14 AM, Jason Wang wrote: >>> >>> >>> On 2018年03月02日 11:46, Jason Baron wrote: >>>> Although linkspeed and duplex

Re: [Qemu-devel] [PATCH 1/4] eth: add speed and duplex definitions

2018-03-06 Thread Jason Baron via Qemu-devel
On 03/02/2018 12:54 PM, Michael S. Tsirkin wrote: > On Thu, Mar 01, 2018 at 10:46:33PM -0500, Jason Baron wrote: >> Pull in definitions for SPEED_UNKNOWN, DUPLEX_UNKNOWN, DUPLEX_HALF, >> and DUPLEX_FULL. >> >> Signed-off-by: Jason Baron >> Cc: "Michael

Re: [Qemu-devel] [PATCH 4/4] virtio-net: add linkspeed and duplex settings to virtio-net

2018-03-02 Thread Jason Baron via Qemu-devel
On 03/02/2018 02:14 AM, Jason Wang wrote: > > > On 2018年03月02日 11:46, Jason Baron wrote: >> Although linkspeed and duplex can be set in a linux guest via 'ethtool >> -s', >> this requires custom ethtool commands for virtio-net by defaul

[Qemu-devel] [PATCH 3/4] virtio-net: use 64-bit values for feature flags

2018-03-01 Thread Jason Baron via Qemu-devel
In prepartion for using some of the high order feature bits, make sure that virtio-net uses 64-bit values everywhere. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org --- hw/net/virtio-net.c

[Qemu-devel] [PATCH 4/4] virtio-net: add linkspeed and duplex settings to virtio-net

2018-03-01 Thread Jason Baron via Qemu-devel
bsequently overwrite it later if desired via: 'ethtool -s'. Linkspeed and duplex settings can be set as: '-device virtio-net,speed=1,duplex=full' where speed is [0...INT_MAX], and duplex is ["half"|"full"]. Signed-off-by: Jason Baron Cc: "Michael

[Qemu-devel] [PATCH 2/4] rocker: drop local duplex definitions

2018-03-01 Thread Jason Baron via Qemu-devel
Make use of duplex definitions from net/eth.h. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: Jiri Pirko Cc: virtio-...@lists.oasis-open.org --- hw/net/rocker/rocker_fp.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/hw/

[Qemu-devel] [PATCH 1/4] eth: add speed and duplex definitions

2018-03-01 Thread Jason Baron via Qemu-devel
Pull in definitions for SPEED_UNKNOWN, DUPLEX_UNKNOWN, DUPLEX_HALF, and DUPLEX_FULL. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org --- include/net/eth.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/n

[Qemu-devel] [PATCH 0/4] virtio-net: allow linkspeed and duplex setting

2018-03-01 Thread Jason Baron via Qemu-devel
.html Thanks, -Jason Jason Baron (4): eth: add speed and duplex definitions rocker: drop local duplex definitions virtio-net: use 64-bit values for feature flags virtio-net: add linkspeed and duplex settings to virtio-net hw/net/rocker/rocker_fp.c | 6 +--- hw/net/virtio-

[Qemu-devel] [PATCH net-next v4 1/3] virtio_net: propagate linkspeed/duplex settings from the hypervisor

2018-01-05 Thread Jason Baron via Qemu-devel
tention is that device feature bits are to grow down from bit 63, since the transports are starting from bit 24 and growing up. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org --- drivers/net/virtio_net.c| 23 ++

[Qemu-devel] [PATCH v4 3/3] qemu: add linkspeed and duplex settings to virtio-net

2018-01-05 Thread Jason Baron via Qemu-devel
bsequently overwrite it later if desired via: 'ethtool -s'. Linkspeed and duplex settings can be set as: '-device virtio-net,speed=1,duplex=full' where speed is [-1...INT_MAX], and duplex is ["half"|"full"]. Signed-off-by: Jason Baron Cc: "Michael

[Qemu-devel] [PATCH v4 0/3] virtio_net: allow hypervisor to indicate linkspeed and duplex setting

2018-01-05 Thread Jason Baron via Qemu-devel
* only do speed/duplex read in virtnet_config_changed_work() on LINK_UP changes from v2: * move speed/duplex read into virtnet_config_changed_work() so link up changes are detected Jason Baron (1): virtio_net: propagate linkspeed/duplex settings from the hypervisor drivers/net/virtio_net.c

[Qemu-devel] [PATCH v4 2/3] qemu: virtio-net: use 64-bit values for feature flags

2018-01-05 Thread Jason Baron via Qemu-devel
In prepartion for using some of the high order feature bits, make sure that virtio-net uses 64-bit values everywhere. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org --- hw/net/virtio-net.c

Re: [Qemu-devel] [PATCH net-next v3 1/3] virtio_net: propagate linkspeed/duplex settings from the hypervisor

2018-01-04 Thread Jason Baron via Qemu-devel
On 01/04/2018 01:22 PM, Michael S. Tsirkin wrote: > On Thu, Jan 04, 2018 at 01:12:30PM -0500, Jason Baron wrote: >> >> >> On 01/04/2018 12:05 PM, Michael S. Tsirkin wrote: >>> On Thu, Jan 04, 2018 at 12:16:44AM -0500, Jason Baron wrote: >>>> The ability

Re: [Qemu-devel] [PATCH net-next v3 1/3] virtio_net: propagate linkspeed/duplex settings from the hypervisor

2018-01-04 Thread Jason Baron via Qemu-devel
On 01/04/2018 12:05 PM, Michael S. Tsirkin wrote: > On Thu, Jan 04, 2018 at 12:16:44AM -0500, Jason Baron wrote: >> The ability to set speed and duplex for virtio_net is useful in various >> scenarios as described here: >> >> 16032be virtio_net: add ethtool support

Re: [Qemu-devel] [PATCH net-next v3 1/3] virtio_net: propagate linkspeed/duplex settings from the hypervisor

2018-01-04 Thread Jason Baron via Qemu-devel
On 01/04/2018 11:27 AM, Michael S. Tsirkin wrote: > On Thu, Jan 04, 2018 at 12:16:44AM -0500, Jason Baron wrote: >> The ability to set speed and duplex for virtio_net is useful in various >> scenarios as described here: >> >> 16032be virtio_net: add ethtool support

[Qemu-devel] [PATCH v3 2/3] qemu: virtio-net: use 64-bit values for feature flags

2018-01-03 Thread Jason Baron via Qemu-devel
In prepartion for using some of the high order feature bits, make sure that virtio-net uses 64-bit values everywhere. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org --- hw/net/virtio-net.c

[Qemu-devel] [PATCH v3 0/3] virtio_net: allow hypervisor to indicate linkspeed and duplex setting

2018-01-03 Thread Jason Baron via Qemu-devel
s, -Jason linux changes: changes from v2: * move speed/duplex read into virtnet_config_changed_work() so link up changes are detected Jason Baron (1): virtio_net: propagate linkspeed/duplex settings from the hypervisor drivers/net/virtio_net.c| 19 ++- include/

[Qemu-devel] [PATCH net-next v3 1/3] virtio_net: propagate linkspeed/duplex settings from the hypervisor

2018-01-03 Thread Jason Baron via Qemu-devel
tention is that device feature bits are to grow down from bit 63, since the transports are starting from bit 24 and growing up. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org --- drivers/net/virtio_net.c| 19 ++

[Qemu-devel] [PATCH v3 3/3] qemu: add linkspeed and duplex settings to virtio-net

2018-01-03 Thread Jason Baron via Qemu-devel
bsequently overwrite it later if desired via: 'ethtool -s'. Linkspeed and duplex settings can be set as: '-device virtio-net,speed=1,duplex=full' where speed is [-1...INT_MAX], and duplex is ["half"|"full"]. Signed-off-by: Jason Baron Cc: "Michael

Re: [Qemu-devel] [PATCH net-next v2 1/3] virtio_net: propagate linkspeed/duplex settings from the hypervisor

2017-12-28 Thread Jason Baron via Qemu-devel
On 12/27/2017 04:43 PM, David Miller wrote: > From: Jason Baron > Date: Fri, 22 Dec 2017 16:54:01 -0500 > >> The ability to set speed and duplex for virtio_net in useful in various >> scenarios as described here: >> >> 16032be virtio_net: add ethtool su

[Qemu-devel] [PATCH v2 0/3] virtio_net: allow hypervisor to indicate linkspeed and duplex setting

2017-12-22 Thread Jason Baron via Qemu-devel
-Jason linux changes: Jason Baron (1): virtio_net: propagate linkspeed/duplex settings from the hypervisor drivers/net/virtio_net.c

[Qemu-devel] [PATCH 2/3] qemu: use 64-bit values for feature flags in virtio-net

2017-12-22 Thread Jason Baron via Qemu-devel
In prepartion for using some of the high order feature bits, make sure that virtio-net uses 64-bit values everywhere. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang --- hw/net/virtio-net.c| 54 +- include/hw/vir

[Qemu-devel] [PATCH net-next v2 1/3] virtio_net: propagate linkspeed/duplex settings from the hypervisor

2017-12-22 Thread Jason Baron via Qemu-devel
htool commands. Introduce a new feature flag, VIRTIO_NET_F_SPEED_DUPLEX, which allows the hypervisor to export a linkspeed and duplex setting. The user can subsequently overwrite it later if desired via: 'ethtool -s'. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc:

[Qemu-devel] [PATCH 3/3] qemu: add linkspeed and duplex settings to virtio-net

2017-12-22 Thread Jason Baron via Qemu-devel
bsequently overwrite it later if desired via: 'ethtool -s'. Linkspeed and duplex settings can be set as: '-device virtio-net,speed=1,duplex=full' where speed is [-1...INT_MAX], and duplex is ["half"|"full"]. Signed-off-by: Jason Baron Cc: "Michae

Re: [Qemu-devel] [PATCH 2/2] qemu: add linkspeed and duplex setting to virtio-net

2017-12-21 Thread Jason Baron via Qemu-devel
On 12/20/2017 09:33 AM, Yan Vugenfirer wrote: > >> On 20 Dec 2017, at 16:31, Michael S. Tsirkin wrote: >> >> On Tue, Dec 19, 2017 at 11:52:39AM -0500, Jason Baron wrote: >>> >>> >>> On 12/19/2017 04:19 AM, Yan Vugenfirer wrote: >>>>

Re: [Qemu-devel] [PATCH net-next 1/2] virtio_net: allow hypervisor to indicate linkspeed and duplex setting

2017-12-20 Thread Jason Baron via Qemu-devel
On 12/20/2017 12:52 PM, Michael S. Tsirkin wrote: > On Wed, Dec 20, 2017 at 12:07:55PM -0500, Jason Baron wrote: >> >> >> On 12/20/2017 09:57 AM, Michael S. Tsirkin wrote: >>> On Thu, Dec 14, 2017 at 02:33:53PM -0500, Jason Baron wrote: >>>> If the hy

Re: [Qemu-devel] [PATCH net-next 1/2] virtio_net: allow hypervisor to indicate linkspeed and duplex setting

2017-12-20 Thread Jason Baron via Qemu-devel
On 12/20/2017 09:57 AM, Michael S. Tsirkin wrote: > On Thu, Dec 14, 2017 at 02:33:53PM -0500, Jason Baron wrote: >> If the hypervisor exports the link and duplex speed, let's use that instead >> of the default unknown speed. The user can still overwrite it later if >>

Re: [Qemu-devel] [PATCH 2/2] qemu: add linkspeed and duplex setting to virtio-net

2017-12-19 Thread Jason Baron via Qemu-devel
On 12/19/2017 04:19 AM, Yan Vugenfirer wrote: > >> On 18 Dec 2017, at 18:04, Jason Baron via Qemu-devel >> mailto:qemu-devel@nongnu.org>> wrote: >> >> >> >> On 12/18/2017 06:34 AM, Yan Vugenfirer wrote: >>> >>>> On 14 Dec

Re: [Qemu-devel] [PATCH 2/2] qemu: add linkspeed and duplex setting to virtio-net

2017-12-18 Thread Jason Baron via Qemu-devel
On 12/18/2017 06:34 AM, Yan Vugenfirer wrote: > >> On 14 Dec 2017, at 21:33, Jason Baron via Qemu-devel >> wrote: >> >> Although they can be currently set in linux via 'ethtool -s', this requires >> guest changes, and thus it would be nice to exten

[Qemu-devel] [PATCH net-next 1/2] virtio_net: allow hypervisor to indicate linkspeed and duplex setting

2017-12-14 Thread Jason Baron via Qemu-devel
EX feature flag, to indicate that a linkspeed and duplex setting are present. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang --- drivers/net/virtio_net.c| 11 ++- include/uapi/linux/virtio_net.h | 4 2 files changed, 14 insertions(+), 1

[Qemu-devel] [PATCH 2/2] qemu: add linkspeed and duplex setting to virtio-net

2017-12-14 Thread Jason Baron via Qemu-devel
vice virtio-net,speed=1,duplex=full' where speed is [-1...INT_MAX], and duplex is ["half"|"full"]. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang --- hw/net/virtio-net.c | 29 +

[Qemu-devel] [PATCH 0/2] virtio_net: allow hypervisor to indicate linkspeed and duplex setting

2017-12-14 Thread Jason Baron via Qemu-devel
meant as a demonstration of how I intend this to work. Thanks, -Jason Jason Baron (2): virtio_net: allow hypervisor to indicate linkspeed and duplex setting qemu: add linkspeed and duplex setting to virtio-net linux changes: drivers/net/virtio_net.c| 11 ++- include/uapi/linux/vi

[Qemu-devel] [PATCH v3 2/2] ahci: add migration support

2013-01-04 Thread Jason Baron
From: Jason Baron I've tested these patches by migrating Windows 7 and Fedora 17 guests (while uunder i/o) on both piix with ahci attached and on q35 (which has a built-in ahci controller). Changes from v2: -migrate all relevant ahci fields -flush any pending i/o in 'post_load'

[Qemu-devel] [PATCH v3 0/2] add ahci migration

2013-01-04 Thread Jason Baron
Hi, Add migration bits for ahci. This allows q35 to be migratable. I also have been working on some qtest migration tests, to show that this does something. I will re-post those separately. Thanks, -Jason Jason Baron (2): ahci: remove unused AHCIDevice fields ahci: add migration support

[Qemu-devel] [PATCH v3 1/2] ahci: remove unused AHCIDevice fields

2013-01-04 Thread Jason Baron
From: Jason Baron 'dma_status' and 'dma_cb' are written to, but never read. Remove these fields in preparation for AHCI migration bits. Signed-off-by: Jason Baron --- hw/ide/ahci.c |8 ++-- hw/ide/ahci.h |2 -- 2 files changed, 2 insertions(+), 8 deletions(-)

Re: [Qemu-devel] [PATCH v2 1/3] qtest: Enable creation of multiple qemu instances

2012-12-20 Thread Jason Baron
On Thu, Dec 20, 2012 at 08:07:02PM +, Blue Swirl wrote: > On Thu, Dec 20, 2012 at 5:14 PM, Jason Baron wrote: > > From: Jason Baron > > > > Currently, the qtest harness can only spawn 1 qemu instance at a time > > because > > the parent pid is used to crea

[Qemu-devel] [PATCH v2 2/3] qtest: extend qtest_qmp() to fill in the reply

2012-12-20 Thread Jason Baron
From: Jason Baron Introduce: Add void qtest_qmp_resp(QTestState *s, QString *resp, const char *fmt, ...) which allows a response string to be filled in. Signed-off-by: Jason Baron --- tests/Makefile |2 +- tests/libqtest.c | 17 ++--- tests/libqtest.h | 15

[Qemu-devel] [PATCH v2 0/3] qtest: add migration testing

2012-12-20 Thread Jason Baron
ere, once those ahci migration patches are accepted. Thanks, -Jason v2: * move core migration functions to libqtest.c * simplify Makefile dependencies * use 'mkdtemp()' instead of child pid to identify qemu instances Jason Baron (3): qtest: Enable creation of multiple qemu instances

[Qemu-devel] [PATCH v2 3/3] qtest: add migrate-test

2012-12-20 Thread Jason Baron
From: Jason Baron Tests a single 'pc' machine migration on the same host. Would be nice to extend the test matrix to various machine versions, but that requires building multiple qemu binaries, which is a bit awkward in the context of qtest. Testing migration between different machin

[Qemu-devel] [PATCH v2 1/3] qtest: Enable creation of multiple qemu instances

2012-12-20 Thread Jason Baron
From: Jason Baron Currently, the qtest harness can only spawn 1 qemu instance at a time because the parent pid is used to create the socket files. Use 'mkdtemp()' in combination with the parent pid to avoid conflicts. Signed-off-by: Jason Baron --- tests/libqtest.c | 15

Re: [Qemu-devel] Q35, Mac OS X, and the War On Entropy

2012-12-19 Thread Jason Baron
On Wed, Dec 19, 2012 at 02:32:42PM -0500, Gabriel L. Somlo wrote: > Hi, > > I'm working on getting OS X to run on KVM, and the latest > q35-qemu tree from GitHub, plus additionally applied commit > 40862309a9d733cb0e878c79f477de003897b5d2 from mainline works > great, with the following command lin

Re: [Qemu-devel] [PATCH 1/3] qtest: Enable creation of multiple qemu instances

2012-12-17 Thread Jason Baron
On Sat, Dec 15, 2012 at 09:20:13AM +, Blue Swirl wrote: > On Sat, Dec 15, 2012 at 9:14 AM, Paolo Bonzini wrote: > >> > +#define QTEST_FILE_TEMP "/tmp/qtest-%d.sock" > >> > +#define QTEST_QMP_FILE_TEMP "/tmp/qtest-%d.qmp" > >> > +#define QTEST_PID_FILE_TEMP "/tmp/qtest-%d.pid" > >> > >> These f

Re: [Qemu-devel] [PATCH 3/3] qtest: add migrate-test

2012-12-14 Thread Jason Baron
gt; > versions with the > > same binary doesn't seem too useful. > > > > Signed-off-by: Jason Baron > > --- > > tests/Makefile |2 + > > tests/migrate-test.c | 140 > > ++ > >

Re: [Qemu-devel] [PATCH 2/3] qtest: extend qtest_qmp() to fill in the reply

2012-12-14 Thread Jason Baron
On Fri, Dec 14, 2012 at 01:07:24AM +0100, Andreas Färber wrote: > Am 13.12.2012 23:02, schrieb Jason Baron: > > diff --git a/tests/Makefile b/tests/Makefile > > index b60f0fb..30a101d 100644 > > --- a/tests/Makefile > > +++ b/tests/Makefile > > @@ -74,10 +74,

[Qemu-devel] [PATCH 3/3] qtest: add migrate-test

2012-12-13 Thread Jason Baron
From: Jason Baron Tests a single 'pc' machine migration on the same host. Currently, the test fail for q35 since the ahci controller doesn't yet migrate. Will add support for q35 once the ahci support is accepted. Would be nice to extend the test matrix to various machine ver

[Qemu-devel] [PATCH 0/3] qtest: add migration testing

2012-12-13 Thread Jason Baron
ere, once those ahci migration patches are accepted. Thanks, -Jason Jason Baron (3): qtest: Enable creation of multiple qemu instances qtest: extend qtest_qmp() to fill in the reply qtest: add migrate-test tests/Makefile |8 ++- tests/libqtest.c | 48 ++ tests/l

[Qemu-devel] [PATCH 1/3] qtest: Enable creation of multiple qemu instances

2012-12-13 Thread Jason Baron
From: Jason Baron Currently, the qtest harness can only spawn 1 qemu instance at a time because the parent pid is used to create the socket files. Use the child pid instead, so we can remove that limitation. Signed-off-by: Jason Baron --- tests/libqtest.c | 31

[Qemu-devel] [PATCH 2/3] qtest: extend qtest_qmp() to fill in the reply

2012-12-13 Thread Jason Baron
From: Jason Baron Introduce: Add void qtest_qmp_resp(QTestState *s, QString *resp, const char *fmt, ...) which allows a response string to be filled in. Signed-off-by: Jason Baron --- tests/Makefile |6 +++--- tests/libqtest.c | 17 ++--- tests/libqtest.h |4 +++- 3

Re: [Qemu-devel] qemu 1.3: windows 2003 SP2 x64 boot crash with hpet enabled (was ok with qemu-kvm 1.2)

2012-12-11 Thread Jason Baron
On Tue, Dec 11, 2012 at 08:02:12AM +0100, Alexandre DERUMIER wrote: > seem to be related to seabios update: > > seabios: q35 update > http://lists.gnu.org/archive/html/qemu-devel/2012-12/msg00113.html > This is most likely due to SeaBIOS commit: commit d9f5cdbdf55d61aef9a1a534d9123ef734427478

Re: [Qemu-devel] [PULL for-1.3 0/3] seabios: q35 update

2012-12-04 Thread Jason Baron
On Tue, Dec 04, 2012 at 08:40:27AM +0100, Gerd Hoffmann wrote: > Hi, > > > 1) legacy ide mode > > > > I can currently create a ide controller on the command-line using > > '-device'. However, on the real h/w there is an IDE compatibility mode > > which essentially advertises an ide controller a

Re: [Qemu-devel] [PULL for-1.3 0/3] seabios: q35 update

2012-12-03 Thread Jason Baron
able. > > With this pull "qemu -M q35" JustWorks[tm]. > > please pull, > Gerd > Hi, Feel free to add my Acked-by: Jason Baron to the series. I've just been looking at WindowsXP support and there are a few issues that need to be resolved to make it work with

Re: [Qemu-devel] [PATCH V10 2/8] hw/apm.c: replace register_ioport*

2012-11-27 Thread Jason Baron
On Tue, Nov 27, 2012 at 01:10:16AM +0100, Andreas Färber wrote: > Am 19.09.2012 13:50, schrieb Julien Grall: > > This patch replaces all register_ioport* by a MemorySection. > > It permits to use the new Memory stuff like listener. > > > > Moreover, the PCI is added as an argument for apm_init, so

Re: [Qemu-devel] [PATCH v4 00/14] Add Q35 base support

2012-11-22 Thread Jason Baron
On Thu, Nov 22, 2012 at 12:24:43PM +0100, Gerd Hoffmann wrote: > > These patches are intened to give us a base set of patches for Q35 upon > > which > > to build. The major change in this series is to add the memory controller > > hub, > > or 'mch' as proper member of the q35 host structure. This

Re: [Qemu-devel] [PATCH v4 07/14 (re-post)] ich9: Add smbus

2012-11-22 Thread Jason Baron
From: Jason Baron Add support for the ich9 smbus chip. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.objs |2 - hw/smbus_ich9.c | 159 ++ 2 files changed, 160 insertions(+), 1 deletions(-) create mode

Re: [Qemu-devel] [PATCH v4 05/14 (re-post)] ich9: Add acpi support and definitions

2012-11-22 Thread Jason Baron
From: Jason Baron Lay the groundwork for subsequent ich9 support. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.objs |2 hw/acpi_ich9.c | 315 ++ hw/acpi_ich9.h | 47 hw/ich9.h| 207

Re: [Qemu-devel] TCP based PCIE request forwarding

2012-11-20 Thread Jason Baron
On Fri, Nov 16, 2012 at 09:39:07AM +0100, lementec fabien wrote: > Hi, > > I am a software engineer who works in an electronic group. Using QEMU > to emulate devices allows me to start writing and testing LINUX software > before the device is actually available. In the group, we are mostly > worki

[Qemu-devel] [PATCH v4 12/14] q35: Add kvmclock support

2012-11-14 Thread Jason Baron
From: Jan Kiszka Reviewed-by: Paolo Bonzini Signed-off-by: Jan Kiszka Signed-off-by: Jason Baron --- hw/pc_q35.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/hw/pc_q35.c b/hw/pc_q35.c index 142bf8a..3429a9a 100644 --- a/hw/pc_q35.c +++ b/hw/pc_q35.c @@ -34,6

[Qemu-devel] [PATCH v4 04/14] pc/piix_pci: factor out smram/pam logic

2012-11-14 Thread Jason Baron
From: Isaku Yamahata Factor out smram/pam logic for use by other chipsets, namely q35 at this point. Note: Should be factored out into a generic North Bridge Class. [jba...@redhat.com: changes for updated memory API] Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw

[Qemu-devel] [PATCH v4 09/14] ich9: Add i82801b11 dmi-to-pci bridge

2012-11-14 Thread Jason Baron
From: Jason Baron Add the dmi-to-pci i82801b11 bridge chip. This is the pci bridge chip that q35 uses on its host bus for PCI bus arbitration. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.objs |1 + hw/i82801b11.c | 125

[Qemu-devel] [PATCH v4 10/14] q35: Suppress SMM BIOS initialization under KVM

2012-11-14 Thread Jason Baron
From: Jan Kiszka Same as for i44fx: KVM does not support SMM yet. Signal it initialized to Seabios to avoid failures. Reviewed-by: Paolo Bonzini Signed-off-by: Jan Kiszka Signed-off-by: Jason Baron --- hw/acpi_ich9.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff

[Qemu-devel] [PATCH v4 05/14] ich9: Add acpi support and definitions

2012-11-14 Thread Jason Baron
From: Jason Baron Lay the groundwork for subsequent ich9 support. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.objs |1 + hw/acpi_ich9.c | 315 ++ hw/acpi_ich9.h | 47 hw/ich9.h| 207

[Qemu-devel] [PATCH v4 08/14] q35: Introduce q35 pc based chipset emulator

2012-11-14 Thread Jason Baron
will begin to version the machine names. Major features which still need to be added: -Migration support (mostly around ahci) -ACPI hotplug support (pcie hotplug support is working) -Passthrough support Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/i386/Makefile.ob

[Qemu-devel] [PATCH v4 13/14] Add a fallback bios file search, if -L fails.

2012-11-14 Thread Jason Baron
From: Jason Baron If -L is specified, and qemu does not find the bios file in , then the search fails. Add infrastructure such that the search will continue in the default paths, if not found in the -L path. Reviewed-by: Paolo Bonzini Signed-off-by: Jason Baron --- vl.c | 36

[Qemu-devel] [PATCH v4 14/14] q35: automatically load the q35 dsdt table

2012-11-14 Thread Jason Baron
From: Jason Baron Automatically, locate the required q35 dsdt table on load. Otherwise we error out. This could be done in the bios, but its harder to produce a good error message. Signed-off-by: Jason Baron --- hw/pc.c | 19 +++ hw/pc.h |2 ++ hw/pc_q35.c |7

[Qemu-devel] [PATCH v4 02/14] pc: Move ioapic_init() from pc_piix.c to pc.c

2012-11-14 Thread Jason Baron
From: Jason Baron Move ioapic_init() from pc_piix.c to pc.c, to make it a common function. Rename ioapic_init() -> ioapic_init_gsi(). Move to pc.h so q35 can use them as well. Reviewed-by: Paolo Bonzini Signed-off-by: Jason Baron --- hw/pc.c | 24 hw/p

[Qemu-devel] [PATCH v4 07/14] ich9: Add smbus

2012-11-14 Thread Jason Baron
From: Jason Baron Add support for the ich9 smbus chip. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.objs |2 +- hw/smbus_ich9.c | 159 ++ 2 files changed, 160 insertions(+), 1 deletions(-) create mode

[Qemu-devel] [PATCH v4 03/14] pc_piix: Move kvm irq routing functions out of pc_piix.c

2012-11-14 Thread Jason Baron
From: Jason Baron Rename: kvm_piix3_gsi_handlei() -> kvm_pc_gsi_handler() kvm_piix3_setup_irq_routing() -> kvm_pc_setup_irq_routing() This is in preparation for other users, namely q35 at this time. Signed-off-by: Jason Baron --- hw/kvm/ioapic.c

[Qemu-devel] [PATCH v4 01/14] pc, pc_piix: split out pc nic initialization

2012-11-14 Thread Jason Baron
From: Isaku Yamahata Factor out pc nic initialization. This simplifies the pc initialization and will reduce the code duplication of q35 pc initialization. Reviewed-by: Paolo Bonzini Reviewed-by: Anthony Liguori Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/pc.c

[Qemu-devel] [PATCH v4 11/14] q35: Fix non-PCI IRQ processing in ich9_lpc_update_apic

2012-11-14 Thread Jason Baron
From: Jan Kiszka Avoid passing a non-PCI IRQ to ich9_gsi_to_pirq. It's wrong and triggers an assertion. Reviewed-by: Paolo Bonzini Signed-off-by: Jan Kiszka Signed-off-by: Jason Baron --- hw/lpc_ich9.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/hw/lpc

[Qemu-devel] [PATCH v4 06/14] ich9: Add the lpc chip

2012-11-14 Thread Jason Baron
From: Jason Baron Add support for the ICH9 LPC chip. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/i386/Makefile.objs |1 + hw/lpc_ich9.c | 523 + 2 files changed, 524 insertions(+), 0 deletions(-) create mode

[Qemu-devel] [PATCH v4 00/14] Add Q35 base support

2012-11-14 Thread Jason Baron
) -removed patches that were merged in v1 Isaku Yamahata (3): pc, pc_piix: split out pc nic initialization pc/piix_pci: factor out smram/pam logic q35: Introduce q35 pc based chipset emulator Jan Kiszka (3): q35: Suppress SMM BIOS initialization under KVM q35: Fix non-PCI IRQ processing

Re: [Qemu-devel] [PATCH v1 00/13] q35 patches for pci tree

2012-10-31 Thread Jason Baron
On Wed, Oct 31, 2012 at 07:55:13AM -0500, Anthony Liguori wrote: > "Michael S. Tsirkin" writes: > > > On Tue, Oct 30, 2012 at 02:20:35PM -0500, Anthony Liguori wrote: > >> Jason Baron writes: > >> > >> > Hi, > >> > > >

Re: [Qemu-devel] [PATCH v1 12/13] q35: fill in usb pci slots with -usb

2012-10-30 Thread Jason Baron
On Tue, Oct 30, 2012 at 05:19:01PM +0100, Gerd Hoffmann wrote: > On 10/30/12 16:19, Jason Baron wrote: > > On Tue, Oct 30, 2012 at 07:34:26AM +0100, Gerd Hoffmann wrote: > >> Hi, > >> > >>> +uhci_devname[sizeof(uhci_devname) - 2] = ((char)'

Re: [Qemu-devel] [PATCH v1 12/13] q35: fill in usb pci slots with -usb

2012-10-30 Thread Jason Baron
On Tue, Oct 30, 2012 at 07:34:26AM +0100, Gerd Hoffmann wrote: > Hi, > > > +uhci_devname[sizeof(uhci_devname) - 2] = ((char)'1') + i; > > snprintf(devname, sizeof(devname), "...%d", i) is more readable. ok. > > > +qdev_prop_set_string(usb_qdev, "masterbus", "ich9-usb-

Re: [Qemu-devel] On block interface types in general, IF_AHCI in particular

2012-10-30 Thread Jason Baron
On Tue, Oct 30, 2012 at 03:43:20PM +0100, Markus Armbruster wrote: > The primary purpose of this memo is a brain dump on how block interface > types are used, and what that means for AHCI. A secondary purpose is to > disabuse Alex of the notion that -drive is simple ;) > > > BlockInterfaceType r

[Qemu-devel] [PATCH v1 12/13] q35: fill in usb pci slots with -usb

2012-10-29 Thread Jason Baron
From: Jason Baron This fills out the usb slots on q35, when -usb is passed. We now have (lspci output): 00:1d.0 USB Controller: Intel Corporation 82801I (ICH9 Family) USB UHCI Controller #1 (rev 03) 00:1d.1 USB Controller: Intel Corporation 82801I (ICH9 Family) USB UHCI Controller #2 (rev 03

[Qemu-devel] [PATCH v1 01/13] pc/piix_pci: factor out smram/pam logic

2012-10-29 Thread Jason Baron
From: Isaku Yamahata Factor out smram/pam logic for later use. Which will be used by q35 too. [jba...@redhat.com: changes for updated memory API] Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/i386/Makefile.objs |1 + hw/pam.c | 87

[Qemu-devel] [PATCH v1 02/13] Back out add of i21154

2012-10-29 Thread Jason Baron
From: Jason Baron As pointed out by Andreas Färber this is covered by dec_pci.c. Signed-off-by: Jason Baron --- hw/Makefile.objs |2 +- hw/i21154.c | 113 -- hw/i21154.h |9 3 files changed, 1 insertions(+), 123

[Qemu-devel] [PATCH v1 07/13] q35: Introduce q35 pc based chipset emulator

2012-10-29 Thread Jason Baron
will begin to version the machine names. Major features which still need to be added: -Migration support (mostly around ahci) -ACPI hotplug support (pcie hotplug support is working) -Passthrough support Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/i386/Makefil

[Qemu-devel] [PATCH v1 13/13] Fixup q35/ich9 Licenses

2012-10-29 Thread Jason Baron
From: Jason Baron Cleanup the q35/ich9 license headers. Signed-off-by: Jason Baron --- hw/acpi_ich9.c | 13 +++-- hw/lpc_ich9.c | 32 hw/smbus_ich9.c | 14 -- 3 files changed, 23 insertions(+), 36 deletions(-) diff --git a/hw

[Qemu-devel] [PATCH v1 11/13] q35: automatically load the q35 dsdt table

2012-10-29 Thread Jason Baron
From: Jason Baron Automatically, locate the required q35 dsdt table on load. Otherwise we error out. This could be done in the bios, but its harder to produce a good error message. Signed-off-by: Jason Baron --- hw/pc.c | 19 +++ hw/pc.h |2 ++ hw/pc_q35.c |7

[Qemu-devel] [PATCH v1 09/13] q35: Add kvmclock support

2012-10-29 Thread Jason Baron
From: Jan Kiszka Reviewed-by: Paolo Bonzini Signed-off-by: Jan Kiszka Signed-off-by: Jason Baron --- hw/pc_q35.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/hw/pc_q35.c b/hw/pc_q35.c index cf0d361..1f31486 100644 --- a/hw/pc_q35.c +++ b/hw/pc_q35.c @@ -52,6

[Qemu-devel] [PATCH v1 08/13] q35: Suppress SMM BIOS initialization under KVM

2012-10-29 Thread Jason Baron
From: Jan Kiszka Same as for i44fx: KVM does not support SMM yet. Signal it initialized to Seabios to avoid failures. Reviewed-by: Paolo Bonzini Signed-off-by: Jan Kiszka Signed-off-by: Jason Baron --- hw/acpi_ich9.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff

[Qemu-devel] [PATCH v1 10/13] Add a fallback bios file search, if -L fails.

2012-10-29 Thread Jason Baron
From: Jason Baron If -L is specified, and qemu does not find the bios file in , then the search fails. Add infrastructure such that the search will continue in the default paths, if not found in the -L path. Reviewed-by: Paolo Bonzini Signed-off-by: Jason Baron --- vl.c | 36

[Qemu-devel] [PATCH v1 04/13] blockdev: Introduce IF_AHCI

2012-10-29 Thread Jason Baron
From: Jason Baron Introduce IF_AHCI so that q35 can differentiate between ide and ahci disks. This allows q35 to specify its default disk type. It also allows q35 to differentiate between ahci and ide disks, such that -drive if=ide does not result in the creating of an ahci disk. This is

[Qemu-devel] [PATCH v1 03/13] blockdev: Introduce QEMUMachine->default_drive_if

2012-10-29 Thread Jason Baron
From: Jason Baron The current QEMUMachine definition has a 'use_scsi' field to indicate if a machine type should use scsi by default. However, Q35 wants to use ahci by default. Thus, introdue a new field in the QEMUMachine defintion, default_drive_if. Please use 'st

[Qemu-devel] [PATCH v1 06/13] pc: Move ioapic_init() from pc_piix.c to pc.c

2012-10-29 Thread Jason Baron
From: Jason Baron Move ioapic_init from pc_piix.c to pc.c, to make it a common function. Rename ioapic_init -> ioapic_init_gsi. Reviewed-by: Paolo Bonzini Signed-off-by: Jason Baron --- hw/pc.c | 24 hw/pc.h |2 ++ hw/pc_piix.c |

[Qemu-devel] [PATCH v1 05/13] pc, pc_piix: split out pc nic initialization

2012-10-29 Thread Jason Baron
From: Isaku Yamahata Factor out pc nic initialization. This simplifies the pc initialization and will reduce the code duplication of q35 pc initialization. Reviewed-by: Paolo Bonzini Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/pc.c | 15 +++ hw/pc.h

[Qemu-devel] [PATCH v1 00/13] q35 patches for pci tree

2012-10-29 Thread Jason Baron
Isaku Yamahata (3): pc/piix_pci: factor out smram/pam logic pc, pc_piix: split out pc nic initialization q35: Introduce q35 pc based chipset emulator Jan Kiszka (2): q35: Suppress SMM BIOS initialization under KVM q35: Add kvmclock support Jason Baron (8): Back out add of i21

Re: [Qemu-devel] q35: usb keyboard trouble

2012-10-26 Thread Jason Baron
On Wed, Oct 24, 2012 at 10:55:28AM -0400, Gabriel L. Somlo wrote: > Jason, > > Commit d8b0dbdba325773469733222a167b54aca74de55 in the q35 tree breaks > '-usbdevice keyboard' for me. Instead of being able to type at the VM, > none of the keypresses make it through, and qemu stderr soon starts > log

Re: [Qemu-devel] [PATCH v3 01/26] blockdev: Introduce a default machine blockdev interface field, QEMUMachine->mach_if

2012-10-24 Thread Jason Baron
On Wed, Oct 24, 2012 at 03:12:36PM +0200, Markus Armbruster wrote: > Jason Baron writes: > > > From: Jason Baron > > > > The current QEMUMachine definition has a 'use_scsi' field to indicate if a > > machine type should use scsi by default. However, Q3

Re: [Qemu-devel] [PATCH v3 02/26] blockdev: Introduce IF_AHCI

2012-10-24 Thread Jason Baron
On Wed, Oct 24, 2012 at 05:50:25PM +0200, Markus Armbruster wrote: > Jason Baron writes: > > > On Mon, Oct 22, 2012 at 01:40:21PM +0200, Kevin Wolf wrote: > >> >> From: Jason Baron > >> >> > >> >> Introduce IF_AHCI so that q35 can differ

Re: [Qemu-devel] [PATCH v3 17/26] Add i21154 bridge chip.

2012-10-22 Thread Jason Baron
On Mon, Oct 22, 2012 at 04:03:05PM +0200, Michael S. Tsirkin wrote: > On Fri, Oct 19, 2012 at 04:43:37PM -0400, Jason Baron wrote: > > From: Jason Baron > > > > This adds support for the DECchip 21154 PCI bridge. > > > > Signed-off-by: Isaku Yamahat

Re: [Qemu-devel] [PATCH v3 17/26] Add i21154 bridge chip.

2012-10-22 Thread Jason Baron
On Mon, Oct 22, 2012 at 06:17:30PM +0200, Michael S. Tsirkin wrote: > On Mon, Oct 22, 2012 at 03:26:24PM +0200, Andreas Färber wrote: > > Am 19.10.2012 22:43, schrieb Jason Baron: > > > From: Jason Baron > > > > > > This adds support for the DECchip 21154 PC

  1   2   3   >