Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1040234AbdDVFID (ORCPT ); Sat, 22 Apr 2017 01:08:03 -0400 Received: from mail-qk0-f193.google.com ([209.85.220.193]:34522 "EHLO mail-qk0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1038859AbdDVFH7 (ORCPT ); Sat, 22 Apr 2017 01:07:59 -0400 MIME-Version: 1.0 In-Reply-To: <20170421165907.GQ30290@intel.com> References: <20170421075825.6307-1-kraxel@redhat.com> <20170421165907.GQ30290@intel.com> From: Ilia Mirkin Date: Sat, 22 Apr 2017 01:07:57 -0400 X-Google-Sender-Auth: 7faSjEFDTTZMESZoq-fyOt4c3j0 Message-ID: Subject: Re: [PATCH] drm: fourcc byteorder: brings header file comments in line with reality. To: =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= Cc: Gerd Hoffmann , "dri-devel@lists.freedesktop.org" , Daniel Vetter , Pekka Paalanen , =?UTF-8?Q?Michel_D=C3=A4nzer?= , Alex Deucher , amd-gfx@lists.freedesktop.org, Jani Nikula , Sean Paul , David Airlie , open list Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v3M58Alh006966 Content-Length: 2894 Lines: 57 On Fri, Apr 21, 2017 at 12:59 PM, Ville Syrjälä wrote: > On Fri, Apr 21, 2017 at 10:49:49AM -0400, Ilia Mirkin wrote: >> On Fri, Apr 21, 2017 at 3:58 AM, Gerd Hoffmann wrote: >> > While working on graphics support for virtual machines on ppc64 (which >> > exists in both little and big endian variants) I've figured the comments >> > for various drm fourcc formats in the header file don't match reality. >> > >> > Comments says the RGB formats are little endian, but in practice they >> > are native endian. Look at the drm_mode_legacy_fb_format() helper. It >> > maps -- for example -- bpp/depth 32/24 to DRM_FORMAT_XRGB8888, no matter >> > whenever the machine is little endian or big endian. The users of this >> > function (fbdev emulation, DRM_IOCTL_MODE_ADDFB) expect the framebuffer >> > is native endian, not little endian. Most userspace also operates on >> > native endian only. >> > >> > So, go update the comments for all 16+24+32 bpp RGB formats. >> > >> > Leaving the yuv formats as-is. I have no idea if and how those are used >> > on bigendian machines. >> >> I think this is premature. The current situation is that I can't get >> modetest to work *at all* on my NV34 / BE setup (I mean, it runs, just >> the colors displayed are wrong). I believe that currently it packs >> things in "cpu native endian". I've tried futzing with that without >> much success, although I didn't spend too much time on it. I have a >> NV34 plugged into my LE setup as well although I haven't tested to >> double-check that it all works there. However I'm quite sure it used >> to, as I used modetest to help develop the YUV overlay support for >> those GPUs. > > I just took a quick stab at fixing modetest to respect the current > wording in drm_fourcc.h: > > git://github.com/vsyrjala/libdrm.git modetest_endian Looks like there was some careless testing on my part :( So ... it looks like the current modetest without those changes does, in fact, work on NV34/BE. With the changes, it breaks (and the handling of the b* modes is a little broken in those patches -- they're not selectable from the cmdline.) Which means that, as Michel & co predicted, it appears to be taking BE input not LE input. This is very surprising to me, but it is what it is. As I mentioned before, the details of how the "BE" mode works on the GPUs is largely unknown to us beyond a few basics. Note that only XR24 works, AR24 ends up with all black displayed. This also happens on LE. Furthermore, all of YUYV, UYVY, and NV12 plane overlays don't appear to work properly. The first half of the overlay is OK (but I think compressed), while the second half is gibberish. Testing this on my board plugged into a LE CPU, I also get the same type of issue, so I'm guessing that it's just bitrot of the feature. (Or modetest gained a bug.) Cheers, -ilia