diff mbox

[buildfix,for-2.0] linux-user: Fix build if headers don't define _LINUX_CAPABILITY_VERSION_1

Message ID 1395245708-22971-1-git-send-email-peter.maydell@linaro.org
State Accepted
Commit ec864874bd1c1e13c82f4096fd3957d26c70a680
Headers show

Commit Message

Peter Maydell March 19, 2014, 4:15 p.m. UTC
Older kernel headers don't define _LINUX_CAPABILITY_VERSION_1.
Switch to using the older _LINUX_CAPABILITY_VERSION; newer headers
still define this for source compatibility.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>
---
 linux-user/syscall.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Laurent Desnogues March 19, 2014, 4:33 p.m. UTC | #1
On Wed, Mar 19, 2014 at 5:15 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
> Older kernel headers don't define _LINUX_CAPABILITY_VERSION_1.
> Switch to using the older _LINUX_CAPABILITY_VERSION; newer headers
> still define this for source compatibility.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>

I'm not sure I should send "Reviewed-by" for issues I reported,
but here it is.

Reviewed-by: Laurent Desnogues <laurent.desnogues@gmail.com>

Thanks,

Laurent

> ---
>  linux-user/syscall.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
> index 366b695..2eac6d5 100644
> --- a/linux-user/syscall.c
> +++ b/linux-user/syscall.c
> @@ -7698,7 +7698,7 @@ abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
>          header.version = tswap32(target_header->version);
>          header.pid = tswap32(target_header->pid);
>
> -        if (header.version != _LINUX_CAPABILITY_VERSION_1) {
> +        if (header.version != _LINUX_CAPABILITY_VERSION) {
>              /* Version 2 and up takes pointer to two user_data structs */
>              data_items = 2;
>          }
> --
> 1.9.0
>
Peter Maydell March 19, 2014, 5:39 p.m. UTC | #2
On 19 March 2014 16:33, Laurent Desnogues <laurent.desnogues@gmail.com> wrote:
> On Wed, Mar 19, 2014 at 5:15 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
>> Older kernel headers don't define _LINUX_CAPABILITY_VERSION_1.
>> Switch to using the older _LINUX_CAPABILITY_VERSION; newer headers
>> still define this for source compatibility.
>>
>> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
>> Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>
>
> I'm not sure I should send "Reviewed-by" for issues I reported,
> but here it is.
>
> Reviewed-by: Laurent Desnogues <laurent.desnogues@gmail.com>

Well, a confirmation that it actually fixes the problem it purports
to fix is always nice :-)

thanks
-- PMM
Peter Maydell March 19, 2014, 6:37 p.m. UTC | #3
On 19 March 2014 16:15, Peter Maydell <peter.maydell@linaro.org> wrote:
> Older kernel headers don't define _LINUX_CAPABILITY_VERSION_1.
> Switch to using the older _LINUX_CAPABILITY_VERSION; newer headers
> still define this for source compatibility.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>

Riku acked this via IRC, so applied to master as a buildfix.

thanks
-- PMM
diff mbox

Patch

diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 366b695..2eac6d5 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -7698,7 +7698,7 @@  abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
         header.version = tswap32(target_header->version);
         header.pid = tswap32(target_header->pid);
 
-        if (header.version != _LINUX_CAPABILITY_VERSION_1) {
+        if (header.version != _LINUX_CAPABILITY_VERSION) {
             /* Version 2 and up takes pointer to two user_data structs */
             data_items = 2;
         }