On Mon, Mar 24, 2014 at 04:14:32PM -0700, Ausmus, James wrote: > On Sat, Mar 22, 2014 at 4:34 AM, Daniel Vetter <daniel@xxxxxxxx> wrote: > > On Fri, Mar 21, 2014 at 05:51:01PM -0700, Ben Widawsky wrote: > >> Let's try this again. I've pushed a branch here: > >> http://cgit.freedesktop.org/~bwidawsk/drm-intel/log/?h=bdw-backports > >> > >> I need to re-review some of the merge conflicts for 4g GGTT, which I > >> will try to do before Monday. > >> > >> Daniel: please make sure this is what you had in mind. I don't know > >> where you want Cc: stable tags. > > > > We don't need cc: stable, we just need to submit it (since it has the > > upstream sha1s already, which is the requirement for stable patches). Cc: > > stable is only for when you want it to get backport anyway. Otherwise > > looks good. I dunno whether git cherry-pick can be told to use the sha1 > > reference layout Greg prefers or not, he uses "This is <sha1> in > > upstream." between the commit header and the actual commit message. But > > ime his scripts reformat your commit messages anyway. > > > >> James: please test this as soon as possible. > > > > Once this is tested and we conclude it's sufficient to get bdw going on > > 3.14 without hilarity I think we should do a quick review on intel-gfx to > > check that the impact outside of bdw is indeed minimal. Then once drm-next > > has landed with all the referenced commits we can submit it to Greg with a > > small cover letter why we want this and that plan B would be to kill bdw > > in 3.14. > > This seems to be working well for me, with the one caveat that on boot > and once per resume I'm hitting the WARN(!i915_preliminary_hw_support, > "GEN8_CENTROID_PIXEL_OPT_DIS not be needed for production") code in > gen8_init_clock_gating - can that WARN be dropped via "drm/i915: Don't > use i915_preliminary_hw_support to mean pre-production" ? > > Both with and without that patch added, the series is: > > Tested-by: James Ausmus <james.ausmus@xxxxxxxxx> Thanks. Daniel, the patch is added to my backports branch. I think given that that it removes a WARN which we know to be bogus, it's a good patch for stable. But it's your call. [snip] -- Ben Widawsky, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
