From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:34208)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <laurent@vivier.eu>) id 1bAZrq-0001Zw-W2
	for qemu-devel@nongnu.org; Wed, 08 Jun 2016 05:33:43 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <laurent@vivier.eu>) id 1bAZrl-0005bx-N6
	for qemu-devel@nongnu.org; Wed, 08 Jun 2016 05:33:40 -0400
Received: from mout.kundenserver.de ([217.72.192.74]:63371)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <laurent@vivier.eu>) id 1bAZrl-0005br-Co
	for qemu-devel@nongnu.org; Wed, 08 Jun 2016 05:33:37 -0400
References: <1465239499-5048-1-git-send-email-peter.maydell@linaro.org>
	<1465239499-5048-15-git-send-email-peter.maydell@linaro.org>
	<8e7c37a6-2df3-0c1c-e52c-07c2676195ea@vivier.eu>
	<CAFEAcA8sR=B9pmhDfJygK6ob59pBk+Hfu0409ZLJp2pDW79+LQ@mail.gmail.com>
From: Laurent Vivier <laurent@vivier.eu>
Message-ID: <6482e445-af59-8973-a1ea-970d3160ccdf@vivier.eu>
Date: Wed, 8 Jun 2016 11:33:26 +0200
MIME-Version: 1.0
In-Reply-To: <CAFEAcA8sR=B9pmhDfJygK6ob59pBk+Hfu0409ZLJp2pDW79+LQ@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
Subject: Re: [Qemu-devel] [PATCH 14/18] linux-user: Use __get_user() and
 __put_user() to handle structs in do_fcntl()
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: QEMU Developers <qemu-devel@nongnu.org>, Patch Tracking <patches@linaro.org>, Riku Voipio <riku.voipio@iki.fi>



Le 07/06/2016 à 23:20, Peter Maydell a écrit :
> On 7 June 2016 at 21:41, Laurent Vivier <laurent@vivier.eu> wrote:
>>> -        fl64.l_type =
>>> -           target_to_host_bitmask(tswap16(target_fl64->l_type), flock_tbl) >> 1;
>>
>> The ">> 1" disappears...
> 
> ...and it's correct that it disappears, right?

Yes. I forgot to remove this when I have understood what was the reason
of the shift.

Thanks,
Laurent