lkml.org 
[lkml]   [2010]   [Jun]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] swiotlb-0.8.2: seperation of physical/virtual address translation and core memory allocation.
On Fri,  4 Jun 2010 13:52:30 -0400
Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> wrote:

> Please include the following patches, located at this git tree:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/konrad/swiotlb-2.6.git swiotlb-0.8.2
>
> in your -mm tree.

I don't put individual git trees into -mm any more - I get them from
linux-next.

I could grab the individual patches but I don't see that this gains
anyone anything. I'd suggest that you ask Stephen to include this tree
in linux-next then ask Linus to pull it into 2.6.36-rc1.

> [nice description]

The sad thing is that this description will get lost because it isn't
attached to a particular patch. So what I do is to copy the nice
covering description into the changelog for [patch 1/n] and then, after
it I put the text

This patch:

and then follow it with the specific description for [patch 1/n]. I
suggest that you make such change in this tree.

It's not a very good solution because anyone who is looking at [patch
5/n] isn't likely to think to look in some other patch for the overall
story. I guess it would be better if patches 2..n included the text
"see <patch name> for a description of this work" or similar.

Even better would be if git understood this workflow - if it was able
to take commentary-only commits and to bidirectionally link those
commentaries to the invividual commits which they describe.

The patches themselves look good and suitably reviewed to me. My one
very minor comment is that include/linux/swiotlb.h ends up with a
mixture of "enum dma_data_direction dir" and "enum dma_data_direction
direction", which could be cleaned up.



\
 
 \ /
  Last update: 2010-06-04 20:47    [W:0.063 / U:0.732 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site