From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932392AbcHHTx4 (ORCPT ); Mon, 8 Aug 2016 15:53:56 -0400 Received: from mail-yw0-f194.google.com ([209.85.161.194]:36718 "EHLO mail-yw0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932343AbcHHTxp (ORCPT ); Mon, 8 Aug 2016 15:53:45 -0400 Date: Mon, 8 Aug 2016 16:53:39 -0300 From: Gustavo Padovan To: Pavel Machek Cc: Gustavo Padovan , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, dri-devel@lists.freedesktop.org, Daniel Stone , Arve =?iso-8859-1?B?SGr4bm5lduVn?= , Riley Andrews , Daniel Vetter , Rob Clark , Greg Hackmann , John Harrison , Maarten Lankhorst , Sumit Semwal , Gustavo Padovan Subject: Re: [PATCH 0/7] de-stage SW_SYNC validation frawework Message-ID: <20160808195339.GB2478@joana> Mail-Followup-To: Gustavo Padovan , Pavel Machek , Gustavo Padovan , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, dri-devel@lists.freedesktop.org, Daniel Stone , Arve =?iso-8859-1?B?SGr4bm5lduVn?= , Riley Andrews , Daniel Vetter , Rob Clark , Greg Hackmann , John Harrison , Maarten Lankhorst , Sumit Semwal , Gustavo Padovan References: <1466437993-21573-1-git-send-email-gustavo@padovan.org> <20160718191245.GA9872@joana> <20160724222111.GA13630@kroah.com> <20160807215158.GB27135@amd> <20160808190812.GN2484@joana> <20160724150030.GA6992@xo-6d-61-c0.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160724150030.GA6992@xo-6d-61-c0.localdomain> User-Agent: Mutt/1.6.2 (2016-07-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2016-07-24 Pavel Machek : > On Mon 2016-08-08 16:08:12, Gustavo Padovan wrote: > > 2016-08-07 Pavel Machek : > > > > > 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?