heir driver and start sending the issues to
their side.
Lucas De Marchi
Regards
Aditya
On Fri, May 31, 2024 at 05:23:09PM GMT, Jeff Johnson wrote:
make allmodconfig && make W=1 C=1 reports:
WARNING: modpost: missing MODULE_DESCRIPTION() in lib/test_kmod.o
Add the missing invocation of the MODULE_DESCRIPTION() macro.
Signed-off-by: Jeff Johnson
Reviewed-by: Lucas D
softdep")
could indeed be solved with a weakdep, so I'm not sure why you'd want to
alias it as a "hard dep".
Lucas De Marchi
sugar, currently implemented as an alias for MODULE_WEAKDEP, so the actual
hard module dependencies can be expressed properly, and possibly handled
d
On Thu, Jul 25, 2024 at 04:39:40PM GMT, Steven Price wrote:
On 25/07/2024 15:29, Lucas De Marchi wrote:
On Thu, Jul 25, 2024 at 01:37:46PM GMT, Dragan Simic wrote:
Panfrost and Lima DRM drivers use devfreq to perform DVFS, which is
supported
on the associated platforms, while using
On Wed, Aug 21, 2024 at 10:58:43AM GMT, Nathan Chancellor wrote:
Did this conversation go anywhere? After the upgrade to kmod 33 in Arch
no, it stalled, thanks for reminding me. I will follow up on it.
Lucas De Marchi
On Tue, Dec 19, 2023 at 05:37:31PM GMT, Masahiro Yamada wrote:
On Thu, Dec 7, 2023 at 3:37 AM Lucas De Marchi wrote:
On Fri, Nov 10, 2023 at 01:13:53PM +0100, Michal Suchanek wrote:
>Hello,
>
>This is resend of the last patch in the series that adds prefix support
>to kernel mod
f. I compile out-of-tree modules just fine. See
https://git.kernel.org/cgit/utils/kernel/kmod/kmod.git/tree/testsuite/module-playground/Makefile
for an example.
> Any ideas?
>
> I'm not sure is this the right group, sorry.
You probably should CC lkml as well. Doing it now.
--
L
Hi Buddy,
This question is more for lkml than linux-modules. I'm CC'ing it.
--
Lucas De Marchi
On Tue, Apr 7, 2015 at 6:48 AM, Buddy wrote:
> HI:
> I want to get information for kernel symbol,so I use array of
> kallsyms_address
> and value of kallsyms_num_syms.But w
k with older versions of the
> hardware? Mika, do you have any memories about this?
I'm not sure if it's a hw bug or if it's just an oversight from previous
versions of this driver. So adding a quirk with correct versions will
be difficult IMO. We've been only testing this wi
f (!i_dev->pm_runtime_disabled)
>> - i2c_dw_init(i_dev);
>> + i2c_dw_enable(i_dev);
>
> This will not work if the device power gets removed (for example being
> put to D3cold) as it looses context.
Do you mean we should keep the i2c_dw_init() here?
On Tue, Jun 23, 2015 at 1:45 PM, wrote:
> Hello,
>
> Christian Ruppert/ALi_GVA/ALi wrote on 10.06.2015 17:05:16:
>> Mika Westerberg wrote on 10.06.
>> 2015 09:07:22:
>> > On Tue, Jun 09, 2015 at 03:29:01PM -0300, Lucas De Marchi wrote:
>> > > Hi Mika,
On Thu, Jun 4, 2015 at 5:22 PM, Rusty Russell wrote:
> Lucas De Marchi writes:
>>> + gzip and xz (but doesn't by default, at least as of version 18!).
>>
>> This is pretty much misleading. There's no such a default option. It's
>> like sayin
From: Lucas De Marchi
Add a CC_FEATURE_APPEND function that we can use to append the features
that are enabled/disabled. This will generate a single string in the
form "+FEATURE1 +FEATURE2 -FEATURE3".
---
m4/features.m4 | 48
1 file c
From: Lucas De Marchi
---
tools/depmod.c | 1 +
tools/insmod.c | 1 +
tools/kmod.c | 1 +
tools/modinfo.c | 1 +
tools/modprobe.c | 1 +
tools/rmmod.c| 1 +
6 files changed, 6 insertions(+)
diff --git a/tools/depmod.c b/tools/depmod.c
index 231b9ab..2a08b6e 100644
--- a/tools
From: Lucas De Marchi
Sample output:
$ ./tools/kmod --version
kmod version 20
+XZ +ZLIB -EXPERIMENTAL
Lucas De Marchi (3):
build: add function to build string of features
build: add relevant features to KMOD_FEATURES in config.h
tools: display features together with version information
From: Lucas De Marchi
Not all the features (i.e. those available in --enable-* or --with-*)
are really relevant to the final user. Create a KMOD_FEATURES definition
in config.h containing these features.
---
configure.ac | 7 +++
1 file changed, 7 insertions(+)
diff --git a/configure.ac b
removes the (non required anymore) enables and
disables.
Signed-off-by: Fabio Mello
Signed-off-by: Lucas De Marchi
---
These pictures explain a little more the consequence of letting the
enable+disable in the code:
http://pub.politreco.com/paste/TEK0011-before.jpg
http
On Sun, Jun 7, 2015 at 6:25 AM, Andreas Mohr wrote:
> Hi,
>
> On Sun, Jun 07, 2015 at 03:27:26AM -0300, lucas.de.mar...@gmail.com wrote:
>> From: Lucas De Marchi
>>
>> Sample output:
>>
>> $ ./tools/kmod --version
>> kmod version 20
>> +XZ +
On Sun, Jun 7, 2015 at 6:17 AM, Andreas Mohr wrote:
> Hi,
>
> On Sun, Jun 07, 2015 at 03:27:29AM -0300, lucas.de.mar...@gmail.com wrote:
>> From: Lucas De Marchi
>> diff --git a/tools/kmod.c b/tools/kmod.c
>> index c9d2cc8..28217df 100644
>> --- a/tools/kmod.c
&
used by
"make check". No changes to the tests are needed and all of them
can run fine.
Shortlog:
Lucas De Marchi (12):
build: use dolt
build: sync dolt with upstream
util: use UINT64_MAX
build: cache modules from playground
tools: Hide
uired
> in the help text though.
kmod supports gz since the first version and xz since version 3. So both
of them can be safely fall into "it's supported since the beginning of
kmod IMO".
Regarding the "default configuration", there's no such thing. Each distribut
On Wed, Jun 3, 2015 at 2:36 PM, Kay Sievers wrote:
> On Wed, Jun 3, 2015 at 7:30 PM, Lucas De Marchi
> wrote:
>> On Mon, Jun 1, 2015 at 3:26 AM, Rusty Russell wrote:
>>> Andreas Mohr writes:
>>>> Hi,
>>>>
>>>> I just had a not so nic
On Wed, Jun 3, 2015 at 10:30 PM, Rusty Russell wrote:
> Lucas De Marchi writes:
>> On Mon, Jun 1, 2015 at 3:26 AM, Rusty Russell wrote:
>>> Andreas Mohr writes:
>>>> Hi,
>>>>
>>>> I just had a not so nice experience
>>>> when f
controller keeps its state
>> between disable/enable. Change the target address is also allowed.
>>
>> So, this patch replaces the initialization on module resume with a
>> simple enable, and removes the (non required anymore) enables and
>> disables.
>>
&
view.
>
> Since kbuild already produces "modules.order" and "modules.builtin"
> files, why not just name it "modules.builtin.modinfo" to keep the
I agree with modules.builtin.modinfo
> names consistent with what is already there? It clearly conveys that
> this file contains modinfo for builtin modules.
>
> I'll leave it up to Lucas to decide if the file format is OK for kmod.
> With the modinfo dump, kmod could then decide what to do with the
> information, append to modules.alias{,.bin}, etc.
I think it's ok with the current format. It's simple enough. I would
delay merging it until
the userspace counterpart is implemented though. So to make sure we
have everything we need.
I can work on that in ~1 or 2 weeks.
thanks
Lucas De Marchi
>
> Also, I think this file needs to be documented in
> Documentation/kbuild/kbuild.txt.
>
> Thanks,
>
> Jessica
--
Lucas De Marchi
configure)
- Bug fixes
- Ignore ill-formed kernel command line, e.g. with
"ivrs_acpihid[00:14.5]=AMD0020:0"
option in it
- Fix some memory leaks
- Fix 0-length builtin.alias.bin: it needs at least the index header
Shortlog is below:
Lucas De Marchi (17):
. I only sent this
> patch since I hadn't found the correct fix.
Feel free to CC kmod mailing list in these cases
(linux-modu...@vger.kernel.org). We've added this message to kmod
exactly to catch problems like this and help people doing the drivers.
Once I found out the problem with vhc
On Wed, Feb 19, 2014 at 07:46:02PM +, Mark Jackson wrote:
> On 18/02/14 18:52, Greg Kroah-Hartman wrote:
> > On Tue, Feb 18, 2014 at 02:27:25AM -0300, Lucas De Marchi wrote:
> >> From: Lucas De Marchi
> >>
> >> Signed-off-by: Lucas De Marchi
> >
eds to be in setup_requires
python: Issue #15: resovled by using setuptools_cython
Lucas De Marchi (23):
man: Change my contact email address
Remove duplicate includes
testsuite: add test for kcmdline params with no value
testsuite: give a more suitable description to
ht
thing.
>
> In fact, kmod's modprobe *never* used the O_NONBLOCK
> (ie. KMOD_REMOVE_NOWAIT) flag, until it was finally enforced in
>
> commit 7ab8804448377fb6b8854f2dd288608db01bb43b
> Author: Lucas De Marchi
> Date: Fri Sep 20 01:30:07 201
you think it'd be still good to
patch kmod on distros, so at least modprobe -r uses O_NONBLOCK? Or
having this patch in kernel is sufficient?
--
Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
e nodes are
created by init even if the module itself is not loaded. The right
module will be loaded when the node is first accessed. And that
includes the "loop" module you mention.
$ kmod static-nodes | grep -A3 loop
Module: loop
Device node: /dev/loop-control
Ty
From: Lucas De Marchi
Commit bfacbb9 (Bluetooth: Use devname:vhci module alias for virtual HCI
driver) added the module alias to hci_vhci module so it's possible to
create the /dev/vhci node. However creating an alias without
specifying the minor doesn't allow us to create the
From: Lucas De Marchi
These patches are not intended for merging as is... clearly we have at least
one problem: fram's minor is clashing with logibm's. I don't know what to do
here.
The first patch also adds comments on those minors that are defined but not
used. Is that ok?
Luc
From: Lucas De Marchi
Signed-off-by: Lucas De Marchi
---
include/linux/miscdevice.h | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/include/linux/miscdevice.h b/include/linux/miscdevice.h
index 7bb6148..51e26f3 100644
--- a/include/linux/miscdevice.h
+++ b
From: Lucas De Marchi
Signed-off-by: Lucas De Marchi
---
Documentation/devices.txt | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devices.txt b/Documentation/devices.txt
index 04356f5..0ea1cc2 100644
--- a/Documentation/devices.txt
+++ b/Documentation/devices.txt
fails.
Close /sys/module/$NAME directory if opening /proc/module fails.
Lucas De Marchi (17):
Update .travis.yml
Add strndupa to missing.h
Make sure there's NUL byte at the end of strndupa
testsuite: Add module playground dir
Add gitignore to module playground
de
ife examples of this, I'm writing the LSM as it currently is, used
> without an initramfs. :)
The way you put now as well as the code are at least different from
what it's written in the commit message. I was expecting that this
modpin would allow module loading only from a trusted
g is below.
Lucas De Marchi
---
Anders Olofsson (1):
build: Allow disabling maintainer mode
John Spencer (1):
testsuite: fix usage of reserved names
Lucas De Marchi (19):
Fix usage of readdir_r()
build: remove check for typeof
util: Add ALIGN_POWER2
libkmod-hash: al
since I'm CC'ing linux-modules.
--
Lucas De Marchi
>
> Signed-off-by: Masami Hiramatsu
> ---
> include/linux/module.h |6 ++
> include/uapi/linux/module.h |1 +
> kernel/module.c | 28
> 3 files chang
ed up :)
>
> Anyway, this is not needed if the lockup option is dropped
> from the series. I send this for testing.
Ok. I'm not sure it's clear... I'm waiting for feedback on the kernel
patches in order to proceed with any review here. I'm not really
convinced we want this
Hi Christian,
On Mon, May 2, 2016 at 7:11 AM, Christian Ruppert
wrote:
> Dear Lucas,
>
> On 22.04.2016 17:19, Lucas De Marchi wrote:
>> CC'ing Christian.
>>
>> On Fri, Apr 22, 2016 at 12:08 PM, Lucas De Marchi
>> wrote:
>>> Disabling the adapter af
f it works for you. I don't have yet the BMM150 evaluation
> board. It should be available in few weeks.
>
> I'll just leave this patch here in case anyone steps in to test it.
It will take a while for my test board to arrive here, so expect delay
on testing this.
thanks
Lucas De Marchi
of the patch. I remember it
being fixed in the last version (or at least I could not reproduce).
I'll test again tomorrow with your script.
Lucas De Marchi
7;s less boilerplate code for
each test.
Shortlog is below:
Cristian Rodríguez (1):
Add missing O_CLOEXEC in kmod_module_get_size()
Holger Obermaier (1):
libkmod-index.c: Fix error message
Lucas De Marchi (35):
Update TODO
Move macro.h to shared directory
On Mon, Feb 23, 2015 at 12:51 PM, Michal Marek wrote:
> On 2015-02-23 15:30, Lucas De Marchi wrote:
>> This could be particularly bad if in a kernel version an option was
>> tristate and in a new version it changed to boolean. I'm not sure if
>> this is common to happe
On Tue, Feb 24, 2015 at 8:42 AM, Harish Jenny Kandiga Nagaraj
wrote:
>
> On Monday 23 February 2015 09:21 PM, Michal Marek wrote:
>> On 2015-02-23 15:30, Lucas De Marchi wrote:
> Can we add some flag like
> KMOD_PROBE_FORCE_DIRECTORY_CHECK = 0x00040,
> and pass it to kmod_mo
is codespell dictionary
I'm not a perl guru, but couldn't we have a single option like
--codespell / --codespell=FILE ?
Other than that,
Acked-By: Lucas De Marchi
--
Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Lucas De Marchi
UAPI headers have a _UAPI_ as prefix, which is removed during
headers_install. If it's put as a suffix it will not be removed and will
be the only header with UAPI in the header guard macro.
Signed-off-by: Lucas De Marchi
---
include/uapi/linux/kdbus.h | 4 ++--
1
From: Lucas De Marchi
UAPI headers have a _UAPI_ as prefix, which is removed during
headers_install. If it's put as a suffix it will not be removed and will
be the only header with UAPI in the header guard macro.
Signed-off-by: Lucas De Marchi
Reviewed-by: David Herrmann
---
include
e elapsed values shorter than the configured interval.
Tracing for wakeup events with ftrace[1] I also get the same results... the time
difference between 2 wakeup events for ~30% of events are shorter the
configured interval.
I'm currently running with kernel version 3.18.6. Is there anythin
On Wed, Mar 4, 2015 at 5:06 AM, Clemens Ladisch wrote:
> Lucas De Marchi wrote:
>> I was debugging my application and noticed that a timerfd event was being
>> triggered *before* the timer expires.
>>
>> I reduced the scope of the program to test a single timerfd and m
On Wed, Feb 18, 2015 at 8:40 PM, Rusty Russell wrote:
> Lucas De Marchi writes:
>> On Wed, Feb 18, 2015 at 2:07 AM, Rusty Russell wrote:
>> Yeah, I just thought (an wanted that) the attributes were being
>> created first and then hooked up in the sysfs tree under
>>
On Thu, Feb 19, 2015 at 12:25 AM, Rusty Russell wrote:
> Lucas De Marchi writes:
>> On Wed, Feb 18, 2015 at 8:40 PM, Rusty Russell wrote:
>>> Lucas De Marchi writes:
>>>> On Wed, Feb 18, 2015 at 2:07 AM, Rusty Russell
>>>> wrote:
>>>> Yea
On Thu, Feb 19, 2015 at 12:25 AM, greg KH wrote:
> On Wed, Feb 18, 2015 at 11:19:14PM -0200, Lucas De Marchi wrote:
>> On Wed, Feb 18, 2015 at 8:40 PM, Rusty Russell wrote:
>> > Lucas De Marchi writes:
>> >> On Wed, Feb 18, 2015 at 2:07 AM, Rusty Russell
&
add the necessary checks for the builtin index.
Otherwise I can take a look on this until the end of this week.
--
Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
On Thu, Feb 19, 2015 at 10:32 AM, Harish Jenny Kandiga Nagaraj
wrote:
>
> On Thursday 19 February 2015 04:00 PM, Lucas De Marchi wrote:
>> On Thu, Feb 19, 2015 at 3:49 AM, Harish Jenny Kandiga Nagaraj
>> wrote:
>>>> Harrish, in your patch if you just change the &q
ules" in the builtin index?
2) should we actually check /sys/module/ to report a
module as builtin or just stop doing that and rely solely in the
index? Initially I'd like to do the opposite, but given the race in
deciding this I'm favoring the index.
thanks
--
Lucas De Marchi
--
On Wed, Feb 25, 2015 at 8:53 AM, Michal Marek wrote:
> On Tue, Feb 24, 2015 at 10:02:55PM -0300, Lucas De Marchi wrote:
>> On Mon, Feb 23, 2015 at 12:51 PM, Michal Marek wrote:
>> > On 2015-02-23 15:30, Lucas De Marchi wrote:
>> >> My questions are:
>> >&g
On Thu, Feb 19, 2015 at 12:35 PM, Harish Jenny Kandiga Nagaraj
wrote:
>
> On Thursday 19 February 2015 07:32 PM, Harish Jenny Kandiga Nagaraj wrote:
>> On Thursday 19 February 2015 06:13 PM, Lucas De Marchi wrote:
>>> On Thu, Feb 19, 2015 at 10:32 AM, Harish Jenny Kandiga Na
ld: make sure to test for -Wflag instead of -Wno-flag
Harish Jenny K N (1):
Fix race while loading modules
Lucas De Marchi (78):
NEWS: Fix typos
README: move mailing list information to top
build-sys: Don't add libkmod subdirectory to include path
build-sys: Don'
usty, was there any changes in this area in the kernel recently?
Or is the creation of /sys/module/ and
/sys/module//initstate not atomic?
See patch below.
--
Lucas De Marchi
>
> Experiments done to prove the issue in kmod.
> Added sleep in kernel module.c at the place of creatio
On Wed, Feb 18, 2015 at 2:07 AM, Rusty Russell wrote:
> Lucas De Marchi writes:
>> On Tue, Feb 17, 2015 at 10:56 AM, Harish Jenny K N
>> wrote:
>>> usecase: two sd cards are being mounted in parallel at same time on
>>> dual core. example modules which are getti
0us
After:
i2c-transfer-time -w -a 0x40 -x 6 -n 2 -- 0 0 0xd0 0x07 0 0 0xd0
0x07 0 0 0xd0 0x07 0 0 0xd0 0x07
num_transfers=2
transfer_time_avg=470.256050us
Signed-off-by: Lucas De Marchi
---
drivers/i2c/busses/i2c-designware-core.c | 48 ---
CC'ing Christian.
On Fri, Apr 22, 2016 at 12:08 PM, Lucas De Marchi
wrote:
> Disabling the adapter after each transfer is pretty bad for sensors and
> other devices doing small transfers at a high rate. It slows down the
> transfer rate a lot since each of them have to wait the
t a package of LSM303D and L3GD20 that could reuse
the drivers already present?
Lucas De Marchi
h bit 5 checked and send an update.
Lucas De Marchi
From: Lucas De Marchi
Disabling the adapter after each transfer is pretty bad for sensors and
other devices doing small transfers at a high rate. It slows down the
transfer rate a lot since each of them have to wait the adapter to be
enabled again.
It was done in order to avoid the adapter to
s
78e1c89 pinctrl: baytrail: Use raw_spinlock for locking
The ones to Cherryview could be sent too but I couldn't test them.
Leaving the patch below for reference.
thanks
Lucas De Marchi
> drivers/pinctrl/intel/pinctrl-baytrail.c | 21 -
> 1 file changed, 16 insert
ython requirements
modprobe: Update error message when path is missing
Change default log level
depmod: Remove unprinted debug messages
depmod: Don't fall back to uname on bad version
Lucas De Marchi (4):
Add README.md file with build details
libkmod-module: fix retu
>
> [1]
> https://lore.kernel.org/linux-modules/20191004094136.166621-1-maenn...@google.com/
>
> Reported-by: Stefan Wahren
> Fixes: 8651ec01daed ("module: add support for symbol namespaces.")
> Cc: Masahiro Yamada
> Cc: Lucas De Marchi
> Cc: Jessica Yu
p
Jan Engelhardt (1):
kmod: build: cure compiler warnings showing up externally
Luca Bruno (1):
libkmod-module: check for NULL before accessing pointers
Lucas De Marchi (17):
testsuite: raise arch image to 3GB
man: depmod: remove deprecated -m option
mkosi: all
DPLL state left over by
> broken SNB BIOSen")
>
> It seems this was cherry-picked incorrectly :-(
while the cherry-pick was correct, the macro is different on
drm-intel-fixes. IS_GEN(dev_priv, 6) needs to be converted to
IS_GEN6(dev_priv).
imply because we don't have
parameters for that module
always annoyed me.
Lucas De Marchi
.modinfo ?
>
> It is much easier to create and it has almost the same format as the
> modules. So I think it will be easier to parse in kmod.
adding a modules.builtin.alias with the same format of modules.alias
means during modprobe we would only need to load one more file to
lookup aliases. That doesn't mean the kernel built system should do it
though. The same way it's depmod job to create the
modules.alias{,.bin}, we could leave this to depmod if it's in fact
useful to split the information.
I think your version is simple enough and we would get more
information that would be useful for modinfo.
It would indeed be nice to output something useful in "modinfo ext4".
In kmod, I think we would create modules.builtin.modinfo{,.bin} and
just add the aliases to modules.alias{,.bin},
This would keep the names consistent with what is already there.
thanks
Lucas De Marchi
>
> --
> Rgrds, legion
>
CC'ing lkml and Rusty to get opinions on this.
On Sat, Apr 23, 2016 at 3:18 PM, Lubomir Rintel wrote:
> On Wed, 2016-04-13 at 01:11 -0300, Lucas De Marchi wrote:
>> On Tue, Mar 29, 2016 at 7:27 AM, Lubomir Rintel
>> wrote:
>> >
>> > On Fri, 2016-03-04
Shortlog is below
Bartosz Golaszewski (1):
module: fix a memory leak
James Minor (1):
libkmod: Fix handling of quotes in kernel command line
Lucas De Marchi (15):
libkmod: fix use of strcpy
util: fix warning of equal values on logical OR
testsuite: include stdio.h
buses created more than 20+ years ago.
(Unfortunately) They are still used today for all kind of sensors.
Boards like RPi, beaglebone, minnowboard expose the pins so we can
actually talk to those sensors, plugging in anyone we'd like to. For
some of them for example there are IIO drivers
Hi!
CC'ing Rusty and mailing lists
On Thu, Jan 28, 2016 at 8:40 PM, Samson Yeung wrote:
>
> On 01/28/2016 11:21 AM, Lucas De Marchi wrote:
>>
>> Hi Samson,
>>
>> On Wed, Jan 27, 2016 at 5:32 PM, Samson Yeung
>> wrote:
>>>
>>> Hi
Hi!
On Fri, Jan 29, 2016 at 3:54 AM, Rusty Russell wrote:
> Lucas De Marchi writes:
>> Hi!
>>
>> CC'ing Rusty and mailing lists
>
> Thanks.
>
>> Rusty and ohers: it looks like both CONFIG_CRC32 and
>> CONFIG_CRYPTO_CRC32 can be compiled as module,
nother id?
Other than that, I tested it on the MinnowBoard Max with a Drone Lure
connected and it works fine.
thanks
Lucas De Marchi
ODULE_DEVICE_TABLE(i2c, ads1015_id);
> +
> +static struct i2c_driver ads1015_driver = {
> + .driver = {
> + .name = ADS1015_DRV_NAME,
same here, otherwise we will have an i2c:ads1015 alias
Lucas De Marchi
"ELK", reg_val);
drivers/gpu/drm/i915/gem/i915_gem_stolen.c: * Whether ILK really reuses
the ELK register for this is unclear.
drivers/gpu/drm/i915/intel_pm.c: * Not 100% sure which way ELK should
go here as the
drivers/gpu/drm/i915/intel_pm.c: * assume ELK doesn&
modules: Add kernel parameter to blacklist modules
as a way to overcome broken initrd generation afaics. Either kernel
command line (using modprobe.blacklist)
or /etc/modprobe.d options are honoured by libkmod and allow a
sufficiently privileged user to bypass it.
+Rusty, +Prarit: is there anything this module parameter is covering
that I'm missing?
For the changes here,
Reviewed-by: Lucas De Marchi
thanks
Lucas De Marchi
> goto free_copy;
> }
>
> --
> 2.27.0
>
On Tue, Sep 1, 2020 at 12:56 PM Prarit Bhargava wrote:
>
>
>
> On 9/1/20 2:50 PM, Lucas De Marchi wrote:
> > On Sat, Aug 29, 2020 at 4:15 AM Qu Wenruo wrote:
> >>
> >> When kernel module loading failed, user space only get one of the
> >> following e
re use cases").
If you look at the definition of 'enum kmod_probe' you will see that
not all flags are returned as positive values - the only values that
have this behavior are the ones related to blacklist.
Could you rather patch the documentation?
thanks
Lucas De Marchi
g, nothing we can do about that :)
It's actually not a bug. For external modules it works pretty much as
intended. See DEPMOD.D(5): the search directive tells what's the
preference among the directories for modules with the same name.
depmod respects that order and put the right one into your
modules.dep.
This allows to put external modules in a different dir and also to
make backports of modules from recent to ancient kernels. These
modules with the same name are usually the same module, with a
different version. Of course what we have here and you are fixing is a
different story.
Reviewed-by: Lucas De Marchi
Lucas De Marchi
bind = ELF64_ST_BIND(info);
>
> - a[count].crc = crc;
> + a[count].crc = kmod_elf_resolve_crc(elf, crc, shndx);
> a[count].bind = kmod_symbol_bind_from_elf(bind);
> a[count].symbol = itr;
> slen = strlen(name);
> --
LGTM. I'll give this a try and apply. Do you think it's possible to
build an out-of-tree module with this option so we can add one to the
testsuite?
Lucas De Marchi
handle dependencies).
For this particular case, not going through the modprobe helper would
be a way to accomplish that since
you wouldn't need the corresponding binfmt module to run modprobe.
Udev handles module
loading via libkmod , but the only way to trigger it is via the rules
rather than via a request from kernel.
Lucas De Marchi
ask is different: "is this a module built-in" and for that we
have userpsace solution already suggested: /lib/modules/*/modules.builtin
yeah, that's right. The kernel build system provides that information
and depmo
ormat change and if possible submit the libkmod
support.
thanks
Lucas De Marchi
then load said module with a kernel *before* EXTENDED_MODVERSIONS
existed, it will see no modversion info on the module to check. This
will be true regardless of symbol length.
I'm not quite sure I understood y
er (2):
meson: Use short options for ln everywhere
meson: Fix setting an absolute customdir
Jan Engelhardt (1):
NEWS: ditch mention of libxz.so
Lucas De Marchi (2):
meson: Fix build with glibc 2.31
kmod 34.1
Tobias Stoeckmann (1):
build: support missing gtkdoci
o,
and \x1f, and whitespace, and similar looking characters
from different languages and emojis (except 🐧obviously).
Signed-off-by: Alexey Dobriyan
But I agree that it would be better to ban these chars from module
names. I don't think we'd ever merge such a module in tree neither.
Luca
likov
135 Tobias Stoeckmann
84 Lucas De Marchi
4 Martin Wilck
4 Martin Wilck
2 Michal Suchanek
2 dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
1 Christian Hesse
1 Enrico Joerns
1 Grayson Nocera
1 Max Kunzelm
not have
autotools and it'd be better to be prepared.
Fix it and also bring a few fixes to weakdep parsing.
Shortlog is below:
Emil Velikov (1):
NEWS: squash a couple of typos
Jakub Ślepecki (1):
libkmod: fix buffer-overflow in weakdep_to_char
Lucas De Marchi (3):
testsuite
T(test_underscores, .description = "check
implementation of underscores()");
I think it's reasonable to apply
https://lore.kernel.org/linux-modules/vqeq3ioklrgrf227zgdfho4virh74qrt5reoyptmzgktyronbr@c2mw32pqikft/
otherwise there's also an implicit conversion needed for
101 - 196 of 196 matches
Mail list logo
