)]}'
{
  "commit": "a298f6a509c934722ea7675561bac9b349ff535a",
  "tree": "bb7365b552c66a3bdff02ee5a4faad1658ca5d64",
  "parents": [
    "61acbdd7a884ffaef2500a3f4cd96b91e56c2c86"
  ],
  "author": {
    "name": "Vikram Gara",
    "email": "vikramgara@google.com",
    "time": "Tue Sep 08 21:59:09 2026 +0000"
  },
  "committer": {
    "name": "Vikram Gara",
    "email": "vikramgara@google.com",
    "time": "Wed Sep 09 19:05:21 2026 +0000"
  },
  "message": "platform-mc / libpldmresponder: Add bypass-terminus-discovery feature and fix event handler NACKs\n\nThis commit introduces a new platform configuration toggle to optimize\nPLDM behavior for lightweight topologies (like crash dump listeners) and\nresolves a critical variable shadowing bug inside the event responder stack.\n\n1. Add `bypass-terminus-discovery` Meson toggle\n   For platforms that only need minimal event routing without massive DBus\n   footprints , this feature flag allows bypassing\n   the traditional heavy discovery phase:\n   - Skips PDR Polling: Short-circuits `getPDRs()` inside the platform\n     manager to suppress heavy I2C cross-traffic for unused sensors and effecters.\n   - Prevents Phantom Chassis Creation: Instantly aborts `createInventoryPath()`\n     in the terminus mapping, preventing bmcweb from blindly exposing the\n     bare terminus (e.g., `MCTP_0`) as a Redfish Chassis object.\n\n2. Fix Completion Code shadowing in `platformEventMessage`\n   A variable shadowing bug (`auto rc \u003d handler(...)`) within the active\n   event handler iteration loop previously resulted in failing handlers\n   falling through. This caused the BMC to illegally return `PLDM_SUCCESS`\n   (0x00) over the wire, even when handlers actively rejected a polled event.\n   The return code variable has been extracted to the outer scope so that any\n   handler failure now immediately breaks the loop and returns the proper NACK.\n\nTested:\n\thttps://paste.googleplex.com/6678851397222400\n\tWith the change we dont see any new entry under Chassis redfish\n\tendpoint and crash dump feature is working fine as well.\n\nGoogle-Bug-Id: 556580550\nChange-Id: I1fcd2cce9fa268054028f3a868abb70863fba7d4\nSigned-off-by: Vikram Gara \u003cvikramgara@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "b414704287f5e525faa78b9d894613848f7369de",
      "old_mode": 33188,
      "old_path": "libpldmresponder/platform.cpp",
      "new_id": "c2f450ff63f6425f9ac829210131ff464ac75878",
      "new_mode": 33188,
      "new_path": "libpldmresponder/platform.cpp"
    },
    {
      "type": "modify",
      "old_id": "53f24dd8c503bd61a031b76ebcfb2be320c0d7a4",
      "old_mode": 33188,
      "old_path": "meson.build",
      "new_id": "e02ccb915013fc8c8e2553d69be40ef1d3b496e7",
      "new_mode": 33188,
      "new_path": "meson.build"
    },
    {
      "type": "modify",
      "old_id": "d7c87725869699c209ab36d1c4288a309bce1a64",
      "old_mode": 33188,
      "old_path": "meson.options",
      "new_id": "74cdb00e9710022768476c82dca40cb600084a7d",
      "new_mode": 33188,
      "new_path": "meson.options"
    },
    {
      "type": "modify",
      "old_id": "548cb1ab2bacc52c79b1ee457fa7cfb4bba03bdf",
      "old_mode": 33188,
      "old_path": "platform-mc/platform_manager.cpp",
      "new_id": "a980741b0706fb4596ee2c125071aa0f2aeda740",
      "new_mode": 33188,
      "new_path": "platform-mc/platform_manager.cpp"
    },
    {
      "type": "modify",
      "old_id": "7a44a5a1fb4b688d39e5b4bbed168bbbb587cef2",
      "old_mode": 33188,
      "old_path": "platform-mc/sensor_manager.cpp",
      "new_id": "31a8581330c3cc387270b5802cb7e757866c1994",
      "new_mode": 33188,
      "new_path": "platform-mc/sensor_manager.cpp"
    },
    {
      "type": "modify",
      "old_id": "fdaacf473b6b066cc829b63ad40ba4bf5257938d",
      "old_mode": 33188,
      "old_path": "platform-mc/terminus.cpp",
      "new_id": "2a25092b5262a1c72259c9499474a448086db947",
      "new_mode": 33188,
      "new_path": "platform-mc/terminus.cpp"
    },
    {
      "type": "modify",
      "old_id": "5270da83d2e0d7151ed15df8571b4a309e713590",
      "old_mode": 33188,
      "old_path": "platform-mc/terminus_manager.cpp",
      "new_id": "a111a540da3f50b19d42a036913a875143a2e911",
      "new_mode": 33188,
      "new_path": "platform-mc/terminus_manager.cpp"
    },
    {
      "type": "modify",
      "old_id": "caa46639e674a45df450c015edca1572e8c420cb",
      "old_mode": 33188,
      "old_path": "requester/test/handler_test.cpp",
      "new_id": "a3c06e52ccc9b7ee73240e617006cece067931f7",
      "new_mode": 33188,
      "new_path": "requester/test/handler_test.cpp"
    }
  ]
}