Subject: Re: virt-manager broken by bind(0) in net-next.

From: Evgeniy Polyakov <[email-masked]>
Date: Sun, 1 Feb 2009 20:40:25 +0300

On Sun, Feb 01, 2009 at 05:12:41PM +0100, Eric Dumazet ([email-masked]) wrote:
> > David applied the patch which fixed the problem, so we can return to the
> > cache line issues. What do you think about the last version where
> > bsockets field was placed at the very end of the structure and with
> > cacheline_aligned_on_smp attribute?
> > 
> 
> Yes, at a minimum, move it away from first cache line.
> 
> And using atomic_t so that we dont have to discuss about accumulated
> errors on SMP on this variable. We will see later if percpu counter
> is wanted or not.
> 
> Thank you
> 
> [PATCH] net: move bsockets outside of read only beginning of struct inet_hashinfo
> 
> And switch bsockets to atomic_t since it might be changed in parallel.
> 
> Signed-off-by: Eric Dumazet <[email-masked]>

Ok, let's do it this way. Ack.
Thank you Eric.

-- 
	Evgeniy Polyakov