Hi Daniel, we’ve resolved this in patch v2. From: Daniel Stone [mailto:[email protected]] Sent: Wednesday, October 29, 2014 0:56 To: Jiang, Fei Cc: Emil Velikov; Cheng, Yao; [email protected]; Vetter, Daniel; [email protected] Subject: Re: [RFC PATCH 3/3] libdrm: user mode helper for ipvr drm driver
Hi, On 17 October 2014 01:36, Jiang, Fei <[email protected]<mailto:[email protected]>> wrote: Thanks for Emil's suggestion. You are right, we need make sure structure size aligned on 8 bytes, which is important for 32bit-64bit compatible case. While you're at it, please don't use enum as a type inside ioctls, since the size can vary by compiler. Please use a uint32_t or whatever instead, assigning enum values to that. Cheers, Daniel Fei -----Original Message----- From: Emil Velikov [mailto:[email protected]<mailto:[email protected]>] Sent: Thursday, October 16, 2014 11:20 PM To: Cheng, Yao; [email protected]<mailto:[email protected]> Cc: [email protected]<mailto:[email protected]>; Jiang, Fei; [email protected]<mailto:[email protected]>; Vetter, Daniel Subject: Re: [RFC PATCH 3/3] libdrm: user mode helper for ipvr drm driver On 16/10/14 15:33, Cheng, Yao wrote: > Hi Emil, > Sorry, what do you mean by "correctly aligned"? does it mean the paddings in > this data structure? > Afaict for compatibility reasons the struct size have to be "aligned" (multiple of 8 bytes), or if you prefer - the struct is missing the required padding :) I've only skimmed through the patch so it may be that other structs are having this issue. Cheers, Emil >> -----Original Message----- >> From: Emil Velikov >> [mailto:[email protected]<mailto:[email protected]>] >> Sent: Wednesday, October 15, 2014 5:24 PM >> To: Cheng, Yao; >> [email protected]<mailto:[email protected]> >> Cc: [email protected]<mailto:[email protected]>; Jiang, Fei; >> [email protected]<mailto:[email protected]>; >> Vetter, Daniel >> Subject: Re: [RFC PATCH 3/3] libdrm: user mode helper for ipvr drm >> driver >> >> Hi Yao, >> >> struct drm_ipvr_gem_userptr does not seem to be correctly aligned - >> is that intentional ? Might be worth checking if anything else in >> ipvr_drm.h and ipvr_bufmgr.h is in the same boat. >> >> Cheers, >> Emil >> _______________________________________________ dri-devel mailing list [email protected]<mailto:[email protected]> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
