GNU bug report logs - #32840
linux-libre 4.18.9: shpchp is no longer a module (now: built-in)

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 32840 in the body.
You can then email your comments to 32840 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Nils Gillmann <gillmann <at> n0.is>
To: bug-guix <at> gnu.org
Subject: linux-libre 4.18.9: shpchp is no longer a module (now: built-in)
Date: Tue, 25 Sep 2018 19:57:20 +0000

Hi,

at taler we need shpchp as a module in the initrd.

Before version 4.18.x, shpchp was a module we've build in guix' linux-libre,
you were able to just specify it for the initrd.

With 4.18.9, this module became a compile-in/built-in (see linux
commit log).  I want to avoid maintaining yet another kernel
definition for future complications if I can.  Can we get this module
built in by default?

I consider it a bug as the functionality was given before and
disappeared now, leaving our system unable to upgrade unless I write
some code around it.

Thanks.

Message #8 received at 32840 <at> debbugs.gnu.org (full text, mbox):

From: Mark H Weaver <mhw <at> netris.org>
To: Nils Gillmann <gillmann <at> n0.is>
Cc: 32840 <at> debbugs.gnu.org
Subject: Re: bug#32840: linux-libre 4.18.9: shpchp is no longer a module (now:
 built-in)
Date: Tue, 25 Sep 2018 19:36:18 -0400

Hi Nils,

Nils Gillmann <gillmann <at> n0.is> writes:

> at taler we need shpchp as a module in the initrd.
>
> Before version 4.18.x, shpchp was a module we've build in guix' linux-libre,
> you were able to just specify it for the initrd.
>
> With 4.18.9, this module became a compile-in/built-in (see linux
> commit log).  I want to avoid maintaining yet another kernel
> definition for future complications if I can.  Can we get this module
> built in by default?

Sure, that sounds sensible.  I'll push a config update soon.

     Thanks,
       Mark

Message #13 received at 32840-done <at> debbugs.gnu.org (full text, mbox):

From: Mark H Weaver <mhw <at> netris.org>
To: Nils Gillmann <gillmann <at> n0.is>
Cc: 32840-done <at> debbugs.gnu.org
Subject: Re: bug#32840: linux-libre 4.18.9: shpchp is no longer a module (now:
 built-in)
Date: Tue, 25 Sep 2018 23:24:09 -0400

Nils Gillmann <gillmann <at> n0.is> writes:

> at taler we need shpchp as a module in the initrd.
>
> Before version 4.18.x, shpchp was a module we've build in guix' linux-libre,
> you were able to just specify it for the initrd.
>
> With 4.18.9, this module became a compile-in/built-in (see linux
> commit log).  I want to avoid maintaining yet another kernel
> definition for future complications if I can.  Can we get this module
> built in by default?

Done in commit fe73352e8073ea0a0e6f6b5591f24395671998ab on master.
Closing this bug now, but please reopen if appropriate.

     Thanks!
       Mark

Message #16 received at 32840-done <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Mark H Weaver <mhw <at> netris.org>
Cc: 32840-done <at> debbugs.gnu.org, Nils Gillmann <gillmann <at> n0.is>
Subject: Re: bug#32840: linux-libre 4.18.9: shpchp is no longer a module (now:
 built-in)
Date: Wed, 26 Sep 2018 08:21:13 +0200

Mark H Weaver <mhw <at> netris.org> writes:

> Nils Gillmann <gillmann <at> n0.is> writes:
>
>> at taler we need shpchp as a module in the initrd.
>>
>> Before version 4.18.x, shpchp was a module we've build in guix' linux-libre,
>> you were able to just specify it for the initrd.
>>
>> With 4.18.9, this module became a compile-in/built-in (see linux
>> commit log).  I want to avoid maintaining yet another kernel
>> definition for future complications if I can.  Can we get this module
>> built in by default?
>
> Done in commit fe73352e8073ea0a0e6f6b5591f24395671998ab on master.
> Closing this bug now, but please reopen if appropriate.

I don’t understand why building it as a module is needed when upstream
decided to turn it into a built-in feature.

Could you please explain?

-- 
Ricardo

Message #19 received at 32840 <at> debbugs.gnu.org (full text, mbox):

From: Nils Gillmann <ng0 <at> n0.is>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: Mark H Weaver <mhw <at> netris.org>, 32840 <at> debbugs.gnu.org,
 Nils Gillmann <gillmann <at> n0.is>
Subject: Re: bug#32840: linux-libre 4.18.9: shpchp is no longer a module
 (now: built-in)
Date: Wed, 26 Sep 2018 16:27:19 +0000

Ricardo Wurmus transcribed 808 bytes:
> 
> Mark H Weaver <mhw <at> netris.org> writes:
> 
> > Nils Gillmann <gillmann <at> n0.is> writes:
> >
> >> at taler we need shpchp as a module in the initrd.
> >>
> >> Before version 4.18.x, shpchp was a module we've build in guix' linux-libre,
> >> you were able to just specify it for the initrd.
> >>
> >> With 4.18.9, this module became a compile-in/built-in (see linux
> >> commit log).  I want to avoid maintaining yet another kernel
> >> definition for future complications if I can.  Can we get this module
> >> built in by default?
> >
> > Done in commit fe73352e8073ea0a0e6f6b5591f24395671998ab on master.
> > Closing this bug now, but please reopen if appropriate.

Thanks! I'll get back if this didn't fix upgrading the server.

> I don’t understand why building it as a module is needed when upstream
> decided to turn it into a built-in feature.
> 
> Could you please explain?

There are modules [M] and boolean y/n "builtins" in Linux. We don't need
to build it as a module now, since upstream changed it to be a builtin.
It is not covered by whatever selection we have in the kernel config as
builtins. Since the module disappeared, users of the module were left
with the choice to remove it and either build their own kernel variation
or ask for it as a feature.

I asked for it, since Taler wanted as little reliance on third-party
repositories as possible, no matter how reliable the repositories are.

> -- 
> Ricardo
> 
> 
> 
> 

Message #24 received at 32840 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Mark H Weaver <mhw <at> netris.org>
Cc: 32840 <at> debbugs.gnu.org, Nils Gillmann <gillmann <at> n0.is>
Subject: Re: bug#32840: linux-libre 4.18.9: shpchp is no longer a module (now:
 built-in)
Date: Thu, 18 Oct 2018 00:59:05 +0200

Hi Mark,

Mark H Weaver <mhw <at> netris.org> skribis:

> Nils Gillmann <gillmann <at> n0.is> writes:
>
>> at taler we need shpchp as a module in the initrd.
>>
>> Before version 4.18.x, shpchp was a module we've build in guix' linux-libre,
>> you were able to just specify it for the initrd.
>>
>> With 4.18.9, this module became a compile-in/built-in (see linux
>> commit log).  I want to avoid maintaining yet another kernel
>> definition for future complications if I can.  Can we get this module
>> built in by default?
>
> Sure, that sounds sensible.  I'll push a config update soon.

I’ve reopened the bug because I think we need the same change as in
fe73352e8073ea0a0e6f6b5591f24395671998ab for AArch64 (at least it
appears to be needed on the OverDrive 1000.)

Could you take a look?

(I’m not entirely sure about the workflow to get the updated config from
“make oldconfig”…)

Thanks,
Ludo’.

Message #27 received at 32840 <at> debbugs.gnu.org (full text, mbox):

From: Mark H Weaver <mhw <at> netris.org>
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 32840 <at> debbugs.gnu.org, Nils Gillmann <gillmann <at> n0.is>
Subject: Re: bug#32840: linux-libre 4.18.9: shpchp is no longer a module (now:
 built-in)
Date: Wed, 17 Oct 2018 20:24:51 -0400

[Message part 1 (text/plain, inline)]

Hi Ludovic,

ludo <at> gnu.org (Ludovic Courtès) writes:

> Mark H Weaver <mhw <at> netris.org> skribis:
>
>> Nils Gillmann <gillmann <at> n0.is> writes:
>>
>>> at taler we need shpchp as a module in the initrd.
>>>
>>> Before version 4.18.x, shpchp was a module we've build in guix' linux-libre,
>>> you were able to just specify it for the initrd.
>>>
>>> With 4.18.9, this module became a compile-in/built-in (see linux
>>> commit log).  I want to avoid maintaining yet another kernel
>>> definition for future complications if I can.  Can we get this module
>>> built in by default?
>>
>> Sure, that sounds sensible.  I'll push a config update soon.
>
> I’ve reopened the bug because I think we need the same change as in
> fe73352e8073ea0a0e6f6b5591f24395671998ab for AArch64 (at least it
> appears to be needed on the OverDrive 1000.)
>
> Could you take a look?

Sure.  I've attached a proposed patch below.  Would you like to try it
and push it to master if it works?

> (I’m not entirely sure about the workflow to get the updated config from
> “make oldconfig”…)

FYI, my current method is:

