* Juan Quintela (quint...@redhat.com) wrote:
> "Dr. David Alan Gilbert (git)" wrote:
> > From: "Dr. David Alan Gilbert"
> >
> > Many users now prefer to use drive_mirror over NBD as an
> > alternative to the older migrate -b option; drive_mirror is
> > more complex to setup but gives you more opt
On Mon, May 15, 2017 at 01:58:40PM +0100, Stefan Hajnoczi wrote:
> On Wed, May 10, 2017 at 10:07:58PM +0300, Michael S. Tsirkin wrote:
> > The following changes since commit 76d20ea0f1b26ebd5da2f5fb2fdf3250cde887bb:
> >
> > Merge remote-tracking branch 'armbru/tags/pull-qapi-2017-05-04-v3' into
On Thu, May 11, 2017 at 02:14:03PM +1000, David Gibson wrote:
> The following changes since commit 76d20ea0f1b26ebd5da2f5fb2fdf3250cde887bb:
>
> Merge remote-tracking branch 'armbru/tags/pull-qapi-2017-05-04-v3' into
> staging (2017-05-09 15:49:14 -0400)
>
> are available in the git repository
Am 15.05.2017 um 14:46 hat Alberto Garcia geschrieben:
> On Mon 15 May 2017 02:39:40 PM CEST, Kevin Wolf wrote:
> > This adds a small test for the image streaming error path for failing
> > block_job_create(), which would have found the null pointer dereference
> > in commit a170a91f.
> >
> > Signe
This patch was detached from the patchset:
"[PATCH v9 0/6] migration/ppc: migrating DRC, ccs_list and pending_events"
Because it is independent and has use outside of the scope of the
pseries DRC migration patchset.
Change log:
v9: no changes
v8: no changes
v7: no changes
v6: - Rebased with
From: Jianjun Duan
In racing situations between hotplug events and migration operation,
a rtas hotplug event could have not yet be delivered to the source
guest when migration is started. In this case the pending_events of
spapr state need be transmitted to the target so that the hotplug
event ca
On Thu, May 11, 2017 at 04:43:35PM +0200, Gerd Hoffmann wrote:
> Hi,
>
> Here is the vga patch queue, with the second patch batch to make display
> updates thread-safe.
>
> please pull,
> Gerd
>
> The following changes since commit 76d20ea0f1b26ebd5da2f5fb2fdf3250cde887bb:
>
> Merge remot
"Dr. David Alan Gilbert" wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> "Dr. David Alan Gilbert (git)" wrote:
>> > From: "Dr. David Alan Gilbert"
>> >
>> > Many users now prefer to use drive_mirror over NBD as an
>> > alternative to the older migrate -b option; drive_mirror is
>> > mor
From: Jianjun Duan
In racing situations between hotplug events and migration operation,
a rtas hotplug event could have not yet be delivered to the source
guest when migration is started. In this case the pending_events of
spapr state need be transmitted to the target so that the hotplug
event ca
This patch was detached from the patchset:
"[PATCH v9 0/6] migration/ppc: migrating DRC, ccs_list and pending_events"
Because it is independent and has use outside of the scope of the
pseries DRC migration patchset.
Change log:
v10: detached from DRC patch set
v9: no changes
v8: no changes
Please ignore this patch. I've resent it again as "v10" to help
differentiate
from the v9 version that is still included in the DRC patch set.
Daniel
On 05/15/2017 10:06 AM, Daniel Henrique Barboza wrote:
This patch was detached from the patchset:
"[PATCH v9 0/6] migration/ppc: migrating DRC
On Fri, 12 May 2017 15:59:29 -0600
Alex Williamson wrote:
> > + if (pasidt_binfo->size >= intel_iommu_get_pts(iommu)) {
> > + pr_err("Invalid gPASID table size %llu, host size
> > %lu\n",
> > + pasidt_binfo->size,
> > + intel_iommu_get_pts(iommu));
On Thu, May 11, 2017 at 04:18:14PM -0300, Eduardo Habkost wrote:
> The following changes since commit 76d20ea0f1b26ebd5da2f5fb2fdf3250cde887bb:
>
> Merge remote-tracking branch 'armbru/tags/pull-qapi-2017-05-04-v3' into
> staging (2017-05-09 15:49:14 -0400)
>
> are available in the git reposit
On Mon, 15 May 2017 14:22:32 +0200
Cédric Le Goater wrote:
> On 05/15/2017 01:40 PM, Greg Kurz wrote:
> > Commit 5bc8d26de20c ("spapr: allocate the ICPState object from under
> > sPAPRCPUCore") moved ICP objects from the machine to CPU cores. This
> > is an improvement since we no longer allocate
MSI should be supported by all interrupt controllers. Switching the old
check for msi_nonbroken into assertion. Do similar thing to
pci_add_capability2() below that. Then time to remove *errp.
Since msi_init() won't fail now, touch up all the callers to avoid
checks against it. One side effect is
On Mon 15 May 2017 03:05:00 PM CEST, Kevin Wolf wrote:
>> Mmm... but does that trigger the bug?
>>
>> The bug happens if the user tries to create a stream job on a BDS that:
>>
>> a) exists
>> b) is not the active node (i.e. needs reopening in read-write mode)
>> c) its node name is not valid for
* Kashyap Chamarthy (kcham...@redhat.com) wrote:
> On Wed, May 03, 2017 at 11:42:57AM +0100, Dr. David Alan Gilbert (git) wrote:
>
> A small comment inline, in the 'ifndef' section.
>
> > From: "Dr. David Alan Gilbert"
> >
> > Many users now prefer to use drive_mirror over NBD as an
> > alterna
On Fri, May 12, 2017 at 01:51:25PM +0200, Gerd Hoffmann wrote:
> Hi,
>
> Here is the ui patch queue with some vnc cleanups, absolute event
> support for input linux, egl bugfixes and the new eg-headless ui.
>
> please pull,
> Gerd
>
> The following changes since commit 76d20ea0f1b26ebd5da2f5
On Fri, May 12, 2017 at 02:21:52PM +0200, Gerd Hoffmann wrote:
> Hi,
>
> Here comes the usb patch queue, with a bugfix collection and a
> documentation update.
>
> please pull,
> Gerd
>
> The following changes since commit 76d20ea0f1b26ebd5da2f5fb2fdf3250cde887bb:
>
> Merge remote-trackin
Hi Gerd, 李林,
On 05/15/2017 07:45 AM, Gerd Hoffmann wrote:
In case the frame timer doesn't run for a while due to the host being
busy skipped_uframes can become big enough that UFRAME_TIMER_NS *
skipped_uframes overflows. Which in turn throws off all subsequent
ehci frame timer calculations.
Re
The netdev_add and netdev_del commands should be used nowadays instead.
Signed-off-by: Thomas Huth
---
hmp-commands.hx | 8
net/net.c | 13 +
2 files changed, 17 insertions(+), 4 deletions(-)
diff --git a/hmp-commands.hx b/hmp-commands.hx
index 0aca984..baeac47 10064
On Mon, 05/15 15:01, Peter Lieven wrote:
> Am 15.05.2017 um 14:52 schrieb Fam Zheng:
> > On Mon, 05/15 14:32, Peter Lieven wrote:
> > > Am 15.05.2017 um 14:28 schrieb Fam Zheng:
> > > > On Mon, 05/15 13:58, Peter Lieven wrote:
> > > > > Am 15.05.2017 um 13:53 schrieb Fam Zheng:
> > > > > > On Mon,
On 05/15/2017 08:39 AM, Greg Kurz wrote:
This function only does hypercall and RTAS-call registration, and thus
never returns an error. This patch adapt the prototype to reflect that.
Signed-off-by: Greg Kurz
Reviewed-by: Philippe Mathieu-Daudé
---
hw/intc/xics_spapr.c |3 +--
hw/ppc
On Fri, May 12, 2017 at 03:19:18PM -0500, Eric Blake wrote:
> On 05/12/2017 09:37 AM, Stefan Hajnoczi wrote:
> > From: "Daniel P. Berrange"
> >
> > The GThread implementation is not functional enough to actually
> > run QEMU reliably. While it was potentially useful for debugging,
> > we have a s
On Fri, May 12, 2017 at 10:38:11AM -0400, Stefan Hajnoczi wrote:
> The following changes since commit ecc1f5adeec4e3324d1b695a7c54e3967c526949:
>
> maintainers: Add myself as linux-user reviewer (2017-05-11 13:31:11 -0400)
>
> are available in the git repository at:
>
> git://github.com/stef
On Fri, May 12, 2017 at 10:37:14AM -0400, Stefan Hajnoczi wrote:
> The following changes since commit ecc1f5adeec4e3324d1b695a7c54e3967c526949:
>
> maintainers: Add myself as linux-user reviewer (2017-05-11 13:31:11 -0400)
>
> are available in the git repository at:
>
> git://github.com/stef
On 05/15/2017 07:13 AM, Dr. David Alan Gilbert wrote:
>>> +#ifndef CONFIG_LIVE_BLOCK_MIGRATION
>>> +if (params.blk || params.shared) {
>>> +error_setg(errp, "QEMU compiled without old-style block migration.
>>> "
>>> + "Use drive_mirror+NBD.");
>>
>> error_setg
The scripts/qemu-gdb.py file is not easily discoverable. Add a .gdbinit
file so GDB either loads qemu-gdb.py automatically or prints a message
informing the user how to enable them (some systems disable ./.gdbinit
loading for security reasons).
Suggested-by: Eric Blake
Signed-off-by: Stefan Hajn
On 05/15/2017 10:47 AM, Stefan Hajnoczi wrote:
The scripts/qemu-gdb.py file is not easily discoverable. Add a .gdbinit
file so GDB either loads qemu-gdb.py automatically or prints a message
informing the user how to enable them (some systems disable ./.gdbinit
loading for security reasons).
Sug
On 05/13/2017 05:04 PM, Philippe Mathieu-Daudé wrote:
> Hi Kamil,
>
> I think it is safer to add it in disas/libvixl/Makefile.objs where
> QEMU_CFLAGS are tuned for libvixl.
> This way you don't need to modify upstream libvixl.
>
Ah, right. disas/libvixl is one of the directories exempt from our
On Fri, May 12, 2017 at 05:52:44PM +0200, Juan Quintela wrote:
> Hi
>
> This pull contains:
> - Fix regression with compression threads (mea culpa)
> - start of includes migration (creating of blocker.h header)
> - Use Error** correctly after several iterations
> - Rename of RAM_SAVE_FLAG_COMPRESS
Marc-André Lureau writes:
> Switch strtoll() usage to qemu_strtoi64() helper while at it.
>
> Replace temporarily the error in qnum_get_int() with values >INT64_MAX
> until the visitor is updated.
>
> Add a few tests for large numbers.
>
> Signed-off-by: Marc-André Lureau
> ---
> qobject/json-l
Am 15.05.2017 um 15:35 schrieb Fam Zheng:
On Mon, 05/15 15:01, Peter Lieven wrote:
Am 15.05.2017 um 14:52 schrieb Fam Zheng:
On Mon, 05/15 14:32, Peter Lieven wrote:
Am 15.05.2017 um 14:28 schrieb Fam Zheng:
On Mon, 05/15 13:58, Peter Lieven wrote:
Am 15.05.2017 um 13:53 schrieb Fam Zheng:
Update to
v1: https://lists.gnu.org/archive/html/qemu-devel/2017-01/msg05699.html
v2: https://lists.gnu.org/archive/html/qemu-devel/2017-02/msg00728.html
v3: https://lists.gnu.org/archive/html/qemu-devel/2017-02/msg04391.html
v4: https://lists.gnu.org/archive/html/qemu-devel/2017-04/msg021
On 05/12/2017 06:38 PM, Philippe Mathieu-Daudé wrote:
> If you have coccinelle installed you can apply this script using:
>
> $ spatch \
> --macro-file scripts/cocci-macro-file.h \
> --dir target --in-place
>
> You can also use directly Peter Senna Tschudin docker image (easie
The '--image-opts' flag indicates whether the source filename
includes options. The target filename has to remain in the
plain filename format though, since it needs to be passed to
bdrv_create(). When using --skip-create though, it would be
possible to use image-opts syntax. This adds --target-im
The qemu-img dd command added --image-opts support, but missed
the corresponding --object support. This prevented passing
secrets (eg auth passwords) needed by certain disk images.
Reviewed-by: Fam Zheng
Reviewed-by: Max Reitz
Reviewed-by: Eric Blake
Signed-off-by: Daniel P. Berrange
---
qemu
The --image-opts flag can only be used to affect the parsing
of the source image. The target image has to be specified in
the traditional style regardless, since it needs to be passed
to the bdrv_create() API which does not support the new style
opts.
Reviewed-by: Fam Zheng
Reviewed-by: Max Reitz
From: "Dr. David Alan Gilbert"
Many users now prefer to use drive_mirror over NBD as an
alternative to the older migrate -b option; drive_mirror is
more complex to setup but gives you more options (e.g. only
migrating some of the disks if some of them are shared).
Allow the large chunk of block
On 15.05.2017 15:57, Eric Blake wrote:
> On 05/13/2017 05:04 PM, Philippe Mathieu-Daudé wrote:
>> Hi Kamil,
>>
>> I think it is safer to add it in disas/libvixl/Makefile.objs where
>> QEMU_CFLAGS are tuned for libvixl.
>> This way you don't need to modify upstream libvixl.
>>
>
> Ah, right. disas/
The qemu-img dd/convert commands will create an image file and
then try to open it. Historically it has been possible to open
new files without passing any options. With encrypted files
though, the *key-secret options are mandatory, so we need to
provide those options when opening the newly created
On 15/05/2017 16:04, Eric Blake wrote:
> On 05/12/2017 06:38 PM, Philippe Mathieu-Daudé wrote:
>> If you have coccinelle installed you can apply this script using:
>>
>> $ spatch \
>> --macro-file scripts/cocci-macro-file.h \
>> --dir target --in-place
>>
>> You can also use d
Reported by Coverity.
Signed-off-by: Fam Zheng
---
qemu-img.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/qemu-img.c b/qemu-img.c
index b506839..8b0dfa7 100644
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -294,6 +294,7 @@ static BlockBackend *img_open_opts(const char *optstr,
if (qdict_
On 15/05/2017 16:06, Paolo Bonzini wrote:
>
>
> On 15/05/2017 16:04, Eric Blake wrote:
>> On 05/12/2017 06:38 PM, Philippe Mathieu-Daudé wrote:
>>> If you have coccinelle installed you can apply this script using:
>>>
>>> $ spatch \
>>> --macro-file scripts/cocci-macro-file.h \
>>>
On Fri, May 12, 2017 at 04:40:00PM -0700, Richard Henderson wrote:
> This is a combination of my queued patches and those
> posted by Aurelien this week.
>
>
> r~
>
>
> The following changes since commit ecc1f5adeec4e3324d1b695a7c54e3967c526949:
>
> maintainers: Add myself as linux-user revi
On Mon, 05/15 16:02, Peter Lieven wrote:
> > > do you have a pointer to the image locking mechanism?
> > It hit qemu.git master just a moment ago. See raw_check_perm.
>
> which master are you looking at?
>
> $ git fetch upstream
> $ git log upstream/master --oneline
> 3a87606 Merge tag 'tracing-p
On Sat, May 13, 2017 at 04:21:43AM +0200, Kamil Rytarowski wrote:
> I volunteer to review NetBSD patches.
> Adding myself will help to not miss some of them.
>
> Restore NetBSD as a maintained host.
>
> All patches to make qemu/pkgsrc building have been emitted to review.
>
> Signed-off-by: Kami
On Sat, May 13, 2017 at 11:32:33AM +0200, Aurelien Jarno wrote:
> The following changes since commit ecc1f5adeec4e3324d1b695a7c54e3967c526949:
>
> maintainers: Add myself as linux-user reviewer (2017-05-11 13:31:11 -0400)
>
> are available in the git repository at:
>
> git://git.aurel32.net/
Stefan Hajnoczi writes:
> The scripts/qemu-gdb.py file is not easily discoverable. Add a .gdbinit
> file so GDB either loads qemu-gdb.py automatically or prints a message
> informing the user how to enable them (some systems disable ./.gdbinit
> loading for security reasons).
>
> Suggested-by:
On 05/15/2017 05:35 AM, Fam Zheng wrote:
> It got lost in commit a8d16f9ca "qemu-img: Update documentation for -U".
>
> Reported-by: Max Reitz
> Signed-off-by: Fam Zheng
> ---
> qemu-img-cmds.hx | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Eric Blake
--
Eric Blak
Am 15.05.2017 um 15:02 schrieb Kevin Wolf:
Am 15.05.2017 um 14:52 hat Fam Zheng geschrieben:
On Mon, 05/15 14:32, Peter Lieven wrote:
Am 15.05.2017 um 14:28 schrieb Fam Zheng:
On Mon, 05/15 13:58, Peter Lieven wrote:
Am 15.05.2017 um 13:53 schrieb Fam Zheng:
On Mon, 05/15 13:26, Peter Lieven
On 05/15/2017 04:46 AM, Dr. David Alan Gilbert wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> Eric Blake wrote:
>>> On 05/11/2017 11:32 AM, Juan Quintela wrote:
Those two capabilities were added through the command line. Notice that
we just created them. This is just the boil
[adding Markus]
On 05/15/2017 04:48 AM, Juan Quintela wrote:
> Eric Blake wrote:
>> On 05/12/2017 05:55 AM, Juan Quintela wrote:
> @@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk,
> bool blk,
> }
>
> if (has_inc && inc) {
> +migrat
On 15.05.2017 16:13, Stefan Hajnoczi wrote:
> On Sat, May 13, 2017 at 04:21:43AM +0200, Kamil Rytarowski wrote:
>> I volunteer to review NetBSD patches.
>> Adding myself will help to not miss some of them.
>>
>> Restore NetBSD as a maintained host.
>>
>> All patches to make qemu/pkgsrc building hav
On 15/05/2017 16:22, Kamil Rytarowski wrote:
> On 15.05.2017 16:13, Stefan Hajnoczi wrote:
>> On Sat, May 13, 2017 at 04:21:43AM +0200, Kamil Rytarowski wrote:
>>> I volunteer to review NetBSD patches.
>>> Adding myself will help to not miss some of them.
>>>
>>> Restore NetBSD as a maintained ho
On Mon, May 15, 2017 at 05:29:15PM +0800, Wei Wang wrote:
> Ping for comments, thanks.
>
> On 05/11/2017 12:57 PM, Wei Wang wrote:
> > The current implementation may change the packet layout when
> > vnet_hdr needs an endianness swap. The layout change causes
> > one more iov to be added to the io
On 05/15/2017 08:58 AM, Stefan Hajnoczi wrote:
>>
>> Juan Quintela (5):
>> migration: Fix regression with compression threads
>> migration: Pass Error ** argument to {save,load}_vmstate
>> ram: Rename RAM_SAVE_FLAG_
Change from v1 to v2:
tswapal_target_sigset() is no longer static since it was breaking the build.
This patch set deals with QEMU Linux user mode functionalities related to
signal handling. It is composed of four sections:
A. Extend strace support (patches 1-2, 8-9):
Strace support is added f
On 05/15/2017 09:05 AM, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert"
>
> Many users now prefer to use drive_mirror over NBD as an
> alternative to the older migrate -b option; drive_mirror is
> more complex to setup but gives you more options (e.g. only
> migrating some of
Improve strace support for syscall tkill(), tgkill() and rt_sigqueueinfo()
by implementing print functions that match arguments types of the system
calls and add them to the corresponding starce.list entry.
tkill:
Prior to this commit, typical strace output used to look like this:
4886 tkill(4886,
Change the unlock_user() argument from arg1 to arg3 to match with
lock_user(), since arg3 contains the pointer to the siginfo_t structure.
Signed-off-by: Miloš Stojanović
---
linux-user/syscall.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/linux-user/syscall.c b/linux-use
On 05/15/2017 09:10 AM, Fam Zheng wrote:
> Reported by Coverity.
>
> Signed-off-by: Fam Zheng
> ---
> qemu-img.c | 1 +
> 1 file changed, 1 insertion(+)
Reviewed-by: Eric Blake
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | lib
Fix the ssetmask() system call by removing the invocation of sigorset().
The ssetmask() system call should replace the old signal mask
with the new and return the old mask. It shouldn't combine
the old and the new mask with sigorset(). Fetching the old
mask for sigorset() is also no longer needed.
This commit improves strace support for syscall rt_tgsigqueueinfo().
Prior to this commit, typical strace output used to look like this:
7775 rt_tgsigqueueinfo(7775,7775,50,1996483164,0,0) = 0
After this commit, it looks like this:
7775 rt_tgsigqueueinfo(7775,7775,50,0x76ffea5c) = 0
Signed-off-b
Change the type of the first argument of rt_sigqueinfo() from int to pid_t
in the syscall declaration to match specifications of the system call.
Proper spacing is added to satisfy checkpatch.pl.
Signed-off-by: Miloš Stojanović
---
linux-user/syscall.c | 2 +-
1 file changed, 1 insertion(+), 1
Improve strace support for syscalls getuid(), gettid(), getppid()
and geteuid(). Since these system calls don't have arguments, "%s()"
is added in the corresponding strace.list entry so that no arguments
are printed.
getuid:
Prior to this commit, typical strace output used to look like this:
4894
On Sun, May 14, 2017 at 5:12 PM, Richard Henderson wrote:
>>
> Surely you'd also want to make this change for 0x11a and 0x11b. Which would
> also simplify that code a bit.
>
> That said, there's *lots* of missing LOCK prefix checks. What brings this
> one in particular to your attention?
>
The
If TRACK_TARGET_SIGMASK is defined, add fields in the TaskState structure
which will hold the target signal and suspend mask and add support for
initialization and forking. No functional changes are being introduced in
this commit. The TRACK_TARGET_SIGMASK will be defined in a later commit
where th
Fix copying between the host and target signal sets for the case when the
target set is larger than the host set.
sigismember() returns 1 if the specified signal number is a member of
the specified signal set, but it can also return -1 if an error occurs
(e.g. an out of range signal number is spec
This commit adds support for printing the content of the target_siginfo_t
structure in a similar way to how it is printed by the host strace. The
pointer to this structure is sent as the last argument of the
rt_sigqueueinfo() and rt_tgsigqueueinfo() system calls.
For this purpose, print_siginfo() i
Add a new system call: rt_tgsigqueueinfo().
This system call is similar to rt_sigqueueinfo(), but instead of
sending the signal and data to the whole thread group with the ID
equal to the argument tgid, it sends it to a single thread within
that thread group. The ID of the thread is specified by t
This patch improves the consistentcy of the output from print_siginfo()
by removing spaces around the equal sign of si_pid, si_uid, si_timer1,
si_timer2, si_band, si_fd, si_addr, si_status and si_sigval. This way
they match si_signo and ci_code. Host strace was used as a reference
for this chage.
Add functions tswapal_target_sigset(), target_to_abi_ulong_old_sigset()
and abi_ulong_to_target_old_sigset().
The tswapal_target_sigset() function transforms target signal sets
from target to host endianness. This is helpful for tracking and working
with the target signal masks in the host endiann
Add support for multiplexing in the host_signal_handler() function and in
system calls rt_sigqueueinfo()/rt_tgsigqueueinfo(), tgkill(), kill() for
the case when pid > 0.
The rt_sigqueueinfo()/rt_tgsigqueueinfo() system calls multiplex target
signals which are out of the host range by setting the s
Add support for tracking the larger target signal mask in system calls
sigprocmask()/rt_sigprocmask(), sigsuspend()/rt_sigsuspend(),
sgetmask()/ssetmask() and in functions do_sigreturn(), do_rt_sigreturn(),
handle_pending_signal(), process_pending_signals().
Add a new function do_target_sigprocmask
Add two inline functions that work with the signal set of the target.
target_sigdelset() removes a signal from target_sigset_t.
target_sigorset() creates a union of two target_sigset_t.
These functions will be used for introducing support for tracking the
target signal set. Functions for emptying
Add MUX_SIG as a multiplex signal number for all target signals which are
out of the host range. Add support for multiplexing in do_sigaction(),
process_pending_signals(), target_set_sigmask() and do_target_sigprocmask().
This patch solves the problem of unusable target signals which are out of
th
"Dr. David Alan Gilbert (git)" wrote:
> From: "Dr. David Alan Gilbert"
>
> Many users now prefer to use drive_mirror over NBD as an
> alternative to the older migrate -b option; drive_mirror is
> more complex to setup but gives you more options (e.g. only
> migrating some of the disks if some of
Eric Blake writes:
> On 05/15/2017 04:46 AM, Dr. David Alan Gilbert wrote:
>> * Juan Quintela (quint...@redhat.com) wrote:
>>> Eric Blake wrote:
On 05/11/2017 11:32 AM, Juan Quintela wrote:
> Those two capabilities were added through the command line. Notice that
> we just created
Eric Blake wrote:
> On 05/15/2017 04:46 AM, Dr. David Alan Gilbert wrote:
>> * Juan Quintela (quint...@redhat.com) wrote:
>>> Eric Blake wrote:
On 05/11/2017 11:32 AM, Juan Quintela wrote:
> Those two capabilities were added through the command line. Notice that
> we just created th
Eric Blake wrote:
> [adding Markus]
>
> On 05/15/2017 04:48 AM, Juan Quintela wrote:
>> Eric Blake wrote:
>>> On 05/12/2017 05:55 AM, Juan Quintela wrote:
>> @@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk,
>> bool blk,
>> }
>>
>> if (has_inc
Juan Quintela writes:
> Eric Blake wrote:
>> On 05/12/2017 05:55 AM, Juan Quintela wrote:
> @@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk,
> bool blk,
> }
>
> if (has_inc && inc) {
> +migrate_set_block_enabled(s, true);
>
Markus Armbruster wrote:
> Eric Blake writes:
>
>> On 05/15/2017 04:46 AM, Dr. David Alan Gilbert wrote:
>>> * Juan Quintela (quint...@redhat.com) wrote:
Eric Blake wrote:
> On 05/11/2017 11:32 AM, Juan Quintela wrote:
>> Those two capabilities were added through the command line.
When using the mapped-file security mode, we shouldn't let the client mess
with the metadata. The current code already tries to hide the metadata dir
from the client by skipping it in local_readdir(). But the client can still
access or modify it through several other operations. This can be used to
On 05/15/2017 03:16 PM, Greg Kurz wrote:
> On Mon, 15 May 2017 14:22:32 +0200
> Cédric Le Goater wrote:
>
>> On 05/15/2017 01:40 PM, Greg Kurz wrote:
>>> Commit 5bc8d26de20c ("spapr: allocate the ICPState object from under
>>> sPAPRCPUCore") moved ICP objects from the machine to CPU cores. This
>
>>> +int smt = kvmppc_smt_threads();
>>> +int nr_servers = DIV_ROUND_UP(max_cpus * smt, smp_threads);
>>
>> may be we should reintroduce nr_servers at the machine level ?
>>
>
> I had reintroduced it but then I realized it was only used in this
> function.
nr_servers is also us
Hello Dave,
It looks like we identified the problem.
We are working on fix and will send it as soon as it is ready.
~Dmitry.
Sent from my iPhone
> On 15 May 2017, at 12:22, Dr. David Alan Gilbert wrote:
>
> * Dmitry Fleytman (dmi...@daynix.com) wrote:
>> Hello Dave,
>
> Hi Dmitry,
> Thanks
* Dmitry Fleytman (dmi...@daynix.com) wrote:
> Hello Dave,
>
> It looks like we identified the problem.
>
> We are working on fix and will send it as soon as it is ready.
Thanks!
Dave
> ~Dmitry.
>
> Sent from my iPhone
>
> > On 15 May 2017, at 12:22, Dr. David Alan Gilbert
> > wrote:
> >
On 05/13/2017 03:16 AM, Gonglei (Arei) wrote:
>
>> From: Halil Pasic [mailto:pa...@linux.vnet.ibm.com]
>> Sent: Friday, May 12, 2017 7:02 PM
>>
>>
>> On 05/08/2017 01:38 PM, Gonglei wrote:
>>> According to the new spec, we should use different
>>> requst structure to store the data request based
On Mon, 15 May 2017 18:09:04 +0200
Cédric Le Goater wrote:
> On 05/15/2017 03:16 PM, Greg Kurz wrote:
> > On Mon, 15 May 2017 14:22:32 +0200
> > Cédric Le Goater wrote:
> >
> >> On 05/15/2017 01:40 PM, Greg Kurz wrote:
> >>> Commit 5bc8d26de20c ("spapr: allocate the ICPState object from und
On Mon, 15 May 2017 18:11:27 +0200
Cédric Le Goater wrote:
> >>> +int smt = kvmppc_smt_threads();
> >>> +int nr_servers = DIV_ROUND_UP(max_cpus * smt, smp_threads);
> >>
> >> may be we should reintroduce nr_servers at the machine level ?
> >>
> >
> > I had reintroduced it
Markus Armbruster wrote:
> Juan Quintela writes:
>
>> Eric Blake wrote:
>>> Or is the proposal that we are also going to simplify the QMP 'migrate'
>>> command to get rid of crufty parameters?
>>
>> I didn't read it that way, but I would not oppose O:-)
>>
>> Later, Juan.
>
> I'm not too famil
* Juan Quintela (quint...@redhat.com) wrote:
> Markus Armbruster wrote:
> > Juan Quintela writes:
> >
> >> Eric Blake wrote:
>
>
> >>> Or is the proposal that we are also going to simplify the QMP 'migrate'
> >>> command to get rid of crufty parameters?
> >>
> >> I didn't read it that way, but
socket_address_flatten() leaks a SocketAddress when its argument is
null. Happens when opening a ChardevBackend of type 'udp' that is
configured without a local address. Screwed up in commit bd269ebc due
to last minute semantic conflict resolution. Spotted by Coverity.
Signed-off-by: Markus Arm
On Sat, May 13, 2017 at 5:42 PM, John Bradley via Qemu-devel
wrote:
> From 7f74f048f135d9c9c230a9e90f72451c841c6d35 Mon Sep 17 00:00:00 2001
> From: John Bradley
> Date: Sat, 13 May 2017 23:07:47 +0100
> Subject: [PATCH] Changes to Broadcom(BCM) files and Raspberry Pi files.
> Addition of PanelEm
Update to
v1: https://lists.gnu.org/archive/html/qemu-devel/2017-01/msg05699.html
v2: https://lists.gnu.org/archive/html/qemu-devel/2017-02/msg00728.html
v3: https://lists.gnu.org/archive/html/qemu-devel/2017-02/msg04391.html
v4: https://lists.gnu.org/archive/html/qemu-devel/2017-04/msg021
The qemu-img dd command added --image-opts support, but missed
the corresponding --object support. This prevented passing
secrets (eg auth passwords) needed by certain disk images.
Reviewed-by: Fam Zheng
Reviewed-by: Max Reitz
Reviewed-by: Eric Blake
Signed-off-by: Daniel P. Berrange
---
qemu
The '--image-opts' flag indicates whether the source filename
includes options. The target filename has to remain in the
plain filename format though, since it needs to be passed to
bdrv_create(). When using --skip-create though, it would be
possible to use image-opts syntax. This adds --target-im
The --image-opts flag can only be used to affect the parsing
of the source image. The target image has to be specified in
the traditional style regardless, since it needs to be passed
to the bdrv_create() API which does not support the new style
opts.
Reviewed-by: Fam Zheng
Reviewed-by: Max Reitz
The qemu-img dd/convert commands will create an image file and
then try to open it. Historically it has been possible to open
new files without passing any options. With encrypted files
though, the *key-secret options are mandatory, so we need to
provide those options when opening the newly created
101 - 200 of 302 matches
Mail list logo
