From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:55929)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <groug@kaod.org>) id 1bkuSF-0003sz-Un
	for qemu-devel@nongnu.org; Fri, 16 Sep 2016 10:49:29 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <groug@kaod.org>) id 1bkuSB-0003E5-Ru
	for qemu-devel@nongnu.org; Fri, 16 Sep 2016 10:49:26 -0400
Received: from 10.mo1.mail-out.ovh.net ([178.32.96.102]:35499)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <groug@kaod.org>) id 1bkuSB-0003DH-LU
	for qemu-devel@nongnu.org; Fri, 16 Sep 2016 10:49:23 -0400
Received: from player770.ha.ovh.net (b7.ovh.net [213.186.33.57])
	by mo1.mail-out.ovh.net (Postfix) with ESMTP id 1AE98FFAB44
	for <qemu-devel@nongnu.org>; Fri, 16 Sep 2016 16:39:59 +0200 (CEST)
Date: Fri, 16 Sep 2016 16:39:55 +0200
From: Greg Kurz <groug@kaod.org>
Message-ID: <20160916163955.3460f28b@bahia>
In-Reply-To: <CAFEAcA8z4vYDWBGxQCUpncwRuDHbmLE22zNgWEcB5gWVW=3P_g@mail.gmail.com>
References: <1474031354-14185-1-git-send-email-groug@kaod.org>
	<CAFEAcA8z4vYDWBGxQCUpncwRuDHbmLE22zNgWEcB5gWVW=3P_g@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PULL 0/8] 9p patches for 2.8 20160916
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>, "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>

On Fri, 16 Sep 2016 15:25:42 +0100
Peter Maydell <peter.maydell@linaro.org> wrote:

> On 16 September 2016 at 14:09, Greg Kurz <groug@kaod.org> wrote:
> > The following changes since commit 5f473241ac595452ae0638dc63e7af2a2294f5ec:
> >
> >   Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2016-09-15 18:12:40 +0100)
> >
> > are available in the git repository at:
> >
> >   https://github.com/gkurz/qemu.git tags/for-upstream
> >
> > for you to fetch changes up to 5c574f1bf92cfaf4aae6cbb66b79066c654920e1:
> >
> >   9pfs: fix potential segfault during walk (2016-09-16 12:16:29 +0200)
> >
> > ----------------------------------------------------------------
> > This pull request contains:
> > - a fix for a regression introduced in 2.7
> > - basic functional testing for virtio-9p
> > - some code cleanups for 9pfs
> >
> > ----------------------------------------------------------------  
> 
> Fails test on ppc64be:
> 
>   /i386/virtio/9p/pci/basic/transaction:                               **
> ERROR:/home/pm215/qemu/tests/virtio-9p-test.c:167:pci_basic_transaction:
> assertion failed (hdr.size
> < (uint32_t) P9_MAX_SIZE): (603979776 < 8192)
> FAIL
> GTester: last random seed: R02S0ce79f32586824bc8cbd5461f009c62a
> (pid=33294)
> 
> Looks like an endianness bug.
> 

Ok, I'll look into it... but the important part in this pull request is
the "9pfs: fix potential segfault during walk" patch. It fixes a regression
introduced in 2.7 by the 9P security fixes. And IIUC, Michael Roth is about
to release 2.6.1.1 with these fixes and the regression...

Should I send a new pull request without the qtest patches ? Or with the
regression fix only ?

Please advise.

Thanks.

--
Greg

> thanks
> -- PMM