-
Notifications
You must be signed in to change notification settings - Fork 356
Feature/add support for boot mailbox #129
Merged
AlexanderAmelkin
merged 7 commits into
master
from
feature/add-support-for-boot-mailbox
Jun 10, 2019
Merged
Feature/add support for boot mailbox #129
AlexanderAmelkin
merged 7 commits into
master
from
feature/add-support-for-boot-mailbox
Jun 10, 2019
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The function converts a set of command line arguments representing byte values into a byte buffer and verifies each individual value to be a valid data byte. Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
The function converts a host 32-bit integer into an IPMI 24-bit value (LSB first). Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
Some commands may return command-specific completion codes. Now they are all reported as 'Unknown'. Add helper functions to support such command-specific codes. Command handlers will need to define their own valstr arrays with completion code descriptions and then use specific_val2str() instead of generic val2str() to convert the completion code into a string. Also reduce code duplication in helper.c Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
Get/set system boot option commands have some command-specific completion codes that are now reported as "Unknown (0080)", etc. Use the previously introduced specific_val2str() to convert those specific error codes to human-readable strings. Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
@vmauery, @pstrinkle, I'd like you to review this change set if you have time. |
pstrinkle
previously approved these changes
Jun 4, 2019
vmauery
previously approved these changes
Jun 5, 2019
Add `chassis bootmbox` command to set and get Boot Initiator Mailbox boot parameter (id 7) the easy way. The command allows for getting and setting the data both in hex and text modes, as well as properly decodes IANA Enterprise number for block 0. It can get/set the whole mailbox at once or operate on separate data blocks. This commit enhances the chassis_get_boot_param() function with extra arguments to re-use its code in handling of the added command. Documentation update will follow. Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an effort to add support for the Boot Initiator Mailbox system boot option.
Before this change the option could only be dumped as a whole in hex dump mode using
bootparam get 7
. With this change set it becomes possible to dump a single block usingbootparam get 7 <id>
as well as usingbootmbox get block <id>
. Also added are text dump function (bootmbox get text
), and the function to set the mailbox value in both text and binary modes (bootmbox set [text] [block <id>]
).