linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: Kalle Valo <kvalo@kernel.org>
Cc: ath11k@lists.infradead.org, linux-wireless@vger.kernel.org,
	debian-kernel@lists.debian.org, debian-boot@lists.debian.org
Subject: Re: failed to insert STA entry for the AP (error -2)
Date: Wed, 7 Dec 2022 14:40:25 +0100	[thread overview]
Message-ID: <20221207134025.GA21691@lst.de> (raw)
In-Reply-To: <87cz8v2xb2.fsf@kernel.org>

Hi all,

adding the debian-kernel list due to issues with using debian-installer
daily snapshot to install on my brand new laptop with an ath11k_pci
supported wifi chip.

It turns out that while d-i comes with the ath11k and ath11k_pci
drivers, but misses the qrtr, qrtr-mki and michael_mic modules that
are needed for the driver to actually work and not just load.

On Wed, Dec 07, 2022 at 02:49:37PM +0200, Kalle Valo wrote:
> Thanks. But this makes me wonder is it sensible to randomly install a
> set of .ko files and drop the rest, like Debian's installer apparently
> does? The dependency for drivers is pretty well documented in Kconfig
> files, thanks to build testers testing with random configurations, but
> if the installer omits all that there will be problems just like you are
> experiencing. So for me MODULE_SOFTDEP() feels just like a band aid and
> not a robust solution.

I think a driver that a driver that has a runtime depedency on a
certain module, but doesn't import symbols is always going to be
somewhat problematic.  But I also agree that the arbitrary splitting
of kernel modules into separate packages for the installer, or
in fact not packaging them at all for the installer is rather
problematic.  I'm not sure what the rationale is behind that, but
I've added the debian-kernel and debian-boot lists.


> Though I am happy to take your MODULE_SOFTDEP() patch, just wondering if
> there is a better way to solve this. For example net/mac80211 (the
> 802.11 stack) has a lot of crypto dependencies:
> 
> 	select CRYPTO
> 	select CRYPTO_LIB_ARC4
> 	select CRYPTO_AES
> 	select CRYPTO_CCM
> 	select CRYPTO_GCM
> 	select CRYPTO_CMAC
> 	select CRC32
> 
> And it's not using MODULE_SOFTDEP() at all.

Yes.  I'm not quite sure how the packages for d-i select which
modules to include where, but given that other wifi hardware
seems to work in the installer they must have figured this out
somehow.

  reply	other threads:[~2022-12-07 13:40 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20221130122807.GA31720@lst.de>
     [not found] ` <87o7so4nr2.fsf@kernel.org>
     [not found]   ` <20221130125236.GA865@lst.de>
2022-11-30 13:12     ` ath11k vs ar_qrtr? Kalle Valo
2022-11-30 13:22       ` Christoph Hellwig
2022-11-30 17:18         ` Kalle Valo
2022-11-30 13:30       ` failed to insert STA entry for the AP (error -2) Christoph Hellwig
2022-12-02 18:46         ` Kalle Valo
2022-12-03 15:52           ` Christoph Hellwig
2022-12-07 12:49             ` Kalle Valo
2022-12-07 13:40               ` Christoph Hellwig [this message]
2022-12-07 14:24                 ` Cyril Brulebois
2022-12-08 18:44                 ` Kalle Valo

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

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

  git send-email \
    --in-reply-to=20221207134025.GA21691@lst.de \
    --to=hch@lst.de \
    --cc=ath11k@lists.infradead.org \
    --cc=debian-boot@lists.debian.org \
    --cc=debian-kernel@lists.debian.org \
    --cc=kvalo@kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    /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.
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 NNTP newsgroup(s).