Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 03ccd8d8 authored by David Anderson's avatar David Anderson Committed by Automerger Merge Worker
Browse files

Merge "adds LO_FLAGS_AUTOCLEAR for loop device of zram backing device" am:... 

Merge "adds LO_FLAGS_AUTOCLEAR for loop device of zram backing device" am: af09c150 am: d50f25be am: d23803f0 am: d4c7f3b2

Original change: https://android-review.googlesource.com/c/platform/system/core/+/1834393

Change-Id: Ia1a011246c990aa70ad2abf630dce63c8c9dad09
parents 0f731232 d4c7f3b2

fs_mgr/fs_mgr.cpp

+3 −0
Original line number Diff line number Diff line
@@ -2119,6 +2119,9 @@ static bool PrepareZramBackingDevice(off64_t size) { 
        PERROR << "Cannot open " << loop_device;

        return false;

    }

    if (!LoopControl::SetAutoClearStatus(loop_fd.get())) {

        PERROR << "Failed set LO_FLAGS_AUTOCLEAR for " << loop_device;

    }

    if (!LoopControl::EnableDirectIo(loop_fd.get())) {

        return false;

    }

fs_mgr/libdm/include/libdm/loop_control.h

+3 −0
Original line number Diff line number Diff line
@@ -46,6 +46,9 @@ class LoopControl final { 
    // Enable Direct I/O on a loop device. This requires kernel 4.9+.

    static bool EnableDirectIo(int fd);



    // Set LO_FLAGS_AUTOCLEAR on a loop device.

    static bool SetAutoClearStatus(int fd);



    LoopControl(const LoopControl&) = delete;

    LoopControl& operator=(const LoopControl&) = delete;

    LoopControl& operator=(LoopControl&&) = default;

fs_mgr/libdm/loop_control.cpp

+10 −0
Original line number Diff line number Diff line
@@ -133,6 +133,16 @@ bool LoopControl::EnableDirectIo(int fd) { 
    return true;

}



bool LoopControl::SetAutoClearStatus(int fd) {

    struct loop_info64 info = {};



    info.lo_flags |= LO_FLAGS_AUTOCLEAR;

    if (ioctl(fd, LOOP_SET_STATUS64, &info)) {

        return false;

    }

    return true;

}



LoopDevice::LoopDevice(android::base::borrowed_fd fd, const std::chrono::milliseconds& timeout_ms,

                       bool auto_close)

    : fd_(fd), owned_fd_(-1) {