Skip to content

pldmd: Changes for 2X1P#79

Open
spargaon wants to merge 1 commit into
integ_sp8from
FWDEV-198903-review
Open

pldmd: Changes for 2X1P#79
spargaon wants to merge 1 commit into
integ_sp8from
FWDEV-198903-review

Conversation

@spargaon

Copy link
Copy Markdown
Collaborator

Changes for 2X1P and Set Event Receiver code.

Rely on interface and endpoint properties to identify local BMC eid for an interface and use the eid for Set Event Receiver command instead of relying bmceid files under /var/run.

Rely on UUID and Network Interface name to identify processor id such as 0 and 1 since in 2x1P mode, both the SoC emit the same UUID.

Changes for 2X1P and Set Event Receiver code.

Rely on interface and endpoint properties to identify
local BMC eid for an interface and use the eid for
Set Event Receiver command instead of relying bmceid
files under /var/run.

Rely on UUID and Network Interface name to identify
processor id such as 0 and 1 since in 2x1P mode, both
the SoC emit the same UUID.

Signed-off-by: Shirish Pargaonkar <Shirish.Pargaonkar@amd.com>
@spargaon

Copy link
Copy Markdown
Collaborator Author

@spargaon

Copy link
Copy Markdown
Collaborator Author

@spargaon

Copy link
Copy Markdown
Collaborator Author

<6> Received pldmMessagePollEvent for terminus 1 and eventid 0x0
<6> Received pldmMessagePollEvent for terminus 1 and eventid 0x4c01
<6> truncate memfd to size 1776
<6> Emitting PLDM Message Poll Event signal: FormatVersion=1, TID=1, TerminusName=Processor0, EventClass=0x5, EventId=0x4c01, FD=12

for (const auto& mctpInfo : removedInfos)
{
auto eidToRemove = std::get<eid>(mctpInfo);
auto eidToRemove = std::get<0>(mctpInfo);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please check if you can access the members with the name instead of using index in a tuple?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants