On 4 August 2013 00:43, Programmingkid wrote:
>
> On Aug 3, 2013, at 7:06 PM, Peter Maydell wrote:
>
>> On 3 August 2013 23:52, G 3 wrote:
>>> This patch adds the ability to use the command key in the guest
>>> operating system. Just add -command-key 55 to the command line
>>> options sent to QEM
I was looking at cross-version migration issues, in the
hope that we can fix most of them for release 1.6.
I noticed that we still use QEMU_VERSION in hardware.
hw/scsi/megasas.c:snprintf(info.package_version, 0x60, "%s-QEMU",
QEMU_VERSION);
hw/usb/redirect.c:#define VERSION "qemu usb-redir g
On 4 August 2013 11:20, Michael S. Tsirkin wrote:
> I was looking at cross-version migration issues, in the
> hope that we can fix most of them for release 1.6.
> I noticed that we still use QEMU_VERSION in hardware.
We fixed most of these back in 2012, but I guess one or
two slipped through the
On Sun, Aug 4, 2013 at 4:39 AM, Peter Maydell wrote:
>
> "Should the QEMU UI use the menu-accelerator key for menus or
> should it pass it through to the guest" is a generic UI front-end
> problem; any solution should not be specific to a single UI,
> we should handle it the same way for all front
Am 04.08.2013 00:02, schrieb Aurelien Jarno:
> Now that this code path is not triggered anymore during the tests,
> revert commit b332d24a8e1290954029814d09156b06ede358e2. Booting a MIPS
> target without kernel nor bios doesn't really make sense.
>
> Signed-off-by: Aurelien Jarno
This is being d
Am 04.08.2013 12:25, schrieb Peter Maydell:
> On 4 August 2013 11:20, Michael S. Tsirkin wrote:
>> I was looking at cross-version migration issues, in the
>> hope that we can fix most of them for release 1.6.
>> I noticed that we still use QEMU_VERSION in hardware.
>
> We fixed most of these back
On 01/08/13 23:08, Anthony Liguori wrote:
Hi,
On behalf of the QEMU Team, I'd like to announce the availability of the
second release candidate for the QEMU 1.6 release. This release is meant
for testing purposes and should not be used in a production environment.
http://wiki.qemu.org/downloa
pxa2xx_i2c_init() creates a pxa2xx-i2c-slave device on a second i2c-bus,
which has a NULL parent device. This causes an assertion in
object_get_canonical_path() when accessing pxa2xx-i2c-slave's
"parent_bus" link property in tosa and likely other PXA2xx machines.
Fix this by using the pxa2xx_i2c d
Am 04.08.2013 15:05, schrieb Andreas Färber:
> pxa2xx_i2c_init() creates a pxa2xx-i2c-slave device on a second i2c-bus,
> which has a NULL parent device. This causes an assertion in
> object_get_canonical_path() when accessing pxa2xx-i2c-slave's
> "parent_bus" link property in tosa and likely other
Hi Mike,
you might be the guy StefanHa was referring to on the qemu-devel mailing-list.
I just made some more tests, so…
Am 02.08.2013 um 23:47 schrieb Mike Dawson :
> Oliver,
>
> We've had a similar situation occur. For about three months, we've run
> several Windows 2008 R2 guests with virt
Hello,
Based on an initial draft from Othmar Pasteka, this series QOM'ifies MicroDrive.
As a prerequisite, PCMCIA must be QOM'ified as pointed out by Peter. We can then
finally drop ide_init2_with_non_qdev_drives(), as desired by Markus for his
IDE refactorings. As a final touch, pxa2xx_pcmcia is
Turn it into a SysBusDevice and use a container MemoryRegion.
Add a link property to the PCMCIACardState.
Signed-off-by: Andreas Färber
---
hw/pcmcia/pxa2xx.c | 78 +-
1 file changed, 66 insertions(+), 12 deletions(-)
diff --git a/hw/pcmcia/p
All its users have finally been converted.
Signed-off-by: Andreas Färber
---
hw/ide/core.c | 49 -
hw/ide/internal.h | 2 --
2 files changed, 51 deletions(-)
diff --git a/hw/ide/core.c b/hw/ide/core.c
index a73af72..ac0ffcf 100644
--- a/hw/id
Turn PCMCIACardState into a device.
Move callbacks to new PCMCIACardClass.
Derive TYPE_MICRODRIVE from TYPE_PCMCIA_CARD.
Replace ide_init2_with_non_qdev_drives().
Signed-off-by: Othmar Pasteka
Signed-off-by: Andreas Färber
---
hw/Makefile.objs | 1 +
hw/ide/microd
Signed-off-by: Andreas Färber
---
hw/misc/pxa2xx_pcmcia.c | 2 +-
include/hw/arm/pxa.h| 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/misc/pxa2xx_pcmcia.c b/hw/misc/pxa2xx_pcmcia.c
index ef71a2a..76724a6 100644
--- a/hw/misc/pxa2xx_pcmcia.c
+++ b/hw/misc/pxa2xx_pcmci
Add missing braces.
Signed-off-by: Andreas Färber
---
hw/ide/microdrive.c | 62 -
1 file changed, 38 insertions(+), 24 deletions(-)
diff --git a/hw/ide/microdrive.c b/hw/ide/microdrive.c
index 0b53649..9e4ee9c 100644
--- a/hw/ide/microdrive.c
Am 04.08.2013 15:17, schrieb Andreas Färber:
> Am 04.08.2013 15:05, schrieb Andreas Färber:
>> pxa2xx_i2c_init() creates a pxa2xx-i2c-slave device on a second i2c-bus,
>> which has a NULL parent device. This causes an assertion in
>> object_get_canonical_path() when accessing pxa2xx-i2c-slave's
>>
On Aug 4, 2013, at 5:39 AM, Peter Maydell wrote:
> On 4 August 2013 00:43, Programmingkid wrote:
>>
>> On Aug 3, 2013, at 7:06 PM, Peter Maydell wrote:
>>
>>> On 3 August 2013 23:52, G 3 wrote:
This patch adds the ability to use the command key in the guest
operating system. Just ad
On 4 August 2013 18:53, Programmingkid wrote:
> On Aug 4, 2013, at 5:39 AM, Peter Maydell wrote
>> Right, but I don't have to specify anything about any other
>> key on the keyboard, why should command be special?
>
> The command key is used to send commands to an application. When
> the user runs
checkpatch seems to do different things if called with a wildcard than
just the patch file.
This is from patch 7 in a series I'm just about to post. It's
now claiming it wants no space after the *, but whilst developing
it it complained the spaces around the * did not balance (as if
it were multi
Add QEMUTimerListGroup and helper functions, to represent
a QEMUTimerList associated with each clock. Add a default
QEMUTimerListGroup representing the default timer lists
which are not associated with any other object (e.g.
an AioContext as added by future patches).
Signed-off-by: Alex Bligh
---
Make qemu_run_timers and qemu_run_all_timers return progress
so that aio_poll etc. can determine whether a timer has been
run.
Signed-off-by: Alex Bligh
---
include/qemu/timer.h |4 ++--
qemu-timer.c | 18 --
2 files changed, 14 insertions(+), 8 deletions(-)
diff -
This patch series adds support for timers attached to an AioContext clock
which get called within aio_poll.
In doing so it removes alarm timers and moves to use ppoll where possible.
This patch set 'sort of' passes make check (see below for caveat)
including a new test harness for the aio timers,
Add qemu_free_clock and expose qemu_new_clock and clock types.
Add utility functions to qemu-timer.c for nanosecond timing.
Add qemu_clock_deadline_ns to calculate deadlines to
nanosecond accuracy.
Add utility function qemu_soonest_timeout to calculate soonest deadline.
Add qemu_timeout_ns_to_m
Where supported, called prctl(PR_SET_TIMERSLACK, 1, ...) to
set one nanosecond timer slack to increase precision of timer
calls.
Signed-off-by: Alex Bligh
---
configure| 18 ++
qemu-timer.c |7 +++
2 files changed, 25 insertions(+)
diff --git a/configure b/configur
Add a QEMUTimerListGroup each AioContext (meaning a QEMUTimerList
associated with each clock is added) and delete it when the
AioContext is freed.
Signed-off-by: Alex Bligh
---
async.c |2 ++
include/block/aio.h |4
tests/test-aio.c |3 +++
tests/te
include/qemu/timer.h has no need to include main-loop.h and
doing so causes an issue for the next patch. Unfortunately
various files assume including timers.h will pull in main-loop.h.
Untangle this mess.
Signed-off-by: Alex Bligh
---
dma-helpers.c |1 +
hw/dma/xilinx_axidma.c
On qemu_mod_timer_ns, ensure qemu_notify or aio_notify is called to
end the appropriate poll(), irrespective of use_icount value.
On qemu_clock_enable, ensure qemu_notify or aio_notify is called for
all QEMUTimerLists attached to the QEMUClock.
Signed-off-by: Alex Bligh
---
include/qemu/timer.h
Remove alarm timers from qemu-timers.c now we use g_poll / ppoll
instead.
Signed-off-by: Alex Bligh
---
include/qemu/timer.h |2 -
main-loop.c |4 -
qemu-timer.c | 500 +-
vl.c |4 +-
4 files changed, 4
Add a notify pointer to QEMUTimerList so it knows what to notify
on a timer change.
Signed-off-by: Alex Bligh
---
async.c |7 ++-
include/qemu/timer.h |7 ++-
qemu-timer.c | 24 ++--
3 files changed, 34 insertions(+), 4 deletions(-)
dif
Convert mainloop to use timeout from default timerlist group
(i.e. the current 3 static timers)
Signed-off-by: Alex Bligh
---
main-loop.c | 45 ++---
1 file changed, 34 insertions(+), 11 deletions(-)
diff --git a/main-loop.c b/main-loop.c
index a44fff6.
Split QEMUClock into QEMUClock and QEMUTimerList so that we can
have more than one QEMUTimerList associated with the same clock.
Introduce a default_timerlist concept and make existing
qemu_clock_* calls that actually should operate on a QEMUTimerList
call the relevant QEMUTimerList implementation
Add a test harness for AioContext timers. The g_source equivalent is
unsatisfactory as it suffers from false wakeups.
Signed-off-by: Alex Bligh
---
tests/test-aio.c | 134 ++
1 file changed, 134 insertions(+)
diff --git a/tests/test-aio.c b/t
Add qemu_poll_ns which works like g_poll but takes a nanosecond
timeout.
Signed-off-by: Alex Bligh
---
configure| 19 +++
include/qemu/timer.h |1 +
qemu-timer.c | 24
3 files changed, 44 insertions(+)
diff --git a/configure b
On Aug 4, 2013, at 7:58 AM, Anthony Liguori wrote:
> On Sun, Aug 4, 2013 at 4:39 AM, Peter Maydell
> wrote:
>>
>> "Should the QEMU UI use the menu-accelerator key for menus or
>> should it pass it through to the guest" is a generic UI front-end
>> problem; any solution should not be specific t
Notify all timerlists derived from vm_clock in icount warp
calculations.
When calculating timer delay based on vm_clock deadline, use
all timerlists.
For compatibility, maintain an apparent bug where when using
icount, if no vm_clock timer was set, qemu_clock_deadline
would return INT32_MAX and a
Convert aio_poll to use deadline based on AioContext's timers.
aio_poll has been changed to return accurately whether progress
has occurred. Prior to this commit, aio_poll always returned
true if g_poll was entered, whether or not any progress was
made. This required a change to tests/test-aio.c w
Calculate the timeout in aio_ctx_prepare taking into account
the timers attached to the AioContext.
Alter aio_ctx_check similarly.
Signed-off-by: Alex Bligh
---
async.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/async.c b/async.c
index c80da2f..8df3e99
On Aug 4, 2013, at 2:07 PM, Peter Maydell wrote:
> On 4 August 2013 18:53, Programmingkid wrote:
>> On Aug 4, 2013, at 5:39 AM, Peter Maydell wrote
>>> Right, but I don't have to specify anything about any other
>>> key on the keyboard, why should command be special?
>>
>> The command key is us
On 4 August 2013 19:09, Alex Bligh wrote:
> checkpatch seems to do different things if called with a wildcard than
> just the patch file.
That's obviously a bug...
> This is from patch 7 in a series I'm just about to post. It's
> now claiming it wants no space after the *, but whilst developing
On Mon, Jul 29, 2013 at 10:35:31PM +0200, Stefan Weil wrote:
> Am 27.07.2013 22:58, schrieb Stefan Weil:
> > Am 27.07.2013 22:43, schrieb Andreas Färber:
> >> Am 27.07.2013 21:37, schrieb Stefan Weil:
> >>> Am 27.07.2013 19:43, schrieb Peter Maydell:
> On 27 July 2013 17:18, Hervé Poussineau
On Sun, Aug 04, 2013 at 02:03:20PM +0200, Andreas Färber wrote:
> Am 04.08.2013 00:02, schrieb Aurelien Jarno:
> > Now that this code path is not triggered anymore during the tests,
> > revert commit b332d24a8e1290954029814d09156b06ede358e2. Booting a MIPS
> > target without kernel nor bios doesn't
On 4 August 2013 23:04, Aurélien Jarno wrote:
> The real hardware probably returns all 1 or all 0 for addresses not
> decoded to a device. This is what QEMU should model, and it should
> not trigger a DBE or IBE exception. Looking at the current MIPS
> documentation, Bus Error is defined as:
>
>
Reads to unassigned memory now return non-zero (since patch
9b8c69243585). This breaks guests runnong on i.MX31 that use the
cache controller --- they poll forever waiting for the L2CC cache
invalidate regsiter to be zero.
This patch adds in an L2 cache controller. It's not quite right ---
it r
When imx_epit.c was last refactored, a common usecase (comparison
register zero) broke. This patch fixes that, and simplifies the code
yet more. It also fixes a major thinko in the reset path --- the
wrong bits in the control register were being cleared.
Signed-off-by: Peter Chubb
Reviewed-by:
On 08/02/2013 09:33 PM, � wrote:
Am 02.08.2013 03:43, schrieb Tiejun Chen:
It makes more sense and simple later.
Signed-off-by: Tiejun Chen
---
v1 -> v2:
To optimize performance slightly, we can reorder the two conditions to
avoid the non-inline function call if cpu->stopped.
Patch doesn't
On Fri, Aug 2, 2013 at 10:43 PM, Stefan Hajnoczi wrote:
> On Fri, Aug 02, 2013 at 11:33:40AM +0800, liu ping fan wrote:
>> On Thu, Aug 1, 2013 at 9:28 PM, Alex Bligh wrote:
>> > Paolo,
>> >
>> >
>> > --On 1 August 2013 08:19:34 -0400 Paolo Bonzini
>> > wrote:
>> >
>> >>> > True, qemu_event basi
On 08/03/2013 03:40 AM, Michael R. Hines wrote:
On 07/25/2013 04:18 PM, Lei Li wrote:
Signed-off-by: Lei Li
---
arch_init.c | 58
++
1 files changed, 58 insertions(+), 0 deletions(-)
diff --git a/arch_init.c b/arch_init.c
index a8b
On 08/03/2013 03:48 AM, Michael R. Hines wrote:
On 07/25/2013 04:18 PM, Lei Li wrote:
Signed-off-by: Lei Li
---
include/sysemu/sysemu.h |1 +
savevm.c| 55
+++
2 files changed, 56 insertions(+), 0 deletions(-)
diff --git a
On 08/03/2013 03:45 AM, Michael R. Hines wrote:
On 07/25/2013 04:18 PM, Lei Li wrote:
Signed-off-by: Lei Li
---
Makefile.objs |1 +
include/migration/migration.h | 15
migration-local.c | 158
+
migration-uni
On 08/03/2013 03:46 AM, Michael R. Hines wrote:
On 07/25/2013 04:18 PM, Lei Li wrote:
Signed-off-by: Lei Li
---
qemu-options.hx |9 +
vl.c| 14 ++
2 files changed, 23 insertions(+), 0 deletions(-)
diff --git a/qemu-options.hx b/qemu-options.hx
index 83
On 08/03/2013 03:47 AM, Michael R. Hines wrote:
On 07/25/2013 04:18 PM, Lei Li wrote:
Signed-off-by: Lei Li
---
hmp-commands.hx | 17 +
hmp.c | 13 +
hmp.h |1 +
3 files changed, 31 insertions(+), 0 deletions(-)
diff --git a/hmp-comm
On 08/03/2013 03:42 AM, Michael R. Hines wrote:
On 07/25/2013 04:18 PM, Lei Li wrote:
Signed-off-by: Lei Li
---
arch_init.c | 32
1 files changed, 32 insertions(+), 0 deletions(-)
diff --git a/arch_init.c b/arch_init.c
index a418071..7eeb52f 100644
--- a/a
Am 05.08.2013 00:37, schrieb Peter Maydell:
> On 4 August 2013 23:04, Aurélien Jarno wrote:
>> The real hardware probably returns all 1 or all 0 for addresses not
>> decoded to a device. This is what QEMU should model, and it should
>> not trigger a DBE or IBE exception. Looking at the current MIP
On Fri, Aug 02, 2013 at 06:14:46PM +0200, Andreas Färber wrote:
> Am 02.08.2013 04:59, schrieb Alexey Kardashevskiy:
> > This patch adds CPU PVR definition for POWER7+.
> >
> > Signed-off-by: Alexey Kardashevskiy
> > ---
> > target-ppc/cpu-models.c | 2 ++
> > target-ppc/cpu-models.h | 1 +
> >
Hi,
>>> 1) Is this difference intentional, or a bug?
>> The vga simply goes into the first free slot. That happens to be #2
>> with i440fx and #1 with q35.
>
> It sounds like it's a safe bet to assume that -vga will put the device
> on slot 2 for pc machinetypes and slot 1 for q35, no matter w
56 matches
Mail list logo
