From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shuah Khan Subject: Re: [PATCH v2 0/2] selftests/pstore: add pstore test script Date: Tue, 6 Oct 2015 11:59:09 -0600 Message-ID: <56140BED.7040507@osg.samsung.com> References: <20151002114637.20957.61769.stgit@arietta> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20151002114637.20957.61769.stgit@arietta> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Hiraku Toyooka , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: Tony Luck , Kees Cook , linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Anton Vorontsov , Mark Salyzyn , Colin Cross , Seiji Aguchi , Shuah Khan List-Id: linux-api@vger.kernel.org On 10/02/2015 05:46 AM, Hiraku Toyooka wrote: > These scripts include test cases which check pstore behavior. This > is useful to avoid regressions of pstore. > > Changes since v1: > * Exit with 1 in pstore/Makefile when pstore_crash_test failed. > * Create helper functions to make the tests much more readable. > * Use /sys/module/pstore/parameters/backend to check pstore backend is > registered. > * Show content of /sys/module/.../backend and /proc/cmdline for debug > * Give UUID to each execution of test script. > * Write unique test string with UUID into pmsg. > * Check only one test string pattern appears in pmsg after crash. > * Check UUID written to pmsg matches across crash. > * Don't touch panic_on_oops because it is not necessary for reboot on crash > > (v1: http://www.kernelhub.org/?msg=831044&p=2) > > I also confirmed that these scripts work fine with kdump reboot with > kernel boot parameter 'crash_kexec_post_notifiers'. Thanks. Applied to linux-kselftest next for 4.4 thanks, -- Shuah -- Shuah Khan Sr. Linux Kernel Developer Open Source Innovation Group Samsung Research America (Silicon Valley) shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org | (970) 217-8978