* colorkey support for intel i915 gpu driver
@ 2019-04-01 20:18 Jim Zhang
2019-04-02 12:31 ` Ville Syrjälä
0 siblings, 1 reply; 14+ messages in thread
From: Jim Zhang @ 2019-04-01 20:18 UTC (permalink / raw)
To: dri-devel@lists.freedesktop.org
[-- Attachment #1.1: Type: text/plain, Size: 602 bytes --]
Hi Sir/Madam:
I am using the open source Baytrail gpu drm driver.
Linux kernel version 3.10.61:
Libdrm package: 2.4.97
When calling function
properties = drmModeObjectGetProperties(drmfd, plane_id, DRM_MODE_OBJECT_PLANE);
it only returns only one property:
This property is:
property->name = "rotation", property->prop_id =4
It looks like that the Baytrail gpu drm driver does not support colorkey.
>From what kernel version , the Baytrail GPU DRM driver starts to support colorkey, and is there a kernel patch available?
Thanks,
Jim
Caterpillar: Confidential Green
[-- Attachment #1.2: Type: text/html, Size: 3905 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: colorkey support for intel i915 gpu driver 2019-04-01 20:18 colorkey support for intel i915 gpu driver Jim Zhang @ 2019-04-02 12:31 ` Ville Syrjälä 2019-04-09 13:29 ` Jim Zhang 0 siblings, 1 reply; 14+ messages in thread From: Ville Syrjälä @ 2019-04-02 12:31 UTC (permalink / raw) To: Jim Zhang; +Cc: intel-gfx, dri-devel@lists.freedesktop.org On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > Hi Sir/Madam: > > I am using the open source Baytrail gpu drm driver. > > Linux kernel version 3.10.61: > Libdrm package: 2.4.97 > > When calling function > properties = drmModeObjectGetProperties(drmfd, plane_id, DRM_MODE_OBJECT_PLANE); > it only returns only one property: > > This property is: > property->name = "rotation", property->prop_id =4 > It looks like that the Baytrail gpu drm driver does not support colorkey. There are two problems currently: - Destination colorkey is not implemented on BYT/CHV. I have patches for it but they have not been reviewed by anyone: https://patchwork.freedesktop.org/series/43902/ - Colorkey can only be set via a custom i915 specific ioctl (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at a generic property based API that never really went anywhere. It's a rather difficult problem making this generic as each hardware has its own peculiar way of specifying colorkeying. The main problem with the custom ioctl is that it's not atomic with other screen updates. So, what kind of use case do you have in mind? -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: colorkey support for intel i915 gpu driver 2019-04-02 12:31 ` Ville Syrjälä @ 2019-04-09 13:29 ` Jim Zhang 2019-04-09 13:46 ` Ville Syrjälä 0 siblings, 1 reply; 14+ messages in thread From: Jim Zhang @ 2019-04-09 13:29 UTC (permalink / raw) To: Ville Syrjälä Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Villie: What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? Thanks, Jim Caterpillar: Confidential Green -----Original Message----- From: Ville Syrjälä <ville.syrjala@linux.intel.com> Sent: Tuesday, April 2, 2019 7:32 AM To: Jim Zhang <Zhang_Jim@cat.com> Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org Subject: Re: colorkey support for intel i915 gpu driver On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > Hi Sir/Madam: > > I am using the open source Baytrail gpu drm driver. > > Linux kernel version 3.10.61: > Libdrm package: 2.4.97 > > When calling function > properties = drmModeObjectGetProperties(drmfd, plane_id, > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > This property is: > property->name = "rotation", property->prop_id =4 It looks like that > the Baytrail gpu drm driver does not support colorkey. There are two problems currently: - Destination colorkey is not implemented on BYT/CHV. I have patches for it but they have not been reviewed by anyone: https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.freedesktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUCy9ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo50PX79myDO5gEYA&e= - Colorkey can only be set via a custom i915 specific ioctl (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at a generic property based API that never really went anywhere. It's a rather difficult problem making this generic as each hardware has its own peculiar way of specifying colorkeying. The main problem with the custom ioctl is that it's not atomic with other screen updates. So, what kind of use case do you have in mind? -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: colorkey support for intel i915 gpu driver 2019-04-09 13:29 ` Jim Zhang @ 2019-04-09 13:46 ` Ville Syrjälä 2019-04-09 13:56 ` Ville Syrjälä 0 siblings, 1 reply; 14+ messages in thread From: Ville Syrjälä @ 2019-04-09 13:46 UTC (permalink / raw) To: Jim Zhang Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > Villie: > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? There is no real plan at this time. But if you have a use case for it I can try to harass people until someone reviews it :) > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? No. We would need to define a new api for it. > > Thanks, > > Jim > > > Caterpillar: Confidential Green > > -----Original Message----- > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > Sent: Tuesday, April 2, 2019 7:32 AM > To: Jim Zhang <Zhang_Jim@cat.com> > Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org > Subject: Re: colorkey support for intel i915 gpu driver > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > Hi Sir/Madam: > > > > I am using the open source Baytrail gpu drm driver. > > > > Linux kernel version 3.10.61: > > Libdrm package: 2.4.97 > > > > When calling function > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > This property is: > > property->name = "rotation", property->prop_id =4 It looks like that > > the Baytrail gpu drm driver does not support colorkey. > > There are two problems currently: > - Destination colorkey is not implemented on BYT/CHV. I have > patches for it but they have not been reviewed by anyone: > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.freedesktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUCy9ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo50PX79myDO5gEYA&e= > - Colorkey can only be set via a custom i915 specific ioctl > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > a generic property based API that never really went anywhere. It's > a rather difficult problem making this generic as each hardware has > its own peculiar way of specifying colorkeying. The main problem > with the custom ioctl is that it's not atomic with other screen > updates. > > So, what kind of use case do you have in mind? > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: colorkey support for intel i915 gpu driver 2019-04-09 13:46 ` Ville Syrjälä @ 2019-04-09 13:56 ` Ville Syrjälä 2019-04-09 13:59 ` [Intel-gfx] " Jim Zhang 2019-04-09 14:14 ` [Intel-gfx] " Jim Zhang 0 siblings, 2 replies; 14+ messages in thread From: Ville Syrjälä @ 2019-04-09 13:56 UTC (permalink / raw) To: Jim Zhang Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > Villie: > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > There is no real plan at this time. But if you have a use case > for it I can try to harass people until someone reviews it :) > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > No. We would need to define a new api for it. I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > Thanks, > > > > Jim > > > > > > Caterpillar: Confidential Green > > > > -----Original Message----- > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Sent: Tuesday, April 2, 2019 7:32 AM > > To: Jim Zhang <Zhang_Jim@cat.com> > > Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org > > Subject: Re: colorkey support for intel i915 gpu driver > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > Hi Sir/Madam: > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > Linux kernel version 3.10.61: > > > Libdrm package: 2.4.97 > > > > > > When calling function > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > This property is: > > > property->name = "rotation", property->prop_id =4 It looks like that > > > the Baytrail gpu drm driver does not support colorkey. > > > > There are two problems currently: > > - Destination colorkey is not implemented on BYT/CHV. I have > > patches for it but they have not been reviewed by anyone: > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.freedesktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUCy9ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo50PX79myDO5gEYA&e= > > - Colorkey can only be set via a custom i915 specific ioctl > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > a generic property based API that never really went anywhere. It's > > a rather difficult problem making this generic as each hardware has > > its own peculiar way of specifying colorkeying. The main problem > > with the custom ioctl is that it's not atomic with other screen > > updates. > > > > So, what kind of use case do you have in mind? > > > > -- > > Ville Syrjälä > > Intel > > -- > Ville Syrjälä > Intel > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [Intel-gfx] colorkey support for intel i915 gpu driver 2019-04-09 13:56 ` Ville Syrjälä @ 2019-04-09 13:59 ` Jim Zhang 2019-04-09 14:17 ` Ville Syrjälä 2019-04-09 14:14 ` [Intel-gfx] " Jim Zhang 1 sibling, 1 reply; 14+ messages in thread From: Jim Zhang @ 2019-04-09 13:59 UTC (permalink / raw) To: Ville Syrjälä Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Nice, do you have any sample code for it? Thanks, Jim Caterpillar: Confidential Green -----Original Message----- From: Ville Syrjälä <ville.syrjala@linux.intel.com> Sent: Tuesday, April 9, 2019 8:57 AM To: Jim Zhang <Zhang_Jim@cat.com> Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > Villie: > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > There is no real plan at this time. But if you have a use case for it > I can try to harass people until someone reviews it :) > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > No. We would need to define a new api for it. I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > Thanks, > > > > Jim > > > > > > Caterpillar: Confidential Green > > > > -----Original Message----- > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Sent: Tuesday, April 2, 2019 7:32 AM > > To: Jim Zhang <Zhang_Jim@cat.com> > > Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org > > Subject: Re: colorkey support for intel i915 gpu driver > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > Hi Sir/Madam: > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > Linux kernel version 3.10.61: > > > Libdrm package: 2.4.97 > > > > > > When calling function > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > This property is: > > > property->name = "rotation", property->prop_id =4 It looks like > > > that the Baytrail gpu drm driver does not support colorkey. > > > > There are two problems currently: > > - Destination colorkey is not implemented on BYT/CHV. I have > > patches for it but they have not been reviewed by anyone: > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.freed > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg-s > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUCy9 > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo50 > > PX79myDO5gEYA&e= > > - Colorkey can only be set via a custom i915 specific ioctl > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > a generic property based API that never really went anywhere. It's > > a rather difficult problem making this generic as each hardware has > > its own peculiar way of specifying colorkeying. The main problem > > with the custom ioctl is that it's not atomic with other screen > > updates. > > > > So, what kind of use case do you have in mind? > > > > -- > > Ville Syrjälä > > Intel > > -- > Ville Syrjälä > Intel > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=Oyn > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzkzNPQ > cu3xTPSCqb5Dnpwk&e= -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: colorkey support for intel i915 gpu driver 2019-04-09 13:59 ` [Intel-gfx] " Jim Zhang @ 2019-04-09 14:17 ` Ville Syrjälä 0 siblings, 0 replies; 14+ messages in thread From: Ville Syrjälä @ 2019-04-09 14:17 UTC (permalink / raw) To: Jim Zhang Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org On Tue, Apr 09, 2019 at 01:59:21PM +0000, Jim Zhang wrote: > Nice, do you have any sample code for it? https://cgit.freedesktop.org/xorg/driver/xf86-video-intel/tree/src/sna/sna_video_sprite.c is the only userspace code we have that uses the colorkey. > > Thanks, > > Jim > > > Caterpillar: Confidential Green > > -----Original Message----- > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > Sent: Tuesday, April 9, 2019 8:57 AM > To: Jim Zhang <Zhang_Jim@cat.com> > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > > Villie: > > > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > > > There is no real plan at this time. But if you have a use case for it > > I can try to harass people until someone reviews it :) > > > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > > > No. We would need to define a new api for it. > > I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > > > > > Thanks, > > > > > > Jim > > > > > > > > > Caterpillar: Confidential Green > > > > > > -----Original Message----- > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > Sent: Tuesday, April 2, 2019 7:32 AM > > > To: Jim Zhang <Zhang_Jim@cat.com> > > > Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org > > > Subject: Re: colorkey support for intel i915 gpu driver > > > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > > Hi Sir/Madam: > > > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > > > Linux kernel version 3.10.61: > > > > Libdrm package: 2.4.97 > > > > > > > > When calling function > > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > > > This property is: > > > > property->name = "rotation", property->prop_id =4 It looks like > > > > that the Baytrail gpu drm driver does not support colorkey. > > > > > > There are two problems currently: > > > - Destination colorkey is not implemented on BYT/CHV. I have > > > patches for it but they have not been reviewed by anyone: > > > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.freed > > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg-s > > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUCy9 > > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo50 > > > PX79myDO5gEYA&e= > > > - Colorkey can only be set via a custom i915 specific ioctl > > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > > a generic property based API that never really went anywhere. It's > > > a rather difficult problem making this generic as each hardware has > > > its own peculiar way of specifying colorkeying. The main problem > > > with the custom ioctl is that it's not atomic with other screen > > > updates. > > > > > > So, what kind of use case do you have in mind? > > > > > > -- > > > Ville Syrjälä > > > Intel > > > > -- > > Ville Syrjälä > > Intel > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop > > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m > > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=Oyn > > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzkzNPQ > > cu3xTPSCqb5Dnpwk&e= > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [Intel-gfx] colorkey support for intel i915 gpu driver 2019-04-09 13:56 ` Ville Syrjälä 2019-04-09 13:59 ` [Intel-gfx] " Jim Zhang @ 2019-04-09 14:14 ` Jim Zhang 2019-04-09 14:19 ` Ville Syrjälä 1 sibling, 1 reply; 14+ messages in thread From: Jim Zhang @ 2019-04-09 14:14 UTC (permalink / raw) To: Ville Syrjälä Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Once I pre-configure the colorkey, am I able to enable and disable it? If colorkey can be enabled/disabled after that might meet my requirement Thanks, Jim Caterpillar: Confidential Green -----Original Message----- From: Ville Syrjälä <ville.syrjala@linux.intel.com> Sent: Tuesday, April 9, 2019 8:57 AM To: Jim Zhang <Zhang_Jim@cat.com> Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > Villie: > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > There is no real plan at this time. But if you have a use case for it > I can try to harass people until someone reviews it :) > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > No. We would need to define a new api for it. I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > Thanks, > > > > Jim > > > > > > Caterpillar: Confidential Green > > > > -----Original Message----- > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Sent: Tuesday, April 2, 2019 7:32 AM > > To: Jim Zhang <Zhang_Jim@cat.com> > > Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org > > Subject: Re: colorkey support for intel i915 gpu driver > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > Hi Sir/Madam: > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > Linux kernel version 3.10.61: > > > Libdrm package: 2.4.97 > > > > > > When calling function > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > This property is: > > > property->name = "rotation", property->prop_id =4 It looks like > > > that the Baytrail gpu drm driver does not support colorkey. > > > > There are two problems currently: > > - Destination colorkey is not implemented on BYT/CHV. I have > > patches for it but they have not been reviewed by anyone: > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.freed > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg-s > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUCy9 > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo50 > > PX79myDO5gEYA&e= > > - Colorkey can only be set via a custom i915 specific ioctl > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > a generic property based API that never really went anywhere. It's > > a rather difficult problem making this generic as each hardware has > > its own peculiar way of specifying colorkeying. The main problem > > with the custom ioctl is that it's not atomic with other screen > > updates. > > > > So, what kind of use case do you have in mind? > > > > -- > > Ville Syrjälä > > Intel > > -- > Ville Syrjälä > Intel > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=Oyn > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzkzNPQ > cu3xTPSCqb5Dnpwk&e= -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: colorkey support for intel i915 gpu driver 2019-04-09 14:14 ` [Intel-gfx] " Jim Zhang @ 2019-04-09 14:19 ` Ville Syrjälä 2019-04-09 14:24 ` [Intel-gfx] " Jim Zhang 0 siblings, 1 reply; 14+ messages in thread From: Ville Syrjälä @ 2019-04-09 14:19 UTC (permalink / raw) To: Jim Zhang Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org On Tue, Apr 09, 2019 at 02:14:49PM +0000, Jim Zhang wrote: > Once I pre-configure the colorkey, am I able to enable and disable it? If colorkey can be enabled/disabled after that might meet my requirement Not atomically with other updates. > > Thanks, > > Jim > > > Caterpillar: Confidential Green > > -----Original Message----- > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > Sent: Tuesday, April 9, 2019 8:57 AM > To: Jim Zhang <Zhang_Jim@cat.com> > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > > Villie: > > > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > > > There is no real plan at this time. But if you have a use case for it > > I can try to harass people until someone reviews it :) > > > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > > > No. We would need to define a new api for it. > > I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > > > > > Thanks, > > > > > > Jim > > > > > > > > > Caterpillar: Confidential Green > > > > > > -----Original Message----- > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > Sent: Tuesday, April 2, 2019 7:32 AM > > > To: Jim Zhang <Zhang_Jim@cat.com> > > > Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org > > > Subject: Re: colorkey support for intel i915 gpu driver > > > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > > Hi Sir/Madam: > > > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > > > Linux kernel version 3.10.61: > > > > Libdrm package: 2.4.97 > > > > > > > > When calling function > > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > > > This property is: > > > > property->name = "rotation", property->prop_id =4 It looks like > > > > that the Baytrail gpu drm driver does not support colorkey. > > > > > > There are two problems currently: > > > - Destination colorkey is not implemented on BYT/CHV. I have > > > patches for it but they have not been reviewed by anyone: > > > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.freed > > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg-s > > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUCy9 > > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo50 > > > PX79myDO5gEYA&e= > > > - Colorkey can only be set via a custom i915 specific ioctl > > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > > a generic property based API that never really went anywhere. It's > > > a rather difficult problem making this generic as each hardware has > > > its own peculiar way of specifying colorkeying. The main problem > > > with the custom ioctl is that it's not atomic with other screen > > > updates. > > > > > > So, what kind of use case do you have in mind? > > > > > > -- > > > Ville Syrjälä > > > Intel > > > > -- > > Ville Syrjälä > > Intel > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop > > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m > > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=Oyn > > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzkzNPQ > > cu3xTPSCqb5Dnpwk&e= > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [Intel-gfx] colorkey support for intel i915 gpu driver 2019-04-09 14:19 ` Ville Syrjälä @ 2019-04-09 14:24 ` Jim Zhang 2019-04-09 15:01 ` Ville Syrjälä 0 siblings, 1 reply; 14+ messages in thread From: Jim Zhang @ 2019-04-09 14:24 UTC (permalink / raw) To: Ville Syrjälä Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org What about if I disable interrupt when changing the colorkey? This will solve the atomic issue. I think we only change colorkey or enable/disable colorkey once a while. If disabling interrupt work, I will disable interrupt and change colorkey. That performance affection could be acceptable Thanks Jim Caterpillar: Confidential Green -----Original Message----- From: Ville Syrjälä <ville.syrjala@linux.intel.com> Sent: Tuesday, April 9, 2019 9:19 AM To: Jim Zhang <Zhang_Jim@cat.com> Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver On Tue, Apr 09, 2019 at 02:14:49PM +0000, Jim Zhang wrote: > Once I pre-configure the colorkey, am I able to enable and disable it? > If colorkey can be enabled/disabled after that might meet my > requirement Not atomically with other updates. > > Thanks, > > Jim > > > Caterpillar: Confidential Green > > -----Original Message----- > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > Sent: Tuesday, April 9, 2019 8:57 AM > To: Jim Zhang <Zhang_Jim@cat.com> > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > > Villie: > > > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > > > There is no real plan at this time. But if you have a use case for > > it I can try to harass people until someone reviews it :) > > > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > > > No. We would need to define a new api for it. > > I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > > > > > Thanks, > > > > > > Jim > > > > > > > > > Caterpillar: Confidential Green > > > > > > -----Original Message----- > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > Sent: Tuesday, April 2, 2019 7:32 AM > > > To: Jim Zhang <Zhang_Jim@cat.com> > > > Cc: dri-devel@lists.freedesktop.org; > > > intel-gfx@lists.freedesktop.org > > > Subject: Re: colorkey support for intel i915 gpu driver > > > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > > Hi Sir/Madam: > > > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > > > Linux kernel version 3.10.61: > > > > Libdrm package: 2.4.97 > > > > > > > > When calling function > > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > > > This property is: > > > > property->name = "rotation", property->prop_id =4 It looks > > > > like that the Baytrail gpu drm driver does not support colorkey. > > > > > > There are two problems currently: > > > - Destination colorkey is not implemented on BYT/CHV. I have > > > patches for it but they have not been reviewed by anyone: > > > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.fre > > > ed > > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg > > > -s > > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUC > > > y9 > > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo > > > 50 > > > PX79myDO5gEYA&e= > > > - Colorkey can only be set via a custom i915 specific ioctl > > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > > a generic property based API that never really went anywhere. It's > > > a rather difficult problem making this generic as each hardware has > > > its own peculiar way of specifying colorkeying. The main problem > > > with the custom ioctl is that it's not atomic with other screen > > > updates. > > > > > > So, what kind of use case do you have in mind? > > > > > > -- > > > Ville Syrjälä > > > Intel > > > > -- > > Ville Syrjälä > > Intel > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedeskt > > op > > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r > > 4m > > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=O > > yn > > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzkzN > > PQ > > cu3xTPSCqb5Dnpwk&e= > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: colorkey support for intel i915 gpu driver 2019-04-09 14:24 ` [Intel-gfx] " Jim Zhang @ 2019-04-09 15:01 ` Ville Syrjälä 2019-04-09 15:17 ` [Intel-gfx] " Jim Zhang 0 siblings, 1 reply; 14+ messages in thread From: Ville Syrjälä @ 2019-04-09 15:01 UTC (permalink / raw) To: Jim Zhang Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org On Tue, Apr 09, 2019 at 02:24:03PM +0000, Jim Zhang wrote: > What about if I disable interrupt when changing the colorkey? This will solve the atomic issue. I think we only change colorkey or enable/disable colorkey once a while. If disabling interrupt work, I will disable interrupt and change colorkey. That performance affection could be acceptable Interrupts don't matter for this. > > Thanks > > Jim > > Caterpillar: Confidential Green > > -----Original Message----- > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > Sent: Tuesday, April 9, 2019 9:19 AM > To: Jim Zhang <Zhang_Jim@cat.com> > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > On Tue, Apr 09, 2019 at 02:14:49PM +0000, Jim Zhang wrote: > > Once I pre-configure the colorkey, am I able to enable and disable it? > > If colorkey can be enabled/disabled after that might meet my > > requirement > > Not atomically with other updates. > > > > > Thanks, > > > > Jim > > > > > > Caterpillar: Confidential Green > > > > -----Original Message----- > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Sent: Tuesday, April 9, 2019 8:57 AM > > To: Jim Zhang <Zhang_Jim@cat.com> > > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > > > On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > > > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > > > Villie: > > > > > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > > > > > There is no real plan at this time. But if you have a use case for > > > it I can try to harass people until someone reviews it :) > > > > > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > > > > > No. We would need to define a new api for it. > > > > I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > > > > > > > > > Thanks, > > > > > > > > Jim > > > > > > > > > > > > Caterpillar: Confidential Green > > > > > > > > -----Original Message----- > > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > Sent: Tuesday, April 2, 2019 7:32 AM > > > > To: Jim Zhang <Zhang_Jim@cat.com> > > > > Cc: dri-devel@lists.freedesktop.org; > > > > intel-gfx@lists.freedesktop.org > > > > Subject: Re: colorkey support for intel i915 gpu driver > > > > > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > > > Hi Sir/Madam: > > > > > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > > > > > Linux kernel version 3.10.61: > > > > > Libdrm package: 2.4.97 > > > > > > > > > > When calling function > > > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > > > > > This property is: > > > > > property->name = "rotation", property->prop_id =4 It looks > > > > > like that the Baytrail gpu drm driver does not support colorkey. > > > > > > > > There are two problems currently: > > > > - Destination colorkey is not implemented on BYT/CHV. I have > > > > patches for it but they have not been reviewed by anyone: > > > > > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.fre > > > > ed > > > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg > > > > -s > > > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUC > > > > y9 > > > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo > > > > 50 > > > > PX79myDO5gEYA&e= > > > > - Colorkey can only be set via a custom i915 specific ioctl > > > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > > > a generic property based API that never really went anywhere. It's > > > > a rather difficult problem making this generic as each hardware has > > > > its own peculiar way of specifying colorkeying. The main problem > > > > with the custom ioctl is that it's not atomic with other screen > > > > updates. > > > > > > > > So, what kind of use case do you have in mind? > > > > > > > > -- > > > > Ville Syrjälä > > > > Intel > > > > > > -- > > > Ville Syrjälä > > > Intel > > > _______________________________________________ > > > Intel-gfx mailing list > > > Intel-gfx@lists.freedesktop.org > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedeskt > > > op > > > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r > > > 4m > > > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=O > > > yn > > > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzkzN > > > PQ > > > cu3xTPSCqb5Dnpwk&e= > > > > -- > > Ville Syrjälä > > Intel > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [Intel-gfx] colorkey support for intel i915 gpu driver 2019-04-09 15:01 ` Ville Syrjälä @ 2019-04-09 15:17 ` Jim Zhang 2019-04-09 19:15 ` Jim Zhang 0 siblings, 1 reply; 14+ messages in thread From: Jim Zhang @ 2019-04-09 15:17 UTC (permalink / raw) To: Ville Syrjälä Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org If I go with pre-configured colorkey, do I need port your kernel patch to i915 driver at kernel 3.10.61 ? Thanks, Jim Caterpillar: Confidential Green -----Original Message----- From: Ville Syrjälä <ville.syrjala@linux.intel.com> Sent: Tuesday, April 9, 2019 10:02 AM To: Jim Zhang <Zhang_Jim@cat.com> Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver On Tue, Apr 09, 2019 at 02:24:03PM +0000, Jim Zhang wrote: > What about if I disable interrupt when changing the colorkey? This > will solve the atomic issue. I think we only change colorkey or > enable/disable colorkey once a while. If disabling interrupt work, I > will disable interrupt and change colorkey. That performance affection > could be acceptable Interrupts don't matter for this. > > Thanks > > Jim > > Caterpillar: Confidential Green > > -----Original Message----- > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > Sent: Tuesday, April 9, 2019 9:19 AM > To: Jim Zhang <Zhang_Jim@cat.com> > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > On Tue, Apr 09, 2019 at 02:14:49PM +0000, Jim Zhang wrote: > > Once I pre-configure the colorkey, am I able to enable and disable it? > > If colorkey can be enabled/disabled after that might meet my > > requirement > > Not atomically with other updates. > > > > > Thanks, > > > > Jim > > > > > > Caterpillar: Confidential Green > > > > -----Original Message----- > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Sent: Tuesday, April 9, 2019 8:57 AM > > To: Jim Zhang <Zhang_Jim@cat.com> > > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > > > On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > > > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > > > Villie: > > > > > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > > > > > There is no real plan at this time. But if you have a use case for > > > it I can try to harass people until someone reviews it :) > > > > > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > > > > > No. We would need to define a new api for it. > > > > I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > > > > > > > > > Thanks, > > > > > > > > Jim > > > > > > > > > > > > Caterpillar: Confidential Green > > > > > > > > -----Original Message----- > > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > Sent: Tuesday, April 2, 2019 7:32 AM > > > > To: Jim Zhang <Zhang_Jim@cat.com> > > > > Cc: dri-devel@lists.freedesktop.org; > > > > intel-gfx@lists.freedesktop.org > > > > Subject: Re: colorkey support for intel i915 gpu driver > > > > > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > > > Hi Sir/Madam: > > > > > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > > > > > Linux kernel version 3.10.61: > > > > > Libdrm package: 2.4.97 > > > > > > > > > > When calling function > > > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > > > > > This property is: > > > > > property->name = "rotation", property->prop_id =4 It looks > > > > > like that the Baytrail gpu drm driver does not support colorkey. > > > > > > > > There are two problems currently: > > > > - Destination colorkey is not implemented on BYT/CHV. I have > > > > patches for it but they have not been reviewed by anyone: > > > > > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.f > > > > re > > > > ed > > > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9Ol > > > > Gg > > > > -s > > > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfg > > > > UC > > > > y9 > > > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3p > > > > Xo > > > > 50 > > > > PX79myDO5gEYA&e= > > > > - Colorkey can only be set via a custom i915 specific ioctl > > > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > > > a generic property based API that never really went anywhere. It's > > > > a rather difficult problem making this generic as each hardware has > > > > its own peculiar way of specifying colorkeying. The main problem > > > > with the custom ioctl is that it's not atomic with other screen > > > > updates. > > > > > > > > So, what kind of use case do you have in mind? > > > > > > > > -- > > > > Ville Syrjälä > > > > Intel > > > > > > -- > > > Ville Syrjälä > > > Intel > > > _______________________________________________ > > > Intel-gfx mailing list > > > Intel-gfx@lists.freedesktop.org > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedes > > > kt > > > op > > > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB > > > 4r > > > 4m > > > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m > > > =O > > > yn > > > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzk > > > zN > > > PQ > > > cu3xTPSCqb5Dnpwk&e= > > > > -- > > Ville Syrjälä > > Intel > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [Intel-gfx] colorkey support for intel i915 gpu driver 2019-04-09 15:17 ` [Intel-gfx] " Jim Zhang @ 2019-04-09 19:15 ` Jim Zhang 2019-04-09 19:44 ` Ville Syrjälä 0 siblings, 1 reply; 14+ messages in thread From: Jim Zhang @ 2019-04-09 19:15 UTC (permalink / raw) To: Ville Syrjälä Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Ville: Yes, if this patch is needed by kernel 3.10.61, please get somebody to review it. What do I need do to speed up the review process? Please generate a patch against kernel 3.10.61 if possible. Thanks, Jim Caterpillar: Confidential Green -----Original Message----- From: Jim Zhang Sent: Tuesday, April 9, 2019 10:18 AM To: 'Ville Syrjälä' <ville.syrjala@linux.intel.com> Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: RE: [Intel-gfx] colorkey support for intel i915 gpu driver If I go with pre-configured colorkey, do I need port your kernel patch to i915 driver at kernel 3.10.61 ? Thanks, Jim Caterpillar: Confidential Green -----Original Message----- From: Ville Syrjälä <ville.syrjala@linux.intel.com> Sent: Tuesday, April 9, 2019 10:02 AM To: Jim Zhang <Zhang_Jim@cat.com> Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver On Tue, Apr 09, 2019 at 02:24:03PM +0000, Jim Zhang wrote: > What about if I disable interrupt when changing the colorkey? This > will solve the atomic issue. I think we only change colorkey or > enable/disable colorkey once a while. If disabling interrupt work, I > will disable interrupt and change colorkey. That performance affection > could be acceptable Interrupts don't matter for this. > > Thanks > > Jim > > Caterpillar: Confidential Green > > -----Original Message----- > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > Sent: Tuesday, April 9, 2019 9:19 AM > To: Jim Zhang <Zhang_Jim@cat.com> > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > On Tue, Apr 09, 2019 at 02:14:49PM +0000, Jim Zhang wrote: > > Once I pre-configure the colorkey, am I able to enable and disable it? > > If colorkey can be enabled/disabled after that might meet my > > requirement > > Not atomically with other updates. > > > > > Thanks, > > > > Jim > > > > > > Caterpillar: Confidential Green > > > > -----Original Message----- > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Sent: Tuesday, April 9, 2019 8:57 AM > > To: Jim Zhang <Zhang_Jim@cat.com> > > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > > > On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > > > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > > > Villie: > > > > > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > > > > > There is no real plan at this time. But if you have a use case for > > > it I can try to harass people until someone reviews it :) > > > > > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > > > > > No. We would need to define a new api for it. > > > > I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > > > > > > > > > Thanks, > > > > > > > > Jim > > > > > > > > > > > > Caterpillar: Confidential Green > > > > > > > > -----Original Message----- > > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > Sent: Tuesday, April 2, 2019 7:32 AM > > > > To: Jim Zhang <Zhang_Jim@cat.com> > > > > Cc: dri-devel@lists.freedesktop.org; > > > > intel-gfx@lists.freedesktop.org > > > > Subject: Re: colorkey support for intel i915 gpu driver > > > > > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > > > Hi Sir/Madam: > > > > > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > > > > > Linux kernel version 3.10.61: > > > > > Libdrm package: 2.4.97 > > > > > > > > > > When calling function > > > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > > > > > This property is: > > > > > property->name = "rotation", property->prop_id =4 It looks > > > > > like that the Baytrail gpu drm driver does not support colorkey. > > > > > > > > There are two problems currently: > > > > - Destination colorkey is not implemented on BYT/CHV. I have > > > > patches for it but they have not been reviewed by anyone: > > > > > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.f > > > > re > > > > ed > > > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9Ol > > > > Gg > > > > -s > > > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfg > > > > UC > > > > y9 > > > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3p > > > > Xo > > > > 50 > > > > PX79myDO5gEYA&e= > > > > - Colorkey can only be set via a custom i915 specific ioctl > > > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > > > a generic property based API that never really went anywhere. It's > > > > a rather difficult problem making this generic as each hardware has > > > > its own peculiar way of specifying colorkeying. The main problem > > > > with the custom ioctl is that it's not atomic with other screen > > > > updates. > > > > > > > > So, what kind of use case do you have in mind? > > > > > > > > -- > > > > Ville Syrjälä > > > > Intel > > > > > > -- > > > Ville Syrjälä > > > Intel > > > _______________________________________________ > > > Intel-gfx mailing list > > > Intel-gfx@lists.freedesktop.org > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedes > > > kt > > > op > > > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB > > > 4r > > > 4m > > > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m > > > =O > > > yn > > > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzk > > > zN > > > PQ > > > cu3xTPSCqb5Dnpwk&e= > > > > -- > > Ville Syrjälä > > Intel > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: colorkey support for intel i915 gpu driver 2019-04-09 19:15 ` Jim Zhang @ 2019-04-09 19:44 ` Ville Syrjälä 0 siblings, 0 replies; 14+ messages in thread From: Ville Syrjälä @ 2019-04-09 19:44 UTC (permalink / raw) To: Jim Zhang Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org On Tue, Apr 09, 2019 at 07:15:20PM +0000, Jim Zhang wrote: > Ville: > > Yes, if this patch is needed by kernel 3.10.61, please get somebody to review it. What do I need do to speed up the review process? > Please generate a patch against kernel 3.10.61 if possible. 3.10 is so ancient I don't even remember what it contains. I have a feeling it's pre-atomic, in which case there is no point in even thinking about a backport. It would simply be too painful. > > Thanks, > > Jim > > > Caterpillar: Confidential Green > > -----Original Message----- > From: Jim Zhang > Sent: Tuesday, April 9, 2019 10:18 AM > To: 'Ville Syrjälä' <ville.syrjala@linux.intel.com> > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > Subject: RE: [Intel-gfx] colorkey support for intel i915 gpu driver > > If I go with pre-configured colorkey, do I need port your kernel patch to i915 driver at kernel 3.10.61 ? > > Thanks, > > Jim > > > > Caterpillar: Confidential Green > > -----Original Message----- > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > Sent: Tuesday, April 9, 2019 10:02 AM > To: Jim Zhang <Zhang_Jim@cat.com> > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > On Tue, Apr 09, 2019 at 02:24:03PM +0000, Jim Zhang wrote: > > What about if I disable interrupt when changing the colorkey? This > > will solve the atomic issue. I think we only change colorkey or > > enable/disable colorkey once a while. If disabling interrupt work, I > > will disable interrupt and change colorkey. That performance affection > > could be acceptable > > Interrupts don't matter for this. > > > > > Thanks > > > > Jim > > > > Caterpillar: Confidential Green > > > > -----Original Message----- > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Sent: Tuesday, April 9, 2019 9:19 AM > > To: Jim Zhang <Zhang_Jim@cat.com> > > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > > > On Tue, Apr 09, 2019 at 02:14:49PM +0000, Jim Zhang wrote: > > > Once I pre-configure the colorkey, am I able to enable and disable it? > > > If colorkey can be enabled/disabled after that might meet my > > > requirement > > > > Not atomically with other updates. > > > > > > > > Thanks, > > > > > > Jim > > > > > > > > > Caterpillar: Confidential Green > > > > > > -----Original Message----- > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > Sent: Tuesday, April 9, 2019 8:57 AM > > > To: Jim Zhang <Zhang_Jim@cat.com> > > > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org > > > Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver > > > > > > On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote: > > > > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote: > > > > > Villie: > > > > > > > > > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release? > > > > > > > > There is no real plan at this time. But if you have a use case for > > > > it I can try to harass people until someone reviews it :) > > > > > > > > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only? > > > > > > > > No. We would need to define a new api for it. > > > > > > I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this. > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > Jim > > > > > > > > > > > > > > > Caterpillar: Confidential Green > > > > > > > > > > -----Original Message----- > > > > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > > Sent: Tuesday, April 2, 2019 7:32 AM > > > > > To: Jim Zhang <Zhang_Jim@cat.com> > > > > > Cc: dri-devel@lists.freedesktop.org; > > > > > intel-gfx@lists.freedesktop.org > > > > > Subject: Re: colorkey support for intel i915 gpu driver > > > > > > > > > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote: > > > > > > Hi Sir/Madam: > > > > > > > > > > > > I am using the open source Baytrail gpu drm driver. > > > > > > > > > > > > Linux kernel version 3.10.61: > > > > > > Libdrm package: 2.4.97 > > > > > > > > > > > > When calling function > > > > > > properties = drmModeObjectGetProperties(drmfd, plane_id, > > > > > > DRM_MODE_OBJECT_PLANE); it only returns only one property: > > > > > > > > > > > > This property is: > > > > > > property->name = "rotation", property->prop_id =4 It looks > > > > > > like that the Baytrail gpu drm driver does not support colorkey. > > > > > > > > > > There are two problems currently: > > > > > - Destination colorkey is not implemented on BYT/CHV. I have > > > > > patches for it but they have not been reviewed by anyone: > > > > > > > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.f > > > > > re > > > > > ed > > > > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9Ol > > > > > Gg > > > > > -s > > > > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfg > > > > > UC > > > > > y9 > > > > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3p > > > > > Xo > > > > > 50 > > > > > PX79myDO5gEYA&e= > > > > > - Colorkey can only be set via a custom i915 specific ioctl > > > > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at > > > > > a generic property based API that never really went anywhere. It's > > > > > a rather difficult problem making this generic as each hardware has > > > > > its own peculiar way of specifying colorkeying. The main problem > > > > > with the custom ioctl is that it's not atomic with other screen > > > > > updates. > > > > > > > > > > So, what kind of use case do you have in mind? > > > > > > > > > > -- > > > > > Ville Syrjälä > > > > > Intel > > > > > > > > -- > > > > Ville Syrjälä > > > > Intel > > > > _______________________________________________ > > > > Intel-gfx mailing list > > > > Intel-gfx@lists.freedesktop.org > > > > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedes > > > > kt > > > > op > > > > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB > > > > 4r > > > > 4m > > > > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m > > > > =O > > > > yn > > > > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzk > > > > zN > > > > PQ > > > > cu3xTPSCqb5Dnpwk&e= > > > > > > -- > > > Ville Syrjälä > > > Intel > > > > -- > > Ville Syrjälä > > Intel > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-04-09 19:44 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-04-01 20:18 colorkey support for intel i915 gpu driver Jim Zhang 2019-04-02 12:31 ` Ville Syrjälä 2019-04-09 13:29 ` Jim Zhang 2019-04-09 13:46 ` Ville Syrjälä 2019-04-09 13:56 ` Ville Syrjälä 2019-04-09 13:59 ` [Intel-gfx] " Jim Zhang 2019-04-09 14:17 ` Ville Syrjälä 2019-04-09 14:14 ` [Intel-gfx] " Jim Zhang 2019-04-09 14:19 ` Ville Syrjälä 2019-04-09 14:24 ` [Intel-gfx] " Jim Zhang 2019-04-09 15:01 ` Ville Syrjälä 2019-04-09 15:17 ` [Intel-gfx] " Jim Zhang 2019-04-09 19:15 ` Jim Zhang 2019-04-09 19:44 ` Ville Syrjälä
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).