Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932585AbbDNPzw (ORCPT ); Tue, 14 Apr 2015 11:55:52 -0400 Received: from blu004-omc4s19.hotmail.com ([65.55.111.158]:52367 "EHLO BLU004-OMC4S19.hotmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753940AbbDNPzn (ORCPT ); Tue, 14 Apr 2015 11:55:43 -0400 X-TMN: [c5nZtgSPe0KdMUipptvUQoubiIW3Beet] X-Originating-Email: [minfei.huang@hotmail.com] Message-ID: Date: Tue, 14 Apr 2015 23:55:36 +0800 From: Minfei Huang To: Josh Poimboeuf CC: sjenning@redhat.com, jkosina@suse.cz, vojtech@suse.cz, live-patching@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] livepatch: Fix the bug if the function name is larger than KSYM_NAME_LEN-1 References: <20150413231305.GD4412@treble.hsd1.ky.comcast.net> <20150414045739.GG4412@treble.hsd1.ky.comcast.net> <20150414051113.GH4412@treble.hsd1.ky.comcast.net> <20150414053247.GI4412@treble.hsd1.ky.comcast.net> <20150414151110.GL4412@treble.hsd1.ky.comcast.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20150414151110.GL4412@treble.hsd1.ky.comcast.net> User-Agent: Mutt/1.5.23 (2014-03-12) X-OriginalArrivalTime: 14 Apr 2015 15:55:41.0477 (UTC) FILETIME=[7574C550:01D076CB] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1976 Lines: 51 On 04/14/15 at 10:11P, Josh Poimboeuf wrote: > On Tue, Apr 14, 2015 at 01:45:49PM +0800, Minfei Huang wrote: > > On 04/14/15 at 12:32P, Josh Poimboeuf wrote: > > > On Tue, Apr 14, 2015 at 01:29:50PM +0800, Minfei Huang wrote: > > > > > > > > For end user, they may know litter about restriction of kallsyms and > > > > livepatch. How can they know the restriction that function name is > > > > limited to 127? > > > > > > As I mentioned above, I think kallsyms.c should fail the build if it > > > encounters a symbol longer than KSYM_NAME_LEN. > > > > > > > I dont think it is a good idea to handle this case like that. The > > function name is only for human recognization. Why the compiler fails > > to build it? > > Well, the function name isn't only for human recognition. kpatch-build > generates patch modules automatically. It assumes that the compiled > function name matches the kallsyms name. And I'd guess that a lot of > other code (both in-kernel and user space tools) make the same > assumption. > > Not to mention that most humans would also make the same assumption... Yes. The assumption is correct for most case. It is significance for livepatch to support extra module, because in my opinion kernel is more stable than the third module. So it is more important, if the livepatch can patch all sorts of patch. For dynamic function name, I think it is simple to avoid it. Usually, we will use ominity to handle a bunch of machines. So it is simple, if we use script to get the function address and build the patch. Josh, is there any chance to accept my patches? It may be important somewhile that system can not restart without schedule to reload the fixed-module. Thanks Minfei > > -- > Josh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/