linux-kernel - Re: Makefile: make modules install output

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAK7LNATFuRk7zeuPec4F2S2C_0y4FcgPhev8A9xixC9O38--DQ@mail.gmail.com>
Date:   Wed, 9 Jun 2021 22:54:54 +0900
From:   Masahiro Yamada <masahiroy@...nel.org>
To:     Julia Lawall <julia.lawall@...ia.fr>
Cc:     Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: Makefile: make modules install output

On Wed, Jun 9, 2021 at 7:16 PM Julia Lawall <julia.lawall@...ia.fr> wrote:
>
> make modules install used to end up with something like
>
>   INSTALL sound/usb/usx2y/snd-usb-usx2y.ko
>   INSTALL sound/x86/snd-hdmi-lpe-audio.ko
>   INSTALL sound/xen/snd_xen_front.ko
>   INSTALL virt/lib/irqbypass.ko
>   DEPMOD  5.12.0-rc2myfix-00015-gc6f886546cb8
>
> From v5.13-rc1, it gives:
>
>   STRIP   /lib/modules/5.13.0-rc1myfix/kernel/fs/btrfs/btrfs.ko
>   STRIP   /lib/modules/5.13.0-rc1myfix/kernel/fs/xfs/xfs.ko
>   STRIP   /lib/modules/5.13.0-rc1myfix/kernel/net/mac80211/mac80211.ko
>   STRIP   /lib/modules/5.13.0-rc1myfix/kernel/drivers/net/ethernet/mellanox/mlx5/core/mlx5_core.ko
>   STRIP   /lib/modules/5.13.0-rc1myfix/kernel/drivers/gpu/drm/amd/amdgpu/amdgpu.ko
>   DEPMOD  /lib/modules/5.13.0-rc1myfix
>
> Is the full path name considered to be an improvement?

I think so.
It is clear where the files are being installed to.

Another motivation was rather to make it aligned with the convention.

Kbuild log convention is this form:

  [ACTION]   [DESTINATION]


For example, when Kbuild is building kernel/foo.c
into kernel/foo.o

  CC      kernel/foo.o

is shown.


In this case, Kbuild is installing kernel/fs/btrfs/btrfs.ko
to /lib/modules/5.13.0-rc1myfix/kernel/fs/btrfs/btrfs.ko

So, I think it is more consistent to show the destination
rather than the source.




> For the names of
> the modules, it doesn't much matter, but the DEPMOD line was a convenient
> way to get the full kernel name for subsequent rebooting.
>
> thanks,
> julia



-- 
Best Regards
Masahiro Yamada