From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e06smtp07.uk.ibm.com ([195.75.94.103]:45903 "EHLO e06smtp07.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751652AbbIOGJd (ORCPT ); Tue, 15 Sep 2015 02:09:33 -0400 Received: from /spool/local by e06smtp07.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 15 Sep 2015 07:09:32 +0100 Date: Tue, 15 Sep 2015 08:09:27 +0200 From: Heiko Carstens To: Ben Hutchings Cc: Willy Tarreau , linux-kernel@vger.kernel.org, stable@vger.kernel.org, Martin Schwidefsky Subject: Re: [PATCH 2.6.32 18/62] s390/hibernate: fix save and restore of kernel text section Message-ID: <20150915060927.GA4051@osiris> References: <20150912225607.341984869@1wt.eu> <1442283045.29883.1.camel@decadent.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1442283045.29883.1.camel@decadent.org.uk> Sender: stable-owner@vger.kernel.org List-ID: On Tue, Sep 15, 2015 at 03:10:45AM +0100, Ben Hutchings wrote: > On Sun, 2015-09-13 at 00:56 +0200, Willy Tarreau wrote: > > 2.6.32-longterm review patch. If anyone has any objections, please let me know. > > > > ------------------ > > > > From: Heiko Carstens > > > > commit d74419495633493c9cd3f2bbeb7f3529d0edded6 upstream. > > > > Sebastian reported a crash caused by a jump label mismatch after resume. > > This happens because we do not save the kernel text section during suspend > > and therefore also do not restore it during resume, but use the kernel image > > that restores the old system. > [...] > > I don't think this is needed because jump labels aren't implemented in > 2.6.32. But maybe there was some other mechanism that changes the > kernel text on s390? Broken jump labels were the symptons when the problem was reported. However we lose everything that modifies kernel text without this patch. If it's not jump labels then there is still kprobes around where we have the same problem. Probably the patch description should have been better.