Re: [Intel-gfx] [PATCH] drm/i915/perf: use DRM_DEBUG for userspace issues

2016-12-07 Thread Daniel Vetter
On Fri, Dec 02, 2016 at 12:18:44PM +, Robert Bragg wrote: > On Fri, Dec 2, 2016 at 8:35 AM, Daniel Vetter wrote: > > > On Thu, Dec 01, 2016 at 05:21:52PM +, Robert Bragg wrote: > > > Avoid using DRM_ERROR for conditions userspace can trigger with a bad > > > config when opening a stream o

Re: [Intel-gfx] [PATCH] drm/i915/perf: use DRM_DEBUG for userspace issues

2016-12-02 Thread Robert Bragg
On Fri, Dec 2, 2016 at 8:35 AM, Daniel Vetter wrote: > On Thu, Dec 01, 2016 at 05:21:52PM +, Robert Bragg wrote: > > Avoid using DRM_ERROR for conditions userspace can trigger with a bad > > config when opening a stream or from not reading data in a timely > > fashion (whereby the OA buffer f

Re: [Intel-gfx] [PATCH] drm/i915/perf: use DRM_DEBUG for userspace issues

2016-12-02 Thread Daniel Vetter
On Thu, Dec 01, 2016 at 05:21:52PM +, Robert Bragg wrote: > Avoid using DRM_ERROR for conditions userspace can trigger with a bad > config when opening a stream or from not reading data in a timely > fashion (whereby the OA buffer fills up). These conditions are tested > by i-g-t which treats e

[Intel-gfx] [PATCH] drm/i915/perf: use DRM_DEBUG for userspace issues

2016-12-01 Thread Robert Bragg
Avoid using DRM_ERROR for conditions userspace can trigger with a bad config when opening a stream or from not reading data in a timely fashion (whereby the OA buffer fills up). These conditions are tested by i-g-t which treats error messages as failures if using the test runner. This wasn't an iss