[POLICY] Re: [PATCH] Fwd: Re: [Intel-gfx] drm/i915 stable patches

[Steve Conklin]

Timo,

It's not just you, but emails for patches to be applied have been
getting less and less like the desired format. Although it's described here:

https://wiki.ubuntu.com/KernelTeam/StableHandbook/StableProcess

In reality it doesn't usually work that way, especially for patches from
upstream. In particular, we often don't have a launchpad bug report.
However, if there is one that can be used to verify that the applied
patch works, please include it.

For these patches in particular, here's what we should have:

1. Submitter has already insured that the patches apply to each series
for which they are being requested.

2. Some justification of what the patches fix. It's ok here to include
links to other bugs and mailing list archives.

3. Series names [QUANTAL], etc in subject line

4. Patch included in the email, not referenced through a pointer to a
bug or other email list. Passing patches by reference makes it possible
to be confused, or for later additions to a referenced bug to confuse
discussion

5. Separate emails for each patch, and for each series if the patches
differ. This makes it possible to ack/nack/discuss each proposed
application of the patches. In the current case, we already have an ack
to the original email which only applies to one of the two patches.

Don't resubmit these, but please follow process in the future - I'll do
the test builds, etc for these, and follow up to the original email.

Steve

On 05/07/2013 01:58 AM, Timo Aaltonen wrote:
> 
> 	Hi!
> 
>   Could you apply the commits referenced in the forwarded email at least
> to the raring kernel?
> 
> 
> -------- Original Message --------
> Subject: Re: [Intel-gfx] drm/i915 stable patches
> Date: Mon, 6 May 2013 10:36:49 +0200
> From: Daniel Vetter <daniel.vetter at ffwll.ch>
> To: stable <stable at vger.kernel.org>
> CC: intel-gfx <intel-gfx at lists.freedesktop.org>
> 
> On Mon, May 6, 2013 at 10:33 AM, Daniel Vetter <daniel.vetter at ffwll.ch>
> wrote:
>> Dear stable team,
>>
>> Please backport
>>
>> commit 4615d4c9e27eda42c3e965f208a4b4065841498c
>> Author: Chris Wilson <chris at chris-wilson.co.uk>
>> Date:   Mon Apr 8 14:28:40 2013 +0100
>>
>>     drm/i915: Use MLC (l3$) for context objects
>>
>> to all supported stable kernels. It's not just a performance
>> optimization, but seems to prevent gpu hangs, too.
>>
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=64073
> 
> And another one for 3.8 and older:
> 
> commit e12a2d53ae45a69aea499b64f75e7222cca0f12f
> Author: Chris Wilson <chris at chris-wilson.co.uk>
> Date:   2012-11-15 11:32:18 +0000
> 
>     drm/i915: Fix detection of base of stolen memory
> 
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=53541
> 
> Cheers, Daniel
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
> 
> 
>