On Thu, May 28, 2015 at 03:37:03PM +1000, Alistair Francis wrote:
> Move the Microblaze PVR registers to the end of the CPUMBState
> and preserve them during reset. This is similar to what the
> QEMU ARM model does with some of it's registers.
>
> This allows the Microblaze PVR registers to only b
On 05/28/2015 02:28 PM, Michal Privoznik wrote:
> On 28.05.2015 05:36, Jason Wang wrote:
>>
>> On 05/28/2015 11:21 AM, Jason Wang wrote:
>>> On 05/27/2015 07:57 PM, Michael S. Tsirkin wrote:
On Wed, May 27, 2015 at 04:45:34PM +0800, Jason Wang wrote:
>> On 05/27/2015 02:26 PM, Pankaj Gup
On Thu, May 28, 2015 at 03:36:25PM +1000, Alistair Francis wrote:
> Fix up the incorrect indentation level in the helper_stackprot() function.
>
> Signed-off-by: Alistair Francis
> Reviewed-by: Peter Crosthwaite
Reviewed-by: Edgar E. Iglesias
> ---
>
> target-microblaze/op_helper.c | 10
Hi!
> IIUC, your implementation also uses 8 cpus per cluster which is not
> consistent with CPU spec.
Yes, i have inherited the value from Shlomo's implementation. To tell the
truth i don't
have time to read through all the docs. And i don't know in which PDF i should
look for
this info. But,
On Thu, May 28, 2015 at 03:38:59PM +1000, Alistair Francis wrote:
> Originally the use-fpu PVR bits were manually set for each machine. This
> is a hassle and difficult to read, instead set them based on the CPU
> properties.
>
> Signed-off-by: Alistair Francis
> Reviewed-by: Peter Crosthwaite
>
On 28.05.2015 05:36, Jason Wang wrote:
>
>
> On 05/28/2015 11:21 AM, Jason Wang wrote:
>>
>> On 05/27/2015 07:57 PM, Michael S. Tsirkin wrote:
>>> On Wed, May 27, 2015 at 04:45:34PM +0800, Jason Wang wrote:
>
> On 05/27/2015 02:26 PM, Pankaj Gupta wrote:
>>> Ping.
>>>
>>> Can
Hi,
> For EVDEV_KPCOMMA it maps to XT KBD code 126 (0x7e in decimal) which
> does not match your patch. I'm not sure where I got that mapping
> from originally - probably from the Linux kernel's XT <-> evdev
> keymapping tables.
Did some more testing, 0x7e seems to be the correct one. Everyth
On Thu, May 28, 2015 at 03:37:42PM +1000, Alistair Francis wrote:
> Microblaze stack protection is configurable and isn't always enabled.
> This patch allows the stack protection to be disabled/enabled from the
> CPU properties.
>
> The stack protection is disabled by default as by default the Mic
On Tue, May 19, 2015 at 07:33:34PM +0100, Peter Maydell wrote:
> From: Greg Bellows
>
> Add support for trapping WFI and WFE instructions to the proper EL when
> SCTLR/SCR/HCR settings apply.
>
> Signed-off-by: Greg Bellows
> [PMM: removed unnecessary tweaking of syn_wfx() prototype;
> use rai
On Tue, May 19, 2015 at 07:33:33PM +0100, Peter Maydell wrote:
> Just NOP the WFI instruction if we have work to do.
> This doesn't make much difference currently (though it does avoid
> jumping out to the top level loop and immediately restarting),
> but the distinction between "halt" and "don't h
On Tue, May 19, 2015 at 07:33:32PM +0100, Peter Maydell wrote:
> Deleting the now-unused ARM_TBFLAG_CPACR_FPEN left a gap in the
> bit usage; move the following ARM_TBFLAG_XSCALE_CPAR and
> ARM_TBFLAG_NS_SHIFT down 3 bits to fill the gap.
>
> Signed-off-by: Peter Maydell
Reviewed-by: Edgar E. Ig
On Tue, May 19, 2015 at 07:33:30PM +0100, Peter Maydell wrote:
> Currently we keep the TB flags PSTATE_SS and SS_ACTIVE in different
> bit positions for AArch64 and AArch32. Replace these separate
> definitions with a single common flag in the upper part of the
> flags word.
>
> Signed-off-by: Pet
On Tue, May 19, 2015 at 07:33:26PM +0100, Peter Maydell wrote:
> Rather than making every caller of raise_exception set the
> syndrome and target EL by hand, make these arguments to
> raise_exception() and have that do the job.
>
> Signed-off-by: Peter Maydell
Reviewed-by: Edgar E. Iglesias
>
On Tue, May 19, 2015 at 07:33:25PM +0100, Peter Maydell wrote:
> Set the exception target EL for MMU faults in tlb_fill.
>
> Signed-off-by: Peter Maydell
Reviewed-by: Edgar E. Iglesias
> ---
> target-arm/op_helper.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/target-arm/op_help
On Wed, May 27, 2015 at 05:27:36PM +1000, Edgar E. Iglesias wrote:
> From: "Edgar E. Iglesias"
Hi,
I just realized I forgot to do the update to use CP_ACCESS_TRAP_ELX
instead of setting exception.target_el...
Will fix for v3...
Cheers,
Edgar
>
> Signed-off-by: Edgar E. Iglesias
> ---
> t
Originally the use-fpu PVR bits were manually set for each machine. This
is a hassle and difficult to read, instead set them based on the CPU
properties.
Signed-off-by: Alistair Francis
Reviewed-by: Peter Crosthwaite
---
V2:
- Remove unnecessary declaration of r
Changes since RFC:
- Tidy up th
Rename the "xlnx.base-vectors" string to "base-vectors" and
move the base_vectors variable into the cfg struct.
Signed-off-by: Alistair Francis
Reviewed-by: Peter Crosthwaite
---
target-microblaze/cpu-qom.h |3 ++-
target-microblaze/cpu.c |4 ++--
target-microblaze/helper.c |8
Microblaze stack protection is configurable and isn't always enabled.
This patch allows the stack protection to be disabled/enabled from the
CPU properties.
The stack protection is disabled by default as by default the Microblaze
machines enable the MMU and stack protection can't be enabled if the
Move the Microblaze PVR registers to the end of the CPUMBState
and preserve them during reset. This is similar to what the
QEMU ARM model does with some of it's registers.
This allows the Microblaze PVR registers to only be set once
at realise instead of constantly at reset.
Signed-off-by: Alista
Fix up the incorrect indentation level in the helper_stackprot() function.
Signed-off-by: Alistair Francis
Reviewed-by: Peter Crosthwaite
---
target-microblaze/op_helper.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/target-microblaze/op_helper.c b/target-m
On Tue, May 19, 2015 at 07:33:24PM +0100, Peter Maydell wrote:
> Move the code which sets exception information out of
> arm_cpu_handle_mmu_fault and into tlb_fill. tlb_fill
> is the only caller which wants to raise_exception()
> so it makes more sense for it to handle the whole of
> the exception
Firstly this patch series tidies up some code and removes
a "xlnx." prefix.
Then it moves the Microblaze PVR registers to the end
of the CPUMBState to preserve them during reset. This
allows most of the operations on them to be moved from
the reset to the realise. Except for the machine specific
o
Signed-off-by: Fam Zheng
Reviewed-by: John Snow
Reviewed-by: Paolo Bonzini
---
tests/qemu-iotests/041| 66 ++-
tests/qemu-iotests/iotests.py | 28 ++
2 files changed, 43 insertions(+), 51 deletions(-)
diff --git a/tests/qemu-iotes
On Tue, May 19, 2015 at 07:33:23PM +0100, Peter Maydell wrote:
> If the SCTLR.UMA trap bit is set then attempts by EL0 to update
> the PSTATE DAIF bits via "MSR DAIFSet, imm" and "MSR DAIFClr, imm"
> instructions will raise an exception. We were failing to set
> the syndrome information for this ex
Only poll the specific type of event we are interested in, to avoid
stealing events that should be consumed by someone else.
Suggested-by: John Snow
Signed-off-by: Fam Zheng
Reviewed-by: John Snow
---
tests/qemu-iotests/iotests.py | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
Unsetting dirty globally with discard is not very correct. The discard may zero
out sectors (depending on can_write_zeroes_with_unmap), we should replicate
this change to destination side to make sure that the guest sees the same data.
Calling bdrv_reset_dirty also troubles mirror job because the
This checks that the discard on mirror source that effectively zeroes
data is also reflected by the data of target.
Signed-off-by: Fam Zheng
Reviewed-by: John Snow
---
tests/qemu-iotests/132 | 59 ++
tests/qemu-iotests/132.out | 5
tests/qem
If specified as "true", it allows discarding on target sectors where source is
not allocated.
Signed-off-by: Fam Zheng
---
block/mirror.c| 7 +--
blockdev.c| 5 +
hmp.c | 2 +-
include/block/block_int.h | 2 ++
qapi/block-core.json | 8
If guest discards a source cluster, mirroring with bdrv_aio_readv is overkill.
Some protocols do zero upon discard, where it's best to use
bdrv_aio_write_zeroes, otherwise, bdrv_aio_discard will be enough.
Signed-off-by: Fam Zheng
---
block/mirror.c | 20 ++--
1 file changed, 18
Using this function would always be wrong because a dirty bitmap must
have a specific owner that consumes the dirty bits and calls
bdrv_reset_dirty_bitmap().
Remove the unused function to avoid future misuse.
Reviewed-by: Eric Blake
Signed-off-by: Fam Zheng
Reviewed-by: John Snow
---
block.c
Like bdrv_is_allocated_above, this function follows the backing chain until
seeing
BDRV_BLOCK_ALLOCATED. Base is not included.
Reimplement bdrv_is_allocated on top.
Signed-off-by: Fam Zheng
---
block/io.c| 56 +--
include/block/block
v6: Fix pnum in bdrv_get_block_status_above. [Paolo]
v5: Rewrite patch 1.
Address Eric's comments on patch 3.
Add Eric's rev-by to patches 2 & 4.
Check BDRV_BLOCK_DATA in patch 3. [Paolo]
This fixes the mirror assert failure reported by wangxiaolong:
https://lists.gnu.org/archive/htm
Ping...
On 05/21/2015 12:52 PM, Wen Congyang wrote:
> Block replication is a very important feature which is used for
> continuous checkpoints(for example: COLO).
>
> Usage:
> Please refer to docs/block-replication.txt
>
> You can get the patch here:
> https://github.com/wencongyang/qemu-colo/co
Hi Kevin:
On 05/26/2015 07:55 PM, Kevin Wolf wrote:
Am 25.05.2015 um 05:30 hat Bo Tu geschrieben:
Bo Tu (3):
qemu-iotests: s390x: fix test 049
qemu-iotests: s390x: fix test 051
qemu-iotests: s390x: fix test 130
Xiao Guang Chen (4):
qemu-iotests: qemu machine type support
qemu-io
On 05/28/2015 11:21 AM, Jason Wang wrote:
>
> On 05/27/2015 07:57 PM, Michael S. Tsirkin wrote:
>> On Wed, May 27, 2015 at 04:45:34PM +0800, Jason Wang wrote:
On 05/27/2015 02:26 PM, Pankaj Gupta wrote:
>> Ping.
>>
>> Can I get any suggestions on this patch.
>>
>> Be
On Fri, 05/22 16:17, John Snow wrote:
> Add a few external dependency groups to the iotests groups list,
> such as "qemu", "nbd", and "scm".
>
> This will assist us in knowing which tests need to be re-run
> when those dependencies are updated, or by helping us to avoid
> tests that we know are br
On Fri, 05/22 16:17, John Snow wrote:
> Like a makefile, try to skip tests if we know they have already been
> executed using the current set of external dependencies.
>
> If a user passes the -ts option to ./check, if a test or its output
> or its dependencies (qemu, qemu-nbd, qemu-io, qemu-img,
On Fri, 05/22 16:17, John Snow wrote:
> Split the help text to highlight the groups of options
> a little better, carving out a clear "format" and
> "protocols" section.
>
> Signed-off-by: John Snow
Reviewed-by: Fam Zheng
> ---
> tests/qemu-iotests/common | 6 +-
> 1 file changed, 5 inser
On 05/27/2015 07:57 PM, Michael S. Tsirkin wrote:
> On Wed, May 27, 2015 at 04:45:34PM +0800, Jason Wang wrote:
>> >
>> >
>> > On 05/27/2015 02:26 PM, Pankaj Gupta wrote:
>>> > > Ping.
>>> > >
>>> > > Can I get any suggestions on this patch.
>>> > >
>>> > > Best regards,
>>> > > Pankaj
>>> > >
On Wed, 05/27 12:43, Paolo Bonzini wrote:
>
>
> On 27/05/2015 12:10, Kevin Wolf wrote:
> > Am 27.05.2015 um 11:50 hat Paolo Bonzini geschrieben:
> >>
> >>
> >> On 27/05/2015 11:07, Kevin Wolf wrote:
> >>> This is the first part of what's troubling me with this series, as it
> >>> makes me doubtfu
On Tue, 05/26 18:54, Paolo Bonzini wrote:
> From: Stefan Hajnoczi
>
> Use atomic_or() for atomic bitmaps where several threads may set bits at
> the same time. This avoids the race condition between threads loading
> an element, bitwise ORing, and then storing the element.
>
> When setting all
This patch has been reviewed. Could it be merged?
Thanks,
Zhu
On 05/16/2015 12:18 AM, Igor Mammedov wrote:
On Wed, 13 May 2015 17:21:36 +0800
Zhu Guihua wrote:
Add a new API named acpi_send_gpe_event() to send hotplug SCI.
This API can be used by pci, cpu and memory hotplug.
This patch is r
> Am 27.05.2015 um 17:57 schrieb Aurelien Jarno :
>
>> On 2015-05-27 07:31, Richard Henderson wrote:
>>> On 05/26/2015 03:03 PM, Aurelien Jarno wrote:
>>> Ok, I understand now. That said I don't see how implementing STFLE will
>>> break that. I think it will actually improve things by enabling m
On Wed, 05/27 11:38, Paolo Bonzini wrote:
>
>
> On 27/05/2015 09:19, Fam Zheng wrote:
> > This series looks at the other side of the broken "qmp transaction" problem
> > with dataplane [1] - the event loop.
> >
> > Before, an ioeventfd of a non-dataplane device is registered via
> > qemu_set_fd_
On 2015/05/26 21:52, Eric Blake wrote:
> On 05/25/2015 10:29 PM, Tetsuya Mukawa wrote:
>
> { 'struct': 'NetdevTapOptions',
> @@ -2259,7 +2261,8 @@
> '*vhostfd':'str',
> '*vhostfds': 'str',
> '*vhostforce': 'bool',
> -'*queues': 'uint32'} }
>
Based on patch by Nikolay Nikolaev:
Vhost-user will implement the multi queue support in a similar way
to what vhost already has - a separate thread for each queue.
To enable the multi queue functionality - a new command line parameter
"queues" is introduced for the vhost-user netdev.
Signed-off-b
On Mon, May 25, 2015 at 2:34 PM, Peter Crosthwaite
wrote:
> On Mon, May 18, 2015 at 4:14 PM, Alistair Francis
> wrote:
>> Originally the use-fpu PVR bits were manually set for each machine. This
>> is a hassle and difficult to read, instead set them based on the CPU
>> properties.
>>
>> Signed-of
On 05/27/2015 08:31 PM, Kevin Wolf wrote:
> Am 21.05.2015 um 07:47 hat Wen Congyang geschrieben:
>> On 05/09/2015 01:21 AM, Kevin Wolf wrote:
>>> For bs->file, using references to existing BDSes has been possible for a
>>> while already. This patch enables the same for bs->backing_hd.
>>
>> 1. We r
On 2015/5/28 2:03, Pavel Fedin wrote:
> Hello!
>
>> I think encoding should be CPU type specific i.e. not defined by what
>> GIC can support and once we add CPU type with 8 cores, it would provide
>> it's own version of mpidr_read since it would be defined by spec
>> how to encode aff0.
>
> I
On Mon, May 25, 2015 at 1:53 PM, Peter Crosthwaite
wrote:
> On Mon, May 18, 2015 at 4:13 PM, Alistair Francis
> wrote:
>> Move the Microblaze PVR registers to the end of the CPUMBState
>> and preserve them during reset. This is similar to what the
>> QEMU ARM model does with some of it's register
On Mon, May 25, 2015 at 1:44 PM, Peter Crosthwaite
wrote:
> On Mon, May 18, 2015 at 4:12 PM, Alistair Francis
> wrote:
>> Fix up the incorrect indentation level in the helper_stackprot() function.
>>
>> Signed-off-by: Alistair Francis
>
> Reviewed-by: Peter Crosthwaite
Thanks
Alistair
>
>> -
Before:
(qemu) drive_add usb_flash_drive
drive_add: string expected
After:
(qemu) drive_add usb_flash_drive
drive_add: string expected
Try "help drive_add" for more information
Reviewed-by: Markus Armbruster
Signed-off-by: Bandan Das
---
monitor.c | 2 ++
1 file changed, 2 insertions(+)
diff -
Hi Markus,
It seems we can directly use cmdline to track current location
when parsing. It's not being reused anywhere else and my unit
tests didn't show any problems. Am I missing something ?
Once everything looks ok, I can rebase this on top of your
patches. Thanks for taking the time to review
There's too much going on in monitor_parse_command().
Split up the arguments parsing bits into a separate function
monitor_parse_arguments(). Let the original function check for
command validity and sub-commands if any and return data (*cmd)
that the newly introduced function can process and return
On Wed, 2015-05-27 at 10:46 +0800, Chen Fan wrote:
> when the vfio device encounters an uncorrectable error in host,
> the vfio_pci driver will signal the eventfd registered by this
> vfio device, the results in the qemu eventfd handler getting
> invoked.
>
> this patch is to pass the error to gue
On Wed, 2015-05-27 at 10:46 +0800, Chen Fan wrote:
> when do virtual secondary bus reset, the vfio device under
> this bus need to do host bus reset to reset the device.
> so add this case.
>
> Signed-off-by: Chen Fan
> ---
> hw/vfio/pci.c | 75
>
On Wed, 2015-05-27 at 10:46 +0800, Chen Fan wrote:
> Signed-off-by: Chen Fan
> ---
> hw/vfio/pci.c | 32
> 1 file changed, 32 insertions(+)
>
> diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c
> index 5934fd7..91ad9ad 100644
> --- a/hw/vfio/pci.c
> +++ b/hw/vfio/pci.c
On Wed, 2015-05-27 at 10:46 +0800, Chen Fan wrote:
> Signed-off-by: Chen Fan
> ---
> hw/vfio/pci.c | 17 +
> 1 file changed, 17 insertions(+)
>
> diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c
> index 91ad9ad..a8c5988 100644
> --- a/hw/vfio/pci.c
> +++ b/hw/vfio/pci.c
> @@ -156,6 +15
On Wed, 2015-05-27 at 10:46 +0800, Chen Fan wrote:
> we introduce a has_bus_reset capability to sign the vfio
> devices if support host bus reset.
>
> Signed-off-by: Chen Fan
> ---
> hw/vfio/pci.c | 123
> ++
> 1 file changed, 123 insertio
On Wed, 2015-05-27 at 10:46 +0800, Chen Fan wrote:
> Calling pcie_aer_init to initilize aer related registers for
> vfio device, then reload physical related registers to expose
> device capability.
>
> Signed-off-by: Chen Fan
> ---
> hw/vfio/pci.c | 83
> +++
On Wed, 2015-05-27 at 10:46 +0800, Chen Fan wrote:
> For vfio pcie device, we could expose the extended capability on
> PCIE bus. in order to avoid config space broken, we introduce
> a copy config for parsing extended caps. and rebuild the pcie
> extended config space.
>
> Signed-off-by: Chen Fan
On Wed, 2015-05-27 at 10:46 +0800, Chen Fan wrote:
> the function is used to get affected devices by bus reset.
> so here extract it, and can used for aer soon.
>
> Signed-off-by: Chen Fan
> ---
> hw/vfio/pci.c | 67
> +++
> 1 file changed
It is Very annoying to carry forward an outdatEd coNtroller with a mOdern
Machine type.
Hence, let us not instantiate the FDC when all of the following apply:
- the machine type is pc-q35-2.4 or later,
- "-device isa-fdc" is not passed on the command line (nor in the config
file),
- no "-drive i
On Wed, 27 May 2015 13:58:57 +0200
Paolo Bonzini wrote:
> the main issue here is lack of migration support. You need to add a
> new subsection to vmstate_ich9_pm that is migrated if the registers
> are different from the default values.
Hrm - OK. I didn't even take that into account. I'll do it
On Wed, 27 May 2015 14:03:59 +0200
Paolo Bonzini wrote:
>
>
> On 27/05/2015 02:29, Paulo Alcantara wrote:
> > Signed-off-by: Paulo Alcantara
> > ---
> > hw/i386/acpi-dsdt-pdrc.dsl| 46
> > ++
>
> Why pdrc and not e.g. ccr (chipset configuration reg
On Wed, 27 May 2015 17:16:52 +0100
Stefan Hajnoczi wrote:
> Drop the union and move the hubport creation into the !is_netdev case.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> net/net.c | 38 --
> 1 file changed, 16 insertions(+), 22 deletions(-)
>
> diff --g
On Wed, 27 May 2015 17:16:51 +0100
Stefan Hajnoczi wrote:
> Both is_netdev and !is_netdev paths already check that
> net_client_init_func[opts->kind] is non-NULL so there is no need for the
> if statement.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> net/net.c | 31 ++
On Wed, 27 May 2015 17:16:50 +0100
Stefan Hajnoczi wrote:
> When a -net type is used that was not compiled into the binary there
> should be an error message.
>
> Note the special case for -net none, which is a no-op.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> net/net.c | 11 +++
> 1
On 27-05-2015 11:32, Gerd Hoffmann wrote:
On Mi, 2015-05-27 at 06:27 -0300, Joao Luis Meloni Assirati wrote:
On 26-05-2015 10:58, Gerd Hoffmann wrote:
This patch adds the two extra brazilian keys to the evdev keymap for
X11. This patch gets the two keys going with the gtk and sdl1 UIs.
With v
From: "Daniel P. Berrange"
It is reasonably common to want to create an object, set a
number of properties, register it in the hierarchy and then
mark it as complete (if a user creatable type). This requires
quite a lot of error prone, verbose, boilerplate code to achieve.
First a pair of functi
From: "Daniel P. Berrange"
The QEMU help for -object is essentially useless, just giving users
the generic syntax. Move it down into its own section and introduce
a nested table where each user creatable object can be documented.
The existing memory-backend-file, rng-random and rng-egd object
typ
From: "Daniel P. Berrange"
A QOM property can be parsed as enum using the visit_type_enum()
helper function, but this forces callers to use the more complex
generic object_property_add() method when registering it. It
also requires that users of that object have access to the
string map when they
From: Paolo Bonzini
This helper adds a read-only link<> property.
Suggested-by: Eduardo Habkost
Signed-off-by: Paolo Bonzini
Reviewed-by: Eduardo Habkost
Signed-off-by: Andreas Färber
---
include/qom/object.h | 18 ++
qom/object.c | 16
2 files chang
From: "Daniel P. Berrange"
The enum string table parameters in various QOM/QAPI methods
are declared 'const char *strings[]'. This results in const
warnings if passed a variable that was declared as
static const char * const strings[] = { };
Add the extra const annotation to the paramet
From: "Daniel P. Berrange"
Add object_get_objects_root() method which is a convience for
obtaining the Object * located at /objects in the object
composition tree. Convert existing code over to use the new
API where appropriate.
Signed-off-by: Daniel P. Berrange
Signed-off-by: Andreas Färber
-
Replace uses of g_test_add_data_func() for QTest test cases.
It is still valid to use it for any non-QTest test cases,
which are not run for multiple target binaries.
Suggested-by: John Snow
Reviewed-by: John Snow
Signed-off-by: Andreas Färber
---
tests/ahci-test.c | 9 ++---
tests
From: "Daniel P. Berrange"
Some types of object must be created before chardevs, other types of
object must be created after chardevs. As such there is no option but
to create objects in two phases.
This takes the decision to create as many object types as possible
right away before anyother bac
From: Eduardo Habkost
Now that object_property_add_alias() strdup()s target_name, we can free
the property names in qdev_pass_gpios().
Signed-off-by: Eduardo Habkost
Reviewed-by: Paolo Bonzini
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Andreas Färber
---
hw/core/qdev.c | 2 ++
1 file chang
From: Martin Cerveny
Add processing of optional argument path as "tree base".
Signed-off-by: Martin Cerveny
Signed-off-by: Andreas Färber
---
scripts/qmp/qom-tree | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/scripts/qmp/qom-tree b/scripts/qmp/qom-tree
index aea
From: "Daniel P. Berrange"
The 'policy' property was being registered with a typename of
'str', but it is in fact an enum of the 'HostMemPolicy' type.
Signed-off-by: Daniel P. Berrange
Reviewed-by: Paolo Bonzini
Signed-off-by: Andreas Färber
---
backends/hostmem.c | 2 +-
1 file changed, 1 i
Hello Peter,
This is my QOM (devices) patch queue. Please pull.
v2 contains changes that hopefully fix warnings on gcc 4.7.
Regards,
Andreas
Cc: Peter Maydell
Cc: Eduardo Habkost
Cc: Paolo Bonzini
Cc: Daniel P. Berrange
The following changes since commit 0915aed5842bd4dbe396b92d4f3b846ae29
From: Eduardo Habkost
With this, object_property_add_alias() callers can safely free the
target property name, like what already happens with the 'name' argument
to all object_property_add*() functions.
Signed-off-by: Eduardo Habkost
Reviewed-by: Paolo Bonzini
Reviewed-by: Stefan Hajnoczi
Sig
Am 21.05.2015 um 13:53 schrieb Daniel P. Berrange:
> On Thu, May 21, 2015 at 12:18:30PM +0100, Peter Maydell wrote:
>> On 20 May 2015 at 16:51, Andreas Färber wrote:
>>> Hello Peter,
>>>
>>> This is my QOM (devices) patch queue. Please pull.
>>>
>>> Regards,
>>> Andreas
>>>
>>> Cc: Peter Maydell
Hello!
> I think encoding should be CPU type specific i.e. not defined by what
> GIC can support and once we add CPU type with 8 cores, it would provide
> it's own version of mpidr_read since it would be defined by spec
> how to encode aff0.
I have redone this thing from scratch:
https://lists.
build_append_namestringv() and aml_string() first calculate the
resulting string's length with vsnprintf(NULL, ...), then allocate,
then print for real. Simply use g_strdup_vprintf() or g_vasprintf()
instead.
Signed-off-by: Markus Armbruster
---
hw/acpi/aml-build.c | 27 +---
Since we now require GLib 2.22+ (commit f40685c), we don't have to
work around lack of g_ptr_array_new_with_free_func() anymore.
This reverts commit 2e5feadbf8133b9b108919a2f576943b448a1a06.
Signed-off-by: Markus Armbruster
---
hw/acpi/aml-build.c | 5 ++---
1 file changed, 2 insertions(+), 3 d
Markus Armbruster (3):
Revert "aml-build: fix build for glib < 2.22"
acpi: Drop superfluous GLIB_CHECK_VERSION()
acpi: Simplify printing to dynamic string
hw/acpi/aml-build.c | 34 +++---
1 file changed, 7 insertions(+), 27 deletions(-)
--
1.9.3
The version check is always true since we require GLib 2.22+ (commit
f40685c). It dates back to commit 8b9c3b8 "acpi-build: fix build on
glib < 2.22", amended in commit fd8f5e3 "acpi-build: fix build on glib
< 2.14". I guess we could revert both of them now, but I'm not sure
it's worth the churn,
Since we now require GLib 2.22+ (commit f40685c), we don't have to
work around lack of g_strcmp0() anymore.
This reverts commit 8f4774789947bc4bc4c8d026a289fe980d3d2ee1.
Conflicts:
qemu-ga.c
Signed-off-by: Markus Armbruster
---
qga/main.c | 2 +-
1 file changed, 1 insertion(+), 1 delet
Since we now require GLib 2.22+ (commit f40685c), we don't have to
work around lack of g_hash_table_iter_init() & friends anymore.
This reverts commit f8833a37c0c6b22ddd57b45e48cfb0f97dbd5af4.
Signed-off-by: Markus Armbruster
---
hw/ppc/spapr_pci.c | 28 +++-
1 file chan
Since we now require GLib 2.22+ (commit f40685c), we don't have to
work around lack of g_hash_table_get_keys() anymore.
This reverts commit 82a3a11897308b606120f7235001e87809708f85.
Signed-off-by: Markus Armbruster
---
target-arm/helper.c | 12 ++--
1 file changed, 2 insertions(+), 10 d
On Wed, 27 May 2015 17:16:49 +0100
Stefan Hajnoczi wrote:
> It's cumbersome to keep the whitelist up-to-date. New netdev backends
> should most likely be allowed so a blacklist makes more sense than a
> whitelist.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> net/net.c | 28 +++--
On Wed, 27 May 2015 17:16:48 +0100
Stefan Hajnoczi wrote:
> Although hmp-commands.hx lists "netmap" as a valid host_net_add type,
> the command rejects it because it's missing from the list.
>
> Signed-off-by: Stefan Hajnoczi
> ---
> net/net.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> di
Hi: Can someone help me to create my account in Qemu?
Thanks.
-ywong
Sent from [ProtonMail](https://protonmail.ch), encrypted email based in
Switzerland.
In the old times, we always had pvpanic in ACPI and a _STA function told
the guest not to use it. Now, we only include pvpanic in ACPI if it is
enabled, so the _STA function is useless from qemu's point of view, but
guests still want to query availability. Make them happy.
This patch extends
e
Huge thanks for detailed explanation.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1458239
Title:
Use qed instead of qcow2 for "-snapshot" functionality
Status in QEMU:
Invalid
Bug description
* Paolo Bonzini (pbonz...@redhat.com) wrote:
>
>
> On 26/05/2015 23:25, Christopher Covington wrote:
> > On 05/25/2015 08:53 AM, Paolo Bonzini wrote:
> >>
> >> On 22/05/2015 13:12, Daniel P. Berrange wrote:
> >>> In
> >>> particular I don't see why we need to have a SATA controller and ISA/LPC
>
When a -net type is used that was not compiled into the binary there
should be an error message.
Note the special case for -net none, which is a no-op.
Signed-off-by: Stefan Hajnoczi
---
net/net.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/net/net.c b/net/net.c
index 3352b
It's cumbersome to keep the whitelist up-to-date. New netdev backends
should most likely be allowed so a blacklist makes more sense than a
whitelist.
Signed-off-by: Stefan Hajnoczi
---
net/net.c | 28 +++-
1 file changed, 3 insertions(+), 25 deletions(-)
diff --git a/ne
Drop the union and move the hubport creation into the !is_netdev case.
Signed-off-by: Stefan Hajnoczi
---
net/net.c | 38 --
1 file changed, 16 insertions(+), 22 deletions(-)
diff --git a/net/net.c b/net/net.c
index cc1793c..e57a089 100644
--- a/net/net.c
+++
1 - 100 of 305 matches
Mail list logo
