apps/init.cpp

@@ -224,7 +224,7 @@ static Result<void> load_services()
 {
     auto dir = TRY(os::Directory::open("/etc/init"));
 
-    auto services = TRY(dir->list_names(os::Directory::Filter::ParentAndBase));
+    auto services = TRY(dir->list(os::Directory::Filter::ParentAndBase));
     sort(services.begin(), services.end(), String::compare);
 
     for (const auto& entry : services) TRY(load_service({ dir->fd(), entry.view() }));

apps/ls.cpp

@@ -1,6 +1,4 @@
 #include <grp.h>
-#include <luna/Sort.h>
-#include <luna/StringBuilder.h>
 #include <luna/Units.h>
 #include <os/ArgumentParser.h>
 #include <os/Directory.h>
@@ -22,48 +20,6 @@ void find_user_and_group(struct stat& st, StringView& owner, StringView& group)
         group = grp->gr_name;
 }
 
-int sort_name(const os::Directory::Entry* a, const os::Directory::Entry* b)
-{
-    return String::compare(&a->name, &b->name);
-}
-
-int sort_size(const os::Directory::Entry* a, const os::Directory::Entry* b)
-{
-    return (a->size < b->size) ? -1 : ((a->size == b->size) ? 0 : 1);
-}
-
-int sort_time(const os::Directory::Entry* a, const os::Directory::Entry* b)
-{
-    return (a->mtime < b->mtime) ? -1 : ((a->mtime == b->mtime) ? 0 : 1);
-}
-
-static int (*sort_function)(const os::Directory::Entry*, const os::Directory::Entry*) = sort_name;
-
-int sort_reverse(const os::Directory::Entry* a, const os::Directory::Entry* b)
-{
-    int rc = sort_function(a, b);
-    if (rc < 0) return 1;
-    if (rc > 0) return -1;
-    return 0;
-}
-
-static Result<String> entry_join(const Vector<os::Directory::Entry>& vec, StringView delim)
-{
-    if (vec.size() == 0) return String {};
-    if (vec.size() == 1) return vec[0].name.clone();
-
-    StringBuilder sb;
-    TRY(sb.add(vec[0].name));
-
-    for (usize i = 1; i < vec.size(); i++)
-    {
-        TRY(sb.add(delim));
-        TRY(sb.add(vec[i].name));
-    }
-
-    return sb.string();
-}
-
 Result<int> luna_main(int argc, char** argv)
 {
     StringView pathname;
@@ -76,12 +32,6 @@ Result<int> luna_main(int argc, char** argv)
     bool one_per_line { false };
     bool list_directories { false };
 
-    StringView sort_type { "name" };
-
-    bool reverse_sort { false };
-
-    bool should_sort { true };
-
     os::ArgumentParser parser;
     parser.add_description("List files contained in a directory (defaults to '.', the current directory)"_sv);
     parser.add_system_program_info("ls"_sv);
@@ -96,26 +46,14 @@ Result<int> luna_main(int argc, char** argv)
                                "follow symbolic links listed as arguments"_sv);
     parser.add_switch_argument(one_per_line, '1', ""_sv, "list one file per line"_sv);
     parser.add_switch_argument(list_directories, 'd', "directory"_sv, "list directories instead of their contents"_sv);
-    parser.add_value_argument(sort_type, ' ', "sort"_sv, true, "sort by name, size or time"_sv);
-    parser.add_switch_argument(reverse_sort, 'r', "reverse"_sv, "reverse order while sorting"_sv);
     parser.parse(argc, argv);
 
-    Vector<os::Directory::Entry> files;
+    Vector<String> files;
     int dirfd = AT_FDCWD;
     SharedPtr<os::Directory> dir;
 
     if (!long_list) follow_symlink_args = true;
 
-    if (sort_type == "none"_sv) { should_sort = false; }
-    else if (sort_type == "name"_sv) { sort_function = sort_name; }
-    else if (sort_type == "size"_sv) { sort_function = sort_size; }
-    else if (sort_type == "time"_sv) { sort_function = sort_time; }
-    else
-    {
-        os::eprintln("%s: unknown sort type: %s", argv[0], sort_type.chars());
-        return 1;
-    }
-
     if (os::FileSystem::is_directory(pathname, follow_symlink_args) && !list_directories)
     {
         dir = TRY(os::Directory::open(pathname));
@@ -130,31 +68,15 @@ Result<int> luna_main(int argc, char** argv)
     }
     else if (os::FileSystem::exists(pathname, follow_symlink_args))
     {
-        struct stat st;
-        TRY(os::FileSystem::stat(pathname, st, false));
-
-        os::Directory::Entry entry = {
-            .name = TRY(String::from_string_view(pathname)),
-            .mode = st.st_mode,
-            .size = (usize)st.st_size,
-            .mtime = st.st_mtime,
-        };
-
-        TRY(files.try_append(move(entry)));
+        auto str = TRY(String::from_string_view(pathname));
+        TRY(files.try_append(move(str)));
     }
     else
         return err(ENOENT);
 
-    if (should_sort)
-    {
-        if (reverse_sort) sort(files.begin(), files.end(), sort_reverse);
-        else
-            sort(files.begin(), files.end(), sort_function);
-    }
-
     if (!long_list)
     {
-        auto list = TRY(entry_join(files, one_per_line ? "\n"_sv : " "_sv));
+        auto list = TRY(String::join(files, one_per_line ? "\n"_sv : " "_sv));
         if (!list.is_empty()) os::println("%s", list.chars());
     }
     else
