The MigrateState struct uses an array for storing migration
parameters. This presumes that all future parameters will
be integers too, which is not going to be the case. There
is no functional reason why an array is used, if anything
it makes the code less clear. The QAPI schema already
defines a s
Now that we don't have have a buffer based QemuFile
implementation, the QEMUSizedBuffer code is also
unused and can be deleted. A simpler buffer class
also exists in util/buffer.c which other code can
used as needed.
Reviewed-by: Dr. David Alan Gilbert
Signed-off-by: Daniel P. Berrange
---
incl
Convert the unix socket migration protocol driver to use
QIOChannel and QEMUFileChannel, instead of plain sockets
APIs. It can be unconditionally built, since the socket
impl of QIOChannel will report a suitable error on platforms
where UNIX sockets are unavailable.
Reviewed-by: Dr. David Alan Gil
On Tue, 16 Feb 2016 16:45:24 +0100
Paolo Bonzini wrote:
> On 15/02/2016 18:58, Cornelia Huck wrote:
> > It seems a bit odd to me that ->started is the only state that is not
> > inside the dataplane struct... this approach saves a function call for
> > an accessor, though.
>
> Actually, I can do
Convert the exec socket migration protocol driver to use
QIOChannel and QEMUFileChannel, instead of the stdio
popen APIs. It can be unconditionally built because the
QIOChannelCommand class can report suitable error messages
on platforms which can't fork processes.
Reviewed-by: Dr. David Alan Gilb
Define two new migration parameters to be used with TLS encryption.
The 'tls-creds' parameter provides the ID of an instance of the
'tls-creds' object type, or rather a subclass such as 'tls-creds-x509'.
Providing these credentials will enable use of TLS on the migration
data stream.
If using x509
This is an update of patches that were previously posted
FYI: https://lists.gnu.org/archive/html/qemu-devel/2015-09/msg00829.html
v1: https://lists.gnu.org/archive/html/qemu-devel/2016-01/msg01914.html
The primary goal of this series of patches is to support TLS on the
migration data channel
Currently if an app initiates an outgoing migration, it
may or may not, get an error reported back on failure. If
the error occurs synchronously to the 'migrate' command
execution, the client app will see the error message. This
is the case for DNS lookup failures. If the error occurs
asynchronousl
Some of the test-vmstate.c test cases use a temporary file
while others use a memory buffer. To facilitate the future
removal of the qemu_bufopen() function, convert all the tests
to use a temporary file.
Reviewed-by: Dr. David Alan Gilbert
Signed-off-by: Daniel P. Berrange
---
tests/Makefile
Instead of relying on the default QEMUFile I/O blocking flag
state, explicitly turn on blocking I/O for outgoing migration
since it takes place in a background thread.
Reviewed-by: Dr. David Alan Gilbert
Signed-off-by: Daniel P. Berrange
---
migration/migration.c | 1 +
1 file changed, 1 insert
Introduce a new QEMUFile implementation that is based on
the QIOChannel objects. This impl is different from existing
impls in that there is no file descriptor that can be made
available, as some channels may be based on higher level
protocols such as TLS.
Although the QIOChannel based implementat
The QEMUFileOps struct contains the I/O subsystem callbacks
and the migration stage hooks. Split the hooks out into a
separate QEMUFileHooks struct to make it easier to refactor
the I/O side of QEMUFile without affecting the hooks.
Reviewed-by: Dr. David Alan Gilbert
Signed-off-by: Daniel P. Berr
Remove the assumption that every QEMUFile implementation has
a file descriptor available by introducing a new function
in QEMUFileOps to change the blocking state of a QEMUFile.
If not set, it will fallback to the original code using
the get_fd method.
Reviewed-by: Dr. David Alan Gilbert
Signed-
The s390 skeys monitor command needs to write out a plain text
file. Currently it is using the QEMUFile class for this, but
work is ongoing to refactor QEMUFile and eliminate much code
related to it. The only feature qemu_fopen() gives over fopen()
is support for QEMU FD passing, but this can be ac
The QEMUFile writev_buffer / put_buffer functions are expected
to write out the full set of requested data, blocking until
complete. The qemu_fflush() caller does not expect to deal with
partial writes. Clarify the function comments and add a sanity
check to the code to catch mistaken implementatio
Eric Blake writes:
> Empty unions serve no purpose, and while we compile with gcc
> which permits them, strict C99 forbids them. We could inject
> a dummy member (and in fact, we do for empty structs), but while
gen_variants() injects void *data.
> empty structs make sense in qapi,
Suggest t
Pranith Kumar writes:
> Hello,
>
> I am interested in working on a portion of the MTTCG project as part
> of GSoC 2016. I am writing to ask for guidance in creating a formal
> proposal.
>
> On IRC, Alex suggested a project to support proper modelling of memory
> consistency between different hos
Oh my, so many stupid little errors. Due to firewall impairment i am sending
these
patches manually. Last time i forgot to disable the newline breaks which made
the
patch v5 unusable. Sorry for this inconvenience.
i2c-tiny-usb is a small usb to i2c bridge:
http://www.harbaum.org/till/i2c_tiny_u
The current documentation of chardev mux=on is rather brief and opaque;
expand it to hopefully be a bit more helpful.
Signed-off-by: Peter Maydell
---
There was some discussion on #qemu yesterday evening about multiplexing,
and "make the docs a bit less confusing" was one suggestion...
---
qemu-
The QIOChannelBuffer struct uses a 'char *' for its data
buffer. It will give simpler type compatibility with the
migration APIs if it uses 'uint8_t *' instead, avoiding
several casts.
Signed-off-by: Daniel P. Berrange
---
include/io/channel-buffer.h | 2 +-
io/channel-buffer.c | 2 +-
2
In the docs for qio_channel_socket_connect_async,
qio_channel_socket_listen_async and
qio_channel_socket_dgram_async, mention that the
SocketAddress parameters are copied, so can be freed
immediately.
Reviewed-by: "Dr. David Alan Gilbert"
Signed-off-by: Daniel P. Berrange
---
include/io/channel
The current implementation of bdrv_qed_drain can cause a double
completion of a request.
The problem is that bdrv_qed_drain calls qed_plug_allocating_write_reqs
unconditionally, but this is not correct if an allocating write
is queued. In this case, qed_unplug_allocating_write_reqs will
restart t
Hi,
-virtio-win-0.1.102.iso
-host archlinux / qemu 2.4.1 / kvm
-guest windows 7 pro x64
-qemu-img create -f raw en_x64_windows_7_ipc2_storage_75 75G
-drive
file=/srv/qemu/vm/en_x64_windows_7_ipc2_storage_75,if=virtio,cache=none,format=raw,discard=unmap
My D: drive (74GB) has 2x 32GB files used
On 16 February 2016 at 13:49, Peter Maydell wrote:
> On 9 February 2016 at 15:24, Peter Maydell wrote:
>> This set of patches does a bunch more cleaning of includes.
>> Specifically it does everything necessary to be able to
>> remove osdep.h includes from header files including qemu-common.h
>
>
The following changes since commit a5af12871fd4601c44f08d9e49131e9ca13ef102:
Merge remote-tracking branch 'remotes/sstabellini/tags/xen-2016-02-12' into
staging (2016-02-12 17:36:12 +)
are available in the git repository at:
git://github.com/berrange/qemu.git tags/pull-io-next-2016-02-1
Depending on what object a file descriptor refers to a different
type of IO channel will be needed - either a QIOChannelFile or
a QIOChannelSocket. Introduce a qio_channel_new_fd() method
which will return the appropriate channel implementation.
Reviewed-by: Eric Blake
Signed-off-by: Daniel P. Be
On 15/02/2016 18:58, Cornelia Huck wrote:
> It seems a bit odd to me that ->started is the only state that is not
> inside the dataplane struct... this approach saves a function call for
> an accessor, though.
Actually, I can do better by moving the flag entirely within
hw/block/virtio-blk.c:
d
On 02/14/2016 01:49 AM, Fam Zheng wrote:
> On Fri, 02/12 18:06, John Snow wrote:
>> During incremental backups, if the target has a cluster size that is
>> larger than the backup cluster size and we are backing up to a target
>> that cannot (for whichever reason) pull clusters up from a backing i
On Mon 15 Feb 2016 05:40:29 PM CET, Stefan Hajnoczi wrote:
> On Fri, Feb 05, 2016 at 12:59:10PM +0200, Alberto Garcia wrote:
>> - With this series we set "a maximum of X operations/second for a
>>period of T seconds". If would also be possible to make it "a
>>maximum of X operations/second
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> This controller is also present in i.MX5X devices but they are not
> yet emulated by QEMU.
>
> Signed-off-by: Jean-Christophe Dubois
> @@ -0,0 +1,353 @@
> +/*
> + * IMX6 System Reset Controller
> + *
> + * Copyright (c) 2015 Jean-Chri
On 02/16/16 15:57, Markus Armbruster wrote:
> We set machine_class to the default first, and update it to the real
> one later. Any use of machine_class in between is almost certainly
> wrong. Set it once and for all instead.
>
> Signed-off-by: Markus Armbruster
> ---
> vl.c | 11 ++-
>
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> For now we only support the following devices:
> * up to 4 Cortex A9 cores
> * A9 MPCORE (SCU, GIC, TWD)
> * 5 i.MX UARTs
> * 2 EPIT timers
> * 1 GPT timer
> * 3 I2C controllers
> * 7 GPIO controllers
> * 6 SDHC controllers
> * 1 CCM de
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> Signed-off-by: Jean-Christophe Dubois
> ---
>
> Changes since V1:
> * Not present on V1
>
> hw/arm/fsl-imx25.c | 1 +
> hw/arm/fsl-imx31.c | 1 +
> hw/i2c/imx_i2c.c | 1 +
> hw/net/imx_fec.c | 1 +
> 4 files changed, 4 insertions
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> Signed-off-by: Jean-Christophe Dubois
> ---
>
> Changes since v1:
> * output a message and exit if RAM size is unsupported.
>
> hw/arm/Makefile.objs | 2 +-
> hw/arm/sabrelite.c | 93
>
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> Signed-off-by: Jean-Christophe Dubois
> ---
>
> Changes since V1:
> * move clk computation to uint64_t to avoid overflow
> * added explanation on _SET, _CLR and _TOG registers
> * move CCM and ANALOG handling in sub memory regions.
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> EPIT, GPT and other i.MX timers are using "abstract" clocks among which
> a CLK_IPG_HIGH clock.
>
> On i.MX25 and i.MX31 CLK_IPG and CLK_IPG_HIGH are mapped to the same clock
> but on other SOC like i.MX6 they are mapped to distinct clo
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> Most clocks supported by the CCM are useless to the qemu framework.
>
> Only clocks related to timers (EPIT, GPT, PWM, WATCHDOG, ...) are usefull
> to QEMU code.
>
> Therefore this patch removes clock computation handling for all clocks
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> This way all CCM clock defines/enums are named CLK_XXX
>
> Signed-off-by: Jean-Christophe Dubois
> ---
>
Reviewed-by: Peter Maydell
thanks
-- PMM
On 8 February 2016 at 22:08, Jean-Christophe Dubois
wrote:
> GPT timer need to rollover when it reaches 0x.
>
> It also need to reset to 0 when in "restart mode" and crossing the
> compare 1 register.
>
> Signed-off-by: Jean-Christophe Dubois
> ---
>
Reviewed-by: Peter Maydell
thanks
On Tue, Feb 16, 2016 at 02:36:49PM +0200, Marcel Apfelbaum wrote:
> >>2. PCI devices with no driver installed are not re-mapped. This can be OK
> >> from the Windows point of view because Resources Window does not show
> >> the MMIO range
> >> for this device.
> >>
> >> If the other (r
We set machine_class to the default first, and update it to the real
one later. Any use of machine_class in between is almost certainly
wrong. Set it once and for all instead.
Signed-off-by: Markus Armbruster
---
vl.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --gi
On Tue, Feb 16, 2016 at 02:51:25PM +0100, Igor Mammedov wrote:
> On Tue, 16 Feb 2016 14:36:49 +0200
> Marcel Apfelbaum wrote:
>
> > On 02/16/2016 02:17 PM, Igor Mammedov wrote:
> > > On Tue, 16 Feb 2016 12:05:33 +0200
> > > Marcel Apfelbaum wrote:
> > >
> > >> On 02/11/2016 06:30 PM, Michael S
On 16 February 2016 at 14:35, Peter Maydell wrote:
> On 1 February 2016 at 20:49, Wei Huang wrote:
>> Current QEMU doesn't clear PL061 state after reset. This causes a
>> weird issue with guest reboot via GPIO. Here is the device state
>> description with two reboot requests:
>
> These reset val
Did you implemented the support for stm32l in qemu?
Hi,
> > When clearing RS in cmd we should also set HALTED in status I think.
> Actually, uhci_frame_timer() had done this work.
>
> if (!(s->cmd & UHCI_CMD_RS)) {
> /* Full stop */
> trace_usb_uhci_schedule_stop();
> qemu_del_timer(s->frame_timer);
> uhci_async_
On 1 February 2016 at 20:49, Wei Huang wrote:
> This patch removes the float_high field of PL061State, which doesn't
> seem to be used anywhere. Because this changes the device state, the
> version ID is also bumped up for the reason of compatiblity.
>
> Signed-off-by: Wei Huang
> ---
> hw/gpio/
On 1 February 2016 at 20:49, Wei Huang wrote:
> Current QEMU doesn't clear PL061 state after reset. This causes a
> weird issue with guest reboot via GPIO. Here is the device state
> description with two reboot requests:
>
> (PL061State fields) data old_in_data istate
> VM boot
> diff --git a/hw/usb/core.c b/hw/usb/core.c
> index d0025db..9d90ec7 100644
> --- a/hw/usb/core.c
> +++ b/hw/usb/core.c
> @@ -128,9 +128,16 @@ static void do_token_setup(USBDevice *s, USBPacket *p)
> }
>
> usb_packet_copy(p, s->setup_buf, p->iov.size);
> +s->setup_index = 0;
>
>
> Hi,
>
> > diff --git a/hw/usb/core.c b/hw/usb/core.c
> > index bea5e1e..6fbcf00 100644
> > --- a/hw/usb/core.c
> > +++ b/hw/usb/core.c
> > @@ -716,7 +716,6 @@ struct USBEndpoint *usb_ep_get(USBDevice *dev,
> int pid, int ep)
> > if (ep == 0) {
> > return &dev->ep_ctl;
> >
On Tue 16 Feb 2016 11:45:32 AM CET, Kevin Wolf wrote:
>> +/* If the bucket is not full yet we have to make sure that we
>> + * fulfill the goal of bkt->max units per second. */
>> +if (bkt->burst_length > 1) {
>> +/* We use 1/10 of the max value to smooth the throttling.
>> +
On Di, 2016-02-16 at 15:34 +0530, P J P wrote:
> When transitioning an OHCI controller to the OHCI_USB_OPERATIONAL
> state, it creates an eof timer object in 'ohci_bus_start'.
> It does not check if one already exists. This results in memory
> leakage and null dereference issue. Add a check to avoi
Hi,
There are several ongoing efforts to implement incremental
backup-related features.
Let's have a voice/video conference to get everyone on the same page,
avoid duplicated work, and get patches merged faster.
Agenda:
* External incremental backup API. Summarize requirements common to
third-
On Tue, Feb 16, 2016 at 03:28:06PM +0200, Victor Kaplansky wrote:
> From: Didier Pallard
>
> Since guest_mask_notifier can not be used in vhost-user mode due
> to buffering implied by unix control socket, force
> use_mask_notifier on virtio devices of vhost-user interfaces, and
> send correct cal
On 10 February 2016 at 13:52, Aaron Lindsay wrote:
> On Feb 09 15:11, Alistair Francis wrote:
>> On Tue, Feb 9, 2016 at 9:19 AM, Peter Maydell
>> wrote:
>> > On 6 February 2016 at 00:55, Alistair Francis
>> > wrote:
>> >> diff --git a/target-arm/cpu.c b/target-arm/cpu.c
>> >> index 7ddbf3d..937
Hi,
> diff --git a/hw/usb/core.c b/hw/usb/core.c
> index bea5e1e..6fbcf00 100644
> --- a/hw/usb/core.c
> +++ b/hw/usb/core.c
> @@ -716,7 +716,6 @@ struct USBEndpoint *usb_ep_get(USBDevice *dev, int pid,
> int ep)
> if (ep == 0) {
> return &dev->ep_ctl;
> }
> -assert(pid =
On Tue, 16 Feb 2016 14:36:49 +0200
Marcel Apfelbaum wrote:
> On 02/16/2016 02:17 PM, Igor Mammedov wrote:
> > On Tue, 16 Feb 2016 12:05:33 +0200
> > Marcel Apfelbaum wrote:
> >
> >> On 02/11/2016 06:30 PM, Michael S. Tsirkin wrote:
> >>> On Thu, Feb 11, 2016 at 04:16:05PM +0100, Igor Mammedo
On 9 February 2016 at 15:24, Peter Maydell wrote:
> This set of patches does a bunch more cleaning of includes.
> Specifically it does everything necessary to be able to
> remove osdep.h includes from header files including qemu-common.h
> Eric Blake (1):
> qapi: Clean up includes in generated
On Do, 2016-02-11 at 16:09 +0100, Tim Sander wrote:
> Probably due to my less then stellar patch to mail handling i think
> this
> patch got forgotten. I think its ok to resend.
Yes, it is ok to avoid it being lost just because the maintainer (/me)
was a bit busy.
I'd strongly suggest to configur
On 02/14/2016 01:46 AM, Fam Zheng wrote:
> On Fri, 02/12 18:06, John Snow wrote:
>> typedef struct CowRequest {
>> @@ -46,6 +43,8 @@ typedef struct BackupBlockJob {
>> CoRwlock flush_rwlock;
>> uint64_t sectors_read;
>> HBitmap *bitmap;
>> +int64_t cluster_size;
>> +int64_
On Do, 2016-02-11 at 16:31 +0530, P J P wrote:
> When processing remote NDIS control message packets, the USB Net
> device emulator checks to see if the USB configuration descriptor
> object is of RNDIS type(2). But it does not check if it is null,
> which leads to a null dereference error. Add che
On Di, 2016-02-09 at 11:59 +0100, Paolo Bonzini wrote:
> Only the echo field is used in the temporary VirtualConsole, so the
> damage was limited. But still, if echo was incorrectly set to true,
> the result would be some puzzling output in VTE monitor and serial
> consoles.
>
Added to ui queue.
On Mi, 2016-02-10 at 17:17 +0100, Paolo Bonzini wrote:
> The "max" value is being compared with >=, but addr + width points to
> the first byte that will _not_ be copied. Laszlo suggested using a
> "greater than" comparison, instead of subtracting one like it is
> already done above for the height
Hi, this patch is a cosmetic rework of two patches originally sent
by Didier Pallard "[PATCH 2/3] virtio-pci: add an option to
bypass guest_notifier_mask" and "[PATCH 3/3] vhost-net: force
guest_notifier_mask bypass in vhost-user case".
The problem the patch solves is described in original posting
From: Didier Pallard
Since guest_mask_notifier can not be used in vhost-user mode due
to buffering implied by unix control socket, force
use_mask_notifier on virtio devices of vhost-user interfaces, and
send correct callfd to the guest at vhost start.
Using guest_notifier_mask function in vhost-
On 9 February 2016 at 18:49, Eric Blake wrote:
> Actively redefining 'inline' is wrong for C++, where gcc has an
> extension 'inline namespace' which fails to compile if the
> keyword 'inline' is replaced by a macro expansion. This will
> matter once we start to include "qemu/osdep.h" first from
Implement a minimal ASPEED AVIC device model, enough to boot a Linux
kernel configured with aspeed_defconfig. The VIC implements the 'new'
register set and expects this to be reflected in the device tree.
The implementation is a little awkward as the hardware uses 32bit
registers to manage 51 IRQs
Implement basic AST2400 timer functionality: Timers can be configured,
enabled, reset and disabled.
A number of hardware features are not implemented:
* Timer Overflow interrupts
* Clock value matching
* Pulse generation
The implementation is enough to boot the Linux kernel configured with
aspee
Hi all,
This is the first time I've implemented QEMU device models so no doubt the
patches contain misunderstandings and/or oversights - happy to hear any
feedback!
This patch series implements enough of the ASPEED AST2400 (ARMv5 SoC) machine
model to boot an aspeed_defconfig Linux kernel[1]. Th
Adds the AST2400 machine type with ASPEED AVIC and timer models. The
new machine type is functional enough to boot Linux to userspace.
Signed-off-by: Andrew Jeffery
---
hw/arm/Makefile.objs | 1 +
hw/arm/ast2400.c | 139 +++
trace-events
On Tue, 2016-02-16 at 12:38 +, Daniel P. Berrange wrote:
> > > Regardless of the way it is exposed in libvirt API, when libvirt probes
> > > for capabilities it will *always* use '-m none'.
> >
> > Domain capabilities are currently derived almost entirely from data
> > taken from virQEMUCaps,
On Tue, Feb 16, 2016 at 01:32:00PM +0100, Andrew Jones wrote:
> On Tue, Feb 16, 2016 at 03:50:53PM +0800, Peter Xu wrote:
> > Instead of malloc/free each time for DumpState, make it
> > static. Added DumpStatus to show status for dump.
>
> I see that the motivation for making DumpState static is f
Andreas Färber writes:
> Am 16.02.2016 um 13:35 schrieb Markus Armbruster:
>> Igor Mammedov writes:
>>
>>> On Mon, 15 Feb 2016 20:43:41 +0100
>>> Markus Armbruster wrote:
>>>
Igor Mammedov writes:
> it will allow mgmt to query present and possible to hotplug CPUs
> it is req
Am 16.02.2016 um 12:20 hat Pavel Dovgalyuk geschrieben:
> > From: Kevin Wolf [mailto:kw...@redhat.com]
> > Am 16.02.2016 um 07:25 hat Pavel Dovgalyuk geschrieben:
> > > > From: Kevin Wolf [mailto:kw...@redhat.com]
> > > > Am 15.02.2016 um 15:24 hat Pavel Dovgalyuk geschrieben:
> > > > > > From: Kev
Markus Armbruster writes:
> Marcel Apfelbaum writes:
>
>> On 02/15/2016 12:20 PM, Markus Armbruster wrote:
>>> Eduardo Habkost writes:
>>>
From: Marcel Apfelbaum
Commit e1ce0c3cb (vl.c: fix regression when reading machine type
from config file) fixed the error message when
From: Alex Bennée
Instead of providing a live version of the source tree to the docker
container we snapshot it with git-archive. This ensure the tree is in a
pristine state for whatever operations the container is going to run on
them.
Uncommitted changes known to files known by the git index w
On Mon, Feb 15, 2016 at 08:00:23PM +0100, Paolo Bonzini wrote:
> This can cause various segmentation faults or aborts in qemu-iotests
> test 091.
>
> Fixes: 5b82b703b69acc67b78b98a5efc897a3912719eb
> Cc: Dave Gilbert
> Cc: Stefan Hajnoczi
> Signed-off-by: Paolo Bonzini
> ---
> migration/ram.c
Specify "PAUSE=1" when invoking make docker-%, and a shell prompt will
show up before the test runs.
Signed-off-by: Fam Zheng
---
tests/docker/Makefile.include | 5 -
tests/docker/run | 8
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/tests/docker/Makef
Signed-off-by: Fam Zheng
Reviewed-by: Alex Bennée
---
MAINTAINERS | 7 +++
1 file changed, 7 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 02710f8..15b8b93 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1584,3 +1584,10 @@ Build system architecture
M: Daniel P. Berrange
S: Od
The configure options are suggested by John Snow .
Signed-off-by: Fam Zheng
---
tests/docker/test-clang | 28
1 file changed, 28 insertions(+)
create mode 100755 tests/docker/test-clang
diff --git a/tests/docker/test-clang b/tests/docker/test-clang
new file mode 10
Docker build requires a "context" directory and we use the
$QEMU_SRC/tests/docker/ directory, and the temoprary dockerfile has to be in
the context.
docker_build normally cleans up this file but let's add an entry here just in
case it fails to.
Signed-off-by: Fam Zheng
---
.gitignore | 1 +
1 f
This builds all available targets.
Signed-off-by: Fam Zheng
---
tests/docker/test-full | 21 +
1 file changed, 21 insertions(+)
create mode 100755 tests/docker/test-full
diff --git a/tests/docker/test-full b/tests/docker/test-full
new file mode 100755
index 000..5708d0b
Signed-off-by: Fam Zheng
---
docs/build-system.txt | 5 +
1 file changed, 5 insertions(+)
diff --git a/docs/build-system.txt b/docs/build-system.txt
index 5ea..2af1e66 100644
--- a/docs/build-system.txt
+++ b/docs/build-system.txt
@@ -438,6 +438,11 @@ top level Makefile, so anything defi
Signed-off-by: Fam Zheng
---
tests/docker/test-quick | 23 +++
1 file changed, 23 insertions(+)
create mode 100755 tests/docker/test-quick
diff --git a/tests/docker/test-quick b/tests/docker/test-quick
new file mode 100755
index 000..ce3cc99
--- /dev/null
+++ b/tests/doc
The script is not named test-travis.sh so it won't run with "make
docker-run", because it can take too long.
Run it with "make docker-run-travis.sh@ubuntu".
Signed-off-by: Fam Zheng
---
tests/docker/travis| 23 +++
tests/docker/travis.py | 48
Signed-off-by: Fam Zheng
---
tests/docker/dockerfiles/centos6.docker | 5 +
tests/docker/dockerfiles/fedora.docker | 7 +++
tests/docker/dockerfiles/ubuntu.docker | 9 +
3 files changed, 21 insertions(+)
create mode 100644 tests/docker/dockerfiles/centos6.docker
create mode 10
Am 16.02.2016 um 13:35 schrieb Markus Armbruster:
> Igor Mammedov writes:
>
>> On Mon, 15 Feb 2016 20:43:41 +0100
>> Markus Armbruster wrote:
>>
>>> Igor Mammedov writes:
>>>
it will allow mgmt to query present and possible to hotplug CPUs
it is required from a target platform that wi
Signed-off-by: Fam Zheng
---
tests/docker/test-mingw | 36
1 file changed, 36 insertions(+)
create mode 100755 tests/docker/test-mingw
diff --git a/tests/docker/test-mingw b/tests/docker/test-mingw
new file mode 100755
index 000..25afa64
--- /dev/null
++
docker_run: A wrapper for "docker run" (or "sudo -n docker run" if
necessary), which takes care of killing and removing the running
container at SIGINT.
docker_clean: A tool to tear down all the containers including inactive
ones that are started by docker_run.
docker_build: A tool to compare an
It's better to have a launcher for all tests, to make it easier to
initialize and manage the environment.
Signed-off-by: Fam Zheng
---
tests/docker/run | 26 ++
1 file changed, 26 insertions(+)
create mode 100755 tests/docker/run
diff --git a/tests/docker/run b/tests/do
"requires" checks the "FEATURE" environment for specified prerequisits,
and skip the execution of test if not found.
Signed-off-by: Fam Zheng
---
tests/docker/common.rc | 29 +
1 file changed, 29 insertions(+)
create mode 100755 tests/docker/common.rc
diff --git a/t
This adds a group of make targets to run docker tests, all are available
in source tree without running ./configure.
The usage is shown by "make docker".
Besides the fixed ones, dynamic targets for building each image and
running each test in each image are generated automatically by make,
scanni
v2: - Fix a few coding style warnings of shellcheck and pylint.
[Marc-André Lureau]
- Rename make targets:
docker-run => docker-test
docker-build => docker-image
docker-run-$TEST.sh@$IMAGE => docker-$TEST@$IMAGE
- In help, document the magic makefile vars which are
On Tue, Feb 16, 2016 at 03:50:58PM +0800, Peter Xu wrote:
> Here, total_size is the size in bytes to be dumped (raw data, which
> means before compression), while written_size are bytes handled (raw
> size too).
>
> Signed-off-by: Peter Xu
> Reviewed-by: Fam Zheng
> ---
> dump.c
On Tue, Feb 16, 2016 at 01:27:55PM +0100, Andrea Bolognani wrote:
> On Tue, 2016-02-16 at 12:15 +, Daniel P. Berrange wrote:
> > On Tue, Feb 16, 2016 at 01:05:45PM +0100, Andrea Bolognani wrote:
> > > On Tue, 2016-02-16 at 10:15 +, Daniel P. Berrange wrote:
> > > > > Back to GIV. Recognize
On 02/16/2016 02:17 PM, Igor Mammedov wrote:
On Tue, 16 Feb 2016 12:05:33 +0200
Marcel Apfelbaum wrote:
On 02/11/2016 06:30 PM, Michael S. Tsirkin wrote:
On Thu, Feb 11, 2016 at 04:16:05PM +0100, Igor Mammedov wrote:
On Tue, 9 Feb 2016 14:17:44 +0200
"Michael S. Tsirkin" wrote:
On Tue, Fe
Igor Mammedov writes:
> On Mon, 15 Feb 2016 20:43:41 +0100
> Markus Armbruster wrote:
>
>> Igor Mammedov writes:
>>
>> > it will allow mgmt to query present and possible to hotplug CPUs
>> > it is required from a target platform that wish to support
>> > command to set board specific MachineCl
On Tue, Feb 16, 2016 at 03:50:53PM +0800, Peter Xu wrote:
> Instead of malloc/free each time for DumpState, make it
> static. Added DumpStatus to show status for dump.
I see that the motivation for making DumpState static is for
dump_in_progress(). DumpState isn't massive, but it isn't tiny
either
On 14.02.2016 18:47, Samuel Thibault wrote:
> From: Guillaume Subiron
>
> This adds the sin6 case in the fhost and lhost unions and related macros.
> It adds udp6_input() and udp6_output().
> It adds the IPv6 case in sorecvfrom().
> Finally, udp_input() is called by ip6_input().
>
> Signed-off-b
On Tue, 2016-02-16 at 12:15 +, Daniel P. Berrange wrote:
> On Tue, Feb 16, 2016 at 01:05:45PM +0100, Andrea Bolognani wrote:
> > On Tue, 2016-02-16 at 10:15 +, Daniel P. Berrange wrote:
> > > > Back to GIV. Recognized values of gic-version are fixed at compile
> > > > time: 2, 3, host. On
On 14.02.2016 18:47, Samuel Thibault wrote:
> From: Yann Bordenave
>
> Disambiguation : icmp_error is renamed into icmp_send_error, since it
> doesn't manage errors, but only sends ICMP Error messages.
>
> Adding icmp6_send_error to send ICMPv6 Error messages. This function is
> simpler than the
201 - 300 of 382 matches
Mail list logo