Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758036Ab2FOWmg (ORCPT ); Fri, 15 Jun 2012 18:42:36 -0400 Received: from hrndva-omtalb.mail.rr.com ([71.74.56.122]:3506 "EHLO hrndva-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754239Ab2FOWme (ORCPT ); Fri, 15 Jun 2012 18:42:34 -0400 X-Authority-Analysis: v=2.0 cv=D8PF24tj c=1 sm=0 a=ZycB6UtQUfgMyuk2+PxD7w==:17 a=XQbtiDEiEegA:10 a=MgfmKKM-MiAA:10 a=5SG0PmZfjMsA:10 a=Q9fys5e9bTEA:10 a=meVymXHHAAAA:8 a=ayC55rCoAAAA:8 a=Kjee0quVX4u8c5mFZegA:9 a=PUjeQqilurYA:10 a=ZycB6UtQUfgMyuk2+PxD7w==:117 X-Cloudmark-Score: 0 X-Originating-IP: 74.67.80.29 Message-ID: <1339800152.25903.27.camel@gandalf.stny.rr.com> Subject: Re: [PATCH 3/6] pstore: Add persistent function tracing From: Steven Rostedt To: Colin Cross Cc: "Luck, Tony" , Anton Vorontsov , Greg Kroah-Hartman , Kees Cook , Frederic Weisbecker , Ingo Molnar , Arnd Bergmann , John Stultz , Shuah Khan , "arve@android.com" , Rebecca Schultz Zavin , Jesper Juhl , Randy Dunlap , Stephen Boyd , Thomas Meyer , Andrew Morton , Marco Stornelli , WANG Cong , "linux-kernel@vger.kernel.org" , "devel@driverdev.osuosl.org" , "linaro-kernel@lists.linaro.org" , "patches@linaro.org" , "kernel-team@android.com" Date: Fri, 15 Jun 2012 18:42:32 -0400 In-Reply-To: References: <20120615170659.GA4060@lizard> <1339780111-12075-3-git-send-email-anton.vorontsov@linaro.org> <3908561D78D1C84285E8C5FCA982C28F1931FC30@ORSMSX104.amr.corp.intel.com> <1339797308.25903.8.camel@gandalf.stny.rr.com> <3908561D78D1C84285E8C5FCA982C28F1931FC85@ORSMSX104.amr.corp.intel.com> <1339798214.25903.17.camel@gandalf.stny.rr.com> <3908561D78D1C84285E8C5FCA982C28F19320CC9@ORSMSX104.amr.corp.intel.com> Content-Type: text/plain; charset="ISO-8859-15" X-Mailer: Evolution 3.2.2-1+b1 Content-Transfer-Encoding: 7bit Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1116 Lines: 25 On Fri, 2012-06-15 at 15:23 -0700, Colin Cross wrote: > The decode also converts PCs to symbols, so for all the data to be > useful you have to boot the exact same kernel. You could > theoretically boot a different kernel with a compatible format, but > you would have to manually extract the PCs from the dump and addr2line > them against the correct vmlinux, which is probably more trouble than > it's worth. Also, the binary data is stored inside a pstore_ram > ringbuffer that may change formats between kernels. I would suggest > documenting that this is only supposed to work booting back to the > same kernel. OK, I'm fine with this too, if it is well documented that the same kernel must be used. Make it similar to hibernating the box. That is, you don't want to boot into a different kernel after you shutdown by hibernating to the hard drive. -- Steve -- 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/