Re: sb_flags vs sb_flagsintr

2017-12-08 Thread Alexander Bluhm
On Thu, Dec 07, 2017 at 01:37:15PM +0100, Martin Pieuchot wrote: > Right, updated diff below. OK bluhm@ > Index: kern/uipc_socket.c > === > RCS file: /cvs/src/sys/kern/uipc_socket.c,v > retrieving revision 1.209 > diff -u -p -r1.209

Re: sb_flags vs sb_flagsintr

2017-12-07 Thread Martin Pieuchot
On 05/12/17(Tue) 14:37, Alexander Bluhm wrote: > On Mon, Dec 04, 2017 at 03:33:56PM +0100, Martin Pieuchot wrote: > > Diff below change the usage of `sb_flags' and `sb_flagsintr'. The > > former will be protected by the socket lock while the latter will > > be using atomic operations. > > I like

Re: sb_flags vs sb_flagsintr

2017-12-05 Thread Alexander Bluhm
On Mon, Dec 04, 2017 at 03:33:56PM +0100, Martin Pieuchot wrote: > Diff below change the usage of `sb_flags' and `sb_flagsintr'. The > former will be protected by the socket lock while the latter will > be using atomic operations. I like this plan. > @@ -381,17 +381,18 @@ sbunlock(struct socket