> From: [email protected] [mailto:[email protected]] On Behalf Of Kees Cook > Sent: Wednesday, June 21, 2017 7:27 AM > > > + > > +static bool efi_capsule_pstore_disable = > > + IS_ENABLED(CONFIG_CAPSULE_PSTORE_DEFAULT_DISABLE); > > This can be marked __ro_after_init.
OK. Will mark it __ro_after_init in next version. > > + > > + rec->type = record->type; > > + rec->timestamp = get_seconds(); > > timestamp will already be correctly populated by the pstore API, so it should > be copied from there rather than using get_seconds() which may not be > functional during resume, etc. Looks like that the timestamp member "time" of struct pstore_record passed from pstore "fs/pstore/platform.c" -> pstore_dump() is not set for DMESG case. BR qiuxu
