2023-10-23 18:43:02

by Mark Brown

[permalink] [raw]
Subject: [PATCH v2] tools/nolibc: Use linux/wait.h rather than duplicating it

Linux defines a few custom flags for waitpid() which aren't currently
provided by nolibc, make them available to nolibc based programs by just
including linux/wait.h where they are defined instead of defining our
own copy of the flags.

Signed-off-by: Mark Brown <[email protected]>
---
Changes in v2:
- Use linux/wait.h rather than defining things locally.
- Link to v1: https://lore.kernel.org/r/[email protected]
---
tools/include/nolibc/types.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/tools/include/nolibc/types.h b/tools/include/nolibc/types.h
index 8cfc4c860fa4..ad0ddaa89e50 100644
--- a/tools/include/nolibc/types.h
+++ b/tools/include/nolibc/types.h
@@ -12,6 +12,7 @@
#include <linux/reboot.h> /* for LINUX_REBOOT_* */
#include <linux/stat.h>
#include <linux/time.h>
+#include <linux/wait.h>


/* Only the generic macros and types may be defined here. The arch-specific
@@ -108,9 +109,6 @@
#define WTERMSIG(status) ((status) & 0x7f)
#define WIFSIGNALED(status) ((status) - 1 < 0xff)

-/* waitpid() flags */
-#define WNOHANG 1
-
/* standard exit() codes */
#define EXIT_SUCCESS 0
#define EXIT_FAILURE 1

---
base-commit: 6465e260f48790807eef06b583b38ca9789b6072
change-id: 20231020-nolibc-waitpid-flags-80ac075ab978

Best regards,
--
Mark Brown <[email protected]>


2023-10-24 15:59:40

by Thomas Weißschuh

[permalink] [raw]
Subject: Re: [PATCH v2] tools/nolibc: Use linux/wait.h rather than duplicating it

Thanks!

Oct 23, 2023 20:42:52 Mark Brown <[email protected]>:

> Linux defines a few custom flags for waitpid() which aren't currently
> provided by nolibc, make them available to nolibc based programs by just
> including linux/wait.h where they are defined instead of defining our
> own copy of the flags.
>
> Signed-off-by: Mark Brown <[email protected]>

Acked-by: Thomas Weißschuh <[email protected]>

I can apply it, but only next week.

Thomas

2023-10-24 16:10:16

by Willy Tarreau

[permalink] [raw]
Subject: Re: [PATCH v2] tools/nolibc: Use linux/wait.h rather than duplicating it

On Tue, Oct 24, 2023 at 05:59:13PM +0200, Thomas Wei?schuh wrote:
> Thanks!
>
> Oct 23, 2023 20:42:52 Mark Brown <[email protected]>:
>
> > Linux defines a few custom flags for waitpid() which aren't currently
> > provided by nolibc, make them available to nolibc based programs by just
> > including linux/wait.h where they are defined instead of defining our
> > own copy of the flags.
> >
> > Signed-off-by: Mark Brown <[email protected]>
>
> Acked-by: Thomas Wei?schuh <[email protected]>
>
> I can apply it, but only next week.

Yeah I'm fine as well, thanks Mark for this. I'm still too short on
time these days but if I manage to upload it sooner, I'll let you know.

Thanks!
Willy

2023-10-29 07:37:59

by Willy Tarreau

[permalink] [raw]
Subject: Re: [PATCH v2] tools/nolibc: Use linux/wait.h rather than duplicating it

On Mon, Oct 23, 2023 at 07:42:45PM +0100, Mark Brown wrote:
> Linux defines a few custom flags for waitpid() which aren't currently
> provided by nolibc, make them available to nolibc based programs by just
> including linux/wait.h where they are defined instead of defining our
> own copy of the flags.
>
> Signed-off-by: Mark Brown <[email protected]>
> ---
> Changes in v2:
> - Use linux/wait.h rather than defining things locally.
> - Link to v1: https://lore.kernel.org/r/[email protected]

Now merged into nolibc/next, thank you Mark!

Willy