[lkml]   [2008]   [Jan]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] KGDB: refactor kconfig menu
    Jason Wessel wrote:
    > Jan Kiszka wrote:
    >> This is an attempt to improve the so far, well, unfortunate Kconfig menu
    >> organization of KGDB. Most notably, it pushes all sub-entries into their
    >> own menuconfig, removes the (IMHO) unneeded "Method for KGDB
    >> communication" choice, and ensures everything is indented properly. This
    >> should keep the original semantic while reducing the number of knobs the
    >> user is confronted with.
    > While it is probably reasonable to put the kgdb options into a sub
    > entry, you cannot get rid of the "Method for KGDB communication"
    > choice, unless you re-implement it another way.
    > It is designed to implement the following:
    > 1) You can chose to build 1 and only 1 KGDB I/O module into the kernel
    > 2) Or you select modules only and can you can build as many additional
    > KGDB I/O modules as you like.
    > 3) Additional I/O modules can be built and you can load one at run time
    > if you want to use another I/O method vs #1

    I see no reason why these cases cannot be enforced later on (when there
    are >1 I/O modules) via Kconfig depends-on rules. But maybe I'm
    overseeing something, please correct me.

    > KGDB's Kconfig options are designed to allow you to have alternate ways
    > to select static or run time configuration of the I/O connection to the
    > KGDB core, IE: over serial, over ethernet, over USB or what ever module
    > you would like to implement.

    Well, I'm looking at this from the user perspective now, and as a user I
    don't understand why I shouldn't be able to configure my setup via the
    tristates of the involved I/O drivers (+ some underlying dependencies).


    [unhandled content-type:application/pgp-signature]
     \ /
      Last update: 2008-01-27 19:17    [W:0.023 / U:0.180 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site