Subject: Re: [PATCH 3/4] [NETLINK]: Dont set socket error for failed event notifications

From: Thomas Graf <[email-masked]>
Date: Thu, 10 Aug 2006 21:04:55 +0200

* Patrick McHardy <[email-masked]> 2006-08-10 20:09
> Thomas Graf wrote:
> > Setting a socket error on all sockets subscribed to a group
> > if an event notificiation of said group fails due to memory
> > pressure only confuses applications and is of no use. 
> > 
> > This patch removes it all together.
> 
> 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.