diff mbox

drm/i915/perf: Treat u64 in uabi as a normal integer

Message ID 20161130164649.26809-1-chris@chris-wilson.co.uk (mailing list archive)
State New, archived
Headers show

Commit Message

Chris Wilson Nov. 30, 2016, 4:46 p.m. UTC
Forgo marking up the u64 integer representing a user pointer as this
just annoys sparse. The conversion from u64 to a user pointer is managed
by u64_to_user_ptr().

Fixes: eec688e1420d ("drm/i915: Add i915 perf infrastructure")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Robert Bragg <robert@sixbynine.org>
Cc: Matthew Auld <matthew.auld@intel.com>
---
 include/uapi/drm/i915_drm.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Matthew Auld Nov. 30, 2016, 4:55 p.m. UTC | #1
On 30 November 2016 at 16:46, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> Forgo marking up the u64 integer representing a user pointer as this
> just annoys sparse. The conversion from u64 to a user pointer is managed
> by u64_to_user_ptr().
>
> Fixes: eec688e1420d ("drm/i915: Add i915 perf infrastructure")
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Robert Bragg <robert@sixbynine.org>
> Cc: Matthew Auld <matthew.auld@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Saarinen, Jani Dec. 1, 2016, 6:54 a.m. UTC | #2
> == Series Details ==

> 

> Series: drm/i915/perf: Treat u64 in uabi as a normal integer

> URL   : https://patchwork.freedesktop.org/series/16171/

> State : warning

> 

> == Summary ==

> 

> Series 16171v1 drm/i915/perf: Treat u64 in uabi as a normal integer

> https://patchwork.freedesktop.org/api/1.0/series/16171/revisions/1/mbox/

> 

>         Subgroup nonblocking-crc-pipe-a:

>                 pass       -> DMESG-WARN (fi-ilk-650)

https://bugs.freedesktop.org/show_bug.cgi?id=98251

> fi-bdw-5557u     total:245  pass:230  dwarn:0   dfail:0   fail:0   skip:15

> fi-bsw-n3050     total:245  pass:205  dwarn:0   dfail:0   fail:0   skip:40

> fi-bxt-t5700     total:245  pass:217  dwarn:0   dfail:0   fail:0   skip:28

> fi-byt-j1900     total:245  pass:217  dwarn:0   dfail:0   fail:0   skip:28

> fi-byt-n2820     total:245  pass:213  dwarn:0   dfail:0   fail:0   skip:32

> fi-hsw-4770      total:245  pass:225  dwarn:0   dfail:0   fail:0   skip:20

> fi-hsw-4770r     total:245  pass:225  dwarn:0   dfail:0   fail:0   skip:20

> fi-ilk-650       total:245  pass:191  dwarn:1   dfail:0   fail:0   skip:53

> fi-ivb-3520m     total:245  pass:223  dwarn:0   dfail:0   fail:0   skip:22

> fi-ivb-3770      total:245  pass:223  dwarn:0   dfail:0   fail:0   skip:22

> fi-kbl-7500u     total:245  pass:223  dwarn:0   dfail:0   fail:0   skip:22

> fi-skl-6260u     total:245  pass:231  dwarn:0   dfail:0   fail:0   skip:14

> fi-skl-6700hq    total:245  pass:224  dwarn:0   dfail:0   fail:0   skip:21

> fi-skl-6700k     total:245  pass:223  dwarn:1   dfail:0   fail:0   skip:21

> fi-skl-6770hq    total:245  pass:231  dwarn:0   dfail:0   fail:0   skip:14

> fi-snb-2520m     total:245  pass:213  dwarn:0   dfail:0   fail:0   skip:32

> fi-snb-2600      total:245  pass:212  dwarn:0   dfail:0   fail:0   skip:33

> 

> 51c6c255aa05775bcfb3f5790a3b9249e5990f3e drm-tip: 2016y-11m-30d-

> 16h-12m-19s UTC integration manifest

> 6287239 drm/i915/perf: Treat u64 in uabi as a normal integer

> 

> == Logs ==

> 



Jani Saarinen
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
diff mbox

Patch

diff --git a/include/uapi/drm/i915_drm.h b/include/uapi/drm/i915_drm.h
index 686e02451789..9ba5a239d099 100644
--- a/include/uapi/drm/i915_drm.h
+++ b/include/uapi/drm/i915_drm.h
@@ -1369,7 +1369,7 @@  struct drm_i915_perf_open_param {
 	 * Pointer to array of u64 (id, value) pairs configuring the stream
 	 * to open.
 	 */
-	__u64 __user properties_ptr;
+	__u64 properties_ptr;
 };
 
 /**