On Thu, 23 Feb 2017 08:17:12 -0800
Laura Abbott <labbott(a)redhat.com> wrote:
On 02/23/2017 06:43 AM, Dan Horák wrote:
> Hi,
>
> I'm in the process to reduce the Fedora s390x kernel's size and
> build time by disabling drivers for devices that can't appear on the
> mainframe. But I'm getting 2 kinds of errors
>
> first is
> ...
> + echo '# s390'
> + cat .config
> + CheckConfigs configs/kernel-4.11.0-s390x.config
> temp-kernel-4.11.0-s390x.config
> + ./check_configs.awk configs/kernel-4.11.0-s390x.config
> temp-kernel-4.11.0-s390x.config
> + '[' -s .mismatches ']'
> + echo 'Error: Mismatches found in configuration files'
> Error: Mismatches found in configuration files
> + cat .mismatches
> Found # CONFIG_HID is not set, after generation, had CONFIG_HID m
> in Fedora tree Found # CONFIG_HWMON is not set, after generation,
> had CONFIG_HWMON m in Fedora tree Found # CONFIG_I2C is not set,
> after generation, had CONFIG_I2C m in Fedora tree Found #
> CONFIG_NEW_LEDS is not set, after generation, had CONFIG_NEW_LEDS y
> in Fedora tree
> + exit 1
>
> This is after setting the options ^^^ to "N" in their files under
> baseconfig/s390x/.
>
This is a check designed to make sure that what we specify in the
Fedora configurations match what is generated via Kconfig. The output
is stating that the Fedora tree has (e.g.) CONFIG_HID=m set but when
run against the kernel configuration generation it comes out as
# CONFIG_HID is not set. The easiest way to fix this is to make
all those options match whatever is being generated (so mark all
those options not set)
>
> the second one is
> ...
> + rm temp-kernel-4.11.0-ppc64p7.config
> + for i in '*.config'
> + cat kernel-4.11.0-s390x.config
> + mv kernel-4.11.0-s390x.config .config
> ++ head -1 .config
> ++ cut -b 3-
> + Arch=s390
> + make ARCH=s390 listnewconfig
> + grep -E '^CONFIG_'
> + '[' -s .newoptions ']'
> + cat .newoptions
> CONFIG_RESET_CONTROLLER
> + exit 1
>
This is saying that CONFIG_RESET_CONTROLLER isn't specified
as a Kconfig value for s390x. You need to explicitly set it.
thanks to both you and Justin (seems it was private reply only), the
updates will follow
Dan