Thank you for helping, modinfo and uname -a doesn't match, first returns 5.9.8-2-MANJARO SMP preempt mod_unload modversions, second returns 5.9.11-3-MANJARO. The Linux kernel and headers installed do match (5.9.11-3). I have a v4l2loopback DKMS module working well.
Search Criteria
Package Details: it87-dkms-git 152.2b8b4fe-3
Package Actions
Git Clone URL: | https://aur.archlinux.org/it87-dkms-git.git (read-only, click to copy) |
---|---|
Package Base: | it87-dkms-git |
Description: | Linux Driver for ITE LPC chips |
Upstream URL: | https://github.com/frankcrawford/it87 |
Keywords: | driver IT87 sensor thermal |
Licenses: | GPL |
Conflicts: | it87-frankcrawford-dkms-git |
Provides: | it87 |
Submitter: | codeworkx |
Maintainer: | codebling |
Last Packager: | codebling |
Votes: | 40 |
Popularity: | 0.065424 |
First Submitted: | 2017-03-30 17:34 (UTC) |
Last Updated: | 2020-11-30 21:00 (UTC) |
opale95 commented on 2020-12-07 18:01 (UTC)
codebling commented on 2020-12-05 18:11 (UTC)
@stardust95 I can't reproduce this, and upstream has not changed. It doesn't seem like this is a problem with this package. Is it possible that your kernel and kernel headers are out of sync? Is it possible you did a partial update of your system? Do you have other DKMS modules, and are these working? Does the text in modinfo it87 | grep vermagic
match uname -a
?
opale95 commented on 2020-12-05 13:21 (UTC)
Since last updates, the module can't be loaded anymore. dmesg outputs this: "it87: disagrees about version of symbol module_layout" and modprobe outputs this "modprobe: ERROR: could not insert 'it87': Exec format error"
codebling commented on 2020-11-30 21:01 (UTC)
@aravance done, thanks for the patch and for spotting the problems! I'll have to go through my other DKMS packages and check to see if they need the same changes
codebling commented on 2020-11-30 20:56 (UTC)
@aravance sorry, this somehow slid through the cracks. I'll make the changes.
aravance commented on 2020-11-24 18:27 (UTC) (edited on 2020-11-24 18:28 (UTC) by aravance)
@codebling Yeah, sorry. I copied the wrong diff. I've edited the previous comment to the correct diff. I'm not entirely sure why some packages have no clean, I'm guessing they may be packages that are overriding an existing module? This one is doing that, so it's possible it is not strictly necessary in this case too.
codebling commented on 2020-11-24 17:24 (UTC)
@aravance looks like a a diff against a different package? But I see what you mean. /extramodules/kernel/drivers/hwmon
should perhaps become /kernel/drivers/hwmon
? With regards to the CLEAN
directive, where would I see files that are not being cleaned? I see that many DKMS packages have no CLEAN
directive at all, does this mean they are ignoring that step completely?
aravance commented on 2020-11-24 16:50 (UTC) (edited on 2020-11-24 18:25 (UTC) by aravance)
I found one more bug, I was using a different git repo, so I missed it with the first revision.
According to https://linux.die.net/man/8/dkms
DEST_MODULE_LOCATION[#]=
This directive specifies the destination where a module should be installed to, once compiled. It also is used for finding original_modules. This is a required directive, except as noted below. This directive must start with the text "/kernel" which is in reference to /lib/modules/<kernelversion>/kernel.
...
This git diff will address that:
EDIT: use the right package
diff --git a/PKGBUILD b/PKGBUILD
index 62033f4..6345de7 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -3,7 +3,7 @@
_pkgbase=it87
pkgname=it87-dkms-git
pkgver=152.2b8b4fe
-pkgrel=2
+pkgrel=3
pkgdesc="Linux Driver for ITE LPC chips"
arch=('x86_64' 'i686')
url="https://github.com/frankcrawford/it87"
@@ -18,7 +18,7 @@ source=("$_pkgbase::git+https://github.com/frankcrawford/it87.git"
"it87.conf")
sha256sums=('SKIP'
- '51d03e3b02223233832bdd4268b088ff28636bc47b682b098f738246b96766d8'
+ 'f325b751c8a81416a75c2c1e7a7bc9ca46ae0fa3b44d4ccc09593274be1b2dc7'
'acdc488d1505e891ed6259b29428d4b27d26d18e3ea170f017b930390d6420e7')
pkgver() {
diff --git a/dkms.conf b/dkms.conf
index b2bcbf1..aa6c5b6 100644
--- a/dkms.conf
+++ b/dkms.conf
@@ -3,5 +3,5 @@ PACKAGE_VERSION="@PKGVER@"
MAKE[0]="make -C $kernel_source_dir M=$dkms_tree/$PACKAGE_NAME/$PACKAGE_VERSION/build modules"
CLEAN="make -C $kernel_source_dir M=$dkms_tree/$PACKAGE_NAME/$PACKAGE_VERSION/build clean"
BUILT_MODULE_NAME[0]="@_PKGBASE@"
-DEST_MODULE_LOCATION[0]="/extramodules/kernel/drivers/hwmon"
+DEST_MODULE_LOCATION[0]="/kernel/drivers/hwmon"
AUTOINSTALL="yes"
codebling commented on 2020-11-23 16:56 (UTC)
@aravance nice catch. Fixed! Thanks.
aravance commented on 2020-11-23 16:12 (UTC) (edited on 2020-11-23 16:13 (UTC) by aravance)
The dkms.conf is leaving a dirty dkms tree for me when upgrading the kernel. The following update to dkms.conf fixed it:
git diff
diff --git a/dkms.conf b/dkms.conf
index 6200bb2..b2bcbf1 100644
--- a/dkms.conf
+++ b/dkms.conf
@@ -1,7 +1,7 @@
PACKAGE_NAME="@_PKGBASE@"
PACKAGE_VERSION="@PKGVER@"
MAKE[0]="make -C $kernel_source_dir M=$dkms_tree/$PACKAGE_NAME/$PACKAGE_VERSION/build modules"
-CLEAN="make clean"
+CLEAN="make -C $kernel_source_dir M=$dkms_tree/$PACKAGE_NAME/$PACKAGE_VERSION/build clean"
BUILT_MODULE_NAME[0]="@_PKGBASE@"
DEST_MODULE_LOCATION[0]="/extramodules/kernel/drivers/hwmon"
AUTOINSTALL="yes"
Pinned Comments
codebling commented on 2021-02-21 09:06 (UTC)
TL;DR for those running
linux
5.11 - you need to install thepahole
package if this package won't build. (see comments below for more details)codebling commented on 2020-10-23 15:36 (UTC)
I've updated this package to use the frankcrawford (most developed fork) upstream and am removing the
it87-frankcrawford-dkms-git
package.If you find another repo on GitHub that is more up to date, please comment.