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

Commit 099e19a9 authored by jioana's avatar jioana
Browse files

Gather latency metrics for key events 

Currently, we are not gathering any latency data for key events. We want
to record latencies and report them using the new
InputEventLatencyReported atom.

In InputDispatcher, call LatencyTracker::trackListener from notifyKey to
gather latency statistics for key events. This will be done only if the
per device input latency metrics flag is enabled because We want to
gather these latency values using the new LatencyAggregator that uses
histograms.

Moved the sources resolution for motion events from InputDispatcher to
LatencyTracker.

Bug: 270049345
Test: atest inputflinger_tests
Flag: com.android.input.flags.enable_per_device_input_latency_metrics
Change-Id: Iadf7b7aecbb14db6f13e4c53ecc11d3ccccd646f
parent de3019c8

services/inputflinger/dispatcher/InputDispatcher.cpp

+9 −3
Original line number Diff line number Diff line
@@ -4511,6 +4511,14 @@ void InputDispatcher::notifyKey(const NotifyKeyArgs& args) { 
            newEntry->traceTracker = mTracer->traceInboundEvent(*newEntry);

        }



        if (input_flags::enable_per_device_input_latency_metrics()) {

            if (args.id != android::os::IInputConstants::INVALID_INPUT_EVENT_ID &&

                IdGenerator::getSource(args.id) == IdGenerator::Source::INPUT_READER &&

                !mInputFilterEnabled) {

                mLatencyTracker.trackNotifyKey(args);

            }

        }



        needWake = enqueueInboundEventLocked(std::move(newEntry));

        mLock.unlock();

    } // release lock
@@ -4643,9 +4651,7 @@ void InputDispatcher::notifyMotion(const NotifyMotionArgs& args) { 
        if (args.id != android::os::IInputConstants::INVALID_INPUT_EVENT_ID &&

            IdGenerator::getSource(args.id) == IdGenerator::Source::INPUT_READER &&

            !mInputFilterEnabled) {

            std::set<InputDeviceUsageSource> sources = getUsageSourcesForMotionArgs(args);

            mLatencyTracker.trackListener(args.id, args.eventTime, args.readTime, args.deviceId,

                                          sources, args.action, InputEventType::MOTION);

            mLatencyTracker.trackNotifyMotion(args);

        }



        needWake = enqueueInboundEventLocked(std::move(newEntry));

services/inputflinger/dispatcher/LatencyTracker.cpp

+20 −0
Original line number Diff line number Diff line
@@ -67,6 +67,26 @@ LatencyTracker::LatencyTracker(InputEventTimelineProcessor* processor) 
    LOG_ALWAYS_FATAL_IF(processor == nullptr);

}



void LatencyTracker::trackNotifyMotion(const NotifyMotionArgs& args) {

    std::set<InputDeviceUsageSource> sources = getUsageSourcesForMotionArgs(args);

    trackListener(args.id, args.eventTime, args.readTime, args.deviceId, sources, args.action,

                  InputEventType::MOTION);

}



void LatencyTracker::trackNotifyKey(const NotifyKeyArgs& args) {

    int32_t keyboardType = AINPUT_KEYBOARD_TYPE_NONE;

    for (auto& inputDevice : mInputDevices) {

        if (args.deviceId == inputDevice.getId()) {

            keyboardType = inputDevice.getKeyboardType();

            break;

        }

    }

    std::set<InputDeviceUsageSource> sources =

            std::set{getUsageSourceForKeyArgs(keyboardType, args)};

    trackListener(args.id, args.eventTime, args.readTime, args.deviceId, sources, args.action,

                  InputEventType::KEY);

}



void LatencyTracker::trackListener(int32_t inputEventId, nsecs_t eventTime, nsecs_t readTime,

                                   DeviceId deviceId,

                                   const std::set<InputDeviceUsageSource>& sources,

services/inputflinger/dispatcher/LatencyTracker.h

+7 −0
Original line number Diff line number Diff line
@@ -59,6 +59,13 @@ public: 
                            nsecs_t deliveryTime, nsecs_t consumeTime, nsecs_t finishTime);

    void trackGraphicsLatency(int32_t inputEventId, const sp<IBinder>& connectionToken,

                              std::array<nsecs_t, GraphicsTimeline::SIZE> timeline);

    /**

     * trackNotifyMotion and trackNotifyKeys are intermediates between InputDispatcher and

     * trackListener. They compute the InputDeviceUsageSource set and call trackListener with

     * the relevant parameters for latency computation.

     */

    void trackNotifyMotion(const NotifyMotionArgs& args);

    void trackNotifyKey(const NotifyKeyArgs& args);



    std::string dump(const char* prefix) const;

    void setInputDevices(const std::vector<InputDeviceInfo>& inputDevices);