From owner-svn-src-head@freebsd.org Sun Jun 19 14:33:51 2016 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 4C563A7A876; Sun, 19 Jun 2016 14:33:51 +0000 (UTC) (envelope-from mavbsd@gmail.com) Received: from mail-lf0-x232.google.com (mail-lf0-x232.google.com [IPv6:2a00:1450:4010:c07::232]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id BE55724F0; Sun, 19 Jun 2016 14:33:50 +0000 (UTC) (envelope-from mavbsd@gmail.com) Received: by mail-lf0-x232.google.com with SMTP id l188so23580222lfe.2; Sun, 19 Jun 2016 07:33:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:subject:to:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=PFgkSJ/ao4/s1xg/wsYbkKKmrVMkj8zIQgpBKKs/OSU=; b=QvLYL9qwpmLWTAY4bmgHAsEo2fVXBOEjmjBWekpidnIxyOTD4rSbuHRqwx45nVwA2L lvveK5HyqQWpc8BrTNCV7I2y9bzv6qCUszHBEl7Odecu8h2WTCnNWmyzi+ewUs4gCoX+ V9Tb7q+iye7d8heG8afQrdNFw45ueaBmyaS7kjxAOPDuEuREbO03IKGDZH0JGV06DWhB /pUtTeKgzc226m2QQ4TU9Ypq9KcgkWtmP+5EWx146vWsYgCwsxqFO5iOM+nsrbn2FX9F OGM2Bv7QnMLE+FYYzR4zYc8Iyz2r+fjVYmqB31b3AKeQkwfcEnl45SDvXdleSUwVt5eS a0zw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:subject:to:references:from:message-id :date:user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=PFgkSJ/ao4/s1xg/wsYbkKKmrVMkj8zIQgpBKKs/OSU=; b=N03Cd8xB9B0014yMbVwGA5BgrG5hxq8qVdJ71zwgg63E7WMDiZUQQw7dNC23E51+s2 9DBS5qMp0CEzN55EIj08E75/+thB2YSN/JVYlJNApeb1UHq0/BkSvmcz2IOoWCEZltF8 t43lGip6GY+XH1TXZ4kZoo+bG4TuHt/5XlPIFlOUzTjIYx6Y2SiYWy2pBRuJ1lcAk/Ml Av2xTAnKVKrpsN1pDrBEc7P4HvOAQb4UjqceFoILvYYc31hXmFHCKY56h8ulgyam8VrS 8nCCsSdkpQRJ50VrLvxEXYH/aFi4NkTeaXETtQ8BHpFmQM9YKSoYPFvc0z87fIa/8mZA OVfA== X-Gm-Message-State: ALyK8tK9FhHExzbIFs2YOsRX7PAORJZfahJNFAaa9vHXdpk+2ivrzQmExi3UJ0TVziz0oQ== X-Received: by 10.46.9.2 with SMTP id 2mr2897272ljj.1.1466346828570; Sun, 19 Jun 2016 07:33:48 -0700 (PDT) Received: from mavbook.mavhome.dp.ua ([134.249.139.101]) by smtp.gmail.com with ESMTPSA id y124sm5960396lfd.0.2016.06.19.07.33.47 (version=TLSv1/SSLv3 cipher=OTHER); Sun, 19 Jun 2016 07:33:47 -0700 (PDT) Sender: Alexander Motin Subject: Re: svn commit: r299448 - in head/sys/cddl/contrib/opensolaris: common/acl uts/common/fs/zfs uts/common/sys To: Cy Schubert , Jan Beich , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201606191428.u5JESbbs053857@slippy.cwsent.com> From: Alexander Motin Message-ID: <49d3d34d-ba91-ebdf-497f-cbe1614bec53@FreeBSD.org> Date: Sun, 19 Jun 2016 17:33:45 +0300 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <201606191428.u5JESbbs053857@slippy.cwsent.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 19 Jun 2016 14:33:51 -0000 On 19.06.16 17:28, Cy Schubert wrote: > In message <20160619080803.GA1638@brick>, Edward Tomasz > =?utf-8?Q?Napiera=C5=82 > a?= writes: >> On 0614T0232, Jan Beich wrote: >>> Alexander Motin writes: >>> >>>> Author: mav >>>> Date: Wed May 11 13:43:20 2016 >>>> New Revision: 299448 >>>> URL: https://svnweb.freebsd.org/changeset/base/299448 >>>> >>>> Log: >>>> MFV r299442: 6762 POSIX write should imply DELETE_CHILD on directories >> - and >>>> some additional considerations >>>> >>>> Reviewed by: Gordon Ross >>>> Reviewed by: Yuri Pankov >>>> Author: Kevin Crowe >>>> >>>> openzfs/openzfs@d316fffc9c361532a482208561bbb614dac7f916 >>> >>> This commit confuses acl_is_trivial_np(3). Notice '+' in ls(1) and 'D' >>> in getfacl(1) outputs. >> >> It's not just that. >> >> Those changes: >> >> 1. Confuse acl_is_trivial_np(3), as you say. It's hard to fix in libc, >> because they make trivial ACLs different for files and directories, >> and acl_is_trivial_np(3) has no way of telling which is which. >> >> 2. They make delete deny permission take precedence over the containing >> directory write allow permission, which is rather different from what >> people expect in unix systems, and is against the NFSv4 specification, >> even though it might be a better fit for Windows. > > This is Windows behavior and inconsistent with the rest of FreeBSD and any > UNIX or Linux system. > >> >> 3. They make umask apply to inherit_only permissions, and >> >> 4. I don't fully understand this one yet, but from the ACL regression >> test suite (which lives in tests/sys/acl/, and I'd appreciate people >> actually ran this before committing ACL-related changes) it looks >> like it makes umask not apply to the stuff it should. >> >> The #1 could be fixed by making ZFS not setting delete_child on write, >> basically reverting to the previous behaviour in that aspect. As for >> the others... I'm not saying each one of those is wrong, but they >> certainly warrant further discussion, especially #2 and #4. > > I think #2 is wrong behavior on any UNIX-like or POSIX system. > >> >> Basically, what I'm trying to say is that we should consider backing >> this out for 11.0-RELEASE, reverting to the previous semantics, verified >> by passing the regression tests. > > Agreed. > > What in FreeBSD was this patch supposed to solve in the first place? Growing divergence from OpenZFS upstream. I am not advocating this patch, but it would be good, if possible, to not revert it completely, but block wrong behavior with some minimal ifdefs to make further ZFS merges easier. Help would be appreciated. ;) -- Alexander Motin