svn commit: r214431 - head/bin/rm

Alexander Best arundel at freebsd.org
Wed Oct 27 21:26:02 UTC 2010


On Wed Oct 27 10, Dag-Erling Smorgrav wrote:
> Author: des
> Date: Wed Oct 27 18:48:23 2010
> New Revision: 214431
> URL: http://svn.freebsd.org/changeset/base/214431
> 
> Log:
>   Language cleanup.
> 
> Modified:
>   head/bin/rm/rm.1
> 
> Modified: head/bin/rm/rm.1
> ==============================================================================
> --- head/bin/rm/rm.1	Wed Oct 27 18:46:56 2010	(r214430)
> +++ head/bin/rm/rm.1	Wed Oct 27 18:48:23 2010	(r214431)
> @@ -231,6 +231,6 @@ The
>  .Fl P
>  option assumes that the underlying file system updates existing blocks
>  in-place and does not store new data in a new location.
> -This is true for UFS but not for ZFS, which is using a Copy-On-Write strategy.
> -In addition, only regular files are overwritten, other types of files
> -are not.
> +This is true for UFS, but not for ZFS or other file systems which use
> +copy-on-write semantics.
> +In addition, only regular files are overwritten.

are in fact COW fs the only exception where the -P flag won't work? before
r213582 LFS was mentioned here and that the block size must be fixed.
also the comment in rm.c says that -P won't work for any logging file systems.
i'm not a fs expert, but i think mentioning that -P won't work for COW fs isn't
enough.

-- 
a13x


More information about the svn-src-head mailing list