Re: panic: share -> excl @r224632

2011-08-05 Thread David Wolfskill
On Fri, Aug 05, 2011 at 10:43:32AM +0200, Martin Matuska wrote: > Patch updated. > ... OK; after reverting kern/vfs_mount.c to its pre-patched status on my laptop, I applied this most recent patch without incident; it built & boots fine, which is now running: FreeBSD g1-222.catwhisker.org. 9.0-BE

Re: panic: share -> excl @r224632

2011-08-05 Thread Martin Matuska
Patch updated. On 05.08.2011 10:26, Kostik Belousov wrote: > On Fri, Aug 05, 2011 at 10:18:49AM +0200, Martin Matuska wrote: >> I agree to Kostik's approach, but I suggest implementing it in a >> separate function and also use for the unmount() part. >> >> Please review attached patch. > Since yo

Re: panic: share -> excl @r224632

2011-08-05 Thread Kostik Belousov
On Fri, Aug 05, 2011 at 10:18:49AM +0200, Martin Matuska wrote: > I agree to Kostik's approach, but I suggest implementing it in a > separate function and also use for the unmount() part. > > Please review attached patch. Since you are moving the fragment to a function, you may somewhat reduce th

Re: panic: share -> excl @r224632

2011-08-05 Thread Martin Matuska
I agree to Kostik's approach, but I suggest implementing it in a separate function and also use for the unmount() part. Please review attached patch. On 04.08.2011 15:59, Kostik Belousov wrote: > On Thu, Aug 04, 2011 at 05:54:54AM -0700, David Wolfskill wrote: >> I've only seen this on my laptop

Re: panic: share -> excl @r224632

2011-08-04 Thread David Wolfskill
On Thu, Aug 04, 2011 at 04:59:32PM +0300, Kostik Belousov wrote: > ... > I am sure that this is caused by r224614. Well, see: you approached it from knowledge, while I made a quasi-educated guess. :-} > I forgot that vn_fullpath cannot operate on the locked vnode. And I expect you have likely fo

Re: panic: share -> excl @r224632

2011-08-04 Thread Kostik Belousov
On Thu, Aug 04, 2011 at 05:54:54AM -0700, David Wolfskill wrote: > I've only seen this on my laptop; my build machine doesn't exhibit the > panic. > > r224602 is the most recent point I've built that does not exhibit the > panic at all. > > The first few lines (hand-transcribed; I have no serial

Re: panic: share -> excl @r224632

2011-08-04 Thread David Wolfskill
On Thu, Aug 04, 2011 at 05:54:54AM -0700, David Wolfskill wrote: > ... > I will see if I can find a commit that affected at least one of the > affected files in the above list that I can revert to avoid the panic, > but I'm a bit slow for a while yet, so I figured I'd finally get around > to postin