[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/19] 9p queue 2022-03-04
From: |
Christian Schoenebeck |
Subject: |
Re: [PULL 00/19] 9p queue 2022-03-04 |
Date: |
Fri, 04 Mar 2022 21:16:41 +0100 |
On Freitag, 4. März 2022 19:42:18 CET Peter Maydell wrote:
> On Fri, 4 Mar 2022 at 12:32, Christian Schoenebeck
>
> <qemu_oss@crudebyte.com> wrote:
> > The following changes since commit
5959ef7d431ffd02db112209cf55e47b677256fd:
> > Merge remote-tracking branch
> > 'remotes/alistair/tags/pull-riscv-to-apply-20220303' into staging
> > (2022-03-03 19:59:38 +0000)>
> > are available in the Git repository at:
> > https://github.com/cschoenebeck/qemu.git tags/pull-9p-20220304
> >
> > for you to fetch changes up to 39edfe337c418995b2932a9a14a612fb0c329dc5:
> > fsdev/p9array.h: convert Doxygen -> kerneldoc format (2022-03-04
> > 13:07:39 +0100)>
> > ----------------------------------------------------------------
> > 9pfs: introduce macOS host support and cleanup
> >
> > * Add support for Darwin (a.k.a. macOS) hosts.
> >
> > * Code cleanup (move qemu_dirent_dup() from osdep -> 9p-util).
> >
> > * API doc cleanup (convert Doxygen -> kerneldoc format).
>
> This fails to build on my OSX box:
>
> In file included from ../../hw/9pfs/9p-util-darwin.c:12:
> ../../hw/9pfs/9p-util.h:57:1: error: unused label 'again'
> [-Werror,-Wunused-label]
> again:
> ^~~~~~
>
> because the use of the label is inside a #ifndef CONFIG_DARWIN
> but the definition is not.
>
> thanks
> -- PMM
So basically it needs this change:
diff --git a/hw/9pfs/9p-util.h b/hw/9pfs/9p-util.h
index cfa7af43c5..97e681e167 100644
--- a/hw/9pfs/9p-util.h
+++ b/hw/9pfs/9p-util.h
@@ -54,7 +54,9 @@ static inline int openat_file(int dirfd, const char *name,
int flags,
{
int fd, serrno, ret;
+#ifndef CONFIG_DARWIN
again:
+#endif
fd = openat(dirfd, name, flags | O_NOFOLLOW | O_NOCTTY | O_NONBLOCK,
mode);
if (fd == -1) {
Will, can you check why this did not fail there and whether there are probably
more issues?
If that's the only one, let me know, then I would fix this on my end and
resend a PR ASAP. Thanks!
Best regards,
Christian Schoenebeck
- [PULL 02/19] 9p: Rename 9p-util -> 9p-util-linux, (continued)
- [PULL 02/19] 9p: Rename 9p-util -> 9p-util-linux, Christian Schoenebeck, 2022/03/04
- [PULL 09/19] 9p: darwin: Implement compatibility for mknodat, Christian Schoenebeck, 2022/03/04
- [PULL 18/19] 9pfs/coth.h: drop Doxygen format on v9fs_co_run_in_worker(), Christian Schoenebeck, 2022/03/04
- [PULL 05/19] 9p: darwin: Ignore O_{NOATIME, DIRECT}, Christian Schoenebeck, 2022/03/04
- [PULL 10/19] 9p: darwin: Adjust assumption on virtio-9p-test, Christian Schoenebeck, 2022/03/04
- [PULL 06/19] 9p: darwin: Move XATTR_SIZE_MAX->P9_XATTR_SIZE_MAX, Christian Schoenebeck, 2022/03/04
- [PULL 14/19] 9pfs/9p.h: convert Doxygen -> kerneldoc format, Christian Schoenebeck, 2022/03/04
- [PULL 03/19] 9p: darwin: Handle struct stat(fs) differences, Christian Schoenebeck, 2022/03/04
- [PULL 12/19] 9pfs: move qemu_dirent_dup() from osdep -> 9p-util, Christian Schoenebeck, 2022/03/04
- Re: [PULL 00/19] 9p queue 2022-03-04, Peter Maydell, 2022/03/04