[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linaro-kernel
Subject:    Re: [LNG] [PATCH] linaro/configs: add hugepage support into linaro-base
From:       Zi Shen Lim <zishen.lim () linaro ! org>
Date:       2013-09-30 20:09:37
Message-ID: CAM1oe50q+Ctz6RpN4LGO91_g9ux1h-U=7D8PAyrej6OzXoQUiw () mail ! gmail ! com
[Download RAW message or body]

On Mon, Sep 30, 2013 at 12:27 PM, Kim Phillips <kim.phillips@linaro.org> wrote:
> On Mon, 30 Sep 2013 12:00:36 -0700
> Zi Shen Lim <zishen.lim@linaro.org> wrote:
>
>> On Mon, Sep 30, 2013 at 12:03 PM, Anders Roxell
>> <anders.roxell@linaro.org> wrote:
>> > Hi,
>> >
>> > On 2013-09-30 11:48, Zi Shen Lim wrote:
>> >> Signed-off-by: Zi Shen Lim <zishen.lim@linaro.org>
>> >> ---
>> >>  linaro/configs/linaro-base.conf | 4 ++++
>> >>  1 file changed, 4 insertions(+)
>> >>
>> >> diff --git a/linaro/configs/linaro-base.conf b/linaro/configs/linaro-base.conf
>> >> index 947ca1f..0093640 100644
>> >> --- a/linaro/configs/linaro-base.conf
>> >> +++ b/linaro/configs/linaro-base.conf
>> >> @@ -92,3 +92,7 @@ CONFIG_HW_PERF_EVENTS=y
>> >>  CONFIG_FUNCTION_TRACER=y
>> >>  CONFIG_ENABLE_DEFAULT_TRACERS=y
>> >>  CONFIG_PROC_DEVICETREE=y
>> >> +CONFIG_HUGETLB_PAGE=y
>> >> +CONFIG_HUGETLBFS=y
>> >> +CONFIG_TRANSPARENT_HUGEPAGE=y
>> >> +CONFIG_TRANSPARENT_HUGEPAGE_MADVISE=y
>> > This CONFIG_* fragments are already in linaro/configs/linaro-base64.conf
>> > right?
>> >
>>
>> Isn't linaro-base64.conf meant for ARMv8?
>>
>> We need hugepage support on existing 32-bit / ARMv7 platforms too, don't we?
>
> [adding linaro-kernel, Andrey]
>
> Yes, it seems the base64 component is a misnomer, at least for now.
>
> Meanwhile, if this patch is targeted to the main linaro kernel, please
> send via the linaro-kernel list, cc'ing linaro-networking.  If not,
> then perhaps a separate .conf file is in order?  Anyway, I think we'd
> all prefer a resolution within the main linaro kernel for hugepage
> support.
>

Andrey, which approach do you prefer?
1. applying this patch in main linaro/configs
2. maintaining hugepage as a separate config fragment
3. applying this patch only in LNG

Let me know if you want me to respin the patch. Thanks.

>> > and we are building with it in our ci/job/linux-lng* scripts...
>
> perhaps those should be patched as well.
>
> Kim

_______________________________________________
linaro-kernel mailing list
linaro-kernel@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-kernel
[prev in list] [next in list] [prev in thread] [next in thread]