unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Leo Famulari <leo@famulari.name>
To: John Kehayias <john.kehayias@protonmail.com>
Cc: "56200@debbugs.gnu.org" <56200@debbugs.gnu.org>
Subject: [bug#56200] [PATCH v2] gnu: linux-libre: Enable the AMD HSA driver.
Date: Tue, 28 Jun 2022 21:05:24 -0400	[thread overview]
Message-ID: <YrulVEX6Mz7uICLZ@jasmine.lan> (raw)
In-Reply-To: <e87igmgsRjri0FJY_3I3lojHFkfLLDjmCR2jUTCX6E384kgr_QAUY0JjK2bxBbg6nOKC6SDXjowggvlL-LTK3a9Crli_fVW5t6XrjLSM930=@protonmail.com>

On Wed, Jun 29, 2022 at 12:38:56AM +0000, John Kehayias wrote:
> I've attached a v2 of the patch that sets it for all the configurations where the option was listed. Note that a few had it "enabled" with setting it to "m", but I don't think that does anything, per my testing. And looking at the documentation, seems like it should be "y": https://www.kernelconfig.io/config_hsa_amd

I think the changes for our configs before version 5.4 are unnecessary.

I checked out the source code of linux-libre 4.9.316 and copied our 4.9
config and removed the CONFIG_HSA_AMD line.

Then, I did `make oldconfig` within ` guix environment linux-libre
--ad-hoc gcc-toolchain`, and it offered me a choice between N and m for
this option.

So, I think they changed this feature so that it can no longer be a
module in version 4.20. Check the "found in Linux kernels" info here:

https://cateee.net/lkddb/web-lkddb/HSA_AMD.html

I removed the changes to the configs for 4.9, 4.14, and 4.19, and pushed
to a WIP branch for final testing:

https://git.savannah.gnu.org/cgit/guix.git/log/?h=wip-linux-libre-5.18

I'll build kernels for x86_64 on the build farm, so substitutes will be
available for testing after a few hours.




  reply	other threads:[~2022-06-29 14:07 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-24 17:38 [bug#56200] Kernel 5.18 update Leo Famulari
2022-06-24 17:41 ` [bug#56200] [PATCH 1/3] gnu: Add linux-libre 5.18 Leo Famulari
2022-06-24 17:41   ` [bug#56200] [PATCH 2/3] gnu: linux-libre: Update to 5.18.6 Leo Famulari
2022-06-24 17:41   ` [bug#56200] [PATCH 3/3] gnu: Remove linux-libre 5.17 Leo Famulari
2022-06-24 18:25 ` [bug#56200] Kernel 5.18 update John Kehayias via Guix-patches via
2022-06-24 18:49   ` Leo Famulari
2022-06-26 14:57   ` John Kehayias via Guix-patches via
2022-06-26 20:02     ` Leo Famulari
2022-06-26 20:30       ` Tobias Geerinckx-Rice via Guix-patches via
2022-06-27  4:43 ` [bug#56200] [PATCH 4/4] linux-libre: Enable the AMD HSA driver John Kehayias via Guix-patches via
2022-06-28 20:30   ` Leo Famulari
2022-06-29  0:38 ` [bug#56200] [PATCH v2] gnu: " John Kehayias via Guix-patches via
2022-06-29  1:05   ` Leo Famulari [this message]
2022-06-29 12:55     ` bug#56200: " Leo Famulari
2022-06-29 16:17       ` [bug#56200] " John Kehayias via Guix-patches via

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YrulVEX6Mz7uICLZ@jasmine.lan \
    --to=leo@famulari.name \
    --cc=56200@debbugs.gnu.org \
    --cc=john.kehayias@protonmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).