ExternalVendorCode/genode
1
0
Fork 0
mirror of https://github.com/genodelabs/genode.git synced 2025-02-06 11:10:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

depot_query: avoid repetitive file accesses

Browse Source 
This patch changes the access of pkg-archives files such that the file
read only once per pkg, not for each ROM module used by the package.

Issue #4611
This commit is contained in:
Norman Feske 2022-09-14 16:58:16 +02:00 committed by Christian Helmuth
parent 851ae574d1
commit ab7c0b40f2
1 changed files with 40 additions and 47 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
repos/gems/src/app/depot_query/main.cc
View File

@ -298,6 +298,7 @@ struct Depot_query::Main
}
catch (File_content::Nonexistent_file) { }
catch (Directory::Nonexistent_directory) { }
catch (File::Truncated_during_read) { }
}
/**
@ -322,8 +323,7 @@ struct Depot_query::Main
});
}
Archive::Path _find_rom_in_pkg(Directory::Path const &, Rom_label const &, Recursion_limit);
Archive::Path _find_rom_in_pkg(File_content const &, Rom_label const &, Recursion_limit);
void _gen_rom_path_nodes(Xml_generator &, Xml_node const &,
Archive::Path const &, Xml_node const &);
void _gen_inherited_rom_path_nodes(Xml_generator &, Xml_node const &,
@ -459,21 +459,10 @@ struct Depot_query::Main
Depot_query::Archive::Path
Depot_query::Main::_find_rom_in_pkg(Directory::Path const &pkg_path,
Depot_query::Main::_find_rom_in_pkg(File_content const &archives,
Rom_label const &rom_label,
Recursion_limit recursion_limit)
{
/*
* \throw Directory::Nonexistent_directory
*/
Directory pkg_dir(_depot_dir, pkg_path);
/*
* \throw Directory::Nonexistent_file
* \throw File::Truncated_during_read
*/
File_content archives(_heap, pkg_dir, "archives", File_content::Limit{16*1024});
Archive::Path result;
archives.for_each_line<Archive::Path>([&] (Archive::Path const &archive_path) {
@ -509,12 +498,14 @@ Depot_query::Main::_find_rom_in_pkg(Directory::Path const &pkg_path,
case Archive::PKG:
_with_file_content(archive_path, "archives", [&] (File_content const &archives) {
Archive::Path const result_from_pkg =
_find_rom_in_pkg(archive_path, rom_label, recursion_limit);
_find_rom_in_pkg(archives, rom_label, recursion_limit);
if (result_from_pkg.valid())
result = result_from_pkg;
});
break;
}
});
@ -527,6 +518,7 @@ void Depot_query::Main::_gen_rom_path_nodes(Xml_generator &xml,
Archive::Path const &pkg_path,
Xml_node const &runtime)
{
_with_file_content(pkg_path, "archives", [&] (File_content const &archives) {
runtime.for_each_sub_node("content", [&] (Xml_node content) {
content.for_each_sub_node([&] (Xml_node node) {
@ -551,7 +543,7 @@ void Depot_query::Main::_gen_rom_path_nodes(Xml_generator &xml,
}
Archive::Path const rom_path =
_find_rom_in_pkg(pkg_path, label, Recursion_limit{8});
_find_rom_in_pkg(archives, label, Recursion_limit{8});
if (rom_path.valid()) {
xml.node("rom", [&] () {
@ -566,6 +558,7 @@ void Depot_query::Main::_gen_rom_path_nodes(Xml_generator &xml,
}
});
});
});
}