From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751690AbcHIGFD (ORCPT ); Tue, 9 Aug 2016 02:05:03 -0400 Received: from mail-wm0-f68.google.com ([74.125.82.68]:35523 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750792AbcHIGFB (ORCPT ); Tue, 9 Aug 2016 02:05:01 -0400 Date: Tue, 9 Aug 2016 08:04:54 +0200 From: Daniel Vetter To: Pavel Machek Cc: Gustavo Padovan , Greg Kroah-Hartman , Gustavo Padovan , 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: <20160809060454.GB6232@phenom.ffwll.local> Mail-Followup-To: Pavel Machek , Gustavo Padovan , Greg Kroah-Hartman , Gustavo Padovan , linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, dri-devel@lists.freedesktop.org, Daniel Stone , Arve =?iso-8859-1?B?SGr4bm5lduVn?= , Riley Andrews , 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> X-Operating-System: Linux phenom 4.6.0-1-amd64 User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Jul 24, 2016 at 05:00:31PM +0200, Pavel Machek wrote: > 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? It's not considered stable. The idea is that we also add the existing testcases to kselftest. It's purely a bit of interface to be able to drive run the test logic for real fences. What it really tests is the fence interface (which is public in the uapi headers and all that), but to be able to do that we need some (hw-independent way) to expose fences, which this provides. Long term we might even do this as a proper interface (with some restrictions to make it safe and avoid userspace pulling the kernel over the table). And then rip out sw_sync entirely. Imo there's no need at all for docs for this. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch