* [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h
@ 2017-04-17 21:33 Alistair Francis
2017-04-17 21:33 ` [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h Alistair Francis
` (2 more replies)
0 siblings, 3 replies; 11+ messages in thread
From: Alistair Francis @ 2017-04-17 21:33 UTC (permalink / raw)
To: ian.jackson, wei.liu2, rcojocaru, tamas, george.dunlap, xen-devel
Cc: alistair23, Alistair Francis
The POSIX spec specifies to use:
#include <poll.h>
instead of:
#include <sys/poll.h>
as seen here:
http://pubs.opengroup.org/onlinepubs/009695399/functions/poll.html
This removes the warning:
#warning redirecting incorrect #include <sys/poll.h> to <poll.h>
when building with the musl C-library.
Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
---
tools/libxl/libxl_internal.h | 2 +-
tools/tests/xen-access/xen-access.c | 2 +-
tools/xenstat/libxenstat/src/xenstat_qmp.c | 2 +-
tools/xentrace/xentrace.c | 2 +-
4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h
index be24b76dfa..5d082c5704 100644
--- a/tools/libxl/libxl_internal.h
+++ b/tools/libxl/libxl_internal.h
@@ -38,7 +38,7 @@
#include <ctype.h>
#include <sys/mman.h>
-#include <sys/poll.h>
+#include <poll.h>
#include <sys/select.h>
#include <sys/stat.h>
#include <sys/time.h>
diff --git a/tools/tests/xen-access/xen-access.c b/tools/tests/xen-access/xen-access.c
index ff4d289b45..238011e010 100644
--- a/tools/tests/xen-access/xen-access.c
+++ b/tools/tests/xen-access/xen-access.c
@@ -36,7 +36,7 @@
#include <signal.h>
#include <unistd.h>
#include <sys/mman.h>
-#include <sys/poll.h>
+#include <poll.h>
#include <xenctrl.h>
#include <xenevtchn.h>
diff --git a/tools/xenstat/libxenstat/src/xenstat_qmp.c b/tools/xenstat/libxenstat/src/xenstat_qmp.c
index a87c9373c2..3fda487d49 100644
--- a/tools/xenstat/libxenstat/src/xenstat_qmp.c
+++ b/tools/xenstat/libxenstat/src/xenstat_qmp.c
@@ -14,7 +14,7 @@
#include <fcntl.h>
#include <sys/types.h>
#include <sys/socket.h>
-#include <sys/poll.h>
+#include <poll.h>
#include <sys/un.h>
#include <stdlib.h>
#include <string.h>
diff --git a/tools/xentrace/xentrace.c b/tools/xentrace/xentrace.c
index f09fe6cf19..364a6fdad5 100644
--- a/tools/xentrace/xentrace.c
+++ b/tools/xentrace/xentrace.c
@@ -24,7 +24,7 @@
#include <getopt.h>
#include <assert.h>
#include <ctype.h>
-#include <sys/poll.h>
+#include <poll.h>
#include <sys/statvfs.h>
#include <xen/xen.h>
--
2.11.0
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h
2017-04-17 21:33 [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h Alistair Francis
@ 2017-04-17 21:33 ` Alistair Francis
2017-04-18 10:03 ` Wei Liu
2017-04-18 13:22 ` [for-4.9] " Ian Jackson
2017-04-18 10:02 ` [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h Wei Liu
2017-04-18 10:10 ` Wei Liu
2 siblings, 2 replies; 11+ messages in thread
From: Alistair Francis @ 2017-04-17 21:33 UTC (permalink / raw)
To: ian.jackson, wei.liu2, rcojocaru, tamas, george.dunlap, xen-devel
Cc: alistair23, Alistair Francis
The POSIX spec specifies to use:
#include <signal.h>
instead of:
#include <sys/signal.h>
as seen here:
http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html
This removes the warning:
#warning redirecting incorrect #include <sys/signal.h> to <signal.h>
when building with the musl C-library.
Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
---
tools/blktap2/drivers/tapdisk-server.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/blktap2/drivers/tapdisk-server.c b/tools/blktap2/drivers/tapdisk-server.c
index eecde3d23f..71315bb069 100644
--- a/tools/blktap2/drivers/tapdisk-server.c
+++ b/tools/blktap2/drivers/tapdisk-server.c
@@ -30,7 +30,7 @@
#include <unistd.h>
#include <stdlib.h>
#include <sys/ioctl.h>
-#include <sys/signal.h>
+#include <signal.h>
#include "tapdisk-utils.h"
#include "tapdisk-server.h"
--
2.11.0
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h
2017-04-17 21:33 [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h Alistair Francis
2017-04-17 21:33 ` [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h Alistair Francis
@ 2017-04-18 10:02 ` Wei Liu
2017-04-18 10:14 ` Razvan Cojocaru
2017-04-18 10:10 ` Wei Liu
2 siblings, 1 reply; 11+ messages in thread
From: Wei Liu @ 2017-04-18 10:02 UTC (permalink / raw)
To: Alistair Francis
Cc: tamas, wei.liu2, rcojocaru, george.dunlap, ian.jackson,
xen-devel, alistair23
On Mon, Apr 17, 2017 at 02:33:10PM -0700, Alistair Francis wrote:
> The POSIX spec specifies to use:
> #include <poll.h>
> instead of:
> #include <sys/poll.h>
> as seen here:
> http://pubs.opengroup.org/onlinepubs/009695399/functions/poll.html
>
> This removes the warning:
> #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
> when building with the musl C-library.
>
> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
FAOD, these are just mechanical changes. I don't think acks from Tamas
and George are strictly required.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h
2017-04-17 21:33 ` [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h Alistair Francis
@ 2017-04-18 10:03 ` Wei Liu
2017-04-18 10:16 ` Razvan Cojocaru
2017-04-18 13:22 ` [for-4.9] " Ian Jackson
1 sibling, 1 reply; 11+ messages in thread
From: Wei Liu @ 2017-04-18 10:03 UTC (permalink / raw)
To: Alistair Francis
Cc: tamas, wei.liu2, rcojocaru, george.dunlap, ian.jackson,
xen-devel, alistair23
On Mon, Apr 17, 2017 at 02:33:11PM -0700, Alistair Francis wrote:
> The POSIX spec specifies to use:
> #include <signal.h>
> instead of:
> #include <sys/signal.h>
> as seen here:
> http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html
>
> This removes the warning:
> #warning redirecting incorrect #include <sys/signal.h> to <signal.h>
> when building with the musl C-library.
>
> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h
2017-04-17 21:33 [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h Alistair Francis
2017-04-17 21:33 ` [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h Alistair Francis
2017-04-18 10:02 ` [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h Wei Liu
@ 2017-04-18 10:10 ` Wei Liu
2017-04-18 10:13 ` Wei Liu
2 siblings, 1 reply; 11+ messages in thread
From: Wei Liu @ 2017-04-18 10:10 UTC (permalink / raw)
To: Alistair Francis
Cc: tamas, wei.liu2, rcojocaru, george.dunlap, ian.jackson,
xen-devel, alistair23
Cc Julien
I think these two patches should be in 4.9.
On Mon, Apr 17, 2017 at 02:33:10PM -0700, Alistair Francis wrote:
> The POSIX spec specifies to use:
> #include <poll.h>
> instead of:
> #include <sys/poll.h>
> as seen here:
> http://pubs.opengroup.org/onlinepubs/009695399/functions/poll.html
>
> This removes the warning:
> #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
> when building with the musl C-library.
>
> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
> ---
> tools/libxl/libxl_internal.h | 2 +-
> tools/tests/xen-access/xen-access.c | 2 +-
> tools/xenstat/libxenstat/src/xenstat_qmp.c | 2 +-
> tools/xentrace/xentrace.c | 2 +-
> 4 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h
> index be24b76dfa..5d082c5704 100644
> --- a/tools/libxl/libxl_internal.h
> +++ b/tools/libxl/libxl_internal.h
> @@ -38,7 +38,7 @@
> #include <ctype.h>
>
> #include <sys/mman.h>
> -#include <sys/poll.h>
> +#include <poll.h>
> #include <sys/select.h>
> #include <sys/stat.h>
> #include <sys/time.h>
> diff --git a/tools/tests/xen-access/xen-access.c b/tools/tests/xen-access/xen-access.c
> index ff4d289b45..238011e010 100644
> --- a/tools/tests/xen-access/xen-access.c
> +++ b/tools/tests/xen-access/xen-access.c
> @@ -36,7 +36,7 @@
> #include <signal.h>
> #include <unistd.h>
> #include <sys/mman.h>
> -#include <sys/poll.h>
> +#include <poll.h>
>
> #include <xenctrl.h>
> #include <xenevtchn.h>
> diff --git a/tools/xenstat/libxenstat/src/xenstat_qmp.c b/tools/xenstat/libxenstat/src/xenstat_qmp.c
> index a87c9373c2..3fda487d49 100644
> --- a/tools/xenstat/libxenstat/src/xenstat_qmp.c
> +++ b/tools/xenstat/libxenstat/src/xenstat_qmp.c
> @@ -14,7 +14,7 @@
> #include <fcntl.h>
> #include <sys/types.h>
> #include <sys/socket.h>
> -#include <sys/poll.h>
> +#include <poll.h>
> #include <sys/un.h>
> #include <stdlib.h>
> #include <string.h>
> diff --git a/tools/xentrace/xentrace.c b/tools/xentrace/xentrace.c
> index f09fe6cf19..364a6fdad5 100644
> --- a/tools/xentrace/xentrace.c
> +++ b/tools/xentrace/xentrace.c
> @@ -24,7 +24,7 @@
> #include <getopt.h>
> #include <assert.h>
> #include <ctype.h>
> -#include <sys/poll.h>
> +#include <poll.h>
> #include <sys/statvfs.h>
>
> #include <xen/xen.h>
> --
> 2.11.0
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h
2017-04-18 10:10 ` Wei Liu
@ 2017-04-18 10:13 ` Wei Liu
2017-04-18 11:30 ` Julien Grall
0 siblings, 1 reply; 11+ messages in thread
From: Wei Liu @ 2017-04-18 10:13 UTC (permalink / raw)
To: Alistair Francis
Cc: tamas, wei.liu2, rcojocaru, george.dunlap, ian.jackson,
xen-devel, Julien Grall, alistair23
Really CC Julien. :-/
On Tue, Apr 18, 2017 at 11:10:59AM +0100, Wei Liu wrote:
> Cc Julien
>
> I think these two patches should be in 4.9.
>
> On Mon, Apr 17, 2017 at 02:33:10PM -0700, Alistair Francis wrote:
> > The POSIX spec specifies to use:
> > #include <poll.h>
> > instead of:
> > #include <sys/poll.h>
> > as seen here:
> > http://pubs.opengroup.org/onlinepubs/009695399/functions/poll.html
> >
> > This removes the warning:
> > #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
> > when building with the musl C-library.
> >
> > Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
> > ---
> > tools/libxl/libxl_internal.h | 2 +-
> > tools/tests/xen-access/xen-access.c | 2 +-
> > tools/xenstat/libxenstat/src/xenstat_qmp.c | 2 +-
> > tools/xentrace/xentrace.c | 2 +-
> > 4 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h
> > index be24b76dfa..5d082c5704 100644
> > --- a/tools/libxl/libxl_internal.h
> > +++ b/tools/libxl/libxl_internal.h
> > @@ -38,7 +38,7 @@
> > #include <ctype.h>
> >
> > #include <sys/mman.h>
> > -#include <sys/poll.h>
> > +#include <poll.h>
> > #include <sys/select.h>
> > #include <sys/stat.h>
> > #include <sys/time.h>
> > diff --git a/tools/tests/xen-access/xen-access.c b/tools/tests/xen-access/xen-access.c
> > index ff4d289b45..238011e010 100644
> > --- a/tools/tests/xen-access/xen-access.c
> > +++ b/tools/tests/xen-access/xen-access.c
> > @@ -36,7 +36,7 @@
> > #include <signal.h>
> > #include <unistd.h>
> > #include <sys/mman.h>
> > -#include <sys/poll.h>
> > +#include <poll.h>
> >
> > #include <xenctrl.h>
> > #include <xenevtchn.h>
> > diff --git a/tools/xenstat/libxenstat/src/xenstat_qmp.c b/tools/xenstat/libxenstat/src/xenstat_qmp.c
> > index a87c9373c2..3fda487d49 100644
> > --- a/tools/xenstat/libxenstat/src/xenstat_qmp.c
> > +++ b/tools/xenstat/libxenstat/src/xenstat_qmp.c
> > @@ -14,7 +14,7 @@
> > #include <fcntl.h>
> > #include <sys/types.h>
> > #include <sys/socket.h>
> > -#include <sys/poll.h>
> > +#include <poll.h>
> > #include <sys/un.h>
> > #include <stdlib.h>
> > #include <string.h>
> > diff --git a/tools/xentrace/xentrace.c b/tools/xentrace/xentrace.c
> > index f09fe6cf19..364a6fdad5 100644
> > --- a/tools/xentrace/xentrace.c
> > +++ b/tools/xentrace/xentrace.c
> > @@ -24,7 +24,7 @@
> > #include <getopt.h>
> > #include <assert.h>
> > #include <ctype.h>
> > -#include <sys/poll.h>
> > +#include <poll.h>
> > #include <sys/statvfs.h>
> >
> > #include <xen/xen.h>
> > --
> > 2.11.0
> >
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h
2017-04-18 10:02 ` [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h Wei Liu
@ 2017-04-18 10:14 ` Razvan Cojocaru
0 siblings, 0 replies; 11+ messages in thread
From: Razvan Cojocaru @ 2017-04-18 10:14 UTC (permalink / raw)
To: Wei Liu, Alistair Francis
Cc: george.dunlap, alistair23, tamas, ian.jackson, xen-devel
On 04/18/2017 01:02 PM, Wei Liu wrote:
> On Mon, Apr 17, 2017 at 02:33:10PM -0700, Alistair Francis wrote:
>> The POSIX spec specifies to use:
>> #include <poll.h>
>> instead of:
>> #include <sys/poll.h>
>> as seen here:
>> http://pubs.opengroup.org/onlinepubs/009695399/functions/poll.html
>>
>> This removes the warning:
>> #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
>> when building with the musl C-library.
>>
>> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
>
> Acked-by: Wei Liu <wei.liu2@citrix.com>
>
> FAOD, these are just mechanical changes. I don't think acks from Tamas
> and George are strictly required.
FWIW:
Acked-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
Thanks,
Razvan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h
2017-04-18 10:03 ` Wei Liu
@ 2017-04-18 10:16 ` Razvan Cojocaru
2017-04-18 11:42 ` Wei Liu
0 siblings, 1 reply; 11+ messages in thread
From: Razvan Cojocaru @ 2017-04-18 10:16 UTC (permalink / raw)
To: Wei Liu, Alistair Francis
Cc: george.dunlap, alistair23, tamas, ian.jackson, xen-devel
On 04/18/2017 01:03 PM, Wei Liu wrote:
> On Mon, Apr 17, 2017 at 02:33:11PM -0700, Alistair Francis wrote:
>> The POSIX spec specifies to use:
>> #include <signal.h>
>> instead of:
>> #include <sys/signal.h>
>> as seen here:
>> http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html
>>
>> This removes the warning:
>> #warning redirecting incorrect #include <sys/signal.h> to <signal.h>
>> when building with the musl C-library.
>>
>> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
>
> Acked-by: Wei Liu <wei.liu2@citrix.com>
FWIW:
Acked-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
Thanks,
Razvan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h
2017-04-18 10:13 ` Wei Liu
@ 2017-04-18 11:30 ` Julien Grall
0 siblings, 0 replies; 11+ messages in thread
From: Julien Grall @ 2017-04-18 11:30 UTC (permalink / raw)
To: Wei Liu, Alistair Francis
Cc: tamas, rcojocaru, george.dunlap, ian.jackson, xen-devel, alistair23
Hi Wei,
Release-acked-by: Julien Grall <julien.grall@arm.com>
Cheers,
On 18/04/17 11:13, Wei Liu wrote:
> Really CC Julien. :-/
>
> On Tue, Apr 18, 2017 at 11:10:59AM +0100, Wei Liu wrote:
>> Cc Julien
>>
>> I think these two patches should be in 4.9.
>>
>> On Mon, Apr 17, 2017 at 02:33:10PM -0700, Alistair Francis wrote:
>>> The POSIX spec specifies to use:
>>> #include <poll.h>
>>> instead of:
>>> #include <sys/poll.h>
>>> as seen here:
>>> http://pubs.opengroup.org/onlinepubs/009695399/functions/poll.html
>>>
>>> This removes the warning:
>>> #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
>>> when building with the musl C-library.
>>>
>>> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
>>> ---
>>> tools/libxl/libxl_internal.h | 2 +-
>>> tools/tests/xen-access/xen-access.c | 2 +-
>>> tools/xenstat/libxenstat/src/xenstat_qmp.c | 2 +-
>>> tools/xentrace/xentrace.c | 2 +-
>>> 4 files changed, 4 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h
>>> index be24b76dfa..5d082c5704 100644
>>> --- a/tools/libxl/libxl_internal.h
>>> +++ b/tools/libxl/libxl_internal.h
>>> @@ -38,7 +38,7 @@
>>> #include <ctype.h>
>>>
>>> #include <sys/mman.h>
>>> -#include <sys/poll.h>
>>> +#include <poll.h>
>>> #include <sys/select.h>
>>> #include <sys/stat.h>
>>> #include <sys/time.h>
>>> diff --git a/tools/tests/xen-access/xen-access.c b/tools/tests/xen-access/xen-access.c
>>> index ff4d289b45..238011e010 100644
>>> --- a/tools/tests/xen-access/xen-access.c
>>> +++ b/tools/tests/xen-access/xen-access.c
>>> @@ -36,7 +36,7 @@
>>> #include <signal.h>
>>> #include <unistd.h>
>>> #include <sys/mman.h>
>>> -#include <sys/poll.h>
>>> +#include <poll.h>
>>>
>>> #include <xenctrl.h>
>>> #include <xenevtchn.h>
>>> diff --git a/tools/xenstat/libxenstat/src/xenstat_qmp.c b/tools/xenstat/libxenstat/src/xenstat_qmp.c
>>> index a87c9373c2..3fda487d49 100644
>>> --- a/tools/xenstat/libxenstat/src/xenstat_qmp.c
>>> +++ b/tools/xenstat/libxenstat/src/xenstat_qmp.c
>>> @@ -14,7 +14,7 @@
>>> #include <fcntl.h>
>>> #include <sys/types.h>
>>> #include <sys/socket.h>
>>> -#include <sys/poll.h>
>>> +#include <poll.h>
>>> #include <sys/un.h>
>>> #include <stdlib.h>
>>> #include <string.h>
>>> diff --git a/tools/xentrace/xentrace.c b/tools/xentrace/xentrace.c
>>> index f09fe6cf19..364a6fdad5 100644
>>> --- a/tools/xentrace/xentrace.c
>>> +++ b/tools/xentrace/xentrace.c
>>> @@ -24,7 +24,7 @@
>>> #include <getopt.h>
>>> #include <assert.h>
>>> #include <ctype.h>
>>> -#include <sys/poll.h>
>>> +#include <poll.h>
>>> #include <sys/statvfs.h>
>>>
>>> #include <xen/xen.h>
>>> --
>>> 2.11.0
>>>
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h
2017-04-18 10:16 ` Razvan Cojocaru
@ 2017-04-18 11:42 ` Wei Liu
0 siblings, 0 replies; 11+ messages in thread
From: Wei Liu @ 2017-04-18 11:42 UTC (permalink / raw)
To: Razvan Cojocaru
Cc: tamas, Wei Liu, george.dunlap, ian.jackson, xen-devel,
alistair23, Alistair Francis
On Tue, Apr 18, 2017 at 01:16:04PM +0300, Razvan Cojocaru wrote:
> On 04/18/2017 01:03 PM, Wei Liu wrote:
> > On Mon, Apr 17, 2017 at 02:33:11PM -0700, Alistair Francis wrote:
> >> The POSIX spec specifies to use:
> >> #include <signal.h>
> >> instead of:
> >> #include <sys/signal.h>
> >> as seen here:
> >> http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html
> >>
> >> This removes the warning:
> >> #warning redirecting incorrect #include <sys/signal.h> to <signal.h>
> >> when building with the musl C-library.
> >>
> >> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
> >
> > Acked-by: Wei Liu <wei.liu2@citrix.com>
>
> FWIW:
>
> Acked-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
>
Thanks!
>
> Thanks,
> Razvan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* [for-4.9] Re: [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h
2017-04-17 21:33 ` [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h Alistair Francis
2017-04-18 10:03 ` Wei Liu
@ 2017-04-18 13:22 ` Ian Jackson
1 sibling, 0 replies; 11+ messages in thread
From: Ian Jackson @ 2017-04-18 13:22 UTC (permalink / raw)
To: Julien Grall, Alistair Francis
Cc: tamas, wei.liu2, rcojocaru, george.dunlap, xen-devel, alistair23
Alistair Francis writes ("[PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h"):
> The POSIX spec specifies to use:
> #include <signal.h>
> instead of:
> #include <sys/signal.h>
> as seen here:
> http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html
Both:
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Julien, I think this pair are a candidate for 4.9.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2017-04-18 13:22 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-17 21:33 [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h Alistair Francis
2017-04-17 21:33 ` [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h Alistair Francis
2017-04-18 10:03 ` Wei Liu
2017-04-18 10:16 ` Razvan Cojocaru
2017-04-18 11:42 ` Wei Liu
2017-04-18 13:22 ` [for-4.9] " Ian Jackson
2017-04-18 10:02 ` [PATCH 1/2] tools: Use POSIX poll.h instead of sys/poll.h Wei Liu
2017-04-18 10:14 ` Razvan Cojocaru
2017-04-18 10:10 ` Wei Liu
2017-04-18 10:13 ` Wei Liu
2017-04-18 11:30 ` Julien Grall
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.