@@ -162,9 +84,9 @@ Result<int> luna_main(int argc, char** argv)
         for (const auto& file : files)
         {
             struct stat st;
-            TRY(os::FileSystem::stat({ dirfd, file.name.view() }, st, false));
+            TRY(os::FileSystem::stat({ dirfd, file.view() }, st, false));
 
-            auto link = TRY(os::FileSystem::readlink({ dirfd, file.name.view() }));
+            auto link = TRY(os::FileSystem::readlink({ dirfd, file.view() }));
 
             StringView owner;
             StringView group;
@@ -177,13 +99,13 @@ Result<int> luna_main(int argc, char** argv)
             if (!human_readable && !si)
             {
                 os::println("%s %u %4s %4s %10lu %s%s%s", formatted_mode, st.st_nlink, owner.chars(), group.chars(),
-                            st.st_size, file.name.chars(), link.is_empty() ? "" : " -> ", link.chars());
+                            st.st_size, file.chars(), link.is_empty() ? "" : " -> ", link.chars());
             }
             else
             {
                 auto size = TRY(to_dynamic_unit(st.st_size, 10, false, si ? Unit::SI : Unit::Binary, false));
                 os::println("%s %u %4s %4s %6s %s%s%s", formatted_mode, st.st_nlink, owner.chars(), group.chars(),
-                            size.chars(), file.name.chars(), link.is_empty() ? "" : " -> ", link.chars());
+                            size.chars(), file.chars(), link.is_empty() ? "" : " -> ", link.chars());
             }
         }
     }

kernel/src/arch/x86_64/CPU.cpp

@@ -73,22 +73,7 @@ void decode_page_fault_error_code(u64 code)
     {
         // FIXME: Kill this process with SIGSEGV once we have signals and all that.
         kerrorln("Current task %zu was terminated because of a page fault", Scheduler::current()->id);
-        if (Scheduler::current()->is_kernel) Scheduler::current()->state = ThreadState::Dying;
-        else
-        {
-            auto* current = Scheduler::current();
-            auto* parent = current->parent;
-            if (parent && parent->state == ThreadState::Waiting)
-            {
-                auto child = *parent->child_being_waited_for;
-                if (child == -1 || child == (pid_t)current->id)
-                {
-                    parent->child_being_waited_for = (pid_t)current->id;
-                    parent->wake_up();
-                }
-            }
-            current->state = ThreadState::Exited;
-        }
+        Scheduler::current()->state = ThreadState::Exited;
         Scheduler::current()->status = 127;
         kernel_yield();
         unreachable();

libluna/src/Sort.cpp

@@ -1,5 +1,4 @@
 #include <luna/Alignment.h>
-#include <luna/DebugLog.h>
 #include <luna/Sort.h>
 
 static void swap_sized(void* ptr1, void* ptr2, usize size)
@@ -43,7 +42,7 @@ static void quicksort_impl(void* base, usize start, usize end, usize size, compa
     {
         usize pivot = partition(base, start, end, size, compar);
         if ((end - start) < 2) return;
-        if (pivot > 0) quicksort_impl(base, start, pivot - 1, size, compar);
+        quicksort_impl(base, start, pivot - 1, size, compar);
         quicksort_impl(base, pivot + 1, end, size, compar);
     }
 }

libos/include/os/Directory.h

@@ -21,17 +21,7 @@ namespace os
 
         Result<String> next(Filter filter);
 
-        Result<Vector<String>> list_names(Filter filter);
-
-        struct Entry
-        {
-            String name;
-            mode_t mode;
-            usize size;
-            time_t mtime;
-        };
-
-        Result<Vector<Entry>> list(Filter filter);
+        Result<Vector<String>> list(Filter filter);
 
         void rewind();
 

libos/src/Directory.cpp

@@ -2,7 +2,6 @@
 #include <fcntl.h>
 #include <luna/CString.h>
 #include <os/Directory.h>
-#include <os/FileSystem.h>
 #include <sys/syscall.h>
 #include <unistd.h>
 
@@ -55,7 +54,7 @@ namespace os
         return String::from_cstring(ent->d_name);
     }
 
-    Result<Vector<String>> Directory::list_names(Filter filter)
+    Result<Vector<String>> Directory::list(Filter filter)
     {
         Vector<String> result;
 
@@ -67,31 +66,6 @@ namespace os
         return result;
     }
 
-    Result<Vector<Directory::Entry>> Directory::list(Filter filter)
-    {
-        Vector<Directory::Entry> result;
-
-        rewind();
-
-        String name {};
-        while ((name = TRY(next(filter))), !name.is_empty())
-        {
-            struct stat st;
-            TRY(FileSystem::stat({ dirfd(m_dirp), name.chars() }, st, false));
-
-            Directory::Entry entry {
-                .name = move(name),
-                .mode = st.st_mode,
-                .size = (usize)st.st_size,
-                .mtime = st.st_mtime,
-            };
-
-            TRY(result.try_append(move(entry)));
-        }
-
-        return result;
-    }
-
     void Directory::rewind()
     {
         rewinddir(m_dirp);

libos/src/FileSystem.cpp

@@ -62,7 +62,7 @@ namespace os::FileSystem
 
         auto dir = TRY(os::Directory::open(path));
 
-        Vector<String> entries = TRY(dir->list_names(os::Directory::Filter::ParentAndBase));
+        Vector<String> entries = TRY(dir->list(os::Directory::Filter::ParentAndBase));
 
         for (const auto& entry : entries) { TRY(remove_tree({ dir->fd(), entry.view() })); }