mbox series

[v6,00/11] 9p: Add support for darwin

Message ID 20220210132946.2303-1-wwcohen@gmail.com (mailing list archive)
Headers show
Series 9p: Add support for darwin | expand

Message

Will Cohen Feb. 10, 2022, 1:29 p.m. UTC
This is a followup to https://lists.gnu.org/archive/html/qemu-devel/2022-02/msg01506.html,
adding 9p server support for Darwin.

Since v5, the following changes have been made to the following patches:

Patch 6/11: 9p: darwin: Move XATTR_SIZE_MAX->P9_XATTR_SIZE_MAX
- For P9_XATTR_SIZE_MAX, ensure that Linux uses XATTR_SIZE_MAX, Darwin uses 64k, and error out for undefined hosts

Patch 9/11: 9p: darwin: Implement compatibility for mknodat
- Move pthread_fchdir_np declaration only to osdep
- Declare pthread_fchdir_np with __attribute__((weak)) to allow checking for its presence before usage
- Move declarations above cplusplus guard

Patch 10/11: 9p: darwin: Adjust assumption on virtio-9p-test (formerly v5 Patch 11/11)
- Move this patch before 9p: darwin: meson patch to avoid qtest breakage during bisecting

Patch 11/11: 9p: darwin: meson: Allow VirtFS on Darwin (formerly v5 Patch 10/11)
- Fully adjust meson virtfs error note to specify macOS

Keno Fischer (10):
  9p: linux: Fix a couple Linux assumptions
  9p: Rename 9p-util -> 9p-util-linux
  9p: darwin: Handle struct stat(fs) differences
  9p: darwin: Handle struct dirent differences
  9p: darwin: Ignore O_{NOATIME, DIRECT}
  9p: darwin: Move XATTR_SIZE_MAX->P9_XATTR_SIZE_MAX
  9p: darwin: *xattr_nofollow implementations
  9p: darwin: Compatibility for f/l*xattr
  9p: darwin: Implement compatibility for mknodat
  9p: darwin: meson: Allow VirtFS on Darwin

Will Cohen (1):
  9p: darwin: Adjust assumption on virtio-9p-test

 fsdev/file-op-9p.h                     |  9 +++-
 fsdev/meson.build                      |  1 +
 hw/9pfs/9p-local.c                     | 27 ++++++++---
 hw/9pfs/9p-proxy.c                     | 38 +++++++++++++--
 hw/9pfs/9p-synth.c                     |  6 +++
 hw/9pfs/9p-util-darwin.c               | 64 ++++++++++++++++++++++++++
 hw/9pfs/{9p-util.c => 9p-util-linux.c} |  2 +-
 hw/9pfs/9p-util.h                      | 35 ++++++++++++++
 hw/9pfs/9p.c                           | 42 ++++++++++++++---
 hw/9pfs/9p.h                           | 18 ++++++++
 hw/9pfs/codir.c                        |  4 +-
 hw/9pfs/meson.build                    |  3 +-
 include/qemu/osdep.h                   | 12 +++++
 include/qemu/xattr.h                   |  4 +-
 meson.build                            | 14 ++++--
 os-posix.c                             | 35 ++++++++++++++
 tests/qtest/virtio-9p-test.c           |  2 +-
 17 files changed, 291 insertions(+), 25 deletions(-)
 create mode 100644 hw/9pfs/9p-util-darwin.c
 rename hw/9pfs/{9p-util.c => 9p-util-linux.c} (97%)

Comments

Will Cohen Feb. 14, 2022, 8:33 p.m. UTC | #1
Hello all,

I'm seeing on the updated release schedule that we're now looking at March
8 for soft feature freeze (
https://wiki.qemu.org/Planning/7.0#Release_Schedule). Is there anything
additional that should be prepared for this patch set to hopefully be able
to target 7.0?

Will

On Thu, Feb 10, 2022 at 8:29 AM Will Cohen <wwcohen@gmail.com> wrote:

> This is a followup to
> https://lists.gnu.org/archive/html/qemu-devel/2022-02/msg01506.html,
> adding 9p server support for Darwin.
>
> Since v5, the following changes have been made to the following patches:
>
> Patch 6/11: 9p: darwin: Move XATTR_SIZE_MAX->P9_XATTR_SIZE_MAX
> - For P9_XATTR_SIZE_MAX, ensure that Linux uses XATTR_SIZE_MAX, Darwin
> uses 64k, and error out for undefined hosts
>
> Patch 9/11: 9p: darwin: Implement compatibility for mknodat
> - Move pthread_fchdir_np declaration only to osdep
> - Declare pthread_fchdir_np with __attribute__((weak)) to allow checking
> for its presence before usage
> - Move declarations above cplusplus guard
>
> Patch 10/11: 9p: darwin: Adjust assumption on virtio-9p-test (formerly v5
> Patch 11/11)
> - Move this patch before 9p: darwin: meson patch to avoid qtest breakage
> during bisecting
>
> Patch 11/11: 9p: darwin: meson: Allow VirtFS on Darwin (formerly v5 Patch
> 10/11)
> - Fully adjust meson virtfs error note to specify macOS
>
> Keno Fischer (10):
>   9p: linux: Fix a couple Linux assumptions
>   9p: Rename 9p-util -> 9p-util-linux
>   9p: darwin: Handle struct stat(fs) differences
>   9p: darwin: Handle struct dirent differences
>   9p: darwin: Ignore O_{NOATIME, DIRECT}
>   9p: darwin: Move XATTR_SIZE_MAX->P9_XATTR_SIZE_MAX
>   9p: darwin: *xattr_nofollow implementations
>   9p: darwin: Compatibility for f/l*xattr
>   9p: darwin: Implement compatibility for mknodat
>   9p: darwin: meson: Allow VirtFS on Darwin
>
> Will Cohen (1):
>   9p: darwin: Adjust assumption on virtio-9p-test
>
>  fsdev/file-op-9p.h                     |  9 +++-
>  fsdev/meson.build                      |  1 +
>  hw/9pfs/9p-local.c                     | 27 ++++++++---
>  hw/9pfs/9p-proxy.c                     | 38 +++++++++++++--
>  hw/9pfs/9p-synth.c                     |  6 +++
>  hw/9pfs/9p-util-darwin.c               | 64 ++++++++++++++++++++++++++
>  hw/9pfs/{9p-util.c => 9p-util-linux.c} |  2 +-
>  hw/9pfs/9p-util.h                      | 35 ++++++++++++++
>  hw/9pfs/9p.c                           | 42 ++++++++++++++---
>  hw/9pfs/9p.h                           | 18 ++++++++
>  hw/9pfs/codir.c                        |  4 +-
>  hw/9pfs/meson.build                    |  3 +-
>  include/qemu/osdep.h                   | 12 +++++
>  include/qemu/xattr.h                   |  4 +-
>  meson.build                            | 14 ++++--
>  os-posix.c                             | 35 ++++++++++++++
>  tests/qtest/virtio-9p-test.c           |  2 +-
>  17 files changed, 291 insertions(+), 25 deletions(-)
>  create mode 100644 hw/9pfs/9p-util-darwin.c
>  rename hw/9pfs/{9p-util.c => 9p-util-linux.c} (97%)
>
> --
> 2.34.1
>
>
Christian Schoenebeck Feb. 16, 2022, 9:40 a.m. UTC | #2
On Montag, 14. Februar 2022 21:33:37 CET Will Cohen wrote:
> Hello all,
> 
> I'm seeing on the updated release schedule that we're now looking at March
> 8 for soft feature freeze (
> https://wiki.qemu.org/Planning/7.0#Release_Schedule). Is there anything
> additional that should be prepared for this patch set to hopefully be able
> to target 7.0?
> 
> Will

That's in three weeks. I'm optimistic that we get this merged before the QEMU 
7.0 soft freeze deadline.

Best regards,
Christian Schoenebeck