mirror of
https://github.com/moparisthebest/pacman
synced 2024-12-22 15:58:50 -05:00
Large performance improvement for check for owned directories
We can take a large shortcut here that saves us a lot of time, especially when upgrading packages with lots of directories. Obviously iterating the full file list of every single package to determine if this directory was present in any other package can take quite some time on a system with many packages installed. We don't need to remove a directory at all if we are upgrading a package and the version we are moving to still had the directory. Also make a small optimization on the package comparsion- we really only care about equality here, not the result of the compare, so we can shortcut using our name_hash. What kind of benefit does this give us? Oh, only a reduction from 295.7 million to 1.4 million strcmp() calls (99.5% fewer) during a `pacman -S linux libreoffice-common` operation. Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
parent
bf84dc4cf1
commit
ae25167bcd
@ -239,8 +239,9 @@ static int can_remove_file(alpm_handle_t *handle, const alpm_file_t *file,
|
|||||||
|
|
||||||
/* Helper function for iterating through a package's file and deleting them
|
/* Helper function for iterating through a package's file and deleting them
|
||||||
* Used by _alpm_remove_commit. */
|
* Used by _alpm_remove_commit. */
|
||||||
static int unlink_file(alpm_handle_t *handle, alpm_pkg_t *info,
|
static int unlink_file(alpm_handle_t *handle, alpm_pkg_t *oldpkg,
|
||||||
const alpm_file_t *fileobj, alpm_list_t *skip_remove, int nosave)
|
alpm_pkg_t *newpkg, const alpm_file_t *fileobj, alpm_list_t *skip_remove,
|
||||||
|
int nosave)
|
||||||
{
|
{
|
||||||
struct stat buf;
|
struct stat buf;
|
||||||
char file[PATH_MAX];
|
char file[PATH_MAX];
|
||||||
@ -274,6 +275,10 @@ static int unlink_file(alpm_handle_t *handle, alpm_pkg_t *info,
|
|||||||
} else if(files < 0) {
|
} else if(files < 0) {
|
||||||
_alpm_log(handle, ALPM_LOG_DEBUG,
|
_alpm_log(handle, ALPM_LOG_DEBUG,
|
||||||
"keeping directory %s (could not count files)\n", file);
|
"keeping directory %s (could not count files)\n", file);
|
||||||
|
} else if(newpkg && _alpm_filelist_contains(alpm_pkg_get_files(newpkg),
|
||||||
|
fileobj->name)) {
|
||||||
|
_alpm_log(handle, ALPM_LOG_DEBUG,
|
||||||
|
"keeping directory %s (in new package)\n", file);
|
||||||
} else {
|
} else {
|
||||||
/* one last check- does any other package own this file? */
|
/* one last check- does any other package own this file? */
|
||||||
alpm_list_t *local, *local_pkgs;
|
alpm_list_t *local, *local_pkgs;
|
||||||
@ -285,7 +290,8 @@ static int unlink_file(alpm_handle_t *handle, alpm_pkg_t *info,
|
|||||||
|
|
||||||
/* we duplicated the package when we put it in the removal list, so we
|
/* we duplicated the package when we put it in the removal list, so we
|
||||||
* so we can't use direct pointer comparison here. */
|
* so we can't use direct pointer comparison here. */
|
||||||
if(_alpm_pkg_cmp(info, local_pkg) == 0) {
|
if(oldpkg->name_hash == local_pkg->name_hash
|
||||||
|
&& strcmp(oldpkg->name, local_pkg->name) == 0) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
filelist = alpm_pkg_get_files(local_pkg);
|
filelist = alpm_pkg_get_files(local_pkg);
|
||||||
@ -308,7 +314,7 @@ static int unlink_file(alpm_handle_t *handle, alpm_pkg_t *info,
|
|||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
/* if the file needs backup and has been modified, back it up to .pacsave */
|
/* if the file needs backup and has been modified, back it up to .pacsave */
|
||||||
alpm_backup_t *backup = _alpm_needbackup(fileobj->name, info);
|
alpm_backup_t *backup = _alpm_needbackup(fileobj->name, oldpkg);
|
||||||
if(backup) {
|
if(backup) {
|
||||||
if(nosave) {
|
if(nosave) {
|
||||||
_alpm_log(handle, ALPM_LOG_DEBUG, "transaction is set to NOSAVE, not backing up '%s'\n", file);
|
_alpm_log(handle, ALPM_LOG_DEBUG, "transaction is set to NOSAVE, not backing up '%s'\n", file);
|
||||||
@ -431,7 +437,7 @@ int _alpm_remove_single_package(alpm_handle_t *handle,
|
|||||||
alpm_file_t *file = filelist->files + i - 1;
|
alpm_file_t *file = filelist->files + i - 1;
|
||||||
int percent;
|
int percent;
|
||||||
/* TODO: check return code and handle accordingly */
|
/* TODO: check return code and handle accordingly */
|
||||||
unlink_file(handle, oldpkg, file, skip_remove,
|
unlink_file(handle, oldpkg, newpkg, file, skip_remove,
|
||||||
handle->trans->flags & ALPM_TRANS_FLAG_NOSAVE);
|
handle->trans->flags & ALPM_TRANS_FLAG_NOSAVE);
|
||||||
|
|
||||||
if(!newpkg) {
|
if(!newpkg) {
|
||||||
|
Loading…
Reference in New Issue
Block a user