Messages in this thread | | | Date | Tue, 21 Dec 2010 22:47:30 -0800 | From | Darren Hart <> | Subject | Re: [PATCH 2/4] futex,plist: remove debug lock assignment for plist_node |
| |
On 12/21/2010 09:57 PM, Lai Jiangshan wrote: > On 12/22/2010 03:14 AM, Darren Hart wrote: >> On 12/21/2010 01:55 AM, Lai Jiangshan wrote: >>> original code use&plist_node->plist as the faked head of >>> the priority list for plist_del(), these debug locks in >>> the faked head are needed for CONFIG_DEBUG_PI_LIST. >>> >>> But now we always pass the real head to plist_del(), the debug locks >>> in plist_node will not be used, so we remove these assignments. >> >> Any reason to keep this separate from patch 1/4 ? >> > > Make the changes clearer and the patches are easier to read. > For me the purposes of patch 1/4 and 2/4 are different, > 1/4 makes the code sensible. 2/4 removes unused code.
Fair enough.
-- Darren Hart Yocto Linux Kernel
| |