lkml.org 
[lkml]   [2011]   [Mar]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [git pull] drm fixes
    On Fri, 25 Mar 2011 10:04:10 -0400
    Jerome Glisse <j.glisse@gmail.com> wrote:

    > My feeling on that is that maybe too much code sharing accross gpu of
    > different generation hurt more than it helps. I have got the feeling
    > that some of the newer Intel asic share some of the bit of older one
    > and that intel is focusing there attention on newer one and obviously
    > doesn't have time or resource to check that change they do don't
    > impact older hw (i don't think such testing is doable without massive
    > investment which is very very unlikely to happen given size of linux
    > market).

    God yes. The mantra of "code sharing is always good" sounds nice, but
    in my experience it's very often not true, especially when you're still
    trying to get things to work.

    We've been slowly splitting code out to make it more robust against
    changes to different generations, but we still have a little ways to
    go. And of course, such splitting introduces problems in itself.

    But as a policy going forward, I'll advocate splitting code whenever a
    new generation requires something slightly different than an old one.
    As an example, rather than adding a few conditionals to our FDI
    training code to support newer chips, I've split it into separate
    functions entirely, leaving the old one alone. If, after awhile we've
    decided that they really are mostly the same and we have things working
    well, we can consider merging them again, but only after extensive
    testing across generations.

    --
    Jesse Barnes, Intel Open Source Technology Center


    \
     
     \ /
      Last update: 2011-03-25 18:49    [W:5.761 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site