* Unpack the latest linux-libre source.
* Copy the existing kernel configuration to .config in the source dir.
* Manually edit .config with desired configuration change.
* Within "guix environment linux-libre" run:

      ARCH=arm64 make oldconfig

  More generally, ARCH should be set to the value returned by
  'system->linux-architecture' in (gnu packages linux).
* Manually edit out some changes that are clearly irrelevant,
  such the addition of a top comment saying which compiler I have in my
  PATH.

Some day, I would prefer to have multiple smaller configuration files,
with a shared common configuration plus smaller architecture-specific
files, but we're not there yet.

     Thanks,
       Mark

[0001-gnu-linux-libre-Enable-SHPC-PCI-Hotplug-driver-on-AR.patch (text/x-patch, inline)]

From 41fdb333b05e300622c4c41aaced70fff2c6df7f Mon Sep 17 00:00:00 2001
From: Mark H Weaver <mhw <at> netris.org>
Date: Wed, 17 Oct 2018 20:09:19 -0400
Subject: [PATCH] gnu: linux-libre: Enable SHPC PCI Hotplug driver on ARM64.

Fixes <https://bugs.gnu.org/32840> on ARM64.

* gnu/packages/aux-files/linux-libre/4.18-arm64.conf: Add
CONFIG_HOTPLUG_PCI_SHPC=y.
---
 gnu/packages/aux-files/linux-libre/4.18-arm64.conf | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/aux-files/linux-libre/4.18-arm64.conf b/gnu/packages/aux-files/linux-libre/4.18-arm64.conf
index ab30f51d2..64b14b52c 100644
--- a/gnu/packages/aux-files/linux-libre/4.18-arm64.conf
+++ b/gnu/packages/aux-files/linux-libre/4.18-arm64.conf
@@ -1,6 +1,6 @@
 #
 # Automatically generated file; DO NOT EDIT.
-# Linux/arm64 4.18.0-gnu Kernel Configuration
+# Linux/arm64 4.18.14-gnu Kernel Configuration
 #
 CONFIG_ARM64=y
 CONFIG_64BIT=y
@@ -481,7 +481,7 @@ CONFIG_HOTPLUG_PCI=y
 CONFIG_HOTPLUG_PCI_ACPI=y
 CONFIG_HOTPLUG_PCI_ACPI_IBM=m
 CONFIG_HOTPLUG_PCI_CPCI=y
-# CONFIG_HOTPLUG_PCI_SHPC is not set
+CONFIG_HOTPLUG_PCI_SHPC=y
 
 #
 # PCI controller drivers
-- 
2.19.1

Message #30 received at 32840 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Mark H Weaver <mhw <at> netris.org>
Cc: 32840 <at> debbugs.gnu.org, Nils Gillmann <gillmann <at> n0.is>
Subject: Re: bug#32840: linux-libre 4.18.9: shpchp is no longer a module (now:
 built-in)
Date: Fri, 19 Oct 2018 00:16:54 +0200

Hi Mark,

Mark H Weaver <mhw <at> netris.org> skribis:

> ludo <at> gnu.org (Ludovic Courtès) writes:

[...]

>> I’ve reopened the bug because I think we need the same change as in
>> fe73352e8073ea0a0e6f6b5591f24395671998ab for AArch64 (at least it
>> appears to be needed on the OverDrive 1000.)
>>
>> Could you take a look?
>
> Sure.  I've attached a proposed patch below.  Would you like to try it
> and push it to master if it works?

It does fix the problem for me so I’ve pushed it as
402e60859da609e95d16a04a253977da88f4077e.

>> (I’m not entirely sure about the workflow to get the updated config from
>> “make oldconfig”…)
>
> FYI, my current method is:
>
> * Unpack the latest linux-libre source.
> * Copy the existing kernel configuration to .config in the source dir.
> * Manually edit .config with desired configuration change.
> * Within "guix environment linux-libre" run:
>
>       ARCH=arm64 make oldconfig
>
>   More generally, ARCH should be set to the value returned by
>   'system->linux-architecture' in (gnu packages linux).
> * Manually edit out some changes that are clearly irrelevant,
>   such the addition of a top comment saying which compiler I have in my
>   PATH.

OK, thanks for explaining!

> Some day, I would prefer to have multiple smaller configuration files,
> with a shared common configuration plus smaller architecture-specific
> files, but we're not there yet.

Yeah…

Anyway I’m happy to say I have GuixSD running on the OverDrive 1000!
There were a few more shenanigans (the openSuSE kernel had a few things
set to “y” that we’re setting to “m”), but then it went smoothly.

Thanks!

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.