[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-backports
Subject: Re: [PATCH] Updating id field in backports genl family
From: Aaron Komisar <aaron.komisar () tandemg ! com>
Date: 2018-06-26 6:54:46
Message-ID: AM4PR0202MB29164C902F2A904E875C123EF8490 () AM4PR0202MB2916 ! eurprd02 ! prod ! outlook ! com
[Download RAW message or body]
On Tue, Jun 26, 2018 at 09:07:13PM +0300, Wan-Hua Chang wrote:
>> Looking at this again ...
>>
>> > I see the patch causes compiling error becasue family->family is not
>> > found when LINUX_VERSION >= 3.13
>>
>> Yeah, that seems right.
>>
>> > backports/backport/backport-include/genetlink.h
>> > struct backport_genl_fmaily is only defined when LINUX_VERSION < 3.13
>> >
>> >
>> > And in backports/backport/compat/backport-3.13.c line 127
>> >
>> > family->id = family->family.id
>> >
>> > it looks like this line has done what the patch want to do. Is the
>> > comprehension correct?
>>
>> I don't think so. We call this function - via
>> __real_bp_extack_genl_register_family() - on copy->family, so we do need
>> to copy like Aaron said, we just need to make it conditional for kernel
>> < 3.13 because on 3.13 we don't use this sub-family thing.
>>
>> I've committed a fix - does that work for both of you? Aaron?
>Hi Johannes,
>Yes, the fix works for me, thanks.
Same here.
Thanks!
>
>>
>> johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic