Re: kern/165863: [panic] [netinet] [patch] in_lltable_prefix_free() races with lla_lookup() and arptimer()

2012-08-03 Thread glebius
Synopsis: [panic] [netinet] [patch] in_lltable_prefix_free() races with lla_lookup() and arptimer() State-Changed-From-To: open->patched State-Changed-By: glebius State-Changed-When: Fri Aug 3 08:01:48 UTC 2012 State-Changed-Why: Fix committed to head/. Responsible-Changed-From-To: freebsd-net

Re: kern/165863: commit references a PR

2012-08-02 Thread dfilter service
The following reply was made to PR kern/165863; it has been noted by GNATS. From: dfil...@freebsd.org (dfilter service) To: bug-follo...@freebsd.org Cc: Subject: Re: kern/165863: commit references a PR Date: Thu, 2 Aug 2012 13:58:02 + (UTC) Author: glebius Date: Thu Aug 2 13:57:49 2012

Re: kern/165863

2012-08-02 Thread Gleb Smirnoff
On Tue, Jul 31, 2012 at 10:43:42PM +0400, Andrey Zonov wrote: A> Please review attached patch. I used Gleb's ideas. He almost fixed the A> issue, but he didn't observe that entry can be safely unlocked in A> arptimer() because it has refcnt incremented and cannot be removed. I A> also fixed e

Re: kern/165863

2012-07-31 Thread Andrey Zonov
On 4/11/12 12:26 AM, Ryan Stone wrote: 2012/4/10 Gleb Smirnoff : Thanks, Ryan! (snip) Looks okay from my viewpoint. Have you stress tested it? My snap patch definitely had problems, AFAIR. If this patch fixes panics observed by kern/165863 and passes stress testing, then it should be commit

Re: kern/165863

2012-04-10 Thread Ryan Stone
2012/4/10 Gleb Smirnoff : >  Thanks, Ryan! (snip) > Looks okay from my viewpoint. Have you stress tested it? My snap patch > definitely had problems, AFAIR. > > If this patch fixes panics observed by kern/165863 and passes stress > testing, then it should be committed ASAP, and shouldn't depend on

Re: kern/165863

2012-04-09 Thread Gleb Smirnoff
Thanks, Ryan! On Thu, Mar 29, 2012 at 05:59:38PM -0400, Ryan Stone wrote: R> Ok, I think that I have an approach that will work. This is heavily R> based off of glebius' proposal. The big difference is that instead of R> initializing the arptimer callout with the ll_entry's lock, I R> initiali

Re: kern/165863

2012-03-29 Thread Ryan Stone
Ok, I think that I have an approach that will work. This is heavily based off of glebius' proposal. The big difference is that instead of initializing the arptimer callout with the ll_entry's lock, I initialize it with the if_afdata_lock. This eliminates the LOR problem in arptimer. It also nic

Re: kern/165863

2012-03-27 Thread Gleb Smirnoff
On Tue, Mar 27, 2012 at 04:36:59PM -0400, Ryan Stone wrote: R> > From: Gleb Smirnoff R> > To: Eric van Gyzen , R> >        Eric van Gyzen , ema...@freebsd.org R> > Cc: bug-follo...@freebsd.org R> > Subject: kern/165863 R> > Date: Fri, 9 Mar 2012 13:20:56 +0400 R> > R> >  --BXVAT5kNtrzKuDFl R> >  C

Re: kern/165863

2012-03-27 Thread Ryan Stone
The following reply was made to PR kern/165863; it has been noted by GNATS. From: Ryan Stone To: Eric van Gyzen Cc: Gleb Smirnoff , Eric van Gyzen , ema...@freebsd.org, bug-follo...@freebsd.org Subject: Re: kern/165863 Date: Tue, 27 Mar 2012 17:24:37 -0400 2012/3/9 Eric van Gyzen

Re: kern/165863

2012-03-27 Thread Ryan Stone
On Fri, Mar 9, 2012 at 4:30 AM, Gleb Smirnoff wrote: > The following reply was made to PR kern/165863; it has been noted by GNATS. > > From: Gleb Smirnoff > To: Eric van Gyzen , >        Eric van Gyzen , ema...@freebsd.org > Cc: bug-follo...@freebsd.org > Subject: kern/165863 > Date: Fri, 9 Mar 2

Re: kern/165863

2012-03-09 Thread Eric van Gyzen
The following reply was made to PR kern/165863; it has been noted by GNATS. From: Eric van Gyzen To: Gleb Smirnoff Cc: Eric van Gyzen , , , Subject: Re: kern/165863 Date: Fri, 9 Mar 2012 09:26:51 -0600 --000307010007030804050405 Content-Type: text/plain; charset="K

Re: kern/165863: [panic] [netinet] [patch] in_lltable_prefix_free() races with lla_lookup() and arptimer()

2012-03-08 Thread Eric van Gyzen
The following reply was made to PR kern/165863; it has been noted by GNATS. From: Eric van Gyzen To: bug-follo...@freebsd.org Cc: Subject: Re: kern/165863: [panic] [netinet] [patch] in_lltable_prefix_free() races with lla_lookup() and arptimer() Date: Thu, 08 Mar 2012 20:23:14 -0600 I just

Re: kern/165863: [panic] [netinet] [patch] in_lltable_prefix_free() races with lla_lookup() and arptimer()

2012-03-08 Thread linimon
Old Synopsis: [panic] [patch] in_lltable_prefix_free() races with lla_lookup() and arptimer() New Synopsis: [panic] [netinet] [patch] in_lltable_prefix_free() races with lla_lookup() and arptimer() Responsible-Changed-From-To: freebsd-bugs->freebsd-net Responsible-Changed-By: linimon Responsible