From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:53575) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UT8KW-0001NV-8H for qemu-devel@nongnu.org; Fri, 19 Apr 2013 06:14:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UT8KT-0001G2-3x for qemu-devel@nongnu.org; Fri, 19 Apr 2013 06:14:08 -0400 Received: from mail-qc0-x22a.google.com ([2607:f8b0:400d:c01::22a]:40438) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UT8HM-0000S2-FB for qemu-devel@nongnu.org; Fri, 19 Apr 2013 06:10:52 -0400 Received: by mail-qc0-f170.google.com with SMTP id d42so1767713qca.29 for ; Fri, 19 Apr 2013 03:10:51 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20130419094210.GF30143@stefanha-thinkpad.redhat.com> References: <1363888656-123910-1-git-send-email-petar.jovanovic@rt-rk.com> <20130419094210.GF30143@stefanha-thinkpad.redhat.com> Date: Fri, 19 Apr 2013 13:10:51 +0300 Message-ID: From: Riku Voipio Content-Type: multipart/alternative; boundary=90e6ba30927e0f533b04dab3efe1 Subject: Re: [Qemu-devel] [PATCH] linux-user: change do_semop to return target errno when unsuccessful List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: petar.jovanovic@imgtec.com, Petar Jovanovic , qemu-devel qemu-devel --90e6ba30927e0f533b04dab3efe1 Content-Type: text/plain; charset=ISO-8859-1 On 19 April 2013 12:42, Stefan Hajnoczi wrote: > On Thu, Mar 21, 2013 at 06:57:36PM +0100, Petar Jovanovic wrote: > > From: Petar Jovanovic > > > > do_semop() is called from two places, and one of these fails to convert > > return error to target errno when semop fails. This patch changes the > > function to always return target errno in case of an unsuccessful call. > > > > Signed-off-by: Petar Jovanovic > > --- > > linux-user/syscall.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > Thanks, applied to the trivial patches tree: > https://github.com/stefanha/qemu/commits/trivial-patches > > Hi, It's also in my linux-user updates set I'm testing now: https://git.linaro.org/gitweb?p=people/rikuvoipio/qemu.git;a=shortlog;h=refs/heads/linux-user-for-upstream --90e6ba30927e0f533b04dab3efe1 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
On 19 April 2013 12:42, Stefan Hajnoczi = <stefanha@gmail.= com> wrote:
On Thu, Mar 21, 2013 at 06:57:36PM +0100, Petar Jovanovic= wrote:
> From: Petar Jovanovic <petar.jovanovic@imgtec.com>
>
> do_semop() is called from two places, and one of these fails to conver= t
> return error to target errno when semop fails. This patch changes the<= br> > function to always return target errno in case of an unsuccessful call= .
>
> Signed-off-by: Petar Jovanovic <petar.jovanovic@imgtec.com>
> ---
> =A0linux-user/syscall.c | =A0 =A04 ++--
> =A01 file changed, 2 insertions(+), 2 deletions(-)

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches
Hi,

It's also in my linux-user updates set I'm = testing now:
--90e6ba30927e0f533b04dab3efe1--