Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1691708

Re: [PATCH 6/8] x86: add MULTIUSER dependency for KVM

From Arnd Bergmann <arnd@arndb.de>
Newsgroups linux.kernel
Subject Re: [PATCH 6/8] x86: add MULTIUSER dependency for KVM
Date 2017-07-19 16:20 +0200
Message-ID <u4XZv-1ff-3@gated-at.bofh.it> (permalink)
References <u4WK5-fH-17@gated-at.bofh.it> <u4WK7-fH-41@gated-at.bofh.it> <u4XZv-1ff-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wed, Jul 19, 2017 at 4:11 PM, Radim Krčmář <rkrcmar@redhat.com> wrote:
> 2017-07-19 14:53+0200, Arnd Bergmann:
>> KVM tries to select 'TASKSTATS', which had additional dependencies:
>>
>> warning: (KVM) selects TASKSTATS which has unmet direct dependencies (NET && MULTIUSER)
>>
>> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>> ---
>
> Hm, do you know why Kconfig warns instead of propagating the
> dependencies?

Kconfig propagates 'depends on' dependencies, but cannot turn a 'select'
into 'depends on', as those two mean different things.

Another solution to the problem would be to use 'depends on TASKSTATS'.

Generally speaking, using 'select' to turn on a user-visible option is a bad
idea, but blindly turning those 'select' into 'depends on' is also dangerous,
as it can break configurations of existing users that would here end up with
neither TASKSTATS nor KVM after a 'make oldconfig'.

      Arnd

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/8] x86: randconfig warning fixes Arnd Bergmann <arnd@arndb.de> - 2017-07-19 15:00 +0200
  [PATCH 3/8] x86: math-emu: avoid bogus -Wint-in-bool-context warning Arnd Bergmann <arnd@arndb.de> - 2017-07-19 15:00 +0200
    [tip:x86/urgent] x86/fpu/math-emu: Avoid bogus  -Wint-in-bool-context warning tip-bot for Arnd Bergmann <tipbot@zytor.com> - 2017-07-20 12:30 +0200
  [PATCH 5/8] x86: silence build with "make -s" Arnd Bergmann <arnd@arndb.de> - 2017-07-19 15:00 +0200
    [tip:x86/urgent] x86/build: Silence the build with "make -s" tip-bot for Arnd Bergmann <tipbot@zytor.com> - 2017-07-20 12:40 +0200
  [PATCH 6/8] x86: add MULTIUSER dependency for KVM Arnd Bergmann <arnd@arndb.de> - 2017-07-19 15:00 +0200
    Re: [PATCH 6/8] x86: add MULTIUSER dependency for KVM Radim Krčmář <rkrcmar@redhat.com> - 2017-07-19 16:20 +0200
      Re: [PATCH 6/8] x86: add MULTIUSER dependency for KVM Arnd Bergmann <arnd@arndb.de> - 2017-07-19 16:20 +0200
        Re: [PATCH 6/8] x86: add MULTIUSER dependency for KVM Radim Krčmář <rkrcmar@redhat.com> - 2017-07-19 18:20 +0200
          Re: [PATCH 6/8] x86: add MULTIUSER dependency for KVM Paolo Bonzini <pbonzini@redhat.com> - 2017-07-23 15:50 +0200
  [PATCH 2/8] x86: math-emu: possible uninitialized variable use Arnd Bergmann <arnd@arndb.de> - 2017-07-19 15:00 +0200
    [tip:x86/urgent] x86/fpu/math-emu: Fix possible uninitialized  variable use tip-bot for Arnd Bergmann <tipbot@zytor.com> - 2017-07-20 12:30 +0200
  [PATCH 1/8] perf/x86: shut up false-positive -Wmaybe-uninitialized warning Arnd Bergmann <arnd@arndb.de> - 2017-07-19 15:00 +0200
    [tip:x86/urgent] perf/x86: Shut up false-positive  -Wmaybe-uninitialized warning tip-bot for Arnd Bergmann <tipbot@zytor.com> - 2017-07-20 12:30 +0200

csiph-web