Re: [PATCH] xfrm: make xfrm_algo.c a module

[Jan Beulich]

>>> On 15.05.12 at 00:37, David Miller <davem@xxxxxxxxxxxxx> wrote:
> From: "Jan Beulich" <JBeulich@xxxxxxxx>
> Date: Wed, 09 May 2012 08:53:51 +0100
> 
>> By making this a standalone config option (selected as needed),
>> selecting CRYPTO from here rather than from XFRM (which is boolean)
>> allows the core crypto code to become a module again even when XFRM=y.
>> 
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
>  ...
>> @@ -15,9 +15,6 @@
>>  #include <linux/crypto.h>
>>  #include <linux/scatterlist.h>
>>  #include <net/xfrm.h>
>> -#if defined(CONFIG_INET_AH) || defined(CONFIG_INET_AH_MODULE) || 
> defined(CONFIG_INET6_AH) || defined(CONFIG_INET6_AH_MODULE)
>> -#include <net/ah.h>
>> -#endif
> 
> This is completely unrelated to the change you are trying to make in
> this patch.
> 
> It belongs in a separate change.

I apologize for that, I meant to split this out and then forgot. I'll
re-submit the two things separately.

Jan

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/