[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 06/13] 9p: darwin: Address minor differences
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH 06/13] 9p: darwin: Address minor differences |
Date: |
Thu, 31 May 2018 21:49:40 +0200 |
On Thu, 31 May 2018 15:23:45 -0400
Keno Fischer <address@hidden> wrote:
> Well, I do have the patch already to switch this and the other patterns,
> so let me know if you want it or not ;).
>
Post it then and we'll see if people are happy with that :)
> On Thu, May 31, 2018 at 3:22 PM, Greg Kurz <address@hidden> wrote:
> > On Thu, 31 May 2018 12:27:35 -0400
> > Keno Fischer <address@hidden> wrote:
> >
> >> >> --- a/hw/9pfs/9p-local.c
> >> >> +++ b/hw/9pfs/9p-local.c
> >> >> @@ -67,7 +67,10 @@ int local_open_nofollow(FsContext *fs_ctx, const
> >> >> char *path, int flags,
> >> >> assert(*path != '/');
> >> >>
> >> >> head = g_strdup(path);
> >> >> - c = strchrnul(path, '/');
> >> >> + /* equivalent to strchrnul(), but that is not available on
> >> >> Darwin */
> >> >
> >> > Please make a qemu_strchrnul() helper with a separate implementation for
> >> > Darwin
> >> > then. I guess you can put it in this file since there aren't any other
> >> > users in
> >> > the QEMU code base.
> >>
> >> There actually are, but they also use this pattern. Could you
> >> suggest the best place to put this utility? I can submit a patch
> >> to switch all instances of this pattern over.
> >
> > Oh if the pattern is already used in other places, it's probably not
> > worth the pain... so please forget this :)
[Qemu-devel] [PATCH 07/13] 9p: darwin: Properly translate AT_REMOVEDIR flag, keno, 2018/05/26
[Qemu-devel] [PATCH 08/13] 9p: darwin: Ignore O_{NOATIME, DIRECT}, keno, 2018/05/26
[Qemu-devel] [PATCH 09/13] 9p: darwin: Provide a compatibility definition for XATTR_SIZE_MAX, keno, 2018/05/26