e
The environment variable HOST_ARCH allows to override 'uname' to generate
configuration files for a different architecture than the current one.
Signed-off-by: Laurent Vivier
---
v2: replace some ERRORS by WARNINGS to be able to use the script inside a
package build
check onl
Le 28/01/2016 23:29, Eric Blake a écrit :
> On 01/28/2016 03:08 PM, Laurent Vivier wrote:
>
> Subject line is TOOO long. I suggest:
>
> linux-user: Fix qemu-binfmt-conf.h to store config across reboot
OK. I was waiting this comment ;)
>
>> This script can conf
one.
Signed-off-by: Laurent Vivier
---
v3: change subject to be shorter
fix typo
remove "!EOF", "echo -n" and "[ ... -o ... ]"
check cpu given by --systemd is in the list
v2: replace some ERRORS by WARNINGS to be able to use the script inside a
package build
sn't work but 'ip' in a chroot works well.
* arm: Raspbian 8.3 (Jessie) works fine.
* s390x: container Debian 8.1 boots well, but "apt-get" hangs on
networking (name resolution?).
"ip link" generates some traces in the kernel log:
"netlink: 8 bytes l
:16:3e:89:6b:d7 brd ff:ff:ff:ff:ff:ff
inet 192.168.122.10/32 scope global eth0
valid_lft forever preferred_lft forever
# ip route add 192.168.122.0/24 via 192.168.122.10
# ip route
192.168.122.0/24 via 192.168.122.10 dev eth0
Signed-off-by: Laurent Vivier
This is the protocol used by udevd to manage kernel events.
Signed-off-by: Laurent Vivier
---
linux-user/syscall.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index a1ed2f5..790ae49 100644
--- a/linux-user/syscall.c
+++ b/linux-user
This is, for instance, needed to log in a container.
Without this, the user cannot be identified and the console login
fails with "Login incorrect".
Signed-off-by: Laurent Vivier
---
linux-user/syscall.c | 57
1 file changed, 57
Ping ?
Le 19/01/2016 00:08, Laurent Vivier a écrit :
> Synchronize with include/uapi/asm/unistd.h from kernel v4.4
>
> This allows to use timerfd_create().
>
> Signed-off-by: Laurent Vivier
> ---
> linux-user/ppc/syscall_nr.h | 14 +-
> 1 file changed, 13 i
Ping ?
Le 18/01/2016 23:50, Laurent Vivier a écrit :
> target_fd_trans is an array of "TargetFdTrans *": compute size
> accordingly. Use g_renew() as proposed by Paolo.
>
> Reported-by: Paolo Bonzini
> Signed-off-by: Laurent Vivier
> ---
> linux-user/syscall.
FPU);
> -INSN(fbcc, f280, ffc0, CF_FPU);
> +INSN(fbcc, f280, ff80, CF_FPU);
> INSN(frestore, f340, ffc0, CF_FPU);
> INSN(fsave, f340, ffc0, CF_FPU);
> INSN(intouch, f340, ffc0, CF_ISA_A);
>
Reviewed-by: Laurent Vivier
/* Build the opcode table only once to avoid
> + issues with multithreading. */
> +if(opcode_table[0] != NULL)
> + return;
> #define INSN(name, opcode, mask, feature) do { \
> if (m68k_feature(env, M68K_FEATURE_##feature)) \
> register_opcode(disas_##name, 0x##opcode, 0x##mask); \
>
Reviewed-by: Laurent Vivier
Le 03/02/2016 10:40, John Paul Adrian Glaubitz a écrit :
> Strange. There should be a cover letter coming along as well which
> explains my changes. Did you get it?
We have the cover letter, but it is never sent to the sender :)
Laurent
>
> On 02/03/2016 10:37 AM, John Paul Adrian Glaubitz wro
Le 03/02/2016 10:39, Laurent Vivier a écrit :
>
>
> Le 03/02/2016 10:37, John Paul Adrian Glaubitz a écrit :
>> Signed-off-by: John Paul Adrian Glaubitz
>> ---
>> target-m68k/translate.c | 4
>> 1 file changed, 4 insertions(+)
>>
>> diff
Le 03/02/2016 11:06, John Paul Adrian Glaubitz a écrit :
> On 02/03/2016 10:57 AM, Laurent Vivier wrote:
>> In fact, the line should be:
>>
>> +if (opcode_table[0] != NULL)
>> +return;
>>
>> thanks to scripts/checkpatch.pl.
>
> Want me t
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
Le 03/02/2016 16:16, Eric Blake a écrit :
> On 02/03/2016 03:14 AM, John Paul Adrian Glaubitz wrote:
>> Signed-off-by: John Paul Adrian Glaubitz
>> --- target-m68k/translate.c | 4
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/target-
://bugs.debian.org/cgi-bin/bugreport.cgi?bug=813698
Signed-off-by: Laurent Vivier
---
linux-user/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/linux-user/main.c b/linux-user/main.c
index e719a2d..2a692e0 100644
--- a/linux-user/main.c
+++ b/linux-user/main.c
@@ -4160,7 +4160,7
On 09/11/2015 17:58, Peter Maydell wrote:
> On 9 November 2015 at 15:33, Paolo Bonzini wrote:
>> You're right. The two are the same then, preference is of course
>> subjective. Feel free to commit yours!
>
> Applied to master, thanks.
>
> -- PMM
When I run configure now, I've the following
On 12/11/2015 17:29, John Snow wrote:
> I broke this when adding checks for clang++.
>
> Reported-by: Laurent Vivier
> Signed-off-by: John Snow
>
> ---
> configure | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/configure b/conf
io/virtio-blk.h
> +++ b/include/hw/virtio/virtio-blk.h
> @@ -70,6 +70,7 @@ typedef struct VirtIOBlockReq {
> size_t in_len;
> struct VirtIOBlockReq *next;
> struct VirtIOBlockReq *mr_next;
> +bool merged;
> BlockAcctCookie acct;
> } VirtIOBlockReq;
>
>
Tested-by: Laurent Vivier
In my case, this patch fixes a memory corruption when the VM is stopped
on io-error (no space) and restarted.
Fix this by marking request R as "merged" and skipping it in
> virtio_blk_handle_rw_error.
>
> Cc: qemu-sta...@nongnu.org
> Signed-off-by: Fam Zheng
>
> ---
>
> v2: Don't lose the request in migration. [Paolo]
Tested-by: Laurent Vivier
> ---
> hw
or it will be
>> restarted and parsed from scratch out of the vq later.
>>
>> Signed-off-by: Fam Zheng
>>
>> ---
>>
>> v3: Fix as Stefan suggested.
>> ---
>> hw/block/virtio-blk.c | 4
>> 1 file changed, 4 insertions(+)
>
that sigaltstack works unless I am
> overseeing anything? Laurent Vivier (CC'ed) who has done some
> extensive qemu development thinks that my change should be enough.
>
> Here's the output of my test case (CC'ing Michael Karcher who
> suggested the test case):
>
>
On 19/11/2015 12:15, Peter Maydell wrote:
> On 19 November 2015 at 10:22, Laurent Vivier wrote:
>> Hi,
>>
>> On 19/11/2015 10:28, John Paul Adrian Glaubitz wrote:
>>> On 11/19/2015 10:17 AM, Peter Maydell wrote:
>>>> Unfortunately this isn't suffic
Le 06/10/2015 10:16, Riku Voipio a écrit :
> On perjantaina 2. lokakuuta 2015 15.48.09 EEST, Laurent Vivier wrote:
>> This patch introduces a system very similar to the one used in the kernel
>> to attach specific functions to a given file descriptor.
>
> Thanks, applie
ff-by: Laurent Vivier
---
linux-user/signal.c | 16 ++--
1 file changed, 6 insertions(+), 10 deletions(-)
diff --git a/linux-user/signal.c b/linux-user/signal.c
index 55e5405..5e8f6d8 100644
--- a/linux-user/signal.c
+++ b/linux-user/signal.c
@@ -3215,7 +3215,6 @@ static void setup_
t that obvious.
>
> Signed-off-by: Greg Kurz
Reviewed-by: Laurent Vivier
> ---
> hw/virtio/vhost.c |9 +++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
> index bb17177f5e7e..9f8ac38ccf2d 100644
&g
ed-by: Cornelia Huck
> Signed-off-by: Greg Kurz
Reviewed-by: Laurent Vivier
> ---
> hw/virtio/vhost.c |9 +
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
> index 6fea07c8ae15..bb17177f5e7e 100644
> --- a
emulated and
> vhost accelerated devices
> - optimize virtio_access_is_big_endian() for little-endian targets
> - various cleanups
>
> This v4 is a respin of the v3, taking into account the following fix:
>
> commit cf0a628f6e81bfc9b7a944fa0b80c3594836df56
> Author: Laurent Vivier
getrandom() has been introduced in kernel 3.17 and is now used during
the boot sequence of Debian unstable (stretch/sid).
Signed-off-by: Laurent Vivier
---
linux-user/syscall.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index
Le 30/01/2016 23:26, Laurent Vivier a écrit :
> Since commit:
> e36800c linux-user: add signalfd/signalfd4 syscalls
>
> It is now possible to register handlers to a file descriptor
> to translate a data stream transiting by this file descriptor.
>
> We can now decode netli
Hi,
I'm playing with a qemu-2.5.0 and pc-i440fx-2.4 machine type, and
perhaps I don't understand correctly the compat_props machinery but
there is something strange for me:
in qemu-2.5.0, hw/virtio/virtio-pci.c:
1880 DEFINE_PROP_BIT("x-disable-pcie", VirtIOPCIProxy, flags,
1881
On 10/02/2016 10:52, Marcel Apfelbaum wrote:
> On 02/09/2016 07:38 PM, Laurent Vivier wrote:
>> Hi,
>>
>> I'm playing with a qemu-2.5.0 and pc-i440fx-2.4 machine type, and
>> perhaps I don't understand correctly the compat_props machinery but
>> there
emulated and
> vhost accelerated devices
> - optimize virtio_access_is_big_endian() for little-endian targets
> - various cleanups
>
> This v4 is a respin of the v3, taking into account the following fix:
>
> commit cf0a628f6e81bfc9b7a944fa0b80c3594836df56
> Author: Laurent Vivier
On 10/02/2016 13:22, Marcel Apfelbaum wrote:
> Commits 1811e64c and a6df8adf use the same virtio feature bit 4
> for different features.
>
> Fix it by using different bits.
>
> Reported-by: Laurent Vivier
> Signed-off-by: Marcel Apfelbaum
> ---
> hw/virtio/vi
On 10/02/2016 13:22, Marcel Apfelbaum wrote:
> Minimizes the possibility to assign
> the same bit to different features.
>
> Signed-off-by: Marcel Apfelbaum
> ---
> hw/virtio/virtio-pci.h | 16 ++--
> 1 file changed, 10 insertions(+), 6 deletions(-)
>
> diff --git a/hw/virtio/virt
On 10/02/2016 13:22, Marcel Apfelbaum wrote:
> Commits 1811e64c and a6df8adf use the same virtio feature bit 4
> for different features.
>
> Fix it by using different bits.
>
> Reported-by: Laurent Vivier
> Signed-off-by: Marcel Apfelbaum
> ---
> hw/virtio/vi
On 10/02/2016 14:31, Marcel Apfelbaum wrote:
> Minimizes the possibility to assign
> the same bit to different features.
>
> Signed-off-by: Marcel Apfelbaum
Reviewed-by: Laurent Vivier
Le 09/02/2016 16:57, Peter Maydell a écrit :
> Our implementation of shmat() and shmdt() for linux-user was
> using "zero guest address" as its marker for "entry in the
> shm_regions[] array is not in use". This meant that if the
> guest did a shmdt(0) we would match on an unused array entry
Is
ser/main.c
> +++ b/linux-user/main.c
> @@ -22,6 +22,7 @@
> #include
> #include
> #include
> +#include
This line needs to be rebased.
Otherwise:
Tested-by: Laurent Vivier
Reviewed-by: Laurent Vivier
> #include
> #include
> #include
> @@ -79,6 +80,7 @@ stat
d match on an unused array entry
> and call page_set_flags() with both start and end addresses zero,
> which causes an assertion failure.
>
> Use an explicit in_use flag to manage the shm_regions[] array,
> so that we avoid this problem.
>
> Signed-off-by: Peter Maydell
>
On 15/02/2016 11:15, Greg Kurz wrote:
> Since QEMU 2.4, the migration stream begins with a configuration section.
> It is known to break migration of pseries machine from older QEMU. It is
> possible to fix this in the pseries compat code but it will then break
> migration of old pseries from lat
Le 15/02/2016 13:46, Riku Voipio a écrit :
> On tiistaina 9. helmikuuta 2016 1.49.20 EET, Laurent Vivier wrote:
>> From kernel v4.4
>
> Thanks. However, while checking throught the syscalls, I'm now opting
> to update syscall numbers for all archs at once.
OK. No probl
Le 15/02/2016 15:00, riku.voi...@linaro.org a écrit :
> From: Riku Voipio
>
> Sync syscall numbers to match the linux v4.5-rc1 kernel.
>
> Cc: Laurent Vivier
> Signed-off-by: Riku Voipio
Reviewed-by: Laurent Vivier
> ---
> linux-user/aarch64/syscall_nr.h|
it appears
> kernel developers reused timerfd syscall number with timerfd_create,
> presumably since no userspace called the old syscall number.
>
> Reportd-by: Laurent Vivier
> Signed-off-by: Riku Voipio
Reviewed-by: Laurent Vivier
> ---
> linux-user/i386/syscall_nr.h | 2
On 15/02/2016 13:58, Greg Kurz wrote:
> On Mon, 15 Feb 2016 12:23:39 +0100
> Laurent Vivier wrote:
>
>> On 15/02/2016 11:15, Greg Kurz wrote:
>>> Since QEMU 2.4, the migration stream begins with a configuration section.
>>> It is known to break migration of ps
Ping?
Le 29/01/2016 17:07, Laurent Vivier a écrit :
> Original qemu-binfmt-conf.h is only able to write configuration
> into /proc/sys/fs/binfmt_misc, and the configuration is lost on reboot.
>
> This script can configure debian and systemd services to restore
> configuration on r
On 16/02/2016 10:09, Greg Kurz wrote:
> On Mon, 15 Feb 2016 15:49:17 +0100
> Laurent Vivier wrote:
>
>> On 15/02/2016 13:58, Greg Kurz wrote:
>>> On Mon, 15 Feb 2016 12:23:39 +0100
>>> Laurent Vivier wrote:
>>>
>>>> On 15/02/20
e and null dereference issue. Add a check to avoid it.
>
> Reported-by: Zuozhi Fzz
> Signed-off-by: Prasad J Pandit
Reviewed-by: Laurent Vivier
> ---
> hw/usb/hcd-ohci.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/hw/usb/hcd-ohci.c
patibility properties which are needed for 2.5 and
> earlier.
>
> This corrects the oversight.
>
> Reported-by: Laszlo Ersek
> Signed-off-by: David Gibson
Reviewed-by: Laurent Vivier
> ---
> hw/ppc/spapr.c | 1 +
> 1 file changed, 1 insertion(+)
>
>
Le 09/12/2015 21:54, Michael Karcher a écrit :
> do_rt_sigreturn forgets to initialize the signal mask variable before
> trying to use it to restore the mask, so the signal mask is undefined
> after do_rt_sigreturn. This bug has been in all the time since
> 7181155d when do_rt_sigreturn was imple
Le 12/12/2015 10:55, Michael Karcher a écrit :
> On 09.12.2015 23:03, Laurent Vivier wrote:
>>
>> Le 09/12/2015 21:54, Michael Karcher a écrit :
>>> do_rt_sigreturn forgets to initialize the signal mask variable before
>>> trying to use it to restore the mask
gt; goto badframe;
>
> -target_to_host_sigset_internal(&set, &target_set);
> +target_to_host_sigset(&set, &frame->uc.tuc_sigmask);
> do_sigprocmask(SIG_SETMASK, &set, NULL);
>
> /* restore registers */
>
Reviewed-by: Laurent Vivier
Tested-by: Laurent Vivier
qemu_log("\nNR\n");
> +qemu_log_mask(CPU_LOG_INT, "\nNR\n");
> break;
> default:
> -qemu_log("\nqemu: unhandled CPU exception %#x - aborting\n",
> +EXCP_DUMP(env, "\nqemu: unhandled CPU exception %#x -
> aborting\n",
> trapnr);
> -cpu_dump_state(cs, stderr, fprintf, 0);
> gdbsig = TARGET_SIGILL;
> break;
> }
> @@ -3047,9 +3040,7 @@ void cpu_loop(CPUM68KState *env)
> }
> break;
> default:
> -fprintf(stderr, "qemu: unhandled CPU exception 0x%x -
> aborting\n",
> -trapnr);
> -cpu_dump_state(cs, stderr, fprintf, 0);
> +EXCP_DUMP(env, "qemu: unhandled CPU exception 0x%x -
> aborting\n", trapnr);
> abort();
> }
> process_pending_signals(env);
>
Reviewed-by: Laurent Vivier
; int i, err = 0;
>
> frame_addr = get_sigframe(ka, env, sizeof(*frame));
> +trace_user_setup_rt_frame(env, frame_addr);
> if (!lock_user_struct(VERIFY_WRITE, frame, frame_addr, 0)) {
> goto give_sigsegv;
> }
> @@ -5515,6 +5512,7 @@ long do_rt_sigreturn(CPUAlphaState *env)
> struct target_rt_sigframe *frame;
> sigset_t set;
>
> +trace_user_do_rt_sigreturn(env, frame_addr);
> if (!lock_user_struct(VERIFY_READ, frame, frame_addr, 1)) {
> goto badframe;
> }
> @@ -5622,6 +5620,7 @@ static void setup_rt_frame(int sig, struct
> target_sigaction *ka,
> unsigned long restorer;
>
> frame_addr = get_sigframe(ka, env, sizeof(*frame));
> +trace_user_setup_rt_frame(env, frame_addr);
> if (!lock_user_struct(VERIFY_WRITE, frame, frame_addr, 0)) {
> goto give_sigsegv;
> }
> @@ -5672,6 +5671,7 @@ long do_rt_sigreturn(CPUTLGState *env)
> struct target_rt_sigframe *frame;
> sigset_t set;
>
> +trace_user_do_rt_sigreturn(env, frame_addr);
> if (!lock_user_struct(VERIFY_READ, frame, frame_addr, 1)) {
> goto badframe;
> }
> @@ -5750,9 +5750,7 @@ void process_pending_signals(CPUArchState *cpu_env)
> return;
>
> handle_signal:
> -#ifdef DEBUG_SIGNAL
> -fprintf(stderr, "qemu: process signal %d\n", sig);
> -#endif
> +trace_user_handle_signal(cpu_env, sig);
> /* dequeue signal */
> q = k->first;
> k->first = q->next;
> diff --git a/trace-events b/trace-events
> index ee890c1..7d02e08 100644
> --- a/trace-events
> +++ b/trace-events
> @@ -1795,3 +1795,14 @@ qcrypto_tls_session_new(void *session, void *creds,
> const char *hostname, const
>
> # net/vhost-user.c
> vhost_user_event(const char *chr, int event) "chr: %s got event: %d"
> +
> +# linux-user/signal.c
> +user_setup_frame(void *env, uint64_t frame_addr) "env=%p frame_addr="PRIx64""
> +user_setup_rt_frame(void *env, uint64_t frame_addr) "env=%p
> frame_addr="PRIx64""
> +user_do_rt_sigreturn(void *env, uint64_t frame_addr) "env=%p
> frame_addr="PRIx64""
> +user_do_sigreturn(void *env, uint64_t frame_addr) "env=%p
> frame_addr="PRIx64""
> +user_force_sig(void *env, int target_sig, int host_sig) "env=%p signal %d
> (host %d)"
> +user_handle_signal(void *env, int target_sig) "env=%p signal %d"
> +user_host_signal(void *env, int host_sig, int target_sig) "env=%p signal %d
> (target %d("
> +user_queue_signal(void *env, int target_sig) "env=%p signal %d"
> +user_s390x_restore_sigregs(void *env, uint64_t sc_psw_addr, uint64_t
> env_psw_addr) "env=%p frame psw.addr "PRIx64 " current psw.addr "PRIx64""
>
Reviewed-by: Laurent Vivier
On 14/12/2015 09:41, Alvise Rigo wrote:
> Use the new slow path for atomic instruction translation when the
> softmmu is enabled.
>
> Suggested-by: Jani Kokkonen
> Suggested-by: Claudio Fontana
> Signed-off-by: Alvise Rigo
> ---
> configure | 4
> 1 file changed, 4 insertions(+)
>
> di
On 13/11/2015 12:23, Mark Cave-Ayland wrote:
> On 13/11/15 09:45, Hervé Poussineau wrote:
>
>> Le 13/11/2015 05:09, Programmingkid a écrit :
>>>
>>> On Nov 12, 2015, at 11:04 PM, qemu-ppc-requ...@nongnu.org wrote:
>>>
Message: 3
Date: Thu, 12 Nov 2015 22:24:08 +0100
From: Herv? Po
On 15/12/2015 15:31, Paolo Bonzini wrote:
>
>
> On 15/12/2015 15:23, alvise rigo wrote:
>> Hi,
>>
>> On Mon, Dec 14, 2015 at 11:14 AM, Laurent Vivier wrote:
>>>
>>>
>>> On 14/12/2015 09:41, Alvise Rigo wrote:
>>>> Use the new
w not in OHCI_RH_RUNNING state.
Signed-off-by: Laurent Vivier
---
hw/usb/hcd-ohci.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/usb/hcd-ohci.c b/hw/usb/hcd-ohci.c
index 5f15ebb..b5a4e39 100644
--- a/hw/usb/hcd-ohci.c
+++ b/hw/usb/hcd-ohci.c
@@ -1438,6 +1438,9 @@ static void ohci_se
to send IRQs. As
the driver is not in OHCI_STATE_RUNNING and not protected against IRQ,
the ohci_irq() can be called and the driver never moved to
OHCI_STATE_RUNNING.
Suggested-by: Gerd Hoffmann
Signed-off-by: Laurent Vivier
---
hw/usb/hcd-ohci.c | 10 ++
1 file changed, 6
overcommitted
CPU, the vCPU becomes slow enough and it happens.
This series fixes a kernel crash on boot (CPU stuck) when the OHCI driver
tries to resume or suspend the device.
Laurent Vivier (2):
ohci: delay first SOF interrupt
ohci: clear pending SOF on suspend
hw/usb/hcd-ohci.c | 13
On 08/01/2016 15:19, Greg Kurz wrote:
> On Thu, 7 Jan 2016 19:32:37 +0100
> Laurent Vivier wrote:
>>
>
> Sorry for the late answer to this one, I got diverted :)
>
>>
>> On 07/01/2016 12:32, Greg Kurz wrote:
>>> When running a fully emulated device i
Michael Tokarev has already enable sigaltstack() for all architecture:
https://patchwork.ozlabs.org/patch/561514/
It is in the "-for-upstream" tree of Riku.
Laurent
Le 10/01/2016 06:56, cheng...@emindsoft.com.cn a écrit :
> From: Chen Gang
>
> Simply enable it, and notice about 80 columns cod
addr_ifname, optlen));
> unlock_user (dev_ifname, optval_addr, 0);
> return ret;
> }
>
Reviewed-by: Laurent Vivier
Le 11/01/2016 07:47, cheng...@emindsoft.com.cn a écrit :
> From: Chen Gang
>
> Just implement it according to the other features implementations.
>
> Signed-off-by: Chen Gang
> ---
> linux-user/syscall.c | 18 +-
> linux-user/syscall_defs.h | 5 +
> 2 files changed,
Le 11/01/2016 09:54, cheng...@emindsoft.com.cn a écrit :
> From: Chen Gang
>
> Implement them according to the other features implementations.
>
> Signed-off-by: Chen Gang
> ---
> linux-user/syscall.c | 27 +--
> 1 file changed, 25 insertions(+), 2 deletions(-)
>
> d
Le 11/01/2016 14:46, Riku Voipio a écrit :
> Hi,
>
> On torstaina 24. joulukuuta 2015 21.04.38 EET, John Paul Adrian Glaubitz
> wrote:
>> Here are, as discussed previously, my proposed patches which change the
>> following:
>>
>> PATCH 1/2 updates the syscall table for m68k to match Linux 4.4. T
On 11/01/2016 17:13, Greg Kurz wrote:
> Indeed vhost doesn't need to ask for vring endian fixing if the device is
> virtio 1.0, since it is already handled by the in-kernel vhost driver. This
> patch simply consolidates the logic into the existing helper.
>
> Signed-off-by: Greg Kurz
> Reviewed
rtio_access_is_big_endian(vdev) ? false : true;
> -#else
> -return virtio_access_is_big_endian(vdev) ? true : false;
> -#endif
> -}
> -
> static inline uint16_t virtio_tswap16(VirtIODevice *vdev, uint16_t s)
> {
> #ifdef HOST_WORDS_BIGENDIAN
> diff --git a/include/hw/virtio/virtio-net.h b/include/hw/virtio/virtio-net.h
> index f3cc25feca2b..27bc868fbc7d 100644
> --- a/include/hw/virtio/virtio-net.h
> +++ b/include/hw/virtio/virtio-net.h
> @@ -94,6 +94,7 @@ typedef struct VirtIONet {
> uint64_t curr_guest_offloads;
> QEMUTimer *announce_timer;
> int announce_counter;
> +bool needs_vnet_hdr_swap;
> } VirtIONet;
>
> void virtio_net_set_netclient_name(VirtIONet *n, const char *name,
>
>
Reviewed-by: Laurent Vivier
vhost_net_set_vnet_endian(dev, ncs[0].peer, false);
> err:
> return r;
> }
> @@ -366,8 +337,6 @@ void vhost_net_stop(VirtIODevice *dev, NetClientState
> *ncs,
> fflush(stderr);
> }
> assert(r >= 0);
> -
> -assert(vhost_net_set_vnet_endian(dev, ncs[0].peer, false) >= 0);
> }
>
> void vhost_net_cleanup(struct vhost_net *net)
>
>
Reviewed-by: Laurent Vivier
l
> virtio_access_is_big_endian(VirtIODevice *vdev)
> #endif
> }
>
> -static inline bool virtio_legacy_is_cross_endian(VirtIODevice *vdev)
> -{
> -#ifdef TARGET_IS_BIENDIAN
> -#ifdef HOST_WORDS_BIGENDIAN
> -return !virtio_is_big_endian(vdev);
> -#else
> - return virtio_is_big_endian(vdev);
> -#endif
> -#else
> -return false;
> -#endif
> -}
> -
> static inline uint16_t virtio_lduw_phys(VirtIODevice *vdev, hwaddr pa)
> {
> if (virtio_access_is_big_endian(vdev)) {
>
>
Reviewed-by: Laurent Vivier
o 1.0,
> since the result is little-endian in all cases.
>
> Signed-off-by: Greg Kurz
> ---
> v2:
> - simplified the logic as suggested by Laurent Vivier
> ---
> include/hw/virtio/virtio-access.h |6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
On 11/01/2016 17:12, Greg Kurz wrote:
> When running a fully emulated device in cross-endian conditions, including
> a virtio 1.0 device offered to a big endian guest, we need to fix the vnet
> headers. This is currently handled by the virtio_net_hdr_swap() function
> in the core virtio-net code
On 13/01/2016 10:06, Greg Kurz wrote:
> On Tue, 12 Jan 2016 18:24:28 +0100
> Laurent Vivier wrote:
>
>> On 11/01/2016 17:12, Greg Kurz wrote:
>>> When running a fully emulated device in cross-endian conditions, including
>>> a virtio 1.0 device offered to a big
of the interface.
Signed-off-by: Laurent Vivier
---
hw/net/vhost_net.c | 23 +++
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
index 318c3e6..10e233a 100644
--- a/hw/net/vhost_net.c
+++ b/hw/net/vhost_net.c
@@ -300,21
Le 14/01/2016 07:24, cheng...@emindsoft.com.cn a écrit :
> From: Chen Gang
>
> After host_to_target_sock_type(), the length of val may be changed, so
> calculate the related lv, too.
>
> Signed-off-by: Chen Gang
> ---
> linux-user/syscall.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --
Le 14/01/2016 10:01, Chen Gang a écrit :
> On 2016年01月14日 16:15, Laurent Vivier wrote:
>> Le 14/01/2016 07:24, cheng...@emindsoft.com.cn a écrit :
>>> From: Chen Gang
>>>
>>> After host_to_target_sock_type(), the length of val may be changed, so
>>>
Le 14/01/2016 10:37, Chen Gang a écrit :
>
> On 2016年01月14日 17:10, Laurent Vivier wrote:
>>
>> Le 14/01/2016 10:01, Chen Gang a écrit :
>>>
>>> I am not quite sure whether kernel always returns sizeof(int) (I guess,
>>> it should be).
>>
&
Le 18/01/2016 11:30, Paolo Bonzini a écrit :
>
>
> On 02/10/2015 14:48, Laurent Vivier wrote:
>> +target_fd_trans = g_realloc(target_fd_trans,
>> +target_fd_max * sizeof(TargetFdTrans));
>
> This should be TargetFdTr
target_fd_trans is an array of "TargetFdTrans *": compute size
accordingly. Use g_renew() as proposed by Paolo.
Reported-by: Paolo Bonzini
Signed-off-by: Laurent Vivier
---
linux-user/syscall.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/linux-user/syscall
Synchronize with include/uapi/asm/unistd.h from kernel v4.4
This allows to use timerfd_create().
Signed-off-by: Laurent Vivier
---
linux-user/ppc/syscall_nr.h | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/linux-user/ppc/syscall_nr.h b/linux-user/ppc
Hi Petros,
Le 18/01/2016 05:33, Petros Angelatos a écrit :
> From: Petros Angelatos
>
> In order for one to use QEMU user mode emulation under a chroot, it is
> required to use binfmt_misc. This can be avoided by QEMU never doing a
> raw execve() to the host system.
Are there some reasons to no
ping
[added Jason in cc:]
On 13/01/2016 20:26, Laurent Vivier wrote:
> commit 5be7d9f1b1452613b95c6ba70b8d7ad3d0797991
>vhost-net: tell tap backend about the vnet endianness
>
> makes vhost net to set the endianness of the device, but only for
> the first device.
Le 18/01/2016 05:33, Petros Angelatos a écrit :
> From: Petros Angelatos
>
> In order for one to use QEMU user mode emulation under a chroot, it is
> required to use binfmt_misc. This can be avoided by QEMU never doing a
> raw execve() to the host system.
>
> Introduce a new option, -execve=pa
arg3, 1))
> goto efault;
> -fl.l_type = tswap16(target_fl->l_type);
> +fl.l_type =
> target_to_host_bitmask(tswap16(target_fl->l_type),
> + flock_tbl);
> fl.l_whence = tswap16(target_fl->l_whence);
> fl.l_start = tswap64(target_fl->l_start);
> fl.l_len = tswap64(target_fl->l_len);
>
This patch looks good to me, except that script/checkpatch.pl complains
about "DOS line ending" and "line over 80 characters".
Reviewed-by: Laurent Vivier
Le 18/12/2015 07:51, Chen Gang a écrit :
>
> I found this issue during my working time, it is about sw_64 (almost the
> same as alpha) host running i386 wine programs.
>
> I also found another issue, but I am not quite sure whether it is worth
> enough for our upstream: The related fix patch is
Le 29/10/2015 00:12, Peter Maydell a écrit :
> On 28 October 2015 at 20:40, Laurent Vivier wrote:
>> This is obsolete, but if we want to use dhcp with some distros (like debian
>> ppc 8.2 jessie), we need it.
>>
>> bind() uses an interface name instead an interface i
Le 06/10/2015 10:16, Riku Voipio a écrit :
> On perjantaina 2. lokakuuta 2015 15.48.09 EEST, Laurent Vivier wrote:
>> This patch introduces a system very similar to the one used in the kernel
>> to attach specific functions to a given file descriptor.
>
> Thanks, applie
Ping ?
Le 23/11/2015 11:38, Laurent Vivier a écrit :
> To return from a signal, setup_frame() puts an instruction to
> be executed in the stack. This sequence calls the syscall sigreturn().
>
> The address of the instruction must be set in the PR register
> to be executed.
>
Le 18/12/2015 22:40, Chen Gang a écrit :
>
> On 12/18/15 17:37, Laurent Vivier wrote:
>>
>> Le 18/12/2015 07:26, Chen Gang a écrit :
>>>
>>> For fcntl, it always needs to notice about it, just like do_fcntl() has
>>> done, or it will cause issue (e.
Le 18/12/2015 23:12, Chen Gang a écrit :
>
> On 12/19/15 05:58, Laurent Vivier wrote:
>>
>> Le 18/12/2015 22:40, Chen Gang a écrit :
>>>
>
> [...]
>
>>> I did not get any script/checkpatch.pl complains, originally.
>>>
>>> Does m
Thank you for the 611 MB tar
The behavior is a little bit different on my system:
root@Quad:~# ls
ghc-7.8.4 ghc_7.8.4-9~bpo8+1.dsc
ghc_7.8.4-9~bpo8+1.debian.tar.xz ghc_7.8.4.orig.tar.xz
root@Quad:~# cd ghc-7.8.4/utils/ghc-p
ghc-pkg/ ghc-pwd/
root@Quad:~# cd ghc-7.8.4
Le 21/12/2015 03:33, cheng...@emindsoft.com.cn a écrit :
> From: Chen Gang
>
> When mapping MAP_ANONYMOUS memory fragments, still need notice about to
> set it zero, or it will cause issues.
Perhaps you can explain in the commit message why this page is not
already filled by zeros ?
> Signed-
Le 21/12/2015 16:14, Riku Voipio a écrit :
> On Fri, Dec 18, 2015 at 04:13:20PM +0100, Laurent Vivier wrote:
>> Le 29/10/2015 00:12, Peter Maydell a écrit :
>>> On 28 October 2015 at 20:40, Laurent Vivier wrote:
>>>> This is obsolete, but if we want to use dhcp
Le 21/12/2015 16:14, Riku Voipio a écrit :
> On Fri, Dec 18, 2015 at 04:13:20PM +0100, Laurent Vivier wrote:
>> Le 29/10/2015 00:12, Peter Maydell a écrit :
>>> On 28 October 2015 at 20:40, Laurent Vivier wrote:
>>>> This is obsolete, but if we want to use dhcp
thus this is not a fragment).
It is consistent with what it is done in the case of one single page.
Reviewed-by: Laurent Vivier
372
> +#define TARGET_NR_userfaultfd 373
> +#define TARGET_NR_membarrier374
> +#define TARGET_NR_mlock2375
>
Reviewed-by: Laurent Vivier
SOCKOP_accept4 18
> +#define SOCKOP_recvmmsg 19
> +#define SOCKOP_sendmmsg 20
>
> #define IPCOP_semop 1
> #define IPCOP_semget 2
>
Reviewed-by: Laurent Vivier
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
Le 24/12/2015 20:04, John Paul Adrian Glaubitz a écrit :
> Hi Laurent!
>
> Here are, as discussed previously, my proposed patches which change
> the following:
>
> PATCH 1/2 updates the syscall table for m68k to match Linux 4.4.
> This one should
if (prot_new & PROT_WRITE) {
> +memset(g2h(start), 0, end - start);
> +}
> }
> return 0;
> }
>
Reviewed-by: Laurent Vivier
On 05/01/2016 14:20, Andrew Jones wrote:
> Fixes compiling with --enable-trace-backends
>
> Signed-off-by: Andrew Jones
> ---
> trace-events | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/trace-events b/trace-events
> index 6f036384a84f8..e42898d5659d5 100
201 - 300 of 10673 matches
Mail list logo
