From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753549AbbDAOA3 (ORCPT ); Wed, 1 Apr 2015 10:00:29 -0400 Received: from mail-ob0-f172.google.com ([209.85.214.172]:36607 "EHLO mail-ob0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753220AbbDAOA1 (ORCPT ); Wed, 1 Apr 2015 10:00:27 -0400 MIME-Version: 1.0 In-Reply-To: References: Date: Wed, 1 Apr 2015 22:00:15 +0800 Message-ID: Subject: Re: [PATCH] livepatch: Enhance livepatch to support remove patch module dynamically From: Minfei Huang To: jpoimboe@redhat.com, sjenning@redhat.com, jkosina@suse.cz, vojtech@suse.cz Cc: live-patching@vger.kernel.org, linux-kernel@vger.kernel.org, Minfei Huang Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2015-04-01 21:35 GMT+08:00 Jiri Kosina : > > On Wed, 1 Apr 2015, Minfei Huang wrote: > > > > Sorry, Use the gmail account to resend this patch, because the yahoo account cannt receive the maillist. > > > > ---- > > > > As mentioned in the annotation, the patch module would not permit to > > be removed once it is loaded. > > > > Kernel may crash in the case that the function in the patch module is > > called during removing. > > > > We will increase the module reference when the patch module is enable, > > so that the module cannt be removed. Once the module is disable, it will > > be removed. > > > > Signed-off-by: Minfei Huang > > --- > > kernel/livepatch/core.c | 47 +++++++++++++++++++++++++++++++---------------- > > 1 file changed, 31 insertions(+), 16 deletions(-) > > > > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > > index 3f9f1d6..0266950 100644 > > --- a/kernel/livepatch/core.c > > +++ b/kernel/livepatch/core.c > > @@ -502,6 +502,17 @@ static int __klp_disable_patch(struct klp_patch *patch) > > return 0; > > } > > > > +static int __klp_disable_patch_nolock(struct klp_patch *patch) > > +{ > > + int ret = 0; > > + > > + ret = __klp_disable_patch(patch); > > + if (ret) > > + return ret; > > + module_put(patch->mod); > > + return ret; > > +} > > + > > Your patch doesn't solve the problem at all. > > There is no guarantee that once __klp_disable_patch() returns noone is > using the old code any more. > Yes, thanks. The __klp_disable_patch only guarantees that we will never call the function in patch module. For now, patch module can never be removed from the kernel once it was loaded. It may be inconvenience if we want re-load a new patch module replace the old patch module. Thanks Minfei > -- > Jiri Kosina > SUSE Labs