On Fri, Apr 07, 2017 at 12:06:26PM +0200, Gerd Hoffmann wrote:
> On Fr, 2017-04-07 at 11:45 +0300, Ville Syrjälä wrote:
> > On Fri, Apr 07, 2017 at 10:29:00AM +0200, Gerd Hoffmann wrote:
> > > Hi,
> > >
> > > > Hmm. Maybe it's still possible to salvage something by redefining the
> > > > BIG_END
On Fr, 2017-04-07 at 11:45 +0300, Ville Syrjälä wrote:
> On Fri, Apr 07, 2017 at 10:29:00AM +0200, Gerd Hoffmann wrote:
> > Hi,
> >
> > > Hmm. Maybe it's still possible to salvage something by redefining the
> > > BIG_ENDIAN format bit to mean the "the other endianness". Ugly but it
> > > might
On Fri, Apr 07, 2017 at 10:29:00AM +0200, Gerd Hoffmann wrote:
> Hi,
>
> > Hmm. Maybe it's still possible to salvage something by redefining the
> > BIG_ENDIAN format bit to mean the "the other endianness". Ugly but it
> > might still result in something usable.
>
> Also at least for the virtua
Hi,
> Hmm. Maybe it's still possible to salvage something by redefining the
> BIG_ENDIAN format bit to mean the "the other endianness". Ugly but it
> might still result in something usable.
Also at least for the virtual machine use case this doesn't buy us much.
The drm drivers (at least the on
Hi,
> > So, bigendian guests assume that DRM_FORMAT_XRGB is big endian not
> > little endian. And given that the fourcc codes are used in the
> > userspace/kernel API too (see DRM_IOCTL_MODE_ADDFB2) I think we can't
> > change that any more ...
>
> Sigh. That makes mixed endian systems pre
On Thu, Apr 06, 2017 at 08:27:47PM +0300, Ville Syrjälä wrote:
> On Thu, Apr 06, 2017 at 10:29:43AM +0200, Gerd Hoffmann wrote:
> > Hi,
> >
> > > > static const uint32_t virtio_gpu_cursor_formats[] = {
> > > > +#ifdef __BIG_ENDIAN
> > > > + DRM_FORMAT_BGRA,
> > > > +#else
> > > >
On Thu, Apr 06, 2017 at 10:29:43AM +0200, Gerd Hoffmann wrote:
> Hi,
>
> > > static const uint32_t virtio_gpu_cursor_formats[] = {
> > > +#ifdef __BIG_ENDIAN
> > > + DRM_FORMAT_BGRA,
> > > +#else
> > > DRM_FORMAT_ARGB,
> > > +#endif
> >
> > DRM formats are supposed to be little endia
Hi,
> > static const uint32_t virtio_gpu_cursor_formats[] = {
> > +#ifdef __BIG_ENDIAN
> > + DRM_FORMAT_BGRA,
> > +#else
> > DRM_FORMAT_ARGB,
> > +#endif
>
> DRM formats are supposed to be little endian, so this isn't really
> correct.
Well, maybe they where *intended* to be lit