[Mesa-dev] [PATCH 03/11] radeon: remove dri_mirror state

Emil Velikov emil.l.velikov at gmail.com
Thu Jul 9 05:55:05 PDT 2015

On 9 July 2015 at 04:45, Michel Dänzer <michel at daenzer.net> wrote:
> On 09.07.2015 02:07, Emil Velikov wrote:
>> Most of the data stored(duplicated) was unused, and for the one that is
>> follow the approach set by other drivers.
>> This eliminates the use of legacy (dri1) types.
> The commentary below should have been after the --- separator, not in
> the actual Git commit log. With that fixed, this patch is
> Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>
>> XXX: The radeon code is the only user of __DRIscreen::drm_version (the
>> only __DRIversion outside of dri1 land). Should we move it into radeon
>> and/or bump the min. required drm module version ?
> Moving this into radeon sounds good to me, but I'm not sure what exactly
> you mean by bumping the minimum required version, or what it's supposed
> to be good for.
> FWIW though, any code which is specific to radeon DRM major version 1
> can be removed, because that's the UMS major version.
Here is what we have atm.


if (major == 1 && minor < 13)
 printf("boho no hyperz for you\n")

if (minor >=15) // major check anyone ?
 r_context.texmicrotile = true; // unused since 2009
ccf7814a315(radeon: major cleanups removing old dead codepaths.)


if (major >= 2)

Considering the amdgpu work (major number), I think we can just error
out if major !=2 and cleanup the above ?


More information about the mesa-dev mailing list