Luiz Capitulino writes:
> It allows for specifying an alias for each option name, see next commits
> examples.
>
> Signed-off-by: Luiz Capitulino
> ---
> qemu-option.c | 9 -
> qemu-option.h | 1 +
> 2 files changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/qemu-option.c b/qemu-o
Il 11/07/2012 03:17, liu ping fan ha scritto:
> OK, I will move the creation in the CPU. But I think as part of qom,
> DeviceState can have a DeviceState child, so there is need for wrapper
> for the function. Maybe just make the qdev_create_kid(Object*) ->
> qdev_create_kid(DeviceState*) ?
You c
Il 11/07/2012 03:17, liu ping fan ha scritto:
>> > It really looks like time is ripe to make CPUs children of Device, so
>> > you can just use qdev_reset_all to reset the CPU.
>> >
> Change CPUState as child of Device? It is cool, but I think it will a
> huge work with argue.
Luckily other people
Luiz Capitulino writes:
> Today, machine options use underscores to separate words (eg. kernel_irqchip),
> however upcoming QOM conversion wants to use dashes instead.
Command line users will appreciate consistent use of '-', too.
> This series converts all machine type options to use dashes. C
Have you tried compiling the latest upstream version to see if this is
still an issue?
** Changed in: qemu
Status: New => Incomplete
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/602336
Title
Blue Swirl writes:
> On Tue, Jul 10, 2012 at 6:34 PM, Markus Armbruster wrote:
>> Scenario:
>>
>> 1. Start a guest with a blank disk (need not be the only disk) and
>>default disk geometry.
>>
>> 2. Examine the disk's physical geometry
>>
>>Details depend on the device model. scsi-hd ex
On Mon, Jul 9, 2012 at 8:57 PM, Andreas Färber wrote:
> Am 09.07.2012 12:59, schrieb igor:
>> On 06/20/2012 03:35 PM, Andreas Färber wrote:
>>> Am 20.06.2012 14:59, schrieb Igor Mammedov:
It's not correct to make CPU runnable (i.e. calling x86_cpu_realize())
when not all properties are s
On Tue, Jul 10, 2012 at 9:35 PM, Igor Mammedov wrote:
> On 07/09/2012 02:57 PM, Andreas Färber wrote:
>>
>> Am 09.07.2012 12:59, schrieb igor:
>>>
>>> On 06/20/2012 03:35 PM, Andreas Färber wrote:
Am 20.06.2012 14:59, schrieb Igor Mammedov:
>
> It's not correct to make CPU runnab
A quick git-grep for more instances of '_' in option parameter names
finds only vcard_emul_options(). Yet another ad hoc option argument
parser. I'm not asking you to do anything about it.
Il 11/07/2012 09:16, Markus Armbruster ha scritto:
>>> C. Do not derive default geometry from DOS partition table
>>>
>>>Can do only for new machine type, because it may break guests.
It will indeed break boot of old images (e.g. FreeDOS). Hence:
C1. Do not derive default geometry from DOS p
On 07/10/2012 12:41 PM, Andreas Färber wrote:
Am 10.07.2012 10:45, schrieb Paolo Bonzini:
Il 10/07/2012 08:16, Liu Ping Fan ha scritto:
Signed-off-by: Liu Ping Fan
---
hw/pc.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/hw/pc.c b/hw/pc.c
index c7e9ab3..
On 07/10/2012 07:29 PM, Andreas Färber wrote:
> Am 10.07.2012 12:43, schrieb Fabien Chouteau:
>> Bridge helper uses ioctl's not available on old Linux versions, we add
>> this flag to disable the build.
>>
>> Signed-off-by: Fabien Chouteau
>> ---
>> Makefile |2 +-
>> configure |8 ++
On 07/06/2012 03:06 PM, Amit Shah wrote:
On (Tue) 26 Jun 2012 [08:01:20], Anthony Liguori wrote:
>On 06/26/2012 05:48 AM, Amit Shah wrote:
> >On (Mon) 25 Jun 2012 [17:59:28], Anthony Liguori wrote:
> >>On 06/25/2012 05:46 PM, Anthony Liguori wrote:
> >>>From: Amit Shah
> >
> >>>diff --git a
On 06/19/2012 08:22 PM, Michael Roth wrote:
On Tue, Jun 19, 2012 at 11:34:42PM +0900, Takuya Yoshikawa wrote:
On Tue, 19 Jun 2012 09:01:36 -0500
Anthony Liguori wrote:
I'm not at all convinced that postcopy is a good idea. There needs a clear
expression of what the value proposition is that'
Patches fixed calculationg for value of year and remove unnecessary code.
Oleg Ogurtsov (2):
hw/exynos4210_rtc.c: Fix calculating for value of year
hw/exynos4210_rtc.c: remove unnecessary code
hw/exynos4210_rtc.c |7 ++-
1 files changed, 2 insertions(+), 5 deletions(-)
--
1.7.5.4
Signed-off-by: Oleg Ogurtsov
---
hw/exynos4210_rtc.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/exynos4210_rtc.c b/hw/exynos4210_rtc.c
index f781020..b42586e 100644
--- a/hw/exynos4210_rtc.c
+++ b/hw/exynos4210_rtc.c
@@ -142,7 +142,7 @@ static const VMStateDescr
Bridge helper uses ioctl's not available on old Linux versions, we add
this flag to disable the build.
Signed-off-by: Fabien Chouteau
---
Makefile |2 +-
configure | 14 ++
2 files changed, 15 insertions(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index 34d6a9e..c50ec5
On 11 July 2012 11:09, Fabien Chouteau wrote:
> Bridge helper uses ioctl's not available on old Linux versions, we add
> this flag to disable the build.
This is still the wrong approach -- you need to be testing for
presence of the ioctls and disabling the bridge helper if they're
not found. Once
Signed-off-by: Oleg Ogurtsov
---
hw/exynos4210_rtc.c |5 +
1 files changed, 1 insertions(+), 4 deletions(-)
diff --git a/hw/exynos4210_rtc.c b/hw/exynos4210_rtc.c
index b42586e..42a4ddc 100644
--- a/hw/exynos4210_rtc.c
+++ b/hw/exynos4210_rtc.c
@@ -510,10 +510,7 @@ static void exynos421
On 05/23/2012 05:07 AM, Andreas Färber wrote:
Also change return type to bool.
Signed-off-by: Andreas Färber
---
hw/apic.h|2 +-
hw/apic_common.c |2 +-
hw/pc.c |6 +++---
target-i386/helper.c |2 +-
target-i386/kvm.c|4 +++-
5 files c
Signed-off-by: Gerd Hoffmann
---
hw/qdev-monitor.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/qdev-monitor.c b/hw/qdev-monitor.c
index 7915b45..2053c4a 100644
--- a/hw/qdev-monitor.c
+++ b/hw/qdev-monitor.c
@@ -545,7 +545,7 @@ static void qdev_print(Monitor *mon,
This allows to extract the beginning, end and name of a Device object.
Signed-off-by: Vasilis Liaskovitis
---
tools/acpi_extract.py | 28
1 files changed, 28 insertions(+), 0 deletions(-)
diff --git a/tools/acpi_extract.py b/tools/acpi_extract.py
index 167a322..cb
A 32-byte register is used to present up to 256 hotplug-able memory devices
to BIOS and OSPM. Hot-add and hot-remove functions trigger an ACPI hotplug
event through these. Only reads are allowed from these registers.
An ACPI hot-remove event but needs to wait for OSPM to eject the device.
We use a
This reverts bitmap state in the case of a failed hot operation, in order to
allow retry of failed hot operations
Signed-off-by: Vasilis Liaskovitis
---
src/acpi-dsdt.dsl |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/src/acpi-dsdt.dsl b/src/acpi-dsdt.dsl
index 1c253
At 07/11/2012 06:31 PM, Vasilis Liaskovitis Wrote:
> The memory device generation is guided by qemu paravirt info. Seabios
> first uses the info to setup SRAT entries for the hotplug-able memory slots.
> Afterwards, build_memssdt uses the created SRAT entries to generate
> appropriate memory device
On 05/23/2012 05:07 AM, Andreas Färber wrote:
Needed for converting cpu_is_bsp().
Signed-off-by: Andreas Färber
Cc: Paolo Bonzini
---
...
@@ -899,7 +900,13 @@ static DeviceState *apic_init(void *env, uint8_t apic_id)
}
qdev_prop_set_uint8(dev, "id", apic_id);
-qdev_prop_se
Dimm physical address offsets are calculated automatically and memory map is
adjusted accordingly. If a DIMM can fit before the PCI_HOLE_START (currently
0xe000), it will be added normally, otherwise its physical address will be
above 4GB.
Signed-off-by: Vasilis Liaskovitis
---
hw/pc.c
Live migration works after memory hot-add events, as long as the
qemu command line "-dimm" arguments are changed on the destination host
to specify "populated=on" for the dimms that have been hot-added.
If a command-line change has not occured, the destination host does not yet
have the correspond
The numa_fw_cfg paravirt interface is extended to include SRAT information for
all hotplug-able dimms. There are 3 words for each hotplug-able memory slot,
denoting start address, size and node proximity. The new info is appended after
existing numa info, so that the fw_cfg layout does not break.
Signed-off-by: Gerd Hoffmann
---
hw/qdev-monitor.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/qdev-monitor.c b/hw/qdev-monitor.c
index 7915b45..e7d2804 100644
--- a/hw/qdev-monitor.c
+++ b/hw/qdev-monitor.c
@@ -543,7 +543,7 @@ static void qdev_print(Monitor *mon,
Implement batch dimm creation command line options. These could be useful for
not bloating the command line with a large number of dimms.
syntax: -dimms pfx=poolid,size=sz,num=n
Will create numdimms dimms with ids poolid0, ..., poolidn-1. Each dimm has a
size of sz.
Implement -dimmpop option
The memory device generation is guided by qemu paravirt info. Seabios
first uses the info to setup SRAT entries for the hotplug-able memory slots.
Afterwards, build_memssdt uses the created SRAT entries to generate
appropriate memory device objects. One memory device (and corresponding SRAT
entry)
Each hotplug-able memory slot is a SysBusDevice. A hot-add operation for a
particular dimm creates a new MemoryRegion of the given physical address
offset, size and node proximity, and attaches it to main system memory as a
sub_region. A hot-remove operation detaches and frees the MemoryRegion from
This implements batch monitor operations for hot-add and hot-remove. These are
probably better suited for a higher-level management layer, but are useful for
testing. Let me know if there is interest for such commands upstream.
syntax: mem_increase poolid num
will hotplug num dimms from pool pooli
trace_megasas_dcmd_dump_frame() takes 9 arguments, which is
rather much. Plus the trace infrastructure doesn't support
it anymore.
As we can get the information via other means it's pointless
to have it in the driver, so rather use some proper trace
point here and remove the old one.
Signed-off-by
In order to hotplug memory between RamSize and BUILD_PCIMEM_START, the pci
window needs to start at BUILD_PCIMEM_START (0xe000).
Otherwise, the guest cannot online new dimms at those ranges due to pci_root
window conflicts. (workaround for linux guest is booting with pci=nocrs)
Signed-off-by:
Current qemu-kvm master merged with latest upstream fails on startup:
(gdb) bt
#0 0x7fdcd4a047a0 in kvm_vcpu_ioctl (env=0x0, type=-1071075694) at
/home/tlv/akivity/qemu/kvm-all.c:1602
#1 0x7fdcd49c9fda in kvm_apic_enable_tpr_reporting
(s=0x7fdcd75af6c0, enable=false) at
/home/tlv/akivity
Extend the DSDT to include methods for handling memory hot-add and hot-remove
notifications and memory device status requests. These functions are called
from the memory device SSDT methods.
Signed-off-by: Vasilis Liaskovitis
---
src/acpi-dsdt.dsl | 70 +
in case of hot-remove or hot-add failure, the dimm bitmaps in qemu and Seabios
are inconsistent with the true state of the DIMM devices. The "populated" field
of the DimmState reflects the true state of the device. This inconsistency means
that a failed operation cannot be retried.
Ths patch updat
Returns total memory of guest in bytes, including hotplugged memory.
Signed-off-by: Vasilis Liaskovitis
---
hmp-commands.hx |2 ++
hmp.c|7 +++
hmp.h|1 +
hw/dimm.c| 15 +++
monitor.c|7 +++
qapi-schema.json | 12 ++
On 2012-07-11 13:46, Avi Kivity wrote:
> Current qemu-kvm master merged with latest upstream fails on startup:
>
> (gdb) bt
> #0 0x7fdcd4a047a0 in kvm_vcpu_ioctl (env=0x0, type=-1071075694) at
> /home/tlv/akivity/qemu/kvm-all.c:1602
> #1 0x7fdcd49c9fda in kvm_apic_enable_tpr_reporting
>
On 07/11/12 12:31, Vasilis Liaskovitis wrote:
> In order to hotplug memory between RamSize and BUILD_PCIMEM_START, the pci
> window needs to start at BUILD_PCIMEM_START (0xe000).
> Otherwise, the guest cannot online new dimms at those ranges due to pci_root
> window conflicts. (workaround for l
On Tue, Jul 10, 2012 at 09:19:37PM -0500, Doug Goldstein wrote:
> On Tue, Jul 10, 2012 at 2:02 PM, Eduardo Habkost wrote:
> > On Tue, Jul 10, 2012 at 11:54:05AM +0200, Christophe Fergeau wrote:
> >> On Sat, Jul 07, 2012 at 07:10:53PM +0300, Zeeshan Ali (Khattak) wrote:
> >> > Hi,
> >> >I'm try
This allows failed hot operations to be retried at anytime. This only
works for guests that use _OST notification. Other guests cannot retry failed
hot operations on same devices until after reboot.
Signed-off-by: Vasilis Liaskovitis
---
hw/acpi_piix4.c | 20 +++-
hw/dimm.c
On 2012-07-11 13:58, Avi Kivity wrote:
> On 07/11/2012 02:55 PM, Jan Kiszka wrote:
>> On 2012-07-11 13:46, Avi Kivity wrote:
>>> Current qemu-kvm master merged with latest upstream fails on startup:
>>>
>>> (gdb) bt
>>> #0 0x7fdcd4a047a0 in kvm_vcpu_ioctl (env=0x0, type=-1071075694) at
>>> /ho
On 07/11/2012 02:59 PM, Jan Kiszka wrote:
>>>
>>> I will try to reproduce. Is there a tree of the merge available?
>>
>> I just merged upstream into qemu-kvm master. For some reason there were
>> no conflicts.
>
> A rare moment, I guess. ;)
I'll put it down to random chance until we can figure
Syntax: "-dimm id=name,size=sz,node=pxm,populated=on|off"
The starting physical address for all dimms is calculated automatically from top
of memory, skipping the pci hole at [PCI_HOLE_START, 4G).
"populated=on" means the dimm is populated at machine startup. Default is off.
"node" is defining nu
On 07/11/2012 03:04 PM, Avi Kivity wrote:
specific command line or guest?
>>>
>>> qemu-system-x86_64
>>
>> Just did the same, but it's all fine here.
>
> Ok, I'll debug it. Probably something stupid like a miscompile.
Indeed, a simple clean build fixed it up. Paolo, it looks like
autodep
Define SSDT hotplug-able memory devices in _SB namespace. The dynamically
generated SSDT includes per memory device hotplug methods. These methods
just call methods defined in the DSDT. Also dynamically generate a MTFY
method and a MEON array of the online/available memory devices. ACPI
extraction
Hot-add hmp syntax: dimm_add dimmid
Hot-remove hmp syntax: dimm_del dimmid
Respective qmp commands are "dimm-add", "dimm-del".
Signed-off-by: Vasilis Liaskovitis
---
hmp-commands.hx | 32
monitor.c | 11 +++
monitor.h |3 +++
qmp-comm
Guest can respond to ACPI hotplug events e.g. with _EJ or _OST method.
This patch implements a tail queue to store guest notifications for memory
hot-add and hot-remove requests.
Guest responses for memory hotplug command on a per-dimm basis can be detected
with the new hmp command "info memhp" or
On 07/11/2012 02:55 PM, Jan Kiszka wrote:
> On 2012-07-11 13:46, Avi Kivity wrote:
>> Current qemu-kvm master merged with latest upstream fails on startup:
>>
>> (gdb) bt
>> #0 0x7fdcd4a047a0 in kvm_vcpu_ioctl (env=0x0, type=-1071075694) at
>> /home/tlv/akivity/qemu/kvm-all.c:1602
>> #1 0x00
This is v2 of the ACPI memory hotplug prototype for x86_64 target.
Changes v1->v2
- memory map is automatically calculated for hotplug dimms. Dimms are added from
top-of-memory skipping the pci hole at [PCI_HOLE_START, 4G).
- Renamed from "-memslot" to "-dimm". Commands changed to "dimm_add",
"d
Add support for _OST method. _OST method will write into the correct I/O byte to
signal success / failure of hot-add or hot-remove to qemu.
Signed-off-by: Vasilis Liaskovitis
---
src/acpi-dsdt.dsl | 46 ++
src/ssdt-mem.dsl |4
2 files chan
This allows qemu to receive notifications from the guest OS on success or
failure of a memory hotplug request. The guest OS needs to implement the _OST
functionality for this to work (linux-next: http://lkml.org/lkml/2012/6/25/321)
Also add new _OST registers in docs/specs/acpi_hotplug.txt
Signed-
On 07/11/2012 09:35 AM, liu ping fan wrote:
On Tue, Jul 10, 2012 at 9:35 PM, Igor Mammedov wrote:
On 07/09/2012 02:57 PM, Andreas Färber wrote:
Am 09.07.2012 12:59, schrieb igor:
On 06/20/2012 03:35 PM, Andreas Färber wrote:
Am 20.06.2012 14:59, schrieb Igor Mammedov:
It's not correct t
On 07/11/2012 03:17 AM, liu ping fan wrote:
On Tue, Jul 10, 2012 at 6:12 PM, Andreas Färber wrote:
Am 10.07.2012 10:41, schrieb Paolo Bonzini:
Il 10/07/2012 08:16, Liu Ping Fan ha scritto:
This will give the embeded logic module, such as apic has the
opportunity to reset.
Signed-off-by: Liu
Am 11.07.2012 07:25, schrieb Michael Tokarev:
> On 11.07.2012 09:09, Stefan Weil wrote:
>> Commit 25e5e4c7 broke compilation for non POSIX hosts (e.g. MinGW)
>> because it partially replaced "ret" by "count".
>>
>> It also changed the handling of EINTR in a wrong way.
>>
>> The patch restores the o
Il 11/07/2012 14:08, Avi Kivity ha scritto:
> specific command line or guest?
>>>
>>> qemu-system-x86_64
>>> >>
>>> >> Just did the same, but it's all fine here.
>> >
>> > Ok, I'll debug it. Probably something stupid like a miscompile.
> Indeed, a simple clean build fixed it up.
Am 08.07.2012 21:22, schrieb blauwir...@gmail.com:
> From: Blue Swirl
>
> Use 'unsigned int' for bit numbers instead of 'unsigned long' or
> 'int'. Adjust asserts.
>
> Signed-off-by: Blue Swirl
I haven't followed the original discussion and therefore don't know what
the controversy is about (n
On Wed, 11 Jul 2012 09:00:59 +0200
Markus Armbruster wrote:
> Luiz Capitulino writes:
>
> > It allows for specifying an alias for each option name, see next commits
> > examples.
> >
> > Signed-off-by: Luiz Capitulino
> > ---
> > qemu-option.c | 9 -
> > qemu-option.h | 1 +
> > 2 fil
Am 08.07.2012 14:09, schrieb Andreas Schwab:
> blauwir...@gmail.com writes:
>
>> +pstrcpy(bs->backing_format, sizeof(bs->backing_format),
>> +backing_fmt ? backing_file : "");
>
> s/backing_file/backing_fmt/
Which is why such changes are probably a bad idea. Even more so if they
Am 10.07.2012 20:06, schrieb MORITA Kazutaka:
> At Tue, 10 Jul 2012 16:12:27 +0200,
> Christoph Hellwig wrote:
>>
>> Only buffers that map to unallocated blocks need to be zeroed.
>>
>> Signed-off-by: Christoph Hellwig
>>
>> ---
>> block/sheepdog.c | 37 ++---
>>
All current users (IDE, SCSI and virtio-blk) happen to share this 20
characters limit. Still, it should be left to device models. They
already enforce their limits. They have to, as the DriveInfo limit
only affects legacy -drive serial=..., not the qdev properties.
usb-storage, which doesn't li
Applies on top of "[PATCH v3 00/29] Disk geometry cleanup".
Markus Armbruster (4):
hw/block-common: Move BlockConf & friends from block.h
hw/block-common: Factor out fall back to legacy -drive serial=...
blockdev: Don't limit DriveInfo serial to 20 characters
hw/block-common: Factor out fa
This stuff doesn't belong to block layer, and was put there only
because a better home didn't exist then. Now it does.
Signed-off-by: Markus Armbruster
---
block.h | 45 -
hw/block-common.h | 45 ++
Signed-off-by: Markus Armbruster
---
hw/Makefile.objs |2 +-
hw/block-common.c| 24
hw/block-common.h|3 +++
hw/ide/qdev.c| 12 ++--
hw/scsi-disk.c |8 +---
hw/usb/dev-storage.c | 10 ++
hw/virtio-blk.c
On 11 July 2012 13:54, Kevin Wolf wrote:
> Does this patch improve anything? Last time I checked, qemu only
> compiled on gcc anyway.
It would be nice to be able to compile with LLVM/Clang; however
since Clang supports the ?: gcc extension this patch doesn't
move us any further in that direction.
Kevin Wolf writes:
> Am 05.07.2012 18:39, schrieb Markus Armbruster:
>> Markus Armbruster writes:
>>
>>> Kevin Wolf writes:
>>>
Am 29.06.2012 17:34, schrieb Markus Armbruster:
> New limits straight from ATA4 6.2 Register delivered data transfer
> command sector addressing.
>
>
On Wed, Jul 11, 2012 at 02:09:53PM +0100, Peter Maydell wrote:
> On 11 July 2012 13:54, Kevin Wolf wrote:
> > Does this patch improve anything? Last time I checked, qemu only
> > compiled on gcc anyway.
>
> It would be nice to be able to compile with LLVM/Clang; however
> since Clang supports the
Il 11/07/2012 14:39, Kevin Wolf ha scritto:
> Am 11.07.2012 07:25, schrieb Michael Tokarev:
>> On 11.07.2012 09:09, Stefan Weil wrote:
>>> Commit 25e5e4c7 broke compilation for non POSIX hosts (e.g. MinGW)
>>> because it partially replaced "ret" by "count".
>>>
>>> It also changed the handling of E
Signed-off-by: Markus Armbruster
---
hw/block-common.c | 40
hw/block-common.h |2 ++
hw/ide/core.c | 24
hw/ide/qdev.c | 19 ++-
hw/scsi-disk.c| 31 +++
hw/virti
On 07/11/2012 08:15 AM, Paolo Bonzini wrote:
Il 11/07/2012 14:39, Kevin Wolf ha scritto:
Am 11.07.2012 07:25, schrieb Michael Tokarev:
On 11.07.2012 09:09, Stefan Weil wrote:
Commit 25e5e4c7 broke compilation for non POSIX hosts (e.g. MinGW)
because it partially replaced "ret" by "count".
It
Am 10.07.2012 11:12, schrieb Markus Armbruster:
> Commit f3d54fc4 factored it out of hw/ide.c for reuse. Sensible,
> except it was put into block.c. Device-specific functionality should
> be kept in device code, not the block layer. Move it to
> hw/hd-geometry.c, and make stylistic changes requi
This patch fix ping issues for linux-user guest.
* The do_setsockopts function in linux-user does not support SOL_RAW
socket which is used in ping net tool.
* The recvmsg in main_loop of ping could not fetch
sockaddr_in struct. That is because do_sendrecvmsg in linux-user does
not pass the msg->m
On Mon, 09 Jul 2012 16:43:40 +0200
Kevin Wolf wrote:
> Am 15.06.2012 23:07, schrieb Eric Blake:
> > On 06/15/2012 02:47 PM, Supriya Kannery wrote:
> >> Enhance "info block" to display hostcache setting for each
> >> block device.
> >>
> >
> >> ##
> >> { 'type': 'BlockInfo',
> >>'data': {'d
On 07/06/2012 08:11 AM, Markus Armbruster wrote:
>
> It seems there could be more knobs to control than just "dump guest
> state yes/no". Therefore, extensible command line syntax like
> "--core-dump guest-ram=off" seems to be advisable. We have too many
> -dont-do-FOO options already.
-m 2T,du
Bridge helper uses ioctl's not available on old Linux versions, we add
this flag to disable the build.
Signed-off-by: Fabien Chouteau
---
Makefile |2 +-
configure | 42 ++
2 files changed, 43 insertions(+), 1 deletion(-)
diff --git a/Makefile b/Ma
On Sat, 16 Jun 2012 02:17:30 +0530
Supriya Kannery wrote:
> New command "block_set_hostcache" added for dynamically changing
> host pagecache setting of a block device.
>
> Usage:
> block_set_hostcache
> = block device
> = on/off
>
> Example:
> (qemu) block_set_hostcache ide0-hd0
On 07/10/2012 09:37 AM, Fabien Chouteau wrote:
On 07/10/2012 01:06 PM, Paolo Bonzini wrote:
Il 10/07/2012 13:02, Fabien Chouteau ha scritto:
Bridge helper uses ioctl's not available on old Linux versions, we add
this flag to disable the build.
Which ioctls? Please detect them, so that we c
Kevin Wolf writes:
> Am 10.07.2012 11:12, schrieb Markus Armbruster:
>> Commit f3d54fc4 factored it out of hw/ide.c for reuse. Sensible,
>> except it was put into block.c. Device-specific functionality should
>> be kept in device code, not the block layer. Move it to
>> hw/hd-geometry.c, and m
Avi and Jan experienced more problem with dependencies. This series
should fix them.
Paolo Bonzini (2):
build: include qapi-generated/ files in qga/Makefile.objs
build: get dependency file directories from object file names
build: do not use hw/kvm/
Makefile |4 ++--
Make
No reason to leave them out, and it will ensure that the dependencies
are picked up. Later we can perhaps move the files to another
directory to avoid ../ usage.
Signed-off-by: Paolo Bonzini
---
Makefile |2 +-
qga/Makefile.objs |2 ++
2 files changed, 3 insertions(+), 1 deleti
After commit dcff25f2cd8c11a9368cc2369aeb0319c32d9e26, Dependency file
are taken from the directories that have a Makefile.objs file. This is
not enough, since files can be included from other directories.
So, pick them from directories that have an object file in them.
Signed-off-by: Paolo Bonzi
Thanks for this patch. Review comments below...
(cc'ing the linux-user maintainer)
On 11 July 2012 14:56, Jing Huang wrote:
> This patch fix ping issues for linux-user guest.
>
> * The do_setsockopts function in linux-user does not support SOL_RAW
> socket which is used in ping net tool.
>
> * T
On 07/11/2012 01:32 PM, Vasilis Liaskovitis wrote:
> Implement batch dimm creation command line options. These could be useful for
> not bloating the command line with a large number of dimms.
IMO this is unneeded. With a management tool there is no problem
generating a long command line; from th
On 07/11/2012 04:22 PM, Corey Bryant wrote:
>
>
> On 07/10/2012 09:37 AM, Fabien Chouteau wrote:
>> On 07/10/2012 01:06 PM, Paolo Bonzini wrote:
>>> Il 10/07/2012 13:02, Fabien Chouteau ha scritto:
Bridge helper uses ioctl's not available on old Linux versions, we add
this flag
On 07/11/2012 04:31 AM, Vasilis Liaskovitis wrote:
> Guest can respond to ACPI hotplug events e.g. with _EJ or _OST method.
> This patch implements a tail queue to store guest notifications for memory
> hot-add and hot-remove requests.
>
> Guest responses for memory hotplug command on a per-dimm b
2012/7/11 Jing Huang :
> This patch fix ping issues for linux-user guest.
>
> * The do_setsockopts function in linux-user does not support SOL_RAW
> socket which is used in ping net tool.
>
> * The recvmsg in main_loop of ping could not fetch
> sockaddr_in struct. That is because do_sendrecvmsg in
On 07/11/2012 04:32 AM, Vasilis Liaskovitis wrote:
> Returns total memory of guest in bytes, including hotplugged memory.
>
> Signed-off-by: Vasilis Liaskovitis
Should this instead be merged with query-balloon output, so that we have
a single command that shows all aspects of memory usage (both
Il 11/07/2012 16:56, Fabien Chouteau ha scritto:
>>> So indeed you could also use SIOCDEVPRIVATE / BRCTL_ADD_IF if you were
>>> inclined to do so...
>>>
>>> >>
>>> >> Unfortunately I don't have time to go deep into the code. It's not a
>>> >> feature I plan to use, so the quickest so
On Wed, Jul 11, 2012 at 04:40:20PM +0200, Paolo Bonzini wrote:
> No reason to leave them out, and it will ensure that the dependencies
> are picked up. Later we can perhaps move the files to another
> directory to avoid ../ usage.
Couldn't we accomplish that by just adding the dependencies to qga
Il 11/07/2012 17:25, Michael Roth ha scritto:
> > No reason to leave them out, and it will ensure that the dependencies
> > are picked up. Later we can perhaps move the files to another
> > directory to avoid ../ usage.
>
> Couldn't we accomplish that by just adding the dependencies to qga-obj-y
On 07/11/2012 10:57 AM, Paolo Bonzini wrote:
Il 11/07/2012 16:56, Fabien Chouteau ha scritto:
So indeed you could also use SIOCDEVPRIVATE / BRCTL_ADD_IF if you were
inclined to do so...
Unfortunately I don't have time to go deep into the code. It's not a
feature I plan to use, so the quicke
Am 10.07.2012 11:12, schrieb Markus Armbruster:
> 29 patches may look discouraging, but most patches are small, and the
> ones that aren't just move code around.
>
> Goals of this series:
>
> 1. One more step towards a clean separation block device host and
>guest part.
>
> 2. Purge CHS geom
Il 15/06/2012 18:45, Eric Blake ha scritto:
> Pre-existing, but do we have any policy on US (canceled) vs. UK
> (cancelled) spelling? Obviously, our API spelling is locked into UK
> spelling, but if the docs generally prefer US spelling, do we need to
> adjust things here? But any such adjustment
On Wed, Jul 11, 2012 at 05:28:26PM +0200, Paolo Bonzini wrote:
> Il 11/07/2012 17:25, Michael Roth ha scritto:
> > > No reason to leave them out, and it will ensure that the dependencies
> > > are picked up. Later we can perhaps move the files to another
> > > directory to avoid ../ usage.
> >
>
Il 15/06/2012 19:32, Eric Blake ha scritto:
> Just so I'm clear, am I right that:
>
> calling block-job-pause multiple times means the subsequent calls are
> successful no-ops; likewise, calling block-job-resume on a job not
> paused (whether by calling multiple times, or if the job never paused i
Il 15/06/2012 19:45, Eric Blake ha scritto:
>> > +++ b/qapi-schema.json
>> > @@ -429,7 +429,7 @@
>> > #
>> > # Since: 1.0
>> > ##
>> > -{ 'enum': 'BlockDeviceIoStatus', 'data': [ 'ok', 'failed', 'nospace' ] }
>> > +{ 'enum': 'BlockDeviceIoStatus', 'data': [ 'ok', 'nospace', 'failed' ] }
> Isn't
Il 15/06/2012 19:50, Eric Blake ha scritto:
>> > { 'type': 'BlockJobInfo',
>> >'data': {'type': 'str', 'device': 'str', 'len': 'int',
>> > - 'offset': 'int', 'paused': 'bool', 'speed': 'int'} }
>> > + 'offset': 'int', 'paused': 'bool', 'speed': 'int',
>> > + 'io-s
1 - 100 of 150 matches
Mail list logo
