On Tue, 19 Aug 2008, Simon Horman wrote: > On Mon, Aug 18, 2008 at 12:52:08AM +0200, Sven Wegener wrote: > > We can't access the cache entry outside of our critical read-locked region, > > because someone may free that entry. And we also need to check in the critical > > region wether the destination is still available, i.e. it's not in the trash. > > If we drop our reference counter, the destination can be purged from the trash > > at any time. Our caller only guarantees that no destination is moved to the > > trash, while we are scheduling. Also there is no need for our own rwlock, > > there is already one in the service structure for use in the schedulers. > > this looks good to me. I have confirmed that it applies against lvs-2.6 > (but not net-2.6 as changes that are already in lvs-2.6 are needed). Yes, I've based it on your tree. > One minor style question. You have changed struct ip_vs_lblc_table to *tbl > in several places, (e.g. sizeof(*tbl) ). I think that I prefer things the > way that they were. Was there a reason for this change? In this patch series, no. I prefer them like sizeof(*tbl) as it's more robust, when changing the type of tbl. Changing the type of tbl and forgeting the sizeof() in kmalloc will result in hard to track down bugs, if the old type is still valid. Using sizeof(*tbl) will nearly always get you the correct value you want, without keeping all the types in sizeof() in sync. The compiler already knows the type of tbl, so no need to be explicit about it, when allocating memory for it. Sven -- To unsubscribe from this list: send the line "unsubscribe lvs-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
