Re: some [big] changes to ZPL (ZFS<->VFS )

2016-08-08 Thread Alan Somers
On r303834 I can no longer reproduce the problem. -Alan On Sat, Aug 6, 2016 at 5:05 AM, Andriy Gapon wrote: > On 05/08/2016 23:31, Alan Somers wrote: >> I'm not certain it's related, but on a head build at r303767 I see a >> LOR and a reproducible panic that involve the snapdir code. > > Alan, >

Re: some [big] changes to ZPL (ZFS<->VFS )

2016-08-06 Thread Andriy Gapon
On 05/08/2016 23:31, Alan Somers wrote: > I'm not certain it's related, but on a head build at r303767 I see a > LOR and a reproducible panic that involve the snapdir code. Alan, thank you very much for the clear report and for the very easy reproduction scenario. I am not sure how I missed this

Re: some [big] changes to ZPL (ZFS<->VFS )

2016-08-05 Thread Alan Somers
I'm not certain it's related, but on a head build at r303767 I see a LOR and a reproducible panic that involve the snapdir code. First, the LOR: $ zpool destroy foo lock order reversal: 1st 0xf800404c8b78 zfs (zfs) @ /usr/home/alans/freebsd/head/sys/kern/vfs_mount.c:1244 2nd 0xf800404c8

Re: some [big] changes to ZPL (ZFS<->VFS )

2016-08-04 Thread Andriy Gapon
On 03/08/2016 17:25, Andriy Gapon wrote: > Another change that was not strictly required and which is probably too > intrusive is killing the support for case insensitive operations. My > thinking was that FreeBSD VFS does not provide support for those anyway. > But I'll probably restore the cod

some [big] changes to ZPL (ZFS<->VFS )

2016-08-03 Thread Andriy Gapon
I would like to get more comments about the following review request: https://reviews.freebsd.org/D6533 Both about the code changes and about the general direction of the changes. I've been asked to try to get this change into 11.0 because of https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=2091