Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932775AbcJLKxo (ORCPT ); Wed, 12 Oct 2016 06:53:44 -0400 Received: from mga11.intel.com ([192.55.52.93]:20995 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932615AbcJLKxi (ORCPT ); Wed, 12 Oct 2016 06:53:38 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,333,1473145200"; d="scan'208";a="771715658" From: Jani Nikula To: "Sun\, Jing A" , Daniel Vetter Cc: Andrzej Hajda , Takashi Iwai , Emil Velikov , "linux-kernel\@vger.kernel.org" , "dri-devel\@lists.freedesktop.org" , "Vetter\, Daniel" , Thierry Reding Subject: RE: [PATCH]"drm: change DRM_MIPI_DSI module type from "bool" to "tristate". In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <87h98ktxow.fsf@intel.com> <87lgxvqlg3.fsf@intel.com> <20161012065136.GH20761@phenom.ffwll.local> Date: Wed, 12 Oct 2016 13:52:46 +0300 Message-ID: <87h98hyh35.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1115 Lines: 29 On Wed, 12 Oct 2016, "Sun, Jing A" 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