From owner-freebsd-bugs@FreeBSD.ORG  Tue May  5 13:23:07 2015
Return-Path: <owner-freebsd-bugs@FreeBSD.ORG>
Delivered-To: freebsd-bugs@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by hub.freebsd.org (Postfix) with ESMTPS id AFA2ACD2
 for <freebsd-bugs@FreeBSD.org>; Tue,  5 May 2015 13:23:07 +0000 (UTC)
Received: from kenobi.freebsd.org (kenobi.freebsd.org
 [IPv6:2001:1900:2254:206a::16:76])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (Client did not present a certificate)
 by mx1.freebsd.org (Postfix) with ESMTPS id 9A16D11F7
 for <freebsd-bugs@FreeBSD.org>; Tue,  5 May 2015 13:23:07 +0000 (UTC)
Received: from bugs.freebsd.org ([127.0.1.118])
 by kenobi.freebsd.org (8.14.9/8.14.9) with ESMTP id t45DN7s5088822
 for <freebsd-bugs@FreeBSD.org>; Tue, 5 May 2015 13:23:07 GMT
 (envelope-from bugzilla-noreply@freebsd.org)
From: bugzilla-noreply@freebsd.org
To: freebsd-bugs@FreeBSD.org
Subject: [Bug 53475] cp(1) copies files in reverse order to destination
Date: Tue, 05 May 2015 13:23:07 +0000
X-Bugzilla-Reason: AssignedTo
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: Base System
X-Bugzilla-Component: bin
X-Bugzilla-Version: 4.8-STABLE
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: Affects Only Me
X-Bugzilla-Who: commit-hook@freebsd.org
X-Bugzilla-Status: In Progress
X-Bugzilla-Priority: Normal
X-Bugzilla-Assigned-To: freebsd-bugs@FreeBSD.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-53475-8-cedHWVBVcJ@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-53475-8@https.bugs.freebsd.org/bugzilla/>
References: <bug-53475-8@https.bugs.freebsd.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-BeenThere: freebsd-bugs@freebsd.org
X-Mailman-Version: 2.1.20
Precedence: list
List-Id: Bug reports <freebsd-bugs.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-bugs>,
 <mailto:freebsd-bugs-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-bugs/>
List-Post: <mailto:freebsd-bugs@freebsd.org>
List-Help: <mailto:freebsd-bugs-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
 <mailto:freebsd-bugs-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 05 May 2015 13:23:07 -0000

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=53475

--- Comment #3 from commit-hook@freebsd.org ---
A commit references this bug:

Author: jilles
Date: Tue May  5 13:23:04 UTC 2015
New revision: 282482
URL: https://svnweb.freebsd.org/changeset/base/282482

Log:
  cp: Remove fts sorting.

  In an attempt to improve performance, cp reordered directories first
  (although the comment says directories last). This is not effective with new
  UFS layout policies.

  The sorting reorders multiple arguments passed to cp, which may be
  undesirable.

  Additionally, the comparison function does not induce a total order. Per
  POSIX, this causes undefined behaviour in qsort().

  NetBSD removed the sorting in 2009.

  On filesystems that return directory entries in hash/btree order, sorting by
  d_fileno before statting improves performance on large directories. However,
  this can only be implemented in fts(3).

  PR:        53475
  Reviewed by:    bde (in 2004)
  MFC after:    1 week

Changes:
  head/bin/cp/cp.c

-- 
You are receiving this mail because:
You are the assignee for the bug.