Re: [PATCH v4 18/19] 9p: Drop superfluous include of linux/limits.h
On Thursday, January 19, 2023 11:37:00 AM CET Markus Armbruster wrote: > Christian Schoenebeck writes: > > > On Thursday, January 19, 2023 7:59:58 AM CET Markus Armbruster wrote: > >> Signed-off-by: Markus Armbruster > >> --- > >> hw/9pfs/9p.c | 3 --- > >> 1 file changed, 3 deletions(-) > >> > >> diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c > >> index 9621ec1341..aa736af380 100644 > >> --- a/hw/9pfs/9p.c > >> +++ b/hw/9pfs/9p.c > >> @@ -17,9 +17,6 @@ > >> */ > >> > >> #include "qemu/osdep.h" > >> -#ifdef CONFIG_LINUX > >> -#include > >> -#endif > >> #include > >> #include "hw/virtio/virtio.h" > >> #include "qapi/error.h" > >> > > > > Where did that base version come from? I don't see it anywhere in history. > > Last relevant change in context was a136d17590a. > > Current master (7ec8aeb6048) has > > #include "qemu/osdep.h" > #ifdef CONFIG_LINUX > #include > #else > #include > #endif > #include > > The previous commit changes it to > > #include "qemu/osdep.h" > #ifdef CONFIG_LINUX > #include > #endif > #include > > because "qemu/osdep.h" already includes . > > Clearer now? Ah, right I missed that in your previous patch. Thanks! Reviewed-by: Christian Schoenebeck Best regards, Christian Schoenebeck
Re: [PATCH v4 18/19] 9p: Drop superfluous include of linux/limits.h
Christian Schoenebeck writes: > On Thursday, January 19, 2023 7:59:58 AM CET Markus Armbruster wrote: >> Signed-off-by: Markus Armbruster >> --- >> hw/9pfs/9p.c | 3 --- >> 1 file changed, 3 deletions(-) >> >> diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c >> index 9621ec1341..aa736af380 100644 >> --- a/hw/9pfs/9p.c >> +++ b/hw/9pfs/9p.c >> @@ -17,9 +17,6 @@ >> */ >> >> #include "qemu/osdep.h" >> -#ifdef CONFIG_LINUX >> -#include >> -#endif >> #include >> #include "hw/virtio/virtio.h" >> #include "qapi/error.h" >> > > Where did that base version come from? I don't see it anywhere in history. > Last relevant change in context was a136d17590a. Current master (7ec8aeb6048) has #include "qemu/osdep.h" #ifdef CONFIG_LINUX #include #else #include #endif #include The previous commit changes it to #include "qemu/osdep.h" #ifdef CONFIG_LINUX #include #endif #include because "qemu/osdep.h" already includes . Clearer now?
Re: [PATCH v4 18/19] 9p: Drop superfluous include of linux/limits.h
On Thursday, January 19, 2023 7:59:58 AM CET Markus Armbruster wrote: > Signed-off-by: Markus Armbruster > --- > hw/9pfs/9p.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c > index 9621ec1341..aa736af380 100644 > --- a/hw/9pfs/9p.c > +++ b/hw/9pfs/9p.c > @@ -17,9 +17,6 @@ > */ > > #include "qemu/osdep.h" > -#ifdef CONFIG_LINUX > -#include > -#endif > #include > #include "hw/virtio/virtio.h" > #include "qapi/error.h" > Where did that base version come from? I don't see it anywhere in history. Last relevant change in context was a136d17590a. Best regards, Christian Schoenebeck
[PATCH v4 18/19] 9p: Drop superfluous include of linux/limits.h
Signed-off-by: Markus Armbruster --- hw/9pfs/9p.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c index 9621ec1341..aa736af380 100644 --- a/hw/9pfs/9p.c +++ b/hw/9pfs/9p.c @@ -17,9 +17,6 @@ */ #include "qemu/osdep.h" -#ifdef CONFIG_LINUX -#include -#endif #include #include "hw/virtio/virtio.h" #include "qapi/error.h" -- 2.39.0