Date: Thu, 7 Feb 2013 18:41:38 -0500 (EST) From: Rick Macklem <rmacklem@uoguelph.ca> To: Sergey Kandaurov <pluknet@gmail.com> Cc: Konstantin Belousov <kostikbel@gmail.com>, FreeBSD Current <freebsd-current@freebsd.org>, Andriy Gapon <avg@freebsd.org> Subject: Re: panic: LK_RETRY set with incompatible flags Message-ID: <1644867646.2814542.1360280498907.JavaMail.root@erie.cs.uoguelph.ca> In-Reply-To: <CAE-mSO%2B46crXpYMsu4B3dkMNmp7S70j_VvmyfxNSeJbskgS2Mw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Sergey Kandaurov wrote: > On 7 February 2013 19:42, Andriy Gapon <avg@freebsd.org> wrote: > > on 07/02/2013 17:36 Sergey Kandaurov said the following: > >> I tested the patch without the (*vpp != dvp) change. > >> It works well. > >> > >> It's something unrelated but when doing ls -l > >> on server (patched) and client (unpatched) sides, > >> I found some inconsistency in returned stats. > >> Or more precisely: > >> > >> NFS server > >> # stat -s /pool1/user1000/.zfs/shares/.. > >> st_dev=2050684725 st_ino=1 st_mode=040555 st_nlink=4 st_uid=0 > >> st_gid=0 > >> st_rdev=0 st_size=4 st_atime=1360251211 st_mtime=1359551493 > >> st_ctime=1359551493 st_birthtime=1359551493 st_blksize=4096 > >> st_blocks=0 st_flags=0 > >> > >> NFS client > >> # stat -s /home/user1000/.zfs/shares/.. > >> st_dev=2050684725 st_ino=7 st_mode=040555 st_nlink=2 st_uid=0 > >> st_gid=0 > >> st_rdev=1377468712 st_size=2 st_atime=1360251104 > >> st_mtime=1359551493 > >> st_ctime=1359551493 st_birthtime=-1 st_blksize=4096 st_blocks=3 > >> st_flags=0 > > > > Hmm, this looks more consistent with the earlier patch. > > Are you sure that you really tested the new kernel (on the server)? > > Sorry, I indeed booted a wrong kernel. > Now tested the really new kernel :) And it works as well. > stat -s looks consistent, thanks. > > # stat -s /home/user1000/.zfs/shares/.. > st_dev=2050684725 st_ino=1 st_mode=040555 st_nlink=4 st_uid=0 st_gid=0 > st_rdev=0 st_size=4 st_atime=1360252962 st_mtime=1359551493 > st_ctime=1359551493 st_birthtime=-1 st_blksize=4096 st_blocks=0 > st_flags=0 > Sounds good. I've attached a slightly updated patch with Andriy's suggested addition of a check for zfsvfs->z_shares_dir != 0. I can't do any commits until April, so if one of you guys is comfortable enough with the patch to commit it, you are more than welcome to do so. Thanks everyone for your help in resolving this, rick > -- > wbr, > pluknet > _______________________________________________ > freebsd-current@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-current > To unsubscribe, send any mail to > "freebsd-current-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1644867646.2814542.1360280498907.JavaMail.root>