[Nouveau] [PATCH] nouveau: add coherent BO attribute

Alexandre Courbot gnurou at gmail.com
Fri Mar 13 02:45:24 PDT 2015


On Fri, Mar 13, 2015 at 3:39 PM, Alexandre Courbot <gnurou at gmail.com> wrote:
> On Fri, Mar 13, 2015 at 3:36 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>> Doesn't this require a kernel version that has your other patch? What
>> happens when this runs on an older kernel? Does it get silently
>> ignored, or does it end up erroring out? If it errors out, that's
>> fine. Otherwise some sort of version check should be put in, no?
>
> The corresponding kernel patch is already merged in Ben's tree. If
> running with an older kernel, this flag will be a no-op, which is fine
> since GK20A's GPU (the reason for this patch as you have guessed :))
> is not enabled for these kernels.
>
> I am fine with adding a version check if you think it is necessary.

So as discussed on IRC, I will check the DRM version from Mesa. This
patch should be good to go - it will require a version increase in
libdrm though. Let me know if you want me to send a patch for this
too.


More information about the Nouveau mailing list