On Wed, Oct 24, 2012 at 2:31 PM, liu ping fan wrote:
> On Tue, Oct 23, 2012 at 8:36 PM, Avi Kivity wrote:
>> On 10/23/2012 02:12 PM, Jan Kiszka wrote:
>>> On 2012-10-22 11:23, Liu Ping Fan wrote:
Without biglock, we try to protect the mr by increase refcnt.
If we can inc refcnt, go back
On Tue, Oct 23, 2012 at 5:37 PM, Avi Kivity wrote:
> On 10/23/2012 11:32 AM, liu ping fan wrote:
>> On Tue, Oct 23, 2012 at 5:07 PM, Jan Kiszka wrote:
>>> On 2012-10-23 07:52, liu ping fan wrote:
On Mon, Oct 22, 2012 at 6:40 PM, Avi Kivity wrote:
> On 10/22/2012 11:23 AM, Liu Ping Fan w
On Tue, Oct 23, 2012 at 5:04 PM, Jan Kiszka wrote:
> On 2012-10-22 11:23, Liu Ping Fan wrote:
>> Use local lock to protect e1000. When calling the system function,
>> dropping the fine lock before acquiring the big lock. This will
>> introduce broken device state, which need extra effort to fix.
>
On Tue, Oct 23, 2012 at 8:36 PM, Avi Kivity wrote:
> On 10/23/2012 02:12 PM, Jan Kiszka wrote:
>> On 2012-10-22 11:23, Liu Ping Fan wrote:
>>> Without biglock, we try to protect the mr by increase refcnt.
>>> If we can inc refcnt, go backward and resort to biglock.
>>>
>>> Another point is memory
On Tue, Oct 23, 2012 at 8:38 PM, Gleb Natapov wrote:
> On Mon, Oct 22, 2012 at 05:23:52PM +0800, Liu Ping Fan wrote:
>> Rejecting the nested mmio request which does not aim at RAM, so we
>> can avoid the potential deadlock caused by the random lock sequence
>> of two device's local lock.
>>
>> Sig
From: Igor Mammedov
it's necessary for making CPU child of DEVICE without
causing circular header deps.
Signed-off-by: Igor Mammedov
[ehabkost: re-added the typedef to hw/irq.h after rebasing]
Signed-off-by: Eduardo Habkost
---
Changes v1 (from Igor) -> v2 (sent by Eduardo):
- Chunk adding ty
The bsd-user/qemu-types.h and linux-user/qemu-types.h files are almost
the same, but linux-user have the additional definitions of tswapal().
This moves the linux-user file to the main directory, so the same file
can be used by linux-user and bsd-user.
Signed-off-by: Eduardo Habkost
---
Cc: Blue
On Wed, Oct 24, 2012 at 07:31:19AM +0530, Avik Sil wrote:
> On 10/24/2012 04:15 AM, David Gibson wrote:
> > On Tue, Oct 23, 2012 at 03:39:35PM +0530, Avik Sil wrote:
> >> Signed-off-by: Avik Sil
> >
> > As Anthony suggests, it would be better to explicitly remove the
> > normal default boot order
Add GCC_WEAK vmstate symbols to qdev.c, so that qdev.o can be used
without savevm.o (i.e. by *-user).
Signed-off-by: Eduardo Habkost
---
Originally submitted as:
Subject: qdev-core: isolate vmstate handling into separate functions
Changes v1 -> v2:
- Add GCC_WEAK_DECL to function declarations
I don't know why it was including it, as I don't see any code that
depends on anything from qemu-common.h.
Signed-off-by: Eduardo Habkost
---
qlist.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/qlist.h b/qlist.h
index ae776f9..7408947 100644
--- a/qlist.h
+++ b/qlist.h
@@ -15,7 +15,6 @@
On 10/24/2012 01:18 AM, Richard Henderson wrote:
On 2012-10-23 16:21, Evgeny Voevodin wrote:
We will use this pointer from functions where we don't have an
interface to pass tcg_ctx as a parameter.
I don't think this is worthwhile. It'll just make the whole thing slower,
passing around unneces
Changes on v5:
- Tons of header cleanups just to eliminate qlist.h <-> cpu-common.h circular
dependency (patches 1-17)
- Add copyright/license information to qdev-properties.c (patch 17)
- Add copyright/license information to qdev-properties-system.c (patch 22)
- use error_report()+abort()
This will help reduce the qemu-common.h dependency hell.
Signed-off-by: Eduardo Habkost
--
Changes v1 -> v2:
- move qemu_open() & qemu_close() to qemu-stdio.h, too
---
qemu-common.h | 59 ++
qemu-stdio.h | 76 +
Instead of keeping all those struct typedefs on qemu-common.h, move it
to a header that can be safely included by other headers, containing
only the struct typedefs and not pulling other dependencies.
Also, move some of the qdev-core.h typedefs to the new file, too, so
other headers don't need to
The code depends on some functions from qemu-option.o, so add
qemu-option.o to universal-obj-y to make sure it's included.
Signed-off-by: Eduardo Habkost
---
Changes v1 -> v2:
- Keep files on the hw/ directory
(it's simply easier to keep them there, as qdev.o depends on irq.o)
- Add a $(hw-c
Include:
- for FILE
- qemu-option.h for QemuOptsList
Some of those headers were probably being included by accident because
some other headers were including qemu-common.h, but those headers
should eventually stop including qemu-common.h.
Signed-off-by: Eduardo Habkost
---
qemu-config.h | 2 ++
This version will be used on cases where sysbus.c is not compiled in
(e.g. *-user).
Signed-off-by: Eduardo Habkost
---
hw/qdev-core.h | 2 +-
hw/qdev.c | 12
2 files changed, 13 insertions(+), 1 deletion(-)
diff --git a/hw/qdev-core.h b/hw/qdev-core.h
index 494b687..19d9e03 1
From: Igor Mammedov
Signed-off-by: Igor Mammedov
[ehabkost: change CPU type declaration to hae TYPE_DEVICE as parent]
Signed-off-by: Eduardo Habkost
---
Yes, there is "changelog" data before the "---" mark, but I believe that
in this case they are important to indicate authorship and the scope
The core qdev code uses the reset handler list from vl.c, so move
qemu_register_reset(), qemu_unregister_reset() and qemu_devices_reset()
to qdev.c.
The function declarations were moved to a new qdev-reset.h file, that is
included by hw.h to keep compatibility, so we don't need to change all
files
The header file is specific for *-user, but I plan to introduce a more
generic qemu-types.h file, so I'm renaming it.
Signed-off-by: Eduardo Habkost
---
Cc: Blue Swirl
Cc: Riku Voipio
---
bsd-user/qemu.h | 2 +-
cpu-all.h | 2 +-
linux-user/qemu.h
From: Igor Mammedov
needed to prevent build breakage when CPU becomes a child of DeviceState
Signed-off-by: Igor Mammedov
[ehabkost: include too]
Signed-off-by: Eduardo Habkost
---
Yes, there is "changelog" data before the "---" mark, but I believe that
in this case they are important to indi
On Wed, Oct 24, 2012 at 12:11:23PM +0900, Yeongkyoon Lee wrote:
> On 2012년 10월 24일 00:25, 陳韋任 (Wei-Ren Chen) wrote:
> > Hi Yeongkyoon,
> >
> >> +#if defined(CONFIG_QEMU_LDST_OPTIMIZATION) && defined(CONFIG_SOFTMMU)
> >> +/* check whether the give addr is in TCG generated code buffer or not */
> >
On 2012년 10월 24일 00:25, 陳韋任 (Wei-Ren Chen) wrote:
Hi Yeongkyoon,
+#if defined(CONFIG_QEMU_LDST_OPTIMIZATION) && defined(CONFIG_SOFTMMU)
+/* check whether the give addr is in TCG generated code buffer or not */
should be given.
Cheers,
chenwj
Thanks a lot,
Include:
- for errno
- & for fcntl()
- "qemu-stdio.h" for qemu_open()
Some of those headers were probably being included by accident because
some other headers were including qemu-common.h, but those headers
should eventually stop including qemu-common.h.
Signed-off-by: Eduardo Habkost
---
hw_error() is specific for fatal hardware emulation errors, not for
internal errors related to the qdev object/class abstraction or object
initialization.
Replace it with an error_report() call, followed by abort().
This will also help reduce dependencies of the qdev code (as hw_error()
is from c
From: Anthony Liguori
Header file dependency is a frickin' nightmare right now. cpu.h tends to get
included in our 'include everything' header files but qdev also needs to include
those headers mainly for qdev-properties since it knows about CharDriverState
and friends.
We can solve this for no
Include:
- for g_malloc0()
- for strcmp()
Some of those headers were probably being included by accident because
some other headers were including qemu-common.h, but those headers
should eventually stop including qemu-common.h.
Signed-off-by: Eduardo Habkost
---
qapi/qmp-registry.c | 2 ++
1
This separates the qdev properties code in two parts:
- qdev-properties.c, that contains most of the qdev properties code;
- qdev-properties-system.c for code specific for qemu-system-*,
containing:
- Property types: drive, chr, netdev, vlan, that depend on code that
won't be included
is needed for the 'bool' type, used in the header.
The header is probably being included by accident because some other
headers are including qemu-common.h, but those headers should eventually
stop including qemu-common.h.
Signed-off-by: Eduardo Habkost
---
ui/vnc-palette.h | 1 +
1 file chang
It just needs the Monitor and DeviceState typedefs, so it doesn't need
all of qemu-common.h.
Signed-off-by: Eduardo Habkost
---
sysemu.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sysemu.h b/sysemu.h
index 0c39a3a..2b103a5 100644
--- a/sysemu.h
+++ b/sysemu.h
@@ -2,7 +2,
Add copyright and license information, based on git log,
and discussion at:
http://article.gmane.org/gmane.comp.emulators.qemu/176405
Message-ID: <20121017201414.ga5...@otherpad.lan.raisama.net>
The list of Copyright owners is huge, but I didn't think I really could
leave any of the authors out
module.h is where machine_init() is defined, but qemu-fsdev-dummy.c
doesn't include it.
The header is probably being included by accident because some other
headers are including qemu-common.h, but those headers should eventually
stop including qemu-common.h.
Signed-off-by: Eduardo Habkost
---
Include:
- for g_malloc0()
- for memset()
Some of those headers were probably being included by accident because
some other headers were including qemu-common.h, but those headers
should eventually stop including qemu-common.h.
Signed-off-by: Eduardo Habkost
---
ui/vnc-palette.c | 2 ++
1 f
Every time we make a tiny change on a header file, we often find
circular header dependency problems. To avoid this nightmare, we need to
stop including qemu-common.h on other headers, and we should gradually
move the declarations from the catchall qemu-common.h header to their
specific headers.
T
On Tue, Oct 23, 2012 at 12:56:18PM -0200, Eduardo Habkost wrote:
> On Wed, Oct 17, 2012 at 01:08:23PM -0500, Anthony Liguori wrote:
> > Eduardo Habkost writes:
> >
> > > The reset register/unregister code is specific to qemu-system-*, so
> > > isolate it so it can be moved to qdev-system.c.
> > >
On 10/24/2012 04:15 AM, David Gibson wrote:
> On Tue, Oct 23, 2012 at 03:39:35PM +0530, Avik Sil wrote:
>> Signed-off-by: Avik Sil
>
> As Anthony suggests, it would be better to explicitly remove the
> normal default boot order in this patch, after adding it for
> absolutely everything in the fir
On 10/24/2012 05:13 AM, David Gibson wrote:
> On Wed, Oct 24, 2012 at 12:47:34AM +0200, Alexander Graf wrote:
>>
>> On 24.10.2012, at 00:43, David Gibson wrote:
>>
>>> On Tue, Oct 23, 2012 at 12:20:08PM +0200, Alexander Graf wrote:
On 23.10.2012, at 12:09, Avik Sil wrote:
> This
The Buildbot has detected a new failure on builder block_mingw32 while building
qemu.
Full details are available at:
http://buildbot.b1-systems.de/qemu/builders/block_mingw32/builds/363
Buildbot URL: http://buildbot.b1-systems.de/qemu/
Buildslave for this Build: kraxel_rhel61
Build Reason: The
On Wed, Oct 24, 2012 at 12:47:34AM +0200, Alexander Graf wrote:
>
> On 24.10.2012, at 00:43, David Gibson wrote:
>
> > On Tue, Oct 23, 2012 at 12:20:08PM +0200, Alexander Graf wrote:
> >>
> >> On 23.10.2012, at 12:09, Avik Sil wrote:
> >>
> >>> This patch makes default boot order machine specif
On 2012-10-22 08:50, Aurelien Jarno wrote:
> OpenBIOS on sparc64 only support Standard VGA and not Cirrus VGA. Don't
> build Cirrus VGA support so that it can't be selected.
>
> This fixes the breakage introduced by commit f2898771.
>
> Reported-by: Richard Henderson
> Cc: Blue Swirl
> Signed-o
qemu-system-sparc64: /home/rth/work/qemu/qemu/memory.c:1022:
memory_region_destroy: Assertion `memory_region_transaction_depth == 0' failed.
Program received signal SIGABRT, Aborted.
0x75234925 in raise () from /lib64/libc.so.6
(gdb) where
#0 0x75234925 in raise () from /lib64/li
On 24.10.2012, at 00:43, David Gibson wrote:
> On Tue, Oct 23, 2012 at 12:20:08PM +0200, Alexander Graf wrote:
>>
>> On 23.10.2012, at 12:09, Avik Sil wrote:
>>
>>> This patch makes default boot order machine specific instead of
>>> set globally. The default boot order can be set per machine in
On Tue, Oct 23, 2012 at 03:39:35PM +0530, Avik Sil wrote:
> Signed-off-by: Avik Sil
As Anthony suggests, it would be better to explicitly remove the
normal default boot order in this patch, after adding it for
absolutely everything in the first patch.
This also needs a better patch comment, expl
On Tue, Oct 23, 2012 at 12:20:08PM +0200, Alexander Graf wrote:
>
> On 23.10.2012, at 12:09, Avik Sil wrote:
>
> > This patch makes default boot order machine specific instead of
> > set globally. The default boot order can be set per machine in
> > QEMUMachine default_machine_opts, or by the com
On 10/22/12 11:03, Igor Mammedov wrote:
It'll keep legacy behavior of kvmclock CPUID feature, which is
toggles on KVM_FEATURE_CLOCKSOURCE and KVM_FEATURE_CLOCKSOURCE2
CPUID feature bits.
Rename feature corresponding to KVM_FEATURE_CLOCKSOURCE to
f-kvmclock1 to free f-kvmclock for use of legacy f
On 10/22/12 11:03, Igor Mammedov wrote:
Signed-off-by: Igor Mammedov
---
target-i386/cpu.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 3131945..dc4fcdf 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -174,7
On 10/22/12 11:02, Igor Mammedov wrote:
Signed-off-by: Igor Mammedov
---
target-i386/cpu.c | 20 +---
1 file changed, 1 insertion(+), 19 deletions(-)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index c9d8dbc..951d12b 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu
On 10/22/12 11:02, Igor Mammedov wrote:
Signed-off-by: Igor Mammedov
---
- convert to static property
---
target-i386/cpu.c | 20 +---
1 file changed, 1 insertion(+), 19 deletions(-)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index fa974e7..c9d8dbc 100644
--- a/tar
On Tue, Oct 23, 2012 at 5:27 PM, Paolo Bonzini wrote:
> Il 23/10/2012 14:34, Paolo Bonzini ha scritto:
>> Fixed like this and pushed to the same place (tip is now commit
>> 8b5c3381d28ebb1765f17a4cbe1852d3b68b7dc4). Sorry.
>>
>> I will try to clean up the dependencies before or during soft featur
On 2012-10-23 16:21, Evgeny Voevodin wrote:
> We will use this pointer from functions where we don't have an
> interface to pass tcg_ctx as a parameter.
I don't think this is worthwhile. It'll just make the whole thing slower,
passing around unnecessary pointers.
r~
Hi,
Gerd Hoffmann a écrit :
On 10/22/12 21:43, Hervé Poussineau wrote:
Hello,
I'm trying to add support for a S3 PCI VGA card to QEMU. It currently
works on some non-x86 systems.
However, on x86, it seems that the card is correctly detected, but there
is no display at all; I think it is due to
On Mon, Oct 22, 2012 at 6:23 AM, Avi Kivity wrote:
> On 10/22/2012 09:04 AM, Philipp Hahn wrote:
>> Hello Doug,
>>
>> On Saturday 20 October 2012 00:46:43 Doug Goldstein wrote:
>>> I'm using libvirt 0.10.2 and I had qemu-kvm 1.1.1 running all my VMs.
>> ...
>>> I had upgraded to qemu-kvm 1.1.2
>>
On 10/23/12 06:29, Igor Mammedov wrote:
On Mon, 22 Oct 2012 19:19:29 -0400
Don Slutz wrote:
On 10/22/12 11:02, Igor Mammedov wrote:
- static properties names of CPUID features are changed to have "f-" prefix,
so that it would be easy to distinguish them from other properties.
- us
Avi Kivity writes:
> Please pull a few memory core fixes from:
>
> git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git memory/urgent
>
> Fixing ioeventfds on big endian systems, and potential use-after-free
> (not really exploitable with the current code, but still).
Pulled. Thanks.
Regards,
Don't explode QEMUMachineInitArgs before passing it to mainstone_init().
Signed-off-by: Peter Maydell
---
hw/mainstone.c | 21 +++--
1 file changed, 7 insertions(+), 14 deletions(-)
diff --git a/hw/mainstone.c b/hw/mainstone.c
index c0d6034..95a6d61 100644
--- a/hw/mainstone.c
Don't explode QEMUMachineInitArgs before passing it to the
omap_sx1 common init function.
Signed-off-by: Peter Maydell
---
hw/omap_sx1.c | 36 +---
1 file changed, 9 insertions(+), 27 deletions(-)
diff --git a/hw/omap_sx1.c b/hw/omap_sx1.c
index ad17487..7474f2
Define a new global dma_context_memory which is a DMAContext corresponding
to the global address_space_memory AddressSpace. This can be used by
sysbus peripherals like sysbus-ohci which need to do DMA.
In particular, use it in the sysbus-ohci device, which fixes a
segfault when attempting to use t
Don't explode QEMUMachineInitArgs in every realview init
function; just pass it to the common realview_init() code
instead.
Signed-off-by: Peter Maydell
---
hw/realview.c | 68 -
1 file changed, 19 insertions(+), 49 deletions(-)
diff --g
Don't explode QEMUMachineInitArgs before passing it to the common
versatile init function.
Signed-off-by: Peter Maydell
---
hw/versatilepb.c | 44 +++-
1 file changed, 11 insertions(+), 33 deletions(-)
diff --git a/hw/versatilepb.c b/hw/versatilepb.c
in
A lot of the ARM board models follow the pattern of having a
single common init function which is called with various
parameters from the QEMUMachine init function for several board
model variants. The change to QEMUMachineInitArgs in commit 5f072e
took the fairly mechanical approach of splitting t
Don't explode QEMUMachineInitArgs before calling common init function.
Signed-off-by: Peter Maydell
---
hw/spitz.c | 45 ++---
1 file changed, 10 insertions(+), 35 deletions(-)
diff --git a/hw/spitz.c b/hw/spitz.c
index 2942626..04f5e85 100644
--- a/hw/
Turns out that patch #32 is the one that causes the command line:
./x86_64-softmmu/qemu-system-x86_64 -cpu
486,+fpu,+vme,+de,+pse,+tsc,+msr,+pae,+mce,+cx8,+apic,+sep,+mtrr,+pge,+mca,+cmov,+pat,+pse36,+pn,+clflush,+ds,+acpi,+mmx,+fxsr,+sse,+sse2,+ss,+ht,+tm,+ia64,+pbe,+pni,+sse3,+pclmulqdq,+pclmu
Don't explode QEMUMachineInitArgs before passing it to the vexpress
common init function.
Signed-off-by: Peter Maydell
---
hw/vexpress.c | 38 +-
1 file changed, 9 insertions(+), 29 deletions(-)
diff --git a/hw/vexpress.c b/hw/vexpress.c
index 36503d6..cd5a
Don't explode QEMUMachineInitArgs before passing it to n8x0_init().
Signed-off-by: Peter Maydell
---
hw/nseries.c | 39 +++
1 file changed, 11 insertions(+), 28 deletions(-)
diff --git a/hw/nseries.c b/hw/nseries.c
index 7ada90d..6560d3c 100644
--- a/hw/nse
On Mon, Oct 15, 2012 at 12:07:37PM -0600, Eric Blake wrote:
> On 10/15/2012 11:29 AM, Alex Bligh wrote:
> > This patch allows qemu-img rebase to rebase an image to
> > have no backing file, as opposed to merely allowing it to
> > rebase to an existing backing file.
>
> You can already do that by r
On 10/23/2012 09:59 AM, Corey Bryant wrote:
> Only network devices are prevented, right?
>
> Also, as I mentioned before, can you limit this to the subset of options
> that cause execve() to be issued? For example, can we allow libvirt to
> pass an fd for hotplugging a network device (e.g. netde
(found while trying to rebase the qemu-linaro OMAP3 patches;
dunno if it's reproducable in plain mainline easily)
Commit 9ac6a217 makes ohci_init_pxa() set up the OHCI code
with a NULL DMAContext*, and asserts in the commit message:
"in the SysBus case, it uses NULL - i.e. assumes for now that th
On 10/23/2012 01:55 AM, Eduardo Otubo wrote:
With the inclusion of the new "double whitelist" seccomp filter, Qemu
won't be able to execve() in runtime, thus, no hotplug net devices
allowed.
v2: * Error messages moved to the backend function, net_init_tap(), recommended
by Paolo Bonzini
On Tue, 23 Oct 2012 11:51:49 -0200
Eduardo Habkost wrote:
> On Tue, Oct 23, 2012 at 03:26:40PM +0200, Igor Mammedov wrote:
> > convert x86_cpu_list() to use QDEV_FIND_PROP_FROM_BIT() for getting
> > CPUID feature name.
> > In addition since x86_cpu_list() was the last user of *feature_name
> > ar
On 10/23/2012 06:00 PM, Peter Maydell wrote:
> (found while trying to rebase the qemu-linaro OMAP3 patches;
> dunno if it's reproducable in plain mainline easily)
>
> Commit 9ac6a217 makes ohci_init_pxa() set up the OHCI code
> with a NULL DMAContext*, and asserts in the commit message:
>
> "in t
On 10/23/2012 05:26 PM, Paolo Bonzini wrote:
>>> Yes, that's the point of doing things asynchronously---you do not need
>>> to do everything within stop_machine, you can start canceling AIO as
>>> soon as the OS sends the hot-unplug request. Then you only proceed with
>>> stop_machine and freeing
On Tue, 23 Oct 2012 11:29:46 -0400
Don Slutz wrote:
> On 10/22/12 11:03, Igor Mammedov wrote:
> > - def->kvm_features was used only for setting hadcoded defaults,
> > with defaults set by static props, we do not need it anymore,
> > ignore it in cpudef_2_x86_cpu().
> > - take in accou
On 10/23/2012 06:23 AM, Benoît Canet wrote:
> Signed-off-by: Benoit Canet
> ---
> block/quorum.c | 13 +
> 1 file changed, 13 insertions(+)
>
> diff --git a/block/quorum.c b/block/quorum.c
> index 878d930..1b40081 100644
> --- a/block/quorum.c
> +++ b/block/quorum.c
> @@ -281,6 +28
On 10/23/2012 06:23 AM, Benoît Canet wrote:
> Valid quorum resources look like
> quorum:threshold/total:path/to/image_1: ... :path/to/image_total
>
> ':' is used as a separator to allow to use networked path
> '\' is the escaping character for filename containing ':'
> '\' escape itself
>
> On th
On 10/22/12 11:03, Igor Mammedov wrote:
- def->kvm_features was used only for setting hadcoded defaults,
with defaults set by static props, we do not need it anymore,
ignore it in cpudef_2_x86_cpu().
- take in account CPUID_EXT_HYPERVISOR feature set by static
property default val
Il 23/10/2012 14:34, Paolo Bonzini ha scritto:
> Fixed like this and pushed to the same place (tip is now commit
> 8b5c3381d28ebb1765f17a4cbe1852d3b68b7dc4). Sorry.
>
> I will try to clean up the dependencies before or during soft feature
> freeze.
Nevermind, still broken. :(
Paolo
Il 23/10/2012 16:49, Avi Kivity ha scritto:
> On 10/23/2012 02:32 PM, Paolo Bonzini wrote:
>> Il 23/10/2012 14:15, Avi Kivity ha scritto:
>>> On 10/23/2012 02:06 PM, Paolo Bonzini wrote:
Il 23/10/2012 14:02, Avi Kivity ha scritto:
> On 10/23/2012 01:57 PM, Paolo Bonzini wrote:
>> Il 23
Hi Yeongkyoon,
> +#if defined(CONFIG_QEMU_LDST_OPTIMIZATION) && defined(CONFIG_SOFTMMU)
> +/* check whether the give addr is in TCG generated code buffer or not */
should be given.
Cheers,
chenwj
--
Wei-Ren Chen (陳韋任)
Computer Systems Lab, Institute of Informatio
On 10/23/2012 01:55 AM, Eduardo Otubo wrote:
This patch includes a second whitelist right before the main loop. It's
a smaller and more restricted whitelist, excluding execve() among many
others.
v2: * ctx changed to main_loop_ctx
* seccomp_on now inside ifdef
* open syscall added to
Nicholas Thomas wrote:
> On Tue, 2012-10-23 at 12:33 +0200, Kevin Wolf wrote:
> > Am 22.10.2012 13:09, schrieb n...@bytemark.co.uk:
> > >
> > > This is unlikely to come up now, but is a necessary prerequisite for
> > > reconnection
> > > behaviour.
> > >
> > > Signed-off-by: Nick Thomas
> > > -
On Wed, Oct 17, 2012 at 01:08:23PM -0500, Anthony Liguori wrote:
> Eduardo Habkost writes:
>
> > The reset register/unregister code is specific to qemu-system-*, so
> > isolate it so it can be moved to qdev-system.c.
> >
> > Signed-off-by: Eduardo Habkost
> > ---
> > hw/qdev-core.c | 24 +++
On 10/23/2012 02:55 PM, Anthony Liguori wrote:
> Avi Kivity writes:
>
>> target_phys_addr_t is unwieldly, violates the C standard (_t suffixes are
>> reserved) and its purpose doesn't match the name (most target_phys_addr_t
>> addresses are not target specific). Replace it with a finger-friendly
On 10/23/2012 03:04 PM, Peter Maydell wrote:
> On 23 October 2012 11:30, Avi Kivity wrote:
>> target_phys_addr_t is unwieldly, violates the C standard (_t suffixes are
>> reserved) and its purpose doesn't match the name (most target_phys_addr_t
>> addresses are not target specific). Replace it wi
On 10/23/2012 02:32 PM, Paolo Bonzini wrote:
> Il 23/10/2012 14:15, Avi Kivity ha scritto:
>> On 10/23/2012 02:06 PM, Paolo Bonzini wrote:
>>> Il 23/10/2012 14:02, Avi Kivity ha scritto:
On 10/23/2012 01:57 PM, Paolo Bonzini wrote:
> Il 23/10/2012 13:55, Avi Kivity ha scritto:
So
Am 18.10.2012 22:41, schrieb Corey Bryant:
> qemu_dup_flags() currently limits the flags that can be set on the
> fcntl() F_SETFL call to those that we currently know can be set with
> fcntl() F_SETFL. The problem with this is that it will prevent use
> of new flags in the future without a code up
The new options are tested with blkdebug on both the source and the
target.
Signed-off-by: Paolo Bonzini
---
The only change is in 041.out.
tests/qemu-iotests/041| 253 ++
tests/qemu-iotests/041.out| 4 +-
tests/qemu-iotests/iotests.
Signed-off-by: Paolo Bonzini
---
tests/qemu-iotests/041 | 362 +
tests/qemu-iotests/041.out | 5 +
tests/qemu-iotests/group | 1 +
3 file modificati, 368 inserzioni(+)
create mode 100755 tests/qemu-iotests/041
create mode 100644 tests/qemu-i
On 10/23/2012 02:40 PM, Jan Kiszka wrote:
> On 2012-10-23 14:28, Avi Kivity wrote:
>> On 10/23/2012 02:16 PM, Jan Kiszka wrote:
>>> On 2012-10-23 14:12, Paolo Bonzini wrote:
Il 23/10/2012 14:04, Jan Kiszka ha scritto:
>
> So the stop_machine idea is thrown away?
>>>
>
Eric Blake writes:
> On 10/23/2012 06:47 AM, Anthony Liguori wrote:
>
This change was postponed to after 1.2 was released to
give libvirt a chance to wean itself off parsing our --help
output.
>>>
>>> Yea, I know this has been the plan for a long time and I agree that it
>>> i
Il 23/10/2012 14:55, Anthony Liguori ha scritto:
>> > target_phys_addr_t is unwieldly, violates the C standard (_t suffixes are
>> > reserved) and its purpose doesn't match the name (most target_phys_addr_t
>> > addresses are not target specific). Replace it with a finger-friendly,
>> > standards
On Tue, 23 Oct 2012 09:32:53 -0400
Don Slutz wrote:
> On 10/22/12 11:02, Igor Mammedov wrote:
> > v5:
> >- Use static properties instead of dynamic ones
> >- Compile in KVM CPUID features only if CONFIG_KVM is defined
> >- Add "f-kvm_steal_tm" and "f-kvmclock_stable" CPUID feature nam
On Tue, Oct 23, 2012 at 03:26:40PM +0200, Igor Mammedov wrote:
> convert x86_cpu_list() to use QDEV_FIND_PROP_FROM_BIT() for getting
> CPUID feature name.
> In addition since x86_cpu_list() was the last user of *feature_name
> arrays, clean them up.
>
> Signed-off-by: Igor Mammedov
> ---
> v2:
>
Juan Quintela wrote:
> Hi
>
> Please send in any agenda topics you are interested in.
No agenda -> no call
Happy hacking, Juan.
On Tue, 23 Oct 2012 14:54:21 +0200
Paolo Bonzini wrote:
> Right now, stop followed by an incoming migration will let the
> virtual machine start. cont before an incoming migration instead
> will fail.
>
> This is bad because the actual behavior is not predictable; it is
> racy with respect to t
On 10/22/12 11:02, Igor Mammedov wrote:
v5:
- Use static properties instead of dynamic ones
- Compile in KVM CPUID features only if CONFIG_KVM is defined
- Add "f-kvm_steal_tm" and "f-kvmclock_stable" CPUID feature names
- Some qdev hacking to:
- allow iterate over Property[] bef
On 23.10.2012, at 15:24, Anthony Liguori wrote:
> Alexander Graf writes:
>
>> On 23.10.2012, at 12:09, Avik Sil wrote:
>>
>>> This patch makes default boot order machine specific instead of
>>> set globally. The default boot order can be set per machine in
>>> QEMUMachine default_machine_opts,
convert x86_cpu_list() to use QDEV_FIND_PROP_FROM_BIT() for getting
CPUID feature name.
In addition since x86_cpu_list() was the last user of *feature_name
arrays, clean them up.
Signed-off-by: Igor Mammedov
---
v2:
* fix x86_cpu_list(), use X86CPU as base to count offset correctly
* Do not f
Signed-off-by: Igor Mammedov
---
v2:
* use separate for() for every feature word
v3:
* fix check_features_against_host(), use X86CPU as base to count
offset correctly
---
target-i386/cpu.c | 72 +--
1 file changed, 43 insertions(+), 29 d
Alexander Graf writes:
> On 23.10.2012, at 12:09, Avik Sil wrote:
>
>> This patch makes default boot order machine specific instead of
>> set globally. The default boot order can be set per machine in
>> QEMUMachine default_machine_opts, or by the command line using
>> -machine ,boot=, or by stan
On Mon, 22 Oct 2012 17:03:21 +0200
Igor Mammedov wrote:
> Signed-off-by: Igor Mammedov
> ---
> v2:
> * use separate for() for every feature word
> ---
> target-i386/cpu.c | 72
> +--
> 1 file changed, 43 insertions(+), 29 deletions(-)
>
>
1 - 100 of 186 matches
Mail list logo
