libui: Change 'into' to 'onto'

No client functionality yet, but it's a start.

kernel/src/arch/x86_64/disk/ATA.cpp

@@ -161,7 +161,7 @@ namespace ATA
     {
         if (!(read_bm(BusmasterRegister::Status) & BMS_IRQPending)) return;
 
-        if (m_current_drive < 2 && m_drives[m_current_drive].has_value()) m_drives[m_current_drive]->irq_handler();
+        if (m_current_drive < 2 && m_drives[m_current_drive]) m_drives[m_current_drive]->irq_handler();
 
         m_irq_called = true;
 
@@ -307,7 +307,14 @@ namespace ATA
 
             kinfoln("ata: Channel %d has a drive on slot %d!", m_channel_index, drive);
 
-            m_drives[drive] = Drive { this, drive, {} };
+            auto rc = adopt_shared_if_nonnull(new (std::nothrow) Drive(this, drive, {}));
+            if (rc.has_error())
+            {
+                kinfoln("ata: Failed to create drive object: %s", rc.error_string());
+                return false;
+            }
+
+            m_drives[drive] = rc.release_value();
 
             if (!m_drives[drive]->initialize())
             {
@@ -320,7 +327,7 @@ namespace ATA
 
         for (u8 drive = 0; drive < 2; drive++)
         {
-            if (m_drives[drive].has_value())
+            if (m_drives[drive])
             {
                 if (!m_drives[drive]->post_initialize())
                 {
@@ -328,7 +335,7 @@ namespace ATA
                     return false;
                 }
 
-                auto rc = ATADevice::create(m_drives[drive].value_ptr());
+                auto rc = ATADevice::create(m_drives[drive]);
 
                 if (rc.has_error())
                 {
@@ -716,7 +723,7 @@ namespace ATA
 
 static u32 next_minor = 0;
 
-Result<String> ATA::Drive::create_drive_name(ATA::Drive* drive)
+Result<String> ATA::Drive::create_drive_name(SharedPtr<ATA::Drive> drive)
 {
     static u32 cd_index = 0;
     static u32 sd_index = 0;
@@ -724,7 +731,7 @@ Result<String> ATA::Drive::create_drive_name(ATA::Drive* drive)
     return String::format("%s%d"_sv, drive->m_is_atapi ? "cd" : "sd", drive->m_is_atapi ? cd_index++ : sd_index++);
 }
 
-Result<SharedPtr<Device>> ATADevice::create(ATA::Drive* drive)
+Result<SharedPtr<Device>> ATADevice::create(SharedPtr<ATA::Drive> drive)
 {
     auto device = TRY(adopt_shared_if_nonnull(new (std::nothrow) ATADevice()));
     device->m_drive = drive;

kernel/src/arch/x86_64/disk/ATA.h

@@ -131,7 +131,7 @@ namespace ATA
 
     static constexpr u16 END_OF_PRDT = (1 << 15);
 
-    class Drive
+    class Drive : public Shareable
     {
       public:
         Drive(Channel* channel, u8 drive_index, Badge<Channel>);
@@ -164,7 +164,7 @@ namespace ATA
 
         Result<void> read_lba(u64 lba, void* out, usize nblocks);
 
-        static Result<String> create_drive_name(ATA::Drive* drive);
+        static Result<String> create_drive_name(SharedPtr<ATA::Drive> drive);
 
       private:
         bool identify_ata();
@@ -271,7 +271,7 @@ namespace ATA
 
         u8 m_current_drive = (u8)-1;
 
-        Option<Drive> m_drives[2];
+        SharedPtr<Drive> m_drives[2];
     };
 
     class Controller : public Shareable
@@ -301,7 +301,7 @@ class ATADevice : public Device
 {
   public:
     // Initializer for DeviceRegistry.
-    static Result<SharedPtr<Device>> create(ATA::Drive* drive);
+    static Result<SharedPtr<Device>> create(SharedPtr<ATA::Drive> drive);
 
     Result<usize> read(u8*, usize, usize) const override;
 
@@ -339,6 +339,6 @@ class ATADevice : public Device
 
   private:
     ATADevice() = default;
-    ATA::Drive* m_drive;
+    SharedPtr<ATA::Drive> m_drive;
     String m_device_path;
 };