[igt-dev] [PATCH 1/3] drm header update: temp commit
Zbigniew Kempczyński
zbigniew.kempczynski at intel.com
Fri Jul 28 08:31:11 UTC 2023
On Fri, Jul 28, 2023 at 09:57:02AM +0200, Albert Esteve wrote:
> On Fri, Jul 28, 2023 at 9:42 AM Zbigniew Kempczyński
> <zbigniew.kempczynski at intel.com> wrote:
>
> On Fri, Jul 21, 2023 at 03:04:56PM +0200, Albert Esteve wrote:
> > Update the drm included header to obtain
> > VIRTUAL_CURSOR_PLANE capability and be
> > able to test cursor hotspot setting.
> >
> > This commit should be removed once the kernel
> > patch is applied and update the headers
> > as appropiate.
> >
> > Signed-off-by: Albert Esteve <aesteve at redhat.com>
> > ---
> > include/drm-uapi/drm.h | 54
> +++++++++++++++++++++++++++++++++++++++++-
> > 1 file changed, 53 insertions(+), 1 deletion(-)
> >
>
> I'm sorry Albert but we don't allow to add not accepted changes
> upstream regarding uapi headers. Only verbatim copy of uapi
> headers may land in igt.
>
> In patch [PATCH 2/3] igt_kms: add hotspot plane property you're
> unconditionally using definition which doesn't exists upstream
> yet. Likely it will just return from ioctl but I got doubts it
> should be written this way.
>
> I would suggest to introduce:
>
> #define LOCAL_DRM_CLIENT_CAP_CURSOR_PLANE_HOTSPOT▸ 6
>
> in igt_kms.h instead drm.h. And when drm.h will land in igt
> remove this localized definition replacing with official one.
>
> Hi Zbigniew,
> Sure, that is why I mention in the commit that it should be removed
> once the kernel patch lands.
> There is a dependency, and I was not expecting this patch to be
> upstreamed before the kernel patch lands.
> But I'm ok with your suggestion. It will result in a cleaner patch to
> review :)
> Thanks!
In the meantime I've just verified I'm not talking nonsense and it
seems I'm talking. At the moment doing verbatim copy of kernel
uapi headers conflicts due to missing __user definition and
unresolved __packed in single place in i915_drm.h. I'm thinking how
to solve this with minimal effort as I would like to avoid
changes in uapi headers or have as minimum changes as possible.
--
Zbigniew
More information about the igt-dev
mailing list