<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 21, 2013 at 1:29 PM, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Long ago, when porting FBO and memory management support to i965, I<br>
merged a bunch of code between the i915 and i965 drivers and put it in<br>
the intel directory.  I think it served us well for a long time, as both<br>
drivers got improvements from shared work on that code.  But since then,<br>
we've talked several times about splitting things back apart (since we<br>
break i915 much more often than we improve it), so I spent yesterday and<br>
today looking at what the impact would be.<br>
<br></blockquote><div><br></div><div style>I'm not a developer, but I like to keep up with the drivers that I have hardware for. Please take my opinions with a grain of salt.</div><div style><br></div><div style>When you say you break i915 more than you improve it, do you mean to say that it is difficult to improve !i915 without breaking i915 and therefore to improve development speed, it should be forked OR that i915 doesn't receive enough testing / have maintainers who can resolve the issues and so it burdens other developers to fix i915 and hence slows development?</div>
<div style><br></div><div style>The reason I ask if because if it is #2, then it sounds like you should be looking for someone to volunteer as the official i915 maintainer [and if none, then fork], but if it is #1, then maintainer or not, it will slow down your efforts.</div>
<div style><br></div><div style><br></div><div style> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
LOC counts (wc -l):<br>
<br>
            intel/     i915/       i965/         total<br>
master:     14751      13458       61109         89318<br>
fork-i915:  0          24322       74978         99300<br>
<br>
We duplicate ~10000 lines of code, but i915 drops ~4000 lines of code<br>
>From its build and i965 drops ~1000.<br>
<br>
context size:<br>
               i915        i965<br>
master:        99512       101456<br>
fork-i915:     99384       100824<br>
<br>
There's a bunch of cleanup I haven't done in the branch, like moving<br>
brw_vtbl.c contents to sensible places, or nuking the intel vs brw split<br>
that doesn't make any sense any more.<br>
<br>
I'm ambivalent about the change.  If the code growth from splitting was<br>
<7000 lines or so, I'd be happy, but this feels pretty big.  On the<br>
other hand, the cleanups feel good to me.  I don't know how other<br>
developers feel.  There's a branch up at fork-i915 of my tree.  If<br>
people are excited about doing this and I get a bunch of acks for the<br>
two "copy the code to my directory" commits, I'll do those two then<br>
start sending out the non-copying changes for review.  If people don't<br>
like it, I won't be hurt.<br>
<br>_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
<br></blockquote></div><br></div></div>