On Fri, 16 May 2008, Jonathan Corbet wrote:May I suggest just adding a comment in those files, just saying something like /* This does not need the BKL, because .. */ where even the "because" part could be dropped when it's really obvious. That way that "list of files I examined and did *not* change" would be obvious in the patch itself, and we also have some documentation that somebody actually looked at the path. Same deal - just document the fact that the BKL isn't needed. Yeah, in the long run that kind of documentation is worthless and we may want to get rid of it again in a year or two, but in the short run it's a good idea. If only to help people who want to review your patches. Btw, do you have gitweb running anywhere? Linus --
| Zhang, Yanmin | AIM7 40% regression with 2.6.26-rc1 |
| Con Kolivas | [PATCH][RSDL-mm 0/7] RSDL cpu scheduler for 2.6.21-rc3-mm2 |
| Nick Piggin | [patch 4/6] mm: merge populate and nopage into fault (fixes nonlinear) |
| Andrew Morton | -mm merge plans for 2.6.23 |
git: | |
| Gerrit Renker | [PATCH 15/37] dccp: Set per-connection CCIDs via socket options |
| David Miller | Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock(). |
| David Miller | [GIT]: Networking |
| Natalie Protasevich | [BUG] New Kernel Bugs |
