From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=48366 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Q75Ma-0007i3-Um for qemu-devel@nongnu.org; Tue, 05 Apr 2011 08:28:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Q75MU-0002dq-PF for qemu-devel@nongnu.org; Tue, 05 Apr 2011 08:28:00 -0400 Received: from mail-gx0-f173.google.com ([209.85.161.173]:62795) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Q75MU-0002da-Lg for qemu-devel@nongnu.org; Tue, 05 Apr 2011 08:27:58 -0400 Received: by gxk26 with SMTP id 26so125549gxk.4 for ; Tue, 05 Apr 2011 05:27:58 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <1301999735-24141-1-git-send-email-harsh@linux.vnet.ibm.com> References: <1301999735-24141-1-git-send-email-harsh@linux.vnet.ibm.com> Date: Tue, 5 Apr 2011 13:27:57 +0100 Message-ID: From: Stefan Hajnoczi Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] Re: [PATCH v4] v9fs_walk: As per 9p2000 RFC, MAXWELEM >= nwnames >= 0. List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Harsh Prateek Bora Cc: jvrao@linux.vnet.ibm.com, qemu-devel@nongnu.org On Tue, Apr 5, 2011 at 11:35 AM, Harsh Prateek Bora wrote: > =A0 =A0The nwnames field in TWALK message is assumed to be >=3D0 and <=3D= MAXWELEM > =A0 =A0which is defined as macro P9_MAXWELEM (16) in virtio-9p.h as per 9= p2000 > =A0 =A0RFC. Appropriate changes are required in V9fsWalkState and v9fs_wa= lk. > > v4: > - rebased on mainline repo, else condition updated as required. > > v3: > - Updated if-else conditions to appropriately handle nwnames =3D 0. > > v2: > - Added check in v9fs_walk_complete as well. > > Signed-off-by: Harsh Prateek Bora > --- > =A0hw/virtio-9p.c | =A0 =A07 +++++-- > =A0hw/virtio-9p.h | =A0 =A02 +- > =A02 files changed, 6 insertions(+), 3 deletions(-) Reviewed-by: Stefan Hajnoczi