Page MenuHomeFreeBSD

ipmi_ssif: Fix inverted for the end of multi-part reads
ClosedPublic

Authored by allanjude on Feb 17 2021, 9:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 20, 9:01 AM
Unknown Object (File)
Wed, Nov 13, 9:15 PM
Unknown Object (File)
Oct 2 2024, 9:13 PM
Unknown Object (File)
Oct 1 2024, 5:30 AM
Unknown Object (File)
Sep 26 2024, 4:01 PM
Unknown Object (File)
Sep 20 2024, 9:59 AM
Unknown Object (File)
Sep 15 2024, 10:56 PM
Unknown Object (File)
Sep 15 2024, 2:38 AM
Subscribers

Details

Summary

Sponsored by: Ampere Computing LLC

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Any docs on what this should contain ?

In D28749#643720, @manu wrote:

Any docs on what this should contain ?

I think specifically this one 0xff indicates 'last block, no more entries', and the logic was accidently inverted before.

You can check the IPMI Specification, sec 12.5 for info on SSIF Multi-part Read Transactions

You can check the IPMI Specification, sec 12.5 for info on SSIF Multi-part Read Transactions

Thanks, that was exactly what I was looking for.

This revision is now accepted and ready to land.Feb 17 2021, 11:24 PM

Do we want to put this link and section reference in the commit message?

Do we want to put this link and section reference in the commit message?

I'm not a fan of putting links as they tend to disappear over time but citing the name of the doc and chapter would be nice to do.