From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965712AbbKDQKK (ORCPT ); Wed, 4 Nov 2015 11:10:10 -0500 Received: from mx1.redhat.com ([209.132.183.28]:32975 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965605AbbKDQKI (ORCPT ); Wed, 4 Nov 2015 11:10:08 -0500 Date: Wed, 4 Nov 2015 10:10:06 -0600 From: Josh Poimboeuf To: Jiri Kosina Cc: Miroslav Benes , Seth Jennings , Vojtech Pavlik , linux-kernel@vger.kernel.org, live-patching@vger.kernel.org, "Cyril B." Subject: Re: [PATCH] livepatch: Cleanup page permission changes Message-ID: <20151104161006.GB29899@treble.redhat.com> References: <132b018cfe2c6e4cef4d1b62ac3ed70333734111.1446494413.git.jpoimboe@redhat.com> <20151103174228.GN27488@treble.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23.1-rc1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 04, 2015 at 10:18:29AM +0100, Jiri Kosina wrote: > On Tue, 3 Nov 2015, Josh Poimboeuf wrote: > > > It's probably a good idea to keep the patches bisectable, so I made this > > a separate patch which applies on top of the first one. > > > > (Note that it completely removes all the code from the first patch, so > > there's no need for a v2 of the first patch which would have had > > Miroslav's suggested style changes.) > > I like this patch and it's something I'd like to queue for 4.4, but given > the fact that the original "[PATCH] x86/livepatch: Fix crash with > !CONFIG_DEBUG_SET_MODULE_RONX" wasn't queued in any tree yet, I don't > think that sending it as a followup patch is of any use. > > So if you agree, I'll just fold the two patches together and use the > changelog below, and queue it for merge with Linus. I kept them separate because the first patch is a low risk bug fix, and the second is a slightly higher risk cleanup. Would it make sense to put the first one into 4.4 and queue the second one for 4.5? -- Josh