Re: [PATCH v4 4/5] pstore: add pmsg

2015-02-03 Thread Lukasz Stelmach
On 03.02.2015 17:05, Mark Salyzyn wrote: > Thanks for your response. [...] > pstore is all about persistence. kmsg is not, until pstore supports > it. A user-space persistent storehouse felt appropriate to support > at the bottom layer. OK. Thanks, for the additional information. Like I said, I

Re: [PATCH v4 4/5] pstore: add pmsg

2015-02-03 Thread Kees Cook
On Tue, Feb 3, 2015 at 8:05 AM, Mark Salyzyn wrote: > Thanks for your response. > > On 01/30/2015 12:57 PM, Lukasz Stelmach wrote: >> >> On 28.01.2015 18:28, Mark Salyzyn wrote: >>> >>> - Precious little user-space content goes to kmsg (otherwise you >>> can ask why is there a syslogd?), there is

Re: [PATCH v4 4/5] pstore: add pmsg

2015-02-03 Thread Mark Salyzyn
Thanks for your response. On 01/30/2015 12:57 PM, Lukasz Stelmach wrote: On 28.01.2015 18:28, Mark Salyzyn wrote: - Precious little user-space content goes to kmsg (otherwise you can ask why is there a syslogd?), there is a reason for this, user space is notorious for containing Personal

Re: [PATCH v4 4/5] pstore: add pmsg

2015-02-03 Thread Lukasz Stelmach
On 03.02.2015 17:05, Mark Salyzyn wrote: Thanks for your response. [...] pstore is all about persistence. kmsg is not, until pstore supports it. A user-space persistent storehouse felt appropriate to support at the bottom layer. OK. Thanks, for the additional information. Like I said, I am no

Re: [PATCH v4 4/5] pstore: add pmsg

2015-02-03 Thread Mark Salyzyn
Thanks for your response. On 01/30/2015 12:57 PM, Lukasz Stelmach wrote: On 28.01.2015 18:28, Mark Salyzyn wrote: - Precious little user-space content goes to kmsg (otherwise you can ask why is there a syslogd?), there is a reason for this, user space is notorious for containing Personal

Re: [PATCH v4 4/5] pstore: add pmsg

2015-02-03 Thread Kees Cook
On Tue, Feb 3, 2015 at 8:05 AM, Mark Salyzyn saly...@android.com wrote: Thanks for your response. On 01/30/2015 12:57 PM, Lukasz Stelmach wrote: On 28.01.2015 18:28, Mark Salyzyn wrote: - Precious little user-space content goes to kmsg (otherwise you can ask why is there a syslogd?), there

Re: [PATCH v4 4/5] pstore: add pmsg

2015-01-30 Thread Lukasz Stelmach
On 28.01.2015 18:28, Mark Salyzyn wrote: > On 01/13/2015 04:16 PM, Łukasz Stelmach wrote: >>> A secured user-space accessible pstore object. Writes >>> to /dev/pmsg0 are appended to the buffer, on reboot >>> the persistent contents are available in >>> /sys/fs/pstore/pmsg-ramoops-[ID]. >>> >>> One

Re: [PATCH v4 4/5] pstore: add pmsg

2015-01-30 Thread Lukasz Stelmach
On 28.01.2015 18:28, Mark Salyzyn wrote: On 01/13/2015 04:16 PM, Łukasz Stelmach wrote: A secured user-space accessible pstore object. Writes to /dev/pmsg0 are appended to the buffer, on reboot the persistent contents are available in /sys/fs/pstore/pmsg-ramoops-[ID]. One possible use is

Re: [PATCH v4 4/5] pstore: add pmsg

2015-01-28 Thread Mark Salyzyn
On 01/13/2015 04:16 PM, Łukasz Stelmach wrote: A secured user-space accessible pstore object. Writes to /dev/pmsg0 are appended to the buffer, on reboot the persistent contents are available in /sys/fs/pstore/pmsg-ramoops-[ID]. One possible use is syslogd, or other daemon, can write messages,

Re: [PATCH v4 4/5] pstore: add pmsg

2015-01-28 Thread Mark Salyzyn
On 01/13/2015 04:16 PM, Łukasz Stelmach wrote: A secured user-space accessible pstore object. Writes to /dev/pmsg0 are appended to the buffer, on reboot the persistent contents are available in /sys/fs/pstore/pmsg-ramoops-[ID]. One possible use is syslogd, or other daemon, can write messages,

RE: [PATCH v4 4/5] pstore: add pmsg

2015-01-16 Thread Luck, Tony
> Cool, seems reasonable to me. Thanks for the respins! I've applied it - but I have a nagging worry about it being user accessible. We've limited access to some pstore features on platforms that only support backend drivers that write to limited lifetime flash storage - efivars & erst.

RE: [PATCH v4 4/5] pstore: add pmsg

2015-01-16 Thread Luck, Tony
Cool, seems reasonable to me. Thanks for the respins! I've applied it - but I have a nagging worry about it being user accessible. We've limited access to some pstore features on platforms that only support backend drivers that write to limited lifetime flash storage - efivars erst. Should

Re: [PATCH v4 4/5] pstore: add pmsg

2015-01-14 Thread Kees Cook
On Tue, Jan 13, 2015 at 4:16 PM, Mark Salyzyn wrote: > A secured user-space accessible pstore object. Writes > to /dev/pmsg0 are appended to the buffer, on reboot > the persistent contents are available in > /sys/fs/pstore/pmsg-ramoops-[ID]. > > One possible use is syslogd, or other daemon, can >

Re: [PATCH v4 4/5] pstore: add pmsg

2015-01-14 Thread Kees Cook
On Tue, Jan 13, 2015 at 4:16 PM, Mark Salyzyn saly...@android.com wrote: A secured user-space accessible pstore object. Writes to /dev/pmsg0 are appended to the buffer, on reboot the persistent contents are available in /sys/fs/pstore/pmsg-ramoops-[ID]. One possible use is syslogd, or other