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

Commit c5cd6b7c authored by Eric Jeong's avatar Eric Jeong
Browse files

Add vehicle properties to integrate car watchdog 

- WATCHDOG_ALIVE to tell car watchdog is alive
- WATCHDOG_TERMINATED_PROCESS to pass a process information terminated
- VHAL_HEARTBEAT to signal that VHAL is alive
by car watchdog

Bug: 159912961
Test: build okay
Change-Id: I0498d525aa6465199f0b0011600234feaf7c4aa4
parent d4b24972

automotive/vehicle/2.0/default/impl/vhal_v2_0/DefaultConfig.h

+24 −0
Original line number Diff line number Diff line
@@ -1074,6 +1074,30 @@ const ConfigDeclaration kVehicleProperties[]{ 
                                .changeMode = VehiclePropertyChangeMode::ON_CHANGE,

                        },

        },

        {

                .config =

                        {

                                .prop = toInt(VehicleProperty::WATCHDOG_ALIVE),

                                .access = VehiclePropertyAccess::WRITE,

                                .changeMode = VehiclePropertyChangeMode::ON_CHANGE,

                        },

        },

        {

                .config =

                        {

                                .prop = toInt(VehicleProperty::WATCHDOG_TERMINATED_PROCESS),

                                .access = VehiclePropertyAccess::WRITE,

                                .changeMode = VehiclePropertyChangeMode::ON_CHANGE,

                        },

        },

        {

                .config =

                        {

                                .prop = toInt(VehicleProperty::VHAL_HEARTBEAT),

                                .access = VehiclePropertyAccess::READ,

                                .changeMode = VehiclePropertyChangeMode::ON_CHANGE,

                        },

        },

};



}  // impl

automotive/vehicle/2.0/types.hal

+66 −0
Original line number Diff line number Diff line
@@ -2892,6 +2892,52 @@ enum VehicleProperty : int32_t { 
        | VehiclePropertyGroup:SYSTEM

        | VehiclePropertyType:MIXED

        | VehicleArea:GLOBAL),



    /**

     * Defines an event that car watchdog updates to tell it's alive.

     *

     * Car watchdog sets this property to system uptime in milliseconds at every 3 second.

     * During the boot, the update may take longer time.

     *

     * @change_mode VehiclePropertyChangeMode:ON_CHANGE

     * @access VehiclePropertyAccess:WRITE

     */

    WATCHDOG_ALIVE = (

        0xF31

        | VehiclePropertyGroup:SYSTEM

        | VehiclePropertyType:INT64

        | VehicleArea:GLOBAL),



    /**

     * Defines a process terminated by car watchdog and the reason of termination.

     *

     * int32Values[0]: 1         // ProcessTerminationReason showing why a process is terminated.

     * string: "/system/bin/log" // Process execution command.

     *

     * @change_mode VehiclePropertyChangeMode:ON_CHANGE

     * @access VehiclePropertyAccess:WRITE

     */

    WATCHDOG_TERMINATED_PROCESS = (

        0x0F32

        | VehiclePropertyGroup:SYSTEM

        | VehiclePropertyType:MIXED

        | VehicleArea:GLOBAL),



    /**

     * Defines an event that VHAL signals to the car watchdog as a heartbeat.

     *

     * VHAL is supposed to write system uptime to this property at every 3 second.

     * Car watchdog subscribes to this property and checks if the property is updated at every 3

     * second. If it isn’t, car watchdog considers VHAL unhealthy and terminates it.

     *

     * @change_mode VehiclePropertyChangeMode:ON_CHANGE

     * @access VehiclePropertyAccess:READ

     */

     VHAL_HEARTBEAT = (

         0x0F33

         | VehiclePropertyGroup:SYSTEM

         | VehiclePropertyType:INT64

         | VehicleArea:GLOBAL),

};



/**
@@ -4790,3 +4836,23 @@ enum RotaryInputType : int32_t { 
    ROTARY_INPUT_TYPE_AUDIO_VOLUME = 1,

};



/**

 * The reason why a process is terminated by car watchdog.

 * This is used with WATCHDOG_TERMINATED_PROCESS property.

 */

enum ProcessTerminationReason : int32_t {

   /**

    * A process doesn't respond to car watchdog within the timeout.

    */

   NOT_RESPONDING = 1,



   /**

    * A process uses more IO operations than what is allowed.

    */

   IO_OVERUSE = 2,



   /**

    * A process uses more memory space than what is allowed.

    */

   MEMORY_OVERUSE = 3,

};