Re: [PATCH 0/7] de-stage SW_SYNC validation frawework

From: Gustavo Padovan
Date: Mon Aug 08 2016 - 15:54:00 EST


2016-07-24 Pavel Machek <pavel@xxxxxx>:

> On Mon 2016-08-08 16:08:12, Gustavo Padovan wrote:
> > 2016-08-07 Pavel Machek <pavel@xxxxxx>:
> >
> > > On Sun 2016-07-24 15:21:11, Greg Kroah-Hartman wrote:
> > > > On Mon, Jul 18, 2016 at 04:12:45PM -0300, Gustavo Padovan wrote:
> > > > > Hi,
> > > > >
> > > > > Do you think there is time to get this in for 4.8?
> > > >
> > > > No, it was too late on my end, due to travel and vacation, sorry. I'll
> > > > queue it up for 4.9-rc1.
> > >
> > > Could we get some documentation what this does? Is it visilble to
> > > userspace?
> >
> > This interface is only intended for testing and validation, there are
> > ioctls on the debugfs file that can be accessed by userspace but there
> > isn't any exported kernel header with this info. The tester should know
> > and add a internal header to be able to access it. We want to prevent
> > people from misusing this feature by not advertising it nor providing
> > documentation.
>
> You are playing dangerous game here. debugfs is not normally considered stable,
> but otoh... ioctls on debugfs?
>
> Anyway, please provide some documentation. Kernel hackers need to know what this does.

Okay, where do you think is the best place? Would documentation inside
the .c file suffice for you?