Re: [Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening

2017-05-18 Thread Greg Kurz
On Thu, 18 May 2017 09:23:07 -0500 Eric Blake wrote: > On 05/09/2017 04:23 AM, Greg Kurz wrote: > > On Fri, 5 May 2017 12:01:55 -0500 > > Eric Blake wrote: > > > >> On 05/05/2017 09:37 AM, Greg Kurz wrote: > >>> All paths in the virtfs directory now

Re: [Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening

2017-05-18 Thread Eric Blake
On 05/09/2017 04:23 AM, Greg Kurz wrote: > On Fri, 5 May 2017 12:01:55 -0500 > Eric Blake wrote: > >> On 05/05/2017 09:37 AM, Greg Kurz wrote: >>> All paths in the virtfs directory now start with "./" (except the virtfs >>> root itself which is exactly "."). >>> >>> We hence

Re: [Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening

2017-05-18 Thread Greg Kurz
On Tue, 9 May 2017 11:23:05 +0200 Greg Kurz wrote: > On Fri, 5 May 2017 12:01:55 -0500 > Eric Blake wrote: > > > On 05/05/2017 09:37 AM, Greg Kurz wrote: > > > All paths in the virtfs directory now start with "./" (except the virtfs > > > root itself which

Re: [Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening

2017-05-09 Thread Greg Kurz
On Fri, 5 May 2017 12:01:55 -0500 Eric Blake wrote: > On 05/05/2017 09:37 AM, Greg Kurz wrote: > > All paths in the virtfs directory now start with "./" (except the virtfs > > root itself which is exactly "."). > > > > We hence don't need to skip leading '/' characters

Re: [Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening

2017-05-05 Thread Eric Blake
On 05/05/2017 09:37 AM, Greg Kurz wrote: > All paths in the virtfs directory now start with "./" (except the virtfs > root itself which is exactly "."). > > We hence don't need to skip leading '/' characters anymore, nor to handle > the empty path case. Also, since virtfs will only ever be

[Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening

2017-05-05 Thread Greg Kurz
All paths in the virtfs directory now start with "./" (except the virtfs root itself which is exactly "."). We hence don't need to skip leading '/' characters anymore, nor to handle the empty path case. Also, since virtfs will only ever be supported on linux+glibc hosts, we can use strchrnul()