Re: [OS-BUILD PATCH] configs/aarch64: Fix CONFIG_FORCE_MAX_ZONEORDER

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 19, 2020 at 10:33:52AM -0400, Don Zickus wrote:
> On Tue, May 19, 2020 at 01:47:09PM -0000, GitLab Bridge on behalf of prarit wrote:
> > From: Prarit Bhargava <prarit@xxxxxxxxxx>
> > 
> > The internal make command 'make rh-brew'/'make dist-brew' is failing with
> > 
> > Processing /home/prarit/git-kernel/kernel-ark/redhat/configs/kernel-aarch64-debug-fedora.config ... Error: Mismatches found in configuration files
> > Found CONFIG_FORCE_MAX_ZONEORDER=11 after generation, had CONFIG_FORCE_MAX_ZONEORDER=13 in Source tree
> > 
> > ARK commit f5ca593e1c2e ("configs: Adjust CONFIG_FORCE_MAX_ZONEORDER for
> > Fedora") set this to 11, and follow-on ARK commit dd028d261347 ("[redhat]
> > Sync up ARK's Fedora config tree with Fedora's dist-git") erroneously
> > overwrote the value back to 13.
> > 
> > Set CONFIG_FORCE_MAX_ZONEORDER back to 11 for aarch64.
> 
> This works for os-build but fails for ark-latest because there is a patch in
> ark-patches that changes the default to 13.
> 
> The problem is we tried to keep the redhat/ changes separate from the
> redhat/fedora only patches in ark-patches.  The goal was os-build could
> build standalone and the combination of ark-patches into ark-latest could
> build standalone.
> 
> Because of that arm64 change of the default to 13, we broke the os-build
> standalone build.  And because the ark-patches does not have the redhat/
> area, we can't make config changes on that branch.

I forgot to point out the patch that causes this:
(git log -p origin/master..origin/ark-patches arch/arm64)


commit df97e71c24526cabc6a434d9f35ef493056b1bcf
Author: Peter Robinson <pbrobinson@xxxxxxxxx>
Date:   Wed Feb 26 13:38:40 2020 -0500

    Add option of 13 for FORCE_MAX_ZONEORDER

    This is a hack, but it's what the other distros currently use
    for aarch64 with 4K pages so we'll do the same while upstream
    decides what the best outcome is (which isn't this).

    Upstream Status: RHEL only
    Signed-off-by: Peter Robinson <pbrobinson@xxxxxxxxx>
    [Add a dependency on RHEL_DIFFERENCES]
    Signed-off-by: Jeremy Cline <jcline@xxxxxxxxxx>

diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig
index 9e87f5847c92..f57bee916600 100644
--- a/arch/arm64/Kconfig
+++ b/arch/arm64/Kconfig
@@ -1125,6 +1125,7 @@ config XEN
 config FORCE_MAX_ZONEORDER
        int
        default "14" if (ARM64_64K_PAGES && TRANSPARENT_HUGEPAGE)
+       default "13" if (ARCH_THUNDER && !ARM64_64K_PAGES && !RHEL_DIFFERENCES)
        default "12" if (ARM64_16K_PAGES && TRANSPARENT_HUGEPAGE)
        default "11"
        help


> 
> Hence the problem and why it hasn't been solved yet.  It is annoying and I
> would love to fix it, but this solution just moves the breakage from
> os-build to ark-latest. :-(
> 
> Therefore,
> 
> Nacked-by: Don Zickus <dzickus@xxxxxxxxxx>
> 
> > 
> > Fixes: dd028d261347 ("[redhat] Sync up ARK's Fedora config tree with Fedora's dist-git")
> > Signed-off-by: Prarit Bhargava <prarit@xxxxxxxxxx>
> > Cc: honli@xxxxxxxxxx
> > Cc: jcline@xxxxxxxxxx
> > ---
> >  .../fedora/generic/arm/aarch64/CONFIG_FORCE_MAX_ZONEORDER       | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/redhat/configs/fedora/generic/arm/aarch64/CONFIG_FORCE_MAX_ZONEORDER b/redhat/configs/fedora/generic/arm/aarch64/CONFIG_FORCE_MAX_ZONEORDER
> > index b730690db048..8220d67904ea 100644
> > --- a/redhat/configs/fedora/generic/arm/aarch64/CONFIG_FORCE_MAX_ZONEORDER
> > +++ b/redhat/configs/fedora/generic/arm/aarch64/CONFIG_FORCE_MAX_ZONEORDER
> > @@ -1,4 +1,4 @@
> >  # We technically want this to be 13 for Fedora with 4K pages but that's only
> >  # an option with an out of tree patch. Keep this 11 for compatibility until
> >  # we figure out what we want here
> > -CONFIG_FORCE_MAX_ZONEORDER=13
> > +CONFIG_FORCE_MAX_ZONEORDER=11
> > -- 
> > 2.26.2
> > _______________________________________________
> > kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
> > To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
> > Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> > List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux