Re: CVS commit: src/sys/netinet

2017-12-26 Thread Ryota Ozaki
On Tue, Dec 26, 2017 at 5:23 PM, Ryota Ozaki wrote: > On Tue, Dec 26, 2017 at 4:50 PM, Tom Ivar Helbekkmo > wrote: >> Ryota Ozaki writes: >> >>> One possible fix has been committed. >>> >>> Can you update the source code and try a

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Ryota Ozaki
On Wed, Dec 27, 2017 at 10:36 AM, Taylor R Campbell wrote: >> Date: Wed, 27 Dec 2017 10:31:25 +0900 >> From: Ryota Ozaki >> >> On Wed, Dec 27, 2017 at 10:27 AM, Taylor R Campbell >>

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Taylor R Campbell
> Date: Wed, 27 Dec 2017 10:31:25 +0900 > From: Ryota Ozaki > > On Wed, Dec 27, 2017 at 10:27 AM, Taylor R Campbell > wrote: > > Can you add this to the collection? > > > > #ifdef NET_MPSAFE > > #define

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Ryota Ozaki
On Wed, Dec 27, 2017 at 10:27 AM, Taylor R Campbell wrote: >> Date: Wed, 27 Dec 2017 10:24:26 +0900 >> From: Ryota Ozaki >> >> On Wed, Dec 27, 2017 at 12:50 AM, Taylor R Campbell >>

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Taylor R Campbell
> Date: Wed, 27 Dec 2017 10:24:26 +0900 > From: Ryota Ozaki > > On Wed, Dec 27, 2017 at 12:50 AM, Taylor R Campbell > wrote: > > Can you just pass the interlock -- softnet_lock or NULL, depending on > > NET_MPSAFE -- as an

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Ryota Ozaki
On Wed, Dec 27, 2017 at 12:50 AM, Taylor R Campbell wrote: >> Date: Tue, 26 Dec 2017 18:42:14 +0900 >> From: Ryota Ozaki >> >> On Tue, Dec 26, 2017 at 6:30 PM, Paul Goyette wrote: >> > I'd really like to see

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Taylor R Campbell
> Date: Tue, 26 Dec 2017 18:42:14 +0900 > From: Ryota Ozaki > > On Tue, Dec 26, 2017 at 6:30 PM, Paul Goyette wrote: > > I'd really like to see other peoples' opinions on this matter before > > making a final decision. (And I'm rather surprised that

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Ryota Ozaki
On Tue, Dec 26, 2017 at 6:30 PM, Paul Goyette wrote: > On Tue, 26 Dec 2017, Ryota Ozaki wrote: > >> On Tue, Dec 26, 2017 at 4:30 PM, Paul Goyette wrote: >>> >>> On Tue, 26 Dec 2017, Ryota Ozaki wrote: >>> > Well, since the lock _might_ be released (and

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Paul Goyette
On Tue, 26 Dec 2017, Ryota Ozaki wrote: On Tue, Dec 26, 2017 at 4:30 PM, Paul Goyette wrote: On Tue, 26 Dec 2017, Ryota Ozaki wrote: Well, since the lock _might_ be released (and subsequently reacquired) by callout_halt(), it might be easiest to modify all the callers to

Re: CVS commit: src/sys/netinet6

2017-12-26 Thread Ryota Ozaki
On Tue, Dec 26, 2017 at 4:30 PM, Paul Goyette wrote: > On Tue, 26 Dec 2017, Ryota Ozaki wrote: > >>> Well, since the lock _might_ be released (and subsequently reacquired) >>> by callout_halt(), it might be easiest to modify all the callers to >>> just unlock it before calling

Re: CVS commit: src/sys/netinet

2017-12-26 Thread Ryota Ozaki
On Tue, Dec 26, 2017 at 4:50 PM, Tom Ivar Helbekkmo wrote: > Ryota Ozaki writes: > >> One possible fix has been committed. >> >> Can you update the source code and try a new kernel? > > Will do. Thanks. > > Meanwhile, before I got around to building