On Tue, Mar 13, 2007 at 07:41:37PM +0530, Srivatsa Vaddagiri wrote: > On Tue, Mar 13, 2007 at 02:55:05PM +0100, Herbert Poetzl wrote: > > yes, tons of locking, complicated indirections and > > a lot of (partially hard to understand) code ... > > Are you referring to these issues in the general Paul Menage's > container code or in the RSS-control code posted by Pavel?
the container code has quite a number of locks, including the container(_manage)_lock/unlock, but those should not hurt that much, as restructuring is not on the hot path (as far as I can tell) but I was more referring to the charge/uncharge(_locked) inlines which will be execised quite often ... and btw, why not use a 'generic' accounting macro/inline to do the accounting for all the different resources like files, rss, sockets (instead of duplicating the code over and over again) that would really simplify improving the accounting by switching to atomics or even per cpu based mechanisms best, Herbert > -- > Regards, > vatsa > _______________________________________________ > Containers mailing list > [EMAIL PROTECTED] > https://lists.osdl.org/mailman/listinfo/containers - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
