Ipmi.SOL: add default values
In the absence of explicit defaults, the implicit defaults applied to
some of these parameters were somewhat less than ideal, such as
AccumulateIntervalMS getting set to zero, leading to netipmid going wild
consuming all available CPU cycles whenever an SOL session was active.
This should also obviate the need for most of the currently-existing
overrides of these settings in the main openbmc repo, which are nearly
all identical.
Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: I745b53bae968976729e2c088c8821bec5f4803f0
diff --git a/yaml/xyz/openbmc_project/Ipmi/SOL.interface.yaml b/yaml/xyz/openbmc_project/Ipmi/SOL.interface.yaml
index 0ef087e..8a0146c 100644
--- a/yaml/xyz/openbmc_project/Ipmi/SOL.interface.yaml
+++ b/yaml/xyz/openbmc_project/Ipmi/SOL.interface.yaml
@@ -8,43 +8,52 @@
properties:
- name: Progress
type: byte
+ default: 0
description: >
Set In Progress property, indicate when any parameters are being
updated.
- name: Enable
type: boolean
+ default: true
description: >
SOL Enable property, this controls whether the SOL payload type can be
activated.
- name: ForceEncryption
type: boolean
+ default: true
description: >
If SOL enable Force Payload Encryption.
- name: ForceAuthentication
type: boolean
+ default: true
description: >
If SOL enable Force Payload Authentication
- name: Privilege
type: byte
+ default: 4
description: >
Sets the minimum operating privilege level that is required to be able
to activate SOL by Activate Payload command.
- name: AccumulateIntervalMS
type: byte
+ default: 20
description: >
Character Accumulate Interval in 5ms increments. BMC will wait this
time before transmitting a packet.
- name: Threshold
type: byte
+ default: 1
description: >
BMC will automatically send an SOL character data packet containing
this number of characters as soon as this number of characters (or
greater) has been accepted from the baseboard serial controller.
- name: RetryCount
type: byte
+ default: 7
description: >
Packet will be dropped if no ACK/NACK received by time retries expire.
- name: RetryIntervalMS
type: byte
+ default: 100
description: >
Retry Interval in 10ms increments.