On Thu, 8 Oct 2009, Peter Zijlstra wrote: > > Right, so I did that full rename just so that people wouldn't get > confused or something, but if both you and Linus think it should remain: > kmap_atomic() and kunmap_atomic(), I can certainly undo that part. I think the renaming probably helps find all the places (simple "grep -w" shows the difference, and no fear of confusion with comma-expressions and multi-line arguments etc). But once they've all been converted, you might as well then do a search-and-replace-back on the patch, and make the end result look like you just removed the (now pointless) argument. In fact, I'd personally be inclined to split the patch into two patches: - one that just ignores the now redundant argument (but still keeps it), and fixes the cases that didn't nest - one that then removes the argument. Why? The _bugs_ are going to be shown by the first patch, and it would be nice to keep that patch small. When a bug shows up, it would be either because there's something wrong in that (much smaller) patch, or because some not-properly-nested casel wasn't fixed. In contrast, the second patch would be large, but if done right, you could then prove that it has no actual semantic changes (ie "binary is same before and after"). That just sounds _much_ nicer from a debug standpoint. Developers would look at the small and concentrated "real changes" patch, rather than be distracted by all the trivial noise. Linus -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
