linux-kernel - RE: [PATCH]"drm: change DRM_MIPI_DSI module type from "bool" to "tristate".

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 for Android: free password hash cracker in your pocket

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

Date:   Wed, 12 Oct 2016 13:52:46 +0300
From:   Jani Nikula <jani.nikula@...ux.intel.com>
To:     "Sun\, Jing A" <jing.a.sun@...el.com>,
        Daniel Vetter <daniel@...ll.ch>
Cc:     Andrzej Hajda <a.hajda@...sung.com>, Takashi Iwai <tiwai@...e.de>,
        Emil Velikov <emil.l.velikov@...il.com>,
        "linux-kernel\@vger.kernel.org" <linux-kernel@...r.kernel.org>,
        "dri-devel\@lists.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
        "Vetter\, Daniel" <daniel.vetter@...el.com>,
        Thierry Reding <treding@...dia.com>
Subject: RE: [PATCH]"drm: change DRM_MIPI_DSI module type from "bool" to "tristate".

On Wed, 12 Oct 2016, "Sun, Jing A" <jing.a.sun@...el.com> wrote:
> I think "installing a kernel with my changes for both drm and i915"
> takes more time and effort to complete than "only updating DRM/i915
> modules without rebuilding the whole kernel". In some cases, that's
> beneficial.

It's possible to change and rebuild and update just the drm and i915,
but you need to be careful to build against the same tree as the ones
you are replacing. This is like using out-of-tree modules (which is
something I can't recommend no matter what, but that's another
discussion).

However, this is completely different from planning to update drm and
i915 modules on a running production system by unloading the old ones
and probing the new ones. Don't do that. It will be a disaster.

> Also reloadablility is always a good thing to have and I truly hope
> Hajda/Iwai's patches would be accepted and merged.  No downside of it
> after all.

I think it's good to be able to unload and reload modules for debugging
and development, but not for normal use.

BR,
Jani.

-- 
Jani Nikula, Intel Open Source Technology Center