From debbugs-submit-bounces@debbugs.gnu.org Sat Apr 04 11:53:57 2020 Received: (at 40274) by debbugs.gnu.org; 4 Apr 2020 15:53:57 +0000 Received: from localhost ([127.0.0.1]:44604 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jKl7N-00065j-84 for submit@debbugs.gnu.org; Sat, 04 Apr 2020 11:53:57 -0400 Received: from mail-wr1-f68.google.com ([209.85.221.68]:41514) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jKl7L-00065G-IA for 40274@debbugs.gnu.org; Sat, 04 Apr 2020 11:53:56 -0400 Received: by mail-wr1-f68.google.com with SMTP id h9so12217489wrc.8 for <40274@debbugs.gnu.org>; Sat, 04 Apr 2020 08:53:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:references:date:in-reply-to:message-id :user-agent:mime-version; bh=8Lmg+FVhrUwrfpISyfOxAOwtA4V1h8upDZQ2PDpPrA8=; b=PWTdSdZ6FiqI3LXLhxDtg/v2NMrDH6H3VNmanHo4w7eVi78ZmQcP/SOsAZVSFglntZ g0N6sAMd/kiq+8tpkUsO4B0CQtHQLmhEAA3GL4P9aOvuboa+bSw9lcLutNPRWCcz/HGg 79SqaLo31AdSGKytkHbanga+Oip9UxwY/Zx2IVApK4a1yNFiApiZZqI/u1Wqo5KBORWu O8IwtTq+M7joK1sVCAQ7YMzG8r3Xu4XKzZNUycsC2DaiROm+xfas4x1u+cU4Cpmvi89p /o0ZpJugcP8lsDFBByhq5FeD4r3OXmnt05ORYkRFjAz0LWuDeLH1E6Uq2c29sWRftYao VBDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:references:date:in-reply-to :message-id:user-agent:mime-version; bh=8Lmg+FVhrUwrfpISyfOxAOwtA4V1h8upDZQ2PDpPrA8=; b=eJOQSQ5RAMJLtkfwEqj9/3Nz7xAZGxnKCS268ZdW+SGiF5oXgK7bTbEnq6NitIeK9r aI6E3g4aPOts80r7zLrpl+yBCO+84kZ3qop1osOCglQUVfZPJ3mf6NSkhb2xi42t8paX PWmDh3CjnSVRebyY8PavTmRi0HuIgWxyCLlno106EW4AyPDhd4z50UwVyA70HEidFYF2 X5k55wObuJk0QUTKYhs+3SkyOrx76jV8Om2+ElA9MXH1j6cfB+6bJclxI3ssGsBy/1O2 ZluURF9uR2PFmmfwiQzAhHowqURJG68gG+OvBTWSA4v+UnpEF2XoYtV7SVTRvTwaehYg 6F8w== X-Gm-Message-State: AGi0Puat9vLYYzrS1rQC8+CqHuxWIaVU0BmSiLNyvgZPBh8+ldA9ga4G HiNKJ0XS5pfiXLp6ELl8AVFDoSv5M/A= X-Google-Smtp-Source: APiQypIb2ria7fCkjbg2uLksF4RldYoTNBjfJ7jZn2dmcYU3aO8grwsjdO1abnLCZZaKGr7A9eC4Sg== X-Received: by 2002:a5d:6441:: with SMTP id d1mr15829478wrw.301.1586015629278; Sat, 04 Apr 2020 08:53:49 -0700 (PDT) Received: from meru ([2a01:cb18:832e:5f00:4928:41d8:3661:4718]) by smtp.gmail.com with ESMTPSA id 127sm17111097wmd.38.2020.04.04.08.53.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 04 Apr 2020 08:53:48 -0700 (PDT) From: Mathieu Othacehe To: Brice Waegeneire Subject: Re: [bug#40274] [PATCH v6 2/2] gnu: Add kernel-module-loader-service. References: <20200328135908.2540-1-brice@waegenei.re> <20200404151732.20341-1-brice@waegenei.re> <20200404151732.20341-3-brice@waegenei.re> Date: Sat, 04 Apr 2020 17:53:47 +0200 In-Reply-To: <20200404151732.20341-3-brice@waegenei.re> (Brice Waegeneire's message of "Sat, 4 Apr 2020 17:17:32 +0200") Message-ID: <87h7xztfg4.fsf@gmail.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: 0.0 (/) X-Debbugs-Envelope-To: 40274 Cc: 40274@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -1.0 (-) Hello Brice, I think what you discussed with Danny, separating module loading from configuration is very wise. > +The kernel module loader service allows one to load, at boot, loadable > +kernel modules---in opposition to built-in kernel modules which should > +be added to the @code{kernel-arguments} field instead. This is Using kernel-arguments you can configure built-in modules but you cannot force them to show-up, at least for most device drivers. For those device drivers, loading of built-in modules is based on hardware probing at boot-time. So I would just stick with: "The kernel module loader service allows one to load, at boot time, specific kernel modules." > +(define ddcci-config > + (plain-file "ddcci.conf" > + "options ddcci dyndbg delay=120")) Having the modprobe-service-type you proposed earlier, would be super nice to do that :) > + > +(operating-system > + ... > + (services (cons* (service kernel-module-loader-service-type > + '("ddcci" "ddcci_backlight")) > + (simple-service 'ddcci-config etc-service-type > + (list `("modprobe.d/ddcci.conf" > + ,ddcci-config))) Great! This seems really nice to me. Mathieu