Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1632764

Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to drm_mode_legacy_fb_format

Show all headers | View raw

  Hi,

> > So just not using the swapping indeed looks like the only sensible
> > option.  Which in turn implies there is no BGRA8888 support for dumb
> > bos.  Hmm, I can see the problem.  Userspace expectation appears to be
> > that ADDFB configures a native endian framebuffer, which the driver
> > simply can't do on bigendian.
> 
> ... with pre-R600 GPUs.

Sure.

> > So, what can/should the driver do here?  Throw errors for ADDFB and
> > force userspace to use ADDFB2?  From a design point of view the best
> > option, but in the other hand I suspect that could break the xorg radeon
> > driver ...
> 
> Yes, it would.

> One thing we could do is provide a way for userspace to query the
> effective format(s) as seen by the GPU and/or CPU.

We already have almost no testing on bigendian.  I doubt adding generic
interfaces specifically to handle this case is going to work because
most userspace will simply not implement that correctly (or at all).

Having support for this in the radeon ioctls might work, because only
radeon kernel + xorg driver have to get things right then.  But I
suspect we already have that.  You've mentioned elsewhere in the thread
that the xorg driver doesn't turn on byteswapping, so the ability to
configure that seems to be somewhere in the API ...

> It might also make sense for the radeon driver to set the
> RADEON_TILING_SWAP_{16,32}BIT flags for dumb BOs.

That could work.  But I guess someone with test hardware needs to try,
to make sure we don't miss corner cases here.

cheers,
  Gerd

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Michel Dänzer <michel@daenzer.net> - 2017-04-25 05:20 +0200
  Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Ville Syrjälä <ville.syrjala@linux.intel.com> - 2017-04-25 12:00 +0200
    Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Michel Dänzer <michel@daenzer.net> - 2017-04-26 04:10 +0200
      Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Ville Syrjälä <ville.syrjala@linux.intel.com> - 2017-04-26 16:40 +0200
  Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Gerd Hoffmann <kraxel@redhat.com> - 2017-04-26 08:00 +0200
    Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Michel Dänzer <michel@daenzer.net> - 2017-04-26 11:40 +0200
      Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Gerd Hoffmann <kraxel@redhat.com> - 2017-04-26 14:20 +0200
        Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Michel Dänzer <michel@daenzer.net> - 2017-04-27 03:00 +0200
          Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Gerd Hoffmann <kraxel@redhat.com> - 2017-04-27 08:50 +0200
            Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Michel Dänzer <michel@daenzer.net> - 2017-04-27 09:10 +0200
              Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Gerd Hoffmann <kraxel@redhat.com> - 2017-04-28 12:10 +0200
    Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Eric Engestrom <eric.engestrom@imgtec.com> - 2017-04-26 15:30 +0200
      Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to  drm_mode_legacy_fb_format Gerd Hoffmann <kraxel@redhat.com> - 2017-04-26 16:00 +0200

csiph-web