From: Erich F. <ef...@es...> - 2002-10-28 16:34:52
|
On Monday 28 October 2002 01:31, Martin J. Bligh wrote: > OK, so I'm trying to read your patch 1, fairly unsucessfully > (seems to be a lot more complex that Michael's). > > Can you explain pool_lock? It does actually seem to work, but > it's rather confusing .... The pool data is needed to be able to loop over the CPUs of one node, only. I'm convinced we'll need to do that sometime, no matter how simple the core of the NUMA scheduler is. The pool_lock is protecting that data while it is built. This can happen in future more often, if somebody starts hotplugging CPUs. > build_pools() has a comment above it saying: > > +/* > + * Call pooldata_lock() before calling this function and > + * pooldata_unlock() after! > + */ > > But then you promptly call pooldata_lock inside build_pools > anyway ... looks like it's just a naff comment, but doesn't > help much. Sorry, the comment came from a former version... > just block). If you really still need to do this, RCU is now > in the kernel ;-) If not, can we just chuck all that stuff? I'm preparing a core patch which doesn't need the pool_lock. I'll send it out today. Regards, Erich |