unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] linux-initrd: Add AHCI modules.
@ 2014-07-18 15:39 David Thompson
  2014-07-18 21:40 ` Mark H Weaver
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: David Thompson @ 2014-07-18 15:39 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 537 bytes --]

One problem that Mark Weaver and I encountered when intalling the GNU
system on our Thinkpad X60s was that initrd couldn't boot from our SATA
hard disks without passing additional kernel modules in the
operating-system declaration.  I think it would make a lot of sense to
have these modules loaded by default, given the widespread use of SATA
disks.

I doubt that hardcoding the AHCI module names into linux-modules is the
preferred way to do this, but I wanted to propose something simple to
get the discussion started. :)

Thoughts?


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-linux-initrd-Add-AHCI-modules.patch --]
[-- Type: text/x-diff, Size: 931 bytes --]

From 2c9599720a8646a79fb3cb9517f7f0e6f1bfa25f Mon Sep 17 00:00:00 2001
From: David Thompson <dthompson2@worcester.edu>
Date: Fri, 18 Jul 2014 11:03:02 -0400
Subject: [PATCH] linux-initrd: Add AHCI modules.

* gnu/system/linux-initrd (linux-modules): Add them.
---
 gnu/system/linux-initrd.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/system/linux-initrd.scm b/gnu/system/linux-initrd.scm
index 9e39f2d..62f95c5 100644
--- a/gnu/system/linux-initrd.scm
+++ b/gnu/system/linux-initrd.scm
@@ -249,7 +249,8 @@ exception and backtrace!)."
 
   (define linux-modules
     ;; Modules added to the initrd and loaded from the initrd.
-    `(,@(if (or virtio? qemu-networking?)
+    `("libahci.ko" "ahci.ko" ; modules for SATA controllers
+      ,@(if (or virtio? qemu-networking?)
             virtio-modules
             '())
       ,@(if (find (file-system-type-predicate "cifs") file-systems)
-- 
2.0.0


[-- Attachment #3: Type: text/plain, Size: 136 bytes --]


-- 
David Thompson
Web Developer - Free Software Foundation - http://fsf.org
GPG Key: 0FF1D807
Support the FSF: https://fsf.org/donate

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-initrd: Add AHCI modules.
  2014-07-18 15:39 [PATCH] linux-initrd: Add AHCI modules David Thompson
@ 2014-07-18 21:40 ` Mark H Weaver
  2014-07-19  1:16 ` Jason Self
  2014-07-19 10:56 ` Ludovic Courtès
  2 siblings, 0 replies; 6+ messages in thread
From: Mark H Weaver @ 2014-07-18 21:40 UTC (permalink / raw)
  To: David Thompson; +Cc: guix-devel

David Thompson <dthompson2@worcester.edu> writes:

> One problem that Mark Weaver and I encountered when intalling the GNU
> system on our Thinkpad X60s was that initrd couldn't boot from our SATA
> hard disks without passing additional kernel modules in the
> operating-system declaration.  I think it would make a lot of sense to
> have these modules loaded by default, given the widespread use of SATA
> disks.
>
> I doubt that hardcoding the AHCI module names into linux-modules is the
> preferred way to do this, but I wanted to propose something simple to
> get the discussion started. :)
>
> Thoughts?

Another option would be to build AHCI support into the kernel.  I'm
currently testing a proposed patch.  I'm not sure whether this is better
or not; I'll leave it to others to decide :)

      Mark

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-initrd: Add AHCI modules.
  2014-07-18 15:39 [PATCH] linux-initrd: Add AHCI modules David Thompson
  2014-07-18 21:40 ` Mark H Weaver
@ 2014-07-19  1:16 ` Jason Self
  2014-07-19 10:59   ` Ludovic Courtès
  2014-07-19 10:56 ` Ludovic Courtès
  2 siblings, 1 reply; 6+ messages in thread
From: Jason Self @ 2014-07-19  1:16 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 610 bytes --]

I would prefer to not maintain different kernel configurations (for
Guix and non-Guix.)

If the initial ramdisk doesn't have enough stuff present to be able to
boot successfully, this seems to provide a good example of entire
directories of stuff to include:

http://www.linuxfromscratch.org/blfs/view/cvs/postlfs/initramfs.html

Even if it might make it bigger than manually tuning it this is
probably easier than figuring out which specific things need copying
and makes it bootable on a wide variety of computers with many
different setups. (And plus I hope for full disk encryption with LVM
some day.)  :)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-initrd: Add AHCI modules.
  2014-07-18 15:39 [PATCH] linux-initrd: Add AHCI modules David Thompson
  2014-07-18 21:40 ` Mark H Weaver
  2014-07-19  1:16 ` Jason Self
@ 2014-07-19 10:56 ` Ludovic Courtès
  2014-07-19 12:48   ` David Thompson
  2 siblings, 1 reply; 6+ messages in thread
From: Ludovic Courtès @ 2014-07-19 10:56 UTC (permalink / raw)
  To: David Thompson; +Cc: guix-devel

David Thompson <dthompson2@worcester.edu> skribis:

> From 2c9599720a8646a79fb3cb9517f7f0e6f1bfa25f Mon Sep 17 00:00:00 2001
> From: David Thompson <dthompson2@worcester.edu>
> Date: Fri, 18 Jul 2014 11:03:02 -0400
> Subject: [PATCH] linux-initrd: Add AHCI modules.
>
> * gnu/system/linux-initrd (linux-modules): Add them.

Ah yes, OK to push!

Ludo’.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-initrd: Add AHCI modules.
  2014-07-19  1:16 ` Jason Self
@ 2014-07-19 10:59   ` Ludovic Courtès
  0 siblings, 0 replies; 6+ messages in thread
From: Ludovic Courtès @ 2014-07-19 10:59 UTC (permalink / raw)
  To: Jason Self; +Cc: guix-devel

"Jason Self" <jason@bluehome.net> skribis:

> I would prefer to not maintain different kernel configurations (for
> Guix and non-Guix.)

This is an argument in favor of adding the modules to the initrd, rather
than changing the kernel config, right?  (If yes, I agree.)

> If the initial ramdisk doesn't have enough stuff present to be able to
> boot successfully, this seems to provide a good example of entire
> directories of stuff to include:
>
> http://www.linuxfromscratch.org/blfs/view/cvs/postlfs/initramfs.html

Yes, it doesn’t support LVM, dm-crypt, etc. but this is documented under
“Limitations”.  :-)

Or did you have something specific in mind?

Ludo’.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-initrd: Add AHCI modules.
  2014-07-19 10:56 ` Ludovic Courtès
@ 2014-07-19 12:48   ` David Thompson
  0 siblings, 0 replies; 6+ messages in thread
From: David Thompson @ 2014-07-19 12:48 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Ludovic Courtès <ludo@gnu.org> writes:

> David Thompson <dthompson2@worcester.edu> skribis:
>
>> From 2c9599720a8646a79fb3cb9517f7f0e6f1bfa25f Mon Sep 17 00:00:00 2001
>> From: David Thompson <dthompson2@worcester.edu>
>> Date: Fri, 18 Jul 2014 11:03:02 -0400
>> Subject: [PATCH] linux-initrd: Add AHCI modules.
>>
>> * gnu/system/linux-initrd (linux-modules): Add them.
>
> Ah yes, OK to push!

Pushed!

-- 
David Thompson
Web Developer - Free Software Foundation - http://fsf.org
GPG Key: 0FF1D807
Support the FSF: https://fsf.org/donate

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-07-19 12:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-18 15:39 [PATCH] linux-initrd: Add AHCI modules David Thompson
2014-07-18 21:40 ` Mark H Weaver
2014-07-19  1:16 ` Jason Self
2014-07-19 10:59   ` Ludovic Courtès
2014-07-19 10:56 ` Ludovic Courtès
2014-07-19 12:48   ` David Thompson

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).