Thomas Graf wrote: > * Patrick McHardy <[EMAIL PROTECTED]> 2006-08-10 20:09 > >>I disagree with this patch, how else are applications supposed >>to know when they missed an update and are not in sync anymore? >>I actually have a half-finished patch to add this in some spots >>where its missing (and uses better error codes). > > > The application has no idea what went wrong nor does it know > for which group so it will have to resync all group subscrptions > and as it only happens due to memory pressure that will fail > anyway.
The error code (-ENOMEM) gives it a pretty good idea what went wrong. Its true that it doesn't know which group was affected (that could be fixed), but at least it knows that something went wrong and it needs to resync. If that fails due to memory shortage as well it can schedule a delayed resync or something, but without getting notified it has no chance of doing anything useful. This makes notification essentially useless. If I can't rely on either getting either a notification or an error, I can't rely on them at all. Please put this back in. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
