[Intel-gfx] [PATCH 1/2] Fix pitchAlignMask on i915

Carl Worth cworth at cworth.org
Thu Mar 18 17:46:31 CET 2010


On Wed, 17 Mar 2010 16:11:28 -0400, Matt Turner <mattst88 at gmail.com> wrote:
> On Wed, Mar 17, 2010 at 3:39 PM, Daniel Vetter <daniel at ffwll.ch> wrote:
> > Acked-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> >
> > This won't break anything, it'll just waste a few bytes of memory due to
> > a too big alignment. So not really 2.11 material.
> 
> Then again, this won't break anything, it'll just save a few bytes, so
> it could be 2.11 material. :)

At this stage of the game, (two release candidates already posted for
the 2.11 release), my primary concern is to avoid slipping in a
last-minute regression that manages to slip past all of the testing
we've done on previous versions.

I'm still happy to accept bug fixes. (They also have a non-zero risk of
causing a regressions, but they're at least fixing a bug at the same
time so the net effect is generally to reduce the bugginess of the
release.) But I agree with Daniel that changes that are primarily code
cleanups should be delayed until after the 2.11 release.

That said, we do appreciate these contributions! I've got both changes
committed to a branch named "after-2.11" in my local repository which
will be easy to merge after the release. Please feel free to ping me if
you don't see me merge it quickly after.

-Carl

PS. What's the purpose of the CC: lines added to the commit-message
portion of the email body? I understand the significance of actually
adding my address to the CC: line of the email itself, and that's
effective. But putting these into the commit message just looks like
noise in our code history. So I've removed those after committing. Did
some tool add those there or was that done manually?

Also, there was a "v2. ..." note in the second commit. Such a note is
useful for someone like me reviewing the patch and deciding whether to
commit it, but again, is not useful as a part of the permanent commit
history.

So if the change described there is just to bring the commit into
conformance with the original commit message, then please just move that
text down below the "---" line, (to just above the diffstat), so that
reviewers can still see the text, but "git am" will automatically ignore
the text and not include it in the commit messages.

If the text really does need to be included in the commit message for
some reason, then consider just updating the commit message to be
correct now, or consider following one commit with an additional commit.

Thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20100318/ffb9484f/attachment.sig>


More information about the Intel-gfx mailing list