On Thu, Feb 09, 2012 at 08:13:52AM +0100, Paolo Bonzini wrote:
> On 02/09/2012 06:46 AM, Hu Tao wrote:
> >I've re-tested today, with host kernel 2.6.35.6-45.fc14.x86_64,
> >2.6.32-71.el6.x86_64, 3.1.0 and 3.3.0-rc1+, qemu version and guest
> >configuration remain the same, includi
On Wed, Feb 08, 2012 at 09:40:08PM -0500, Xin Tong wrote:
> can you show me where these threads get created in the qemu code ?
If you mean vcpu thread, see cpu_x86_init (target-i386/helper.c).
At the end of this function,
CPUX86State *cpu_x86_init(const char *cpu_model)
{
qemu_init_vcpu(en
I want to run a application using qemu-sparc,for example "$qemu-sparc
hello",where hello is an application compiled for sparc platform. And the host
os and architecture are linux and x86. But it catches an error when loading elf
file "hello",the spesific error is as follow:
// in load_elf_image
Looks like the subject would warmly welcome a "kvm:" tag.
On 2012-02-09 05:22, David Gibson wrote:
> In kvm-all.c we store an ioctl cmd number in the irqchip_inject_ioctl field
> of KVMState, which has type 'int'. This seems to make sense since the
> ioctl() man page says that the cmd parameter h
Avi,
Before I forget:
On 2012-02-05 13:39, Jan Kiszka wrote:
> +static void vapic_map_rom_writable(VAPICROMState *s)
> +{
> +target_phys_addr_t rom_paddr = s->rom_state_paddr & ROM_BLOCK_MASK;
> +MemoryRegionSection section;
> +MemoryRegion *as;
> +size_t rom_size;
> +uint8_t
This set of patches adds support for Samsung Exynos4210-based boards NURI and
SMDKC210.
Tested on Linux kernel v3.x series.
Usage:
Exynos4210 SOC boards are modelled to run two CPU cores..
Linux kernel v3.x configured with exynos4_defconfig can be used for boards
models testing with rootfs
on in
From: Maksim Kozlov
Patch adds basic model for Exynos4210 SoC PMU.
This model implements PMU registers just as a bulk of memory. Currently,
the only reason this device exists is that secondary CPU boot loader
uses PMU INFORM5 register as a holding pen.
Signed-off-by: Maksim Kozlov
Signed-off-by
Signed-off-by: Evgeny Voevodin
---
Makefile.target |2 +-
hw/exynos4210.c | 19 +
hw/exynos4210_mct.c | 1488 +++
3 files changed, 1508 insertions(+), 1 deletions(-)
create mode 100644 hw/exynos4210_mct.c
diff --git a/Makefile.target
On Wed, Feb 08, 2012 at 12:00:14PM +0100, Gerd Hoffmann wrote:
> This patch adds some infrastructure to handle suspend and resume to
> qemu. First there are two functions to switch state and second there
> is a suspend notifier:
>
> * qemu_system_suspend_request() is supposed to be called when t
On Wed, Feb 08, 2012 at 12:00:15PM +0100, Gerd Hoffmann wrote:
> This patch switches pc s3 suspend over to the new infrastructure.
> The cmos_s3 qemu_irq is killed, the new notifier is used instead.
> The xen hack goes away with that too, the hypercall can simply be
> done in a notifier function no
On Wed, Feb 08, 2012 at 12:00:16PM +0100, Gerd Hoffmann wrote:
> This patch adds the system_wakeup monitor command which will simply
> wake up suspended guests.
>
We can report this one as power button wakeup.
> Signed-off-by: Gerd Hoffmann
> ---
> hmp-commands.hx | 14 ++
> hmp.
On Wed, Feb 08, 2012 at 12:00:19PM +0100, Gerd Hoffmann wrote:
> Add a 'wakeup' property to the mc146818rtc. It is on by default.
> When enabled the rtc will wake up the guest when the alarm fires.
>
> Signed-off-by: Gerd Hoffmann
> ---
> hw/mc146818rtc.c |5 +
> 1 files changed, 5 inse
From: Maksim Kozlov
Add basic support of exynos4210 UART
Signed-off-by: Maksim Kozlov
Signed-off-by: Evgeny Voevodin
---
Makefile.target |2 +-
hw/exynos4210.c | 29 +++
hw/exynos4210.h |9 +
hw/exynos4210_uart.c | 676
Signed-off-by: Evgeny Voevodin
Reviewed-by: Peter Maydell
---
hw/lan9118.c | 124 +++---
1 files changed, 118 insertions(+), 6 deletions(-)
diff --git a/hw/lan9118.c b/hw/lan9118.c
index 78777c7..31fd2dd 100644
--- a/hw/lan9118.c
+++ b/hw/lan
SMDKC210 uses lan9215 chip, but lan9118 in 16-bit mode seems to
be enough.
Signed-off-by: Evgeny Voevodin
Reviewed-by: Peter Maydell
---
hw/exynos4_boards.c | 27 +--
1 files changed, 25 insertions(+), 2 deletions(-)
diff --git a/hw/exynos4_boards.c b/hw/exynos4_board
Am 08.02.2012 22:41, schrieb Stefan Weil:
> Makefile, Makefile.hw, Makefile.target and libcacard/Makefile
> added GLIB_CFLAGS to QEMU_CFLAGS.
>
> Makefile.objs does this, too, and is included by all other
> Makefiles, so GLIB_CFLAGS were added twice (reported by malc).
>
> Signed-off-by: Stefan W
Signed-off-by: Evgeny Voevodin
---
Makefile.target |1 +
hw/exynos4210.h | 82
hw/exynos4210_combiner.c | 469 ++
hw/exynos4210_gic.c | 458
4 files changed, 1010 inser
On Tue, Jan 31, 2012 at 12:43:16PM +0100, Ulrich Hecht wrote:
> If the host's page size is equal to or smaller than the target's, native
> execve() will fail appropriately with E2BIG if called with too big an
> environment for the target to handle. It may falsely succeed, however, if
> the host's p
On Mon, Feb 06, 2012 at 10:19:42PM +0100, Hervé Poussineau wrote:
>
> Signed-off-by: Hervé Poussineau
> ---
> hw/ide/pci.c |2 +-
> hw/ide/piix.c |4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
The types in this code are a mess but your patch doesn't make it worse.
It would
On Mon, Feb 06, 2012 at 02:02:55PM +0800, 陳韋任 wrote:
> Each target use #define marco (in target-xxx/cpu.h) to rename cpu_exec
> (cpu-exec.c) to cpu_xxx_exec, then defines its own cpu_loop which calls
> cpu_xxx_exec. So basically, cpu-exec.c is not i386 (only) emulator main
> execution loop. This
On Sat, Feb 04, 2012 at 03:51:43PM +0100, Jan Kiszka wrote:
> On 2012-02-04 13:32, Blue Swirl wrote:
> > On Sat, Feb 4, 2012 at 12:23, Jan Kiszka wrote:
> >> On 2012-02-04 13:12, Blue Swirl wrote:
> >>> On Fri, Feb 3, 2012 at 12:02, Jan Kiszka wrote:
> Helpful to understand guest configurati
On Tue, Feb 07, 2012 at 10:26:29PM +0100, Stefan Weil wrote:
> Fix a typo in a local function name.
>
> Signed-off-by: Stefan Weil
> ---
> hw/fmopl.c |4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/com
On 02/09/2012 09:58 AM, Paolo Bonzini wrote:
> On 02/08/2012 04:27 PM, Avi Kivity wrote:
>> +static void core_log_stop(MemoryListener *listener,
>> + MemoryRegionSection *section)
>> +{
>> +}
>> +
>> +static void core_log_sync(MemoryListener *listener,
>> +
On 02/09/2012 10:35 AM, Jan Kiszka wrote:
> Avi,
>
> Before I forget:
>
> On 2012-02-05 13:39, Jan Kiszka wrote:
> > +static void vapic_map_rom_writable(VAPICROMState *s)
> > +{
> > +target_phys_addr_t rom_paddr = s->rom_state_paddr & ROM_BLOCK_MASK;
> > +MemoryRegionSection section;
> > +
On Mon, Jan 23, 2012 at 03:42:38AM +0900, Benjamin MARSILI wrote:
>
> Signed-off-by: Benjamin MARSILI
> ---
> net/socket.c |8
> 1 files changed, 4 insertions(+), 4 deletions(-)
Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches
S
Signed-off-by: Evgeny Voevodin
---
MAINTAINERS |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 173e893..e55ea0f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -183,6 +183,14 @@ F: *win32*
ARM Machines
+Exynos
+M: Evg
On Tue, Feb 07, 2012 at 08:57:27PM +, Peter Maydell wrote:
> Fix a typo in a local variable name.
>
> Signed-off-by: Peter Maydell
> ---
> vl.c | 10 +-
> 1 files changed, 5 insertions(+), 5 deletions(-)
Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu
On Fri, Jan 13, 2012 at 08:29:49PM +, Peter Maydell wrote:
> Clarify that enum type names and function type names should follow
> the CamelCase style used for structured type names.
>
> Signed-off-by: Peter Maydell
> ---
> During a conversation on IRC with Anthony, I realised that the coding
On 02/09/2012 07:11 AM, Peter Maydell wrote:
The ARM devboard models (vexpress-a9, realview, versatilepb, etc)
were accidentally trying to set one of the arm_sysctl properties
after device init. This has now become a fatal error; set the property
before device init where it should be done instead
Add initial support of NURI and SMDKC210 boards
Signed-off-by: Evgeny Voevodin
---
Makefile.target |3 +-
hw/exynos4210.c | 193 +++
hw/exynos4210.h | 40 +++
hw/exynos4_boards.c | 153 +++
Signed-off-by: Evgeny Voevodin
---
Makefile.target |2 +-
hw/exynos4210.c | 12 ++
hw/exynos4210_pwm.c | 422 +++
3 files changed, 435 insertions(+), 1 deletions(-)
create mode 100644 hw/exynos4210_pwm.c
diff --git a/Makefile.target
Anthony Liguori writes:
> On 02/08/2012 09:04 AM, malc wrote:
>> On Wed, 8 Feb 2012, Andreas F?rber wrote:
>>
>>> malc,
>>>
>>> Arbitrarily reformatting your files is not okay. If you want a different
>>> formatting, you need to fix checkpatch.pl first to not error on that
>>> formatting in your
On Thu, Feb 09, 2012 at 09:30:09AM +0100, Jan Kiszka wrote:
> Looks like the subject would warmly welcome a "kvm:" tag.
>
> On 2012-02-09 05:22, David Gibson wrote:
[snip]
>
> What about naming the problem instead:
>
> /* Comparison with IOCTL macros on 32-bit hosts requires unsigned. */
Just o
On 2012-02-09 10:58, David Gibson wrote:
> On Thu, Feb 09, 2012 at 09:30:09AM +0100, Jan Kiszka wrote:
>> Looks like the subject would warmly welcome a "kvm:" tag.
>>
>> On 2012-02-09 05:22, David Gibson wrote:
> [snip]
>>
>> What about naming the problem instead:
>>
>> /* Comparison with IOCTL mac
On 2012-02-09 09:35, Stefan Hajnoczi wrote:
> On Sat, Feb 04, 2012 at 03:51:43PM +0100, Jan Kiszka wrote:
>> On 2012-02-04 13:32, Blue Swirl wrote:
>>> On Sat, Feb 4, 2012 at 12:23, Jan Kiszka wrote:
On 2012-02-04 13:12, Blue Swirl wrote:
> On Fri, Feb 3, 2012 at 12:02, Jan Kiszka wrote:
Hi folks,
There is a vgabios sitting implementation in the seabios repository for
a while. Recently it got a huge overhaul and should be on-par with the
lgpl vgabios implementation. I'd like to see qemu switch to that
implementation, for very simliar reasons which made us switch from bochs
bio
Signed-off-by: Gerd Hoffmann
---
pc-bios/vgabios-cirrus.bin | Bin 35840 -> 35328 bytes
roms/seabios |2 +-
2 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/pc-bios/vgabios-cirrus.bin b/pc-bios/vgabios-cirrus.bin
index
424dd0c70c4a52e1a98d8a64ebf4aa382b85d5cf..48
On 02/09/12 09:48, Gleb Natapov wrote:
> On Wed, Feb 08, 2012 at 12:00:14PM +0100, Gerd Hoffmann wrote:
>> * qemu_system_wakeup_request is supposed to be called on events which
>>should wake up the guest.
>>
> qemu_system_wakeup_request() should get wakeup source as a parameter.
> There are wa
Signed-off-by: Gerd Hoffmann
---
roms/Makefile |6 ++
roms/config.seabios |1 +
2 files changed, 7 insertions(+), 0 deletions(-)
create mode 100644 roms/config.seabios
diff --git a/roms/Makefile b/roms/Makefile
index 5c3d5f4..b24c728 100644
--- a/roms/Makefile
+++ b/roms/Makef
Hi,
>> Pretend that resume was caused by power button */
>> pm1a->sts |=
>> (ACPI_BITMASK_WAKE_STATUS |
>> ACPI_BITMASK_POWER_BUTTON_STATUS);
> Here we should report real reason for a wakeup (if it can be reported in
> mp1sts that is).
These are av
Signed-off-by: Gerd Hoffmann
---
roms/Makefile | 17 +
roms/config.vga.cirrus|3 +++
roms/config.vga.isavga|3 +++
roms/config.vga.qxl |6 ++
roms/config.vga.stdvga|3 +++
roms/config.vga.vmware|6 ++
roms/configure-s
On Wed, 8 Feb 2012, Gerd Hoffmann wrote:
> This patch switches pc s3 suspend over to the new infrastructure.
> The cmos_s3 qemu_irq is killed, the new notifier is used instead.
> The xen hack goes away with that too, the hypercall can simply be
> done in a notifier function now.
nice cleanup
On Thu, Feb 09, 2012 at 11:51:45AM +0100, Gerd Hoffmann wrote:
> Hi,
>
> >> Pretend that resume was caused by power button */
> >> pm1a->sts |=
> >> (ACPI_BITMASK_WAKE_STATUS |
> >> ACPI_BITMASK_POWER_BUTTON_STATUS);
> > Here we should report real r
On 02/08/2012 12:00 PM, Gerd Hoffmann wrote:
+/* set CMOS shutdown status register (index 0xF) as S3_resume(0xFE)
+ BIOS will read it and start S3 resume at POST Entry */
+static void rtc_notify_suspend(Notifier *notifier, void *data)
+{
+RTCState *s = container_of(notifier, RTCState, suspe
On Thu, Feb 09, 2012 at 11:45:21AM +0100, Gerd Hoffmann wrote:
> On 02/09/12 09:48, Gleb Natapov wrote:
> > On Wed, Feb 08, 2012 at 12:00:14PM +0100, Gerd Hoffmann wrote:
> >> * qemu_system_wakeup_request is supposed to be called on events which
> >>should wake up the guest.
> >>
> > qemu_syst
On Thu, Feb 09, 2012 at 08:39:28AM +1100, Benjamin Herrenschmidt wrote:
> On Wed, 2012-02-08 at 16:27 +0100, Joerg Roedel wrote:
> > Again, device grouping is done by the IOMMU drivers, so this all
> > belongs
> > into the generic iommu-code rather than the driver core.
>
> Except that there isn't
On Thu, Feb 9, 2012 at 10:08 AM, Jan Kiszka wrote:
> On 2012-02-09 09:35, Stefan Hajnoczi wrote:
>> On Sat, Feb 04, 2012 at 03:51:43PM +0100, Jan Kiszka wrote:
>>> On 2012-02-04 13:32, Blue Swirl wrote:
On Sat, Feb 4, 2012 at 12:23, Jan Kiszka wrote:
> On 2012-02-04 13:12, Blue Swirl wro
On 02/09/12 09:56, Gleb Natapov wrote:
> On Wed, Feb 08, 2012 at 12:00:19PM +0100, Gerd Hoffmann wrote:
>> Add a 'wakeup' property to the mc146818rtc. It is on by default.
>> When enabled the rtc will wake up the guest when the alarm fires.
>>
>> Signed-off-by: Gerd Hoffmann
>> ---
>> hw/mc14681
On 02/09/12 12:19, Gleb Natapov wrote:
> On Thu, Feb 09, 2012 at 11:45:21AM +0100, Gerd Hoffmann wrote:
>> On 02/09/12 09:48, Gleb Natapov wrote:
>>> On Wed, Feb 08, 2012 at 12:00:14PM +0100, Gerd Hoffmann wrote:
* qemu_system_wakeup_request is supposed to be called on events which
sh
> /* multicore boards that use the default secondary core boot functions
> * can ignore these two function calls. If the default functions won't
> * work, then write_secondary_boot() should write a suitable blob of
> * code mimicing the secondary CPU startup process used by the b
> >> What about naming the problem instead:
> >>
> >> /* Comparison with IOCTL macros on 32-bit hosts requires unsigned. */
> >
> > Just once, it would be nice to post something to this list and get a
> > substantive comment _before_ the bitching about minutiae.
> >
> > Oh, and it's not just 32-
> The ARM devboard models (vexpress-a9, realview, versatilepb, etc)
> were accidentally trying to set one of the arm_sysctl properties
> after device init. This has now become a fatal error; set the property
> before device init where it should be done instead.
>
> Signed-off-by: Peter Maydell
>
> the results of readelf -l hello are as follow:
> program headers:
> TYPE offsetvirtAddr phyAddr FileSiz
>MemSiz LOAD 0x0540x4000 0x4000 0x02bf0
> 0x02bf0
Clearly not a linux binary. Those have file offsets and vi
On Thu, Feb 09, 2012 at 01:00:47PM +0100, Gerd Hoffmann wrote:
> On 02/09/12 09:56, Gleb Natapov wrote:
> > On Wed, Feb 08, 2012 at 12:00:19PM +0100, Gerd Hoffmann wrote:
> >> Add a 'wakeup' property to the mc146818rtc. It is on by default.
> >> When enabled the rtc will wake up the guest when the
On Thu, Feb 09, 2012 at 12:17:33PM +0100, Paolo Bonzini wrote:
> On 02/08/2012 12:00 PM, Gerd Hoffmann wrote:
> >+/* set CMOS shutdown status register (index 0xF) as S3_resume(0xFE)
> >+ BIOS will read it and start S3 resume at POST Entry */
> >+static void rtc_notify_suspend(Notifier *notifier,
Am 09.02.2012 13:15, schrieb Paul Brook:
>> The ARM devboard models (vexpress-a9, realview, versatilepb, etc)
>> were accidentally trying to set one of the arm_sysctl properties
>> after device init. This has now become a fatal error; set the property
>> before device init where it should be done i
Am 09.02.2012 13:34, schrieb Andreas Färber:
> Am 09.02.2012 13:15, schrieb Paul Brook:
>>> The ARM devboard models (vexpress-a9, realview, versatilepb, etc)
>>> were accidentally trying to set one of the arm_sysctl properties
>>> after device init. This has now become a fatal error; set the proper
On Thu, Feb 09, 2012 at 01:02:47PM +0100, Gerd Hoffmann wrote:
> On 02/09/12 12:19, Gleb Natapov wrote:
> > On Thu, Feb 09, 2012 at 11:45:21AM +0100, Gerd Hoffmann wrote:
> >> On 02/09/12 09:48, Gleb Natapov wrote:
> >>> On Wed, Feb 08, 2012 at 12:00:14PM +0100, Gerd Hoffmann wrote:
> * qemu_
Hi,
>> Incremental patch (just infrastructure, no acpi windup yet) attached.
>> Something like this?
>>
> We need to give ACPI ability to prevent wakeup. So, for instance, if RTC
> alarm calls wakeup but ACPI detects that RTC_EN is cleared it can
> prevent it.
Yea, already figured that after re
On 02/09/2012 01:31 PM, Gleb Natapov wrote:
Real HW may have other ways to notify BIOS that system was S3 suspended
(special chipset register for instance). I think we can write DSDT magic
to write RTC for us, but I prefer to do it from inside QEMU.
Heh, I would prefer DSDT magic, but I wouldn
On Thu, Feb 09, 2012 at 01:47:30PM +0100, Paolo Bonzini wrote:
> On 02/09/2012 01:31 PM, Gleb Natapov wrote:
> >Real HW may have other ways to notify BIOS that system was S3 suspended
> >(special chipset register for instance). I think we can write DSDT magic
> >to write RTC for us, but I prefer t
Am 09.02.2012 07:59, schrieb Peter Maydell:
> Drop the legacy init function arm_sysctl_init(), since it has no
> users left any more. This allows us to drop the awkward '1' from
> the actual device init function.
>
> Signed-off-by: Peter Maydell
No other occurrences in the code,
Acked-by: Andre
On Thu, Feb 09, 2012 at 01:46:07PM +0100, Gerd Hoffmann wrote:
> Hi,
>
> >> Incremental patch (just infrastructure, no acpi windup yet) attached.
> >> Something like this?
> >>
> > We need to give ACPI ability to prevent wakeup. So, for instance, if RTC
> > alarm calls wakeup but ACPI detects th
> Am 09.02.2012 13:15, schrieb Paul Brook:
> >> The ARM devboard models (vexpress-a9, realview, versatilepb, etc)
> >> were accidentally trying to set one of the arm_sysctl properties
> >> after device init. This has now become a fatal error; set the property
> >> before device init where it should
All, I am reading this code.
static inline Int128 int128_add(Int128 a, Int128 b){ Int128 r = {
a.lo + b.lo, a.hi + b.hi }; r.hi += (r.lo < a.lo) || (r.lo < b.lo);
return r;}
This assumes the result r will not exceed 128 bits?--
Wizard
Am 08.02.2012 08:55, schrieb Peter A. G. Crosthwaite:
> From: "Peter A. G. Crosthwaite"
>
> Create a QOM device for bootstrapping linux on arm. Wraps the existing
> arm_boot code and calls arm_load_kernel() at device init. Allows booting
> of linux without -kernel -initrd -append arguments. The m
Hi,
>>> We need to give ACPI ability to prevent wakeup. So, for instance, if RTC
>>> alarm calls wakeup but ACPI detects that RTC_EN is cleared it can
>>> prevent it.
>>
>> Yea, already figured that after reading your rtc reply ...
>>
>> One more incremental attached for review.
>>
> We can star
Signed-off-by: Luiz Capitulino
---
hw/virtio.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/hw/virtio.c b/hw/virtio.c
index 74cc038..064aecf 100644
--- a/hw/virtio.c
+++ b/hw/virtio.c
@@ -845,8 +845,7 @@ int virtio_load(VirtIODevice *vdev, QEMUFile *f)
void virtio_
Paul Brook writes:
>> > starting your own toy kernel is a fun thing to do and there are many
>> > tutorials out there on how to do it. Unfortunately when one wants to
>> > write a kernel in 64bit it becomes much harder because one can't
>> > compile 64bit code as elf32 image and converting a elf6
> Paul Brook writes:
> >> > starting your own toy kernel is a fun thing to do and there are many
> >> > tutorials out there on how to do it. Unfortunately when one wants to
> >> > write a kernel in 64bit it becomes much harder because one can't
> >> > compile 64bit code as elf32 image and converti
On 02/09/2012 03:48 AM, Markus Armbruster wrote:
Anthony Liguori writes:
On 02/08/2012 09:04 AM, malc wrote:
On Wed, 8 Feb 2012, Andreas F?rber wrote:
malc,
Arbitrarily reformatting your files is not okay. If you want a different
formatting, you need to fix checkpatch.pl first to not error
On 09.02.2012, at 08:43, Thomas Huth wrote:
> Am Thu, 9 Feb 2012 11:26:09 +1100
> schrieb David Gibson :
>
>> On Wed, Feb 08, 2012 at 02:27:35PM +0100, Thomas Huth wrote:
>>> Am Wed, 8 Feb 2012 21:48:40 +1100
>>> schrieb David Gibson :
>>>
On Wed, Feb 08, 2012 at 10:54:21AM +0400, malc wro
On 2012-02-09 13:07, Paul Brook wrote:
What about naming the problem instead:
/* Comparison with IOCTL macros on 32-bit hosts requires unsigned. */
>>>
>>> Just once, it would be nice to post something to this list and get a
>>> substantive comment _before_ the bitching about minutia
On 2012-02-09 10:32, Avi Kivity wrote:
> On 02/09/2012 10:35 AM, Jan Kiszka wrote:
>> Avi,
>>
>> Before I forget:
>>
>> On 2012-02-05 13:39, Jan Kiszka wrote:
>>> +static void vapic_map_rom_writable(VAPICROMState *s)
>>> +{
>>> +target_phys_addr_t rom_paddr = s->rom_state_paddr & ROM_BLOCK_MASK
Am 30.01.2012 23:29, schrieb Alexander Graf:
> When using Windows 8 with an AHCI disk drive, it issues a blue screen.
> The reason is that WIN_SECURITY_FREEZE_LOCK / CFA_WEAR_LEVEL is not
> supported by our ATA implementation, but Windows expects it to be there.
Is there anything that makes Window
This patch substitutes more qdev code with shared infrastructure. The
code is now ripe enough that we can attack command-line parsing.
Parsing to a string is replaced with a StringInputVisitor (there is also
a StringOutputVisitor, but it's not used yet). This lets us remove most
integer parsing/
Most visitors will use the same code for enum parsing. Move it to
the core.
Signed-off-by: Paolo Bonzini
---
qapi/qapi-visit-core.c| 51 +
qapi/qapi-visit-impl.h| 23
qapi/qmp-input-visitor.c | 34 +-
Visitors allow a limited form of polymorphism. Exploit it to support
setting the non-legacy PCI address property both as a DD.F string
and as an 8-bit integer.
The 8-bit integer form is just too clumsy, it is unlikely that we will
ever drop it.
Signed-off-by: Paolo Bonzini
---
hw/qdev-properti
Am 30.01.2012 23:29, schrieb Alexander Graf:
> When masking IRQ lines, we should actually mask them out and not declare
> them active anymore. Once we mask them in again, they are allowed to trigger
> again.
>
> Signed-off-by: Alexander Graf
> ---
> hw/ide/ahci.c |2 ++
> 1 files changed, 2
Am 30.01.2012 23:29, schrieb Alexander Graf:
> bdrv_aio_cancel() can trigger bdrv_aio_flush() which makes all aio
> that is currently in flight finish. So what we do is:
>
> port reset
> detect ncq in flight
> cancel ncq
> delete ncq sg list
>
> at which point we have double freed the sg
Am 30.01.2012 18:10, schrieb MORITA Kazutaka:
> The co_recv coroutine has two things that will try to enter it:
>
> 1. The select(2) read callback on the sheepdog socket.
> 2. The aio_add_request() blocking operations, including a coroutine
> mutex.
>
> This patch fixes it by setting NUL
On 09.02.2012, at 15:28, Kevin Wolf wrote:
> Am 30.01.2012 23:29, schrieb Alexander Graf:
>> When using Windows 8 with an AHCI disk drive, it issues a blue screen.
>> The reason is that WIN_SECURITY_FREEZE_LOCK / CFA_WEAR_LEVEL is not
>> supported by our ATA implementation, but Windows expects it
Signed-off-by: Paolo Bonzini
---
.gitignore |2 +
test-string-input-visitor.c | 160 +++
test-string-output-visitor.c | 188 ++
tests/Makefile | 12 ++-
4 files changed, 360 insertions
On 09.02.2012, at 15:41, Kevin Wolf wrote:
> Am 30.01.2012 23:29, schrieb Alexander Graf:
>> When masking IRQ lines, we should actually mask them out and not declare
>> them active anymore. Once we mask them in again, they are allowed to trigger
>> again.
>>
>> Signed-off-by: Alexander Graf
>>
Am 09.02.2012 15:49, schrieb Alexander Graf:
>
> On 09.02.2012, at 15:28, Kevin Wolf wrote:
>
>> Am 30.01.2012 23:29, schrieb Alexander Graf:
>>> When using Windows 8 with an AHCI disk drive, it issues a blue screen.
>>> The reason is that WIN_SECURITY_FREEZE_LOCK / CFA_WEAR_LEVEL is not
>>> supp
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Signed-off-by: Andreas Färber
Cc: Blue Swirl
Cc: malc
---
scripts/checkpatch.pl |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/scripts/checkpatch.pl b/scripts/chec
Add generic property accessors that take a string and parse it
appropriately for the property type. All the magic here is done
by the new string-based visitors.
Signed-off-by: Paolo Bonzini
---
include/qemu/object.h | 24
qom/object.c | 24 +
Luiz Capitulino writes:
> I've tried to implement a BLOCK_MEDIUM_EJECT event that, as we discussed[1],
> would be emitted by guest-initiated ejects and by the QMP/HMP eject and change
> commands.
>
> However, that turned to be a bit problematic, because the eject and change
> commands don't exact
On 09.02.2012, at 15:59, Kevin Wolf wrote:
> Am 09.02.2012 15:49, schrieb Alexander Graf:
>>
>> On 09.02.2012, at 15:28, Kevin Wolf wrote:
>>
>>> Am 30.01.2012 23:29, schrieb Alexander Graf:
When using Windows 8 with an AHCI disk drive, it issues a blue screen.
The reason is that WIN_
Am 09.02.2012 15:59, schrieb Andreas Färber:
> Disable warnings for spaces before opening parenthesis in
> hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
>
> Signed-off-by: Andreas Färber
> Cc: Blue Swirl
> Cc: malc
> ---
For testing I used the following pseudo-patch:
--8<--
--- a/hw/sb16.c
+++
Hex properties are an obstacle to removal of old qdev string parsing, but
even here we can lay down the foundations for future simplification. In
general, they are rarely used and their printed form is more interesting
than the parsing. For example you'd usually set isa-serial.index
instead of is
On 02/09/2012 04:23 PM, Jan Kiszka wrote:
> On 2012-02-09 10:32, Avi Kivity wrote:
> > On 02/09/2012 10:35 AM, Jan Kiszka wrote:
> >> Avi,
> >>
> >> Before I forget:
> >>
> >> On 2012-02-05 13:39, Jan Kiszka wrote:
> >>> +static void vapic_map_rom_writable(VAPICROMState *s)
> >>> +{
> >>> +targ
More qdev printers could have been removed in the previous series, and
object_property_parse also made several parsers unnecessary. In fact,
the new code is even more robust with respect to overflows, so clean
them up!
Signed-off-by: Paolo Bonzini
---
hw/qdev-properties.c | 134 ---
object_property_parse lets us drop the legacy setters when their task
is done just as well by the string visitors.
Signed-off-by: Paolo Bonzini
---
hw/qdev-properties.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/qdev-properties.c b/hw/qdev-properties.c
index d
String based visitors provide a consistent interface for parsing
strings to C values, as well as consuming C values as strings.
They will be used to parse command-line options.
Signed-off-by: Paolo Bonzini
---
Makefile.objs|5 +-
qapi/string-input-visitor.c | 136 ++
Kevin Wolf writes:
> Am 07.02.2012 15:09, schrieb Markus Armbruster:
>> Stash away the option argument with add_device_config(), so we still
>> have its location when we get around to parsing it.
>>
>> This doesn't improve any messages I can see just yet, but that'll
>> change shortly.
>>
>> Si
This method is optional, do not implement it if it is empty.
Signed-off-by: Paolo Bonzini
---
qapi/qmp-input-visitor.c |5 -
1 files changed, 0 insertions(+), 5 deletions(-)
diff --git a/qapi/qmp-input-visitor.c b/qapi/qmp-input-visitor.c
index 012cd0a..e6b6152 100644
--- a/qapi/qmp-inp
Kevin Wolf writes:
> Am 07.02.2012 15:09, schrieb Markus Armbruster:
>> This part takes care of backends "file", "pipe", "pty" and "stdio".
>> Unlike many other backends, these leave open error reporting to their
>> caller. Because the caller doesn't know what went wrong, this results
>> in a pr
On 02/05/2012 02:39 PM, Jan Kiszka wrote:
> From: Jan Kiszka
>
> This enables acceleration for MMIO-based TPR registers accesses of
> 32-bit Windows guest systems. It is mostly useful with KVM enabled,
> either on older Intel CPUs (without flexpriority feature, can also be
> manually disabled for
On Thu, Feb 09, 2012 at 11:29:42AM -0200, Luiz Capitulino wrote:
>
> Signed-off-by: Luiz Capitulino
> ---
> hw/virtio.c |3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
Tweaked commit message to be more conventional with "virtio: Remove
unneeded g_free() check in virtio_cleanup()"
1 - 100 of 215 matches
Mail list logo