)]}' { "log": [ { "commit": "14658a9854eeda1def8ea95863dabf4c6f3d833a", "tree": "647c1ad3850937f8271d80e75af2ab7783a1cf4f", "parents": [ "b8cce8fa88df910ea62099c828514c2042a1bace" ], "author": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Fri Jul 17 17:41:53 2026 +0000" }, "committer": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Fri Oct 23 16:33:17 2026 +0000" }, "message": "mappertool: add getancestors subcommand\n\nAdd a \u0027getancestors\u0027 subcommand that wraps the mapper\u0027s GetAncestors\nD-Bus method. Given an object path, it prints each ancestor object\nfound in the mapper, along with the services that own it and the\ninterfaces each service implements there.\n\nAn optional -i/--interface flag (repeatable) restricts the output\nto ancestors that implement a specific interface.\n\nUsage: mappertool getancestors \u003cpath\u003e\n mappertool getancestors \u003cpath\u003e -i \u003cinterface\u003e\nChange-Id: I1c5d70b4fdc4a922835aba6f0a0eaa90936c7fdb\nSigned-off-by: Yash Patel \u003cyash.pateltx@gmail.com\u003e\n" }, { "commit": "b8cce8fa88df910ea62099c828514c2042a1bace", "tree": "533aa40582d06551276b05a34277782765639638", "parents": [ "279821651b6c2be3f97751c145c917feac217dd0" ], "author": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Fri Jul 17 17:23:42 2026 +0000" }, "committer": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Fri Oct 23 16:32:58 2026 +0000" }, "message": "mappertool: add getsubtree subcommand\n\nAdd a \u0027getsubtree\u0027 subcommand that wraps the mapper\u0027s GetSubTree\nD-Bus method. Given a subtree root path, it prints each object path\nfound under that subtree, along with the services that own it and\nthe interfaces each service implements there.\n\nAn optional -d/--depth flag limits how many levels deep to search\n(0 \u003d unlimited, which is the default). An optional -i/--interface\nflag (repeatable) restricts the output to objects that implement a\nspecific interface.\n\nUsage: mappertool getsubtree \u003csubtree\u003e\n mappertool getsubtree \u003csubtree\u003e -d \u003cdepth\u003e\n mappertool getsubtree \u003csubtree\u003e -i \u003cinterface\u003e\nChange-Id: Ia928ad581884f4b8b01c656531d6a1064756b322\nSigned-off-by: Yash Patel \u003cyash.pateltx@gmail.com\u003e\n" }, { "commit": "279821651b6c2be3f97751c145c917feac217dd0", "tree": "833c142da5cc979d2599e1c473143bbf20897c71", "parents": [ "195f0bd0251aeb2496a7ea932d252aa14170be7f" ], "author": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Fri Jul 17 16:52:42 2026 +0000" }, "committer": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Fri Oct 23 16:28:19 2026 +0000" }, "message": "mappertool: add getsubtreepaths subcommand\n\nAdd a \u0027getsubtreepaths\u0027 subcommand that wraps the mapper\u0027s\nGetSubTreePaths D-Bus method. Given a subtree root path, it prints\nall object paths registered under that subtree.\n\nAn optional -d/--depth flag limits how many levels deep to search\n(0 \u003d unlimited, which is the default). An optional -i/--interface\nflag (repeatable) restricts the output to objects that implement a\nspecific interface, mirroring the interface list argument of\nGetSubTreePaths itself.\n\nUsage: mappertool getsubtreepaths \u003csubtree\u003e\n mappertool getsubtreepaths \u003csubtree\u003e -d \u003cdepth\u003e\n mappertool getsubtreepaths \u003csubtree\u003e -i \u003cinterface\u003e\n mappertool getsubtreepaths \u003csubtree\u003e -d \u003cdepth\u003e\n -i \u003ciface1\u003e -i \u003ciface2\u003e\nChange-Id: I7498370a0b4a266ab0039bea87f397c264028dc2\nSigned-off-by: Yash Patel \u003cyash.pateltx@gmail.com\u003e\n" }, { "commit": "195f0bd0251aeb2496a7ea932d252aa14170be7f", "tree": "c3f9b5f80733a77bd1f39bd30dd6dba1febc58df", "parents": [ "cd0264d92aacfce7c64018721c1326650069a4ba" ], "author": { "name": "Georg Hackel", "email": "georg.hackel@9elements.com", "time": "Tue Sep 08 12:16:26 2026 +0200" }, "committer": { "name": "Tan Siewert", "email": "tan.siewert@9elements.com", "time": "Wed Oct 14 22:08:51 2026 +0200" }, "message": "build: make mappertool optional\n\nOn space-constrainted platforms (i.e. any 32MB platform), mappertool\neats valuable space for a convenience feature.\n\nAdd a meson option to disable mappertool. By default, the option is\nenabled to not break any assumptions.\n\nChange-Id: I11bc6ba083b352ddfd479c21c209620a2274e3a9\nSigned-off-by: Georg Hackel \u003cgeorg.hackel@9elements.com\u003e\nSigned-off-by: Tan Siewert \u003ctan.siewert@9elements.com\u003e\n" }, { "commit": "cd0264d92aacfce7c64018721c1326650069a4ba", "tree": "04336f134aa7dca601f30e71ffa7d8959e266fb1", "parents": [ "c0ad7b4ff925e01d4e2dfaa57271372e028497d6" ], "author": { "name": "Janet Adkins", "email": "janeta@us.ibm.com", "time": "Wed Sep 23 11:12:17 2026 -0500" }, "committer": { "name": "Janet Adkins", "email": "janeta@us.ibm.com", "time": "Wed Oct 07 08:26:41 2026 -0500" }, "message": "mapper: Add optional interfaces to get-service\n\nThe mapper get-service \u003cpath\u003e command returns the first listed service\nimplementing the specified \u003cpath\u003e.\n\nThis can cause a problem if the \u003cpath\u003e is implemented by multiple\nservices. The caller may wish to act on a property of a particular\ninterface. But the interface may actually belong to one of the other\nservices for that \u003cpath\u003e.\n\nE.g.\nAdding associations to the /xyz/openbmc_project/state/chassis0 changes\nthe first service returned by the GetObject DBus call to be\nxyz.openbmc_project.ObjectMapper instead of\nxyz.openbmc_project.State.Chassis\n\nThis results in failures to the users such as\nphosphor-set-chassis-transition-to-off@.service [1],\nphosphor-set-chassis-transition-to-on@.service [2] and obmcutil [3].\n\n```\nbusctl --json\u003dpretty call xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/object_mapper xyz.openbmc_project.ObjectMapper GetObject sas \"/xyz/openbmc_project/state/chassis0\" 0\n{\n \"type\" : \"a{sas}\",\n \"data\" : [\n {\n \"xyz.openbmc_project.ObjectMapper\" : [],\n \"xyz.openbmc_project.State.Chassis\" : [\n \"org.freedesktop.DBus.Introspectable\",\n \"org.freedesktop.DBus.Peer\",\n \"org.freedesktop.DBus.Properties\",\n \"xyz.openbmc_project.State.Chassis\",\n \"xyz.openbmc_project.State.PowerOnHours\"\n ],\n \"xyz.openbmc_project.State.Chassis0\" : [\n \"org.freedesktop.DBus.Introspectable\",\n \"org.freedesktop.DBus.Peer\",\n \"org.freedesktop.DBus.Properties\",\n \"xyz.openbmc_project.State.Chassis\",\n \"xyz.openbmc_project.State.PowerOnHours\"\n ]\n }\n ]\n}\n```\n\nThis commit extends the mapper get-service command to accept an optional\ninterface. If an interface is specified the first service implementing\nthe specified interface for the \u003cpath\u003e will be returned.\n\n[1] https://github.com/openbmc/phosphor-state-manager/blob/86a4eaebbe9617d36590749f28d7e79abff5a9f1/service_files/phosphor-set-chassis-transition-to-off%40.service#L14\n[2] https://github.com/openbmc/phosphor-state-manager/blob/86a4eaebbe9617d36590749f28d7e79abff5a9f1/service_files/phosphor-set-chassis-transition-to-on%40.service#L15\n[3] https://github.com/openbmc/phosphor-state-manager/blob/86a4eaebbe9617d36590749f28d7e79abff5a9f1/obmcutil#L494\n\nTested:\n - Added association to /xyz/openbmc_project/state/chassis0\n - Modified phosphor-set-chassis-transition-to-off@.service,\n phosphor-set-chassis-transition-to-on@.service, and obmcutil to\n specify the interface.\n - Built p10bmc and confirmed expected service was returned when the\n additional interface was specified. Confirmed able to power on and\n power off with changes.\n\nChange-Id: I77ee4ea41ddebb467083874adeac1df6085eb967\nSigned-off-by: Janet Adkins \u003cjaneta@us.ibm.com\u003e\n" }, { "commit": "c0ad7b4ff925e01d4e2dfaa57271372e028497d6", "tree": "de68b24984be831cf0ad24452b73a3fe5a9b3e95", "parents": [ "05c83917dc980bc7a0e49f905fe0ae594a9317e6" ], "author": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Thu Jul 16 21:01:58 2026 +0000" }, "committer": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Wed Jul 22 16:47:24 2026 +0000" }, "message": "mappertool: add getobject subcommand\n\nAdd a \u0027getobject\u0027 subcommand that wraps the mapper\u0027s GetObject D-Bus\nmethod. Given an object path, it prints each service that owns that\npath along with the interfaces that service implements there.\n\nAn optional -i/--interface flag (repeatable) restricts the output to\nservices that implement a specific interface on that path, mirroring\nthe optional interface list argument of GetObject itself.\n\nServices with no interfaces listed for the path are filtered out of\nthe output as they provide no actionable information.\n\nUsage: mappertool getobject \u003cpath\u003e\n mappertool getobject \u003cpath\u003e -i \u003cinterface\u003e\n mappertool getobject \u003cpath\u003e -i \u003ciface1\u003e -i \u003ciface2\u003e\nChange-Id: I535dbd981f7821194cbc282c6f82f64fa336008f\nSigned-off-by: Yash Patel \u003cyash.pateltx@gmail.com\u003e\n" }, { "commit": "05c83917dc980bc7a0e49f905fe0ae594a9317e6", "tree": "e2aafa0d1e968a28b5dbbeb9388d7da3fd4f68e8", "parents": [ "51afbb3d33ce78783bb9a7498dfb6e4ad8e19998" ], "author": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Wed Jul 01 19:54:08 2026 +0000" }, "committer": { "name": "Yash Patel", "email": "yash.pateltx@gmail.com", "time": "Tue Jul 14 15:50:26 2026 +0000" }, "message": "mappertool: add association inspection CLI tool\n\nAdds a new mappertool binary to phosphor-objmgr that makes it\neasier to inspect D-Bus associations on a running BMC.\n\nCalls GetManagedObjects on xyz.openbmc_project.ObjectMapper\nand pairs forward/reverse association objects for display.\n\nUsage:\n mappertool assocs dump all associations\n mappertool assocs -n \u003cstr\u003e filter by path substring\n mappertool assocs -t \u003ctype\u003e filter by association type\n mappertool assocs -p show raw association paths\n\nExample output (mappertool assocs -t probing):\n\n```\n/xyz/openbmc_project/inventory/system/board/Ingraham_Board probing \u003c-\u003e probed_by /xyz/openbmc_project/inventory/system/chassis/motherboard/ebmc_card_bmc\n/xyz/openbmc_project/inventory/system/board/Nisqually_Backplane probing \u003c-\u003e probed_by /xyz/openbmc_project/inventory/system/chassis/motherboard\n/xyz/openbmc_project/inventory/system/chassis/Rainier_2U_Chassis probing \u003c-\u003e probed_by /xyz/openbmc_project/inventory/system/chassis/motherboard/tod_battery\n```\n\nTested: ran on p10bmc Rainier 2S2U in Simics, output verified\nfor assocs, -n, -t, and -p flags.\n\nChange-Id: I9940f58390f27194f1f915bd4f88a16478a01794\nSigned-off-by: Yash Patel \u003cyash.pateltx@gmail.com\u003e\n" }, { "commit": "51afbb3d33ce78783bb9a7498dfb6e4ad8e19998", "tree": "ee8bc315817b33b5ab1e7d1ab1736583e5ecf701", "parents": [ "3fb58d2325d52f57a4185c66e8a8a9067d2cc127" ], "author": { "name": "Alexander Hansen", "email": "alexander.hansen@9elements.com", "time": "Tue Jun 16 17:40:25 2026 +0200" }, "committer": { "name": "Alexander Hansen", "email": "alexander.hansen@9elements.com", "time": "Tue Jun 23 14:41:15 2026 +0200" }, "message": "fix object_path encapsulation\n\nsdbusplus has updated their documentation to discourage direct access to\nthe mutable string member.\n\nUse the new `object_path::string()` instead or existing sdbusplus APIs,\npreferring to pass the object_path directly where possible.\n\nTested: on anacapa qemu\n\nchecked mapper tree is full, also checked some mapper calls and\nintrospections.\n\n```\nroot@anacapa:~# busctl get-property xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/FruDevice/Anacapa_BMC_Storage_Module/probed_by xyz.openbmc_project.Association endpoints\nas 1 \"/xyz/openbmc_project/inventory/system/board/BMC_Storage_Module\"\n```\n\n```\nroot@anacapa:~# busctl call xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/object_mapper xyz.openbmc_project.ObjectMapper GetSubTreePaths sias /xyz/openbmc_project/inventory 0 2 xyz.openbmc_project.Inventory.Item.Board xyz.openbmc_project.Inventory.Item.Chassis\nas 7 \"/xyz/openbmc_project/inventory/system/board/Anacapa_SCM\" \"/xyz/openbmc_project/inventory/system/board/BMC_Storage_Module\" \"/xyz/openbmc_project/inventory/system/board/BRB_L\" \"/xyz/openbmc_project/inventory/system/board/BRB_R\" \"/xyz/openbmc_project/inventory/system/board/MB\" \"/xyz/openbmc_project/inventory/system/board/PDB_L\" \"/xyz/openbmc_project/inventory/system/board/PDB_R\"\n```\n\nChange-Id: Iba33b7cfc391794258d027aa6680f211da566255\nSigned-off-by: Alexander Hansen \u003calexander.hansen@9elements.com\u003e\n" }, { "commit": "3fb58d2325d52f57a4185c66e8a8a9067d2cc127", "tree": "9a8deaf3788716f8c562fb8c9bc39e1e66ed3860", "parents": [ "62ad8a8767872a85f60351cde019c94c450a19b7" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Jun 19 07:48:29 2026 -0400" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Jun 19 07:48:29 2026 -0400" }, "message": "sdbusplus: use shorter type aliases\n\nThe sdbusplus headers provide shortened aliases for many types.\nSwitch to using them to provide better code clarity and shorter\nlines. Possible replacements are for:\n * exception_t\n * manager_t\n * match\n * match_rules\n * message_t\n * object_t\n * slot_t\n\nChange-Id: I5a141a18f864f5742ddb2b371f76b4510ee08b17\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "62ad8a8767872a85f60351cde019c94c450a19b7", "tree": "94906fb0c6bd792386906b329039fc18242a8652", "parents": [ "dce7a3286d37966ed5f60e2a206b035293e88283" ], "author": { "name": "Alexander Hansen", "email": "alexander.hansen@9elements.com", "time": "Wed Apr 08 14:44:16 2026 +0200" }, "committer": { "name": "Alexander", "email": "alexander.hansen@9elements.com", "time": "Wed May 20 18:06:10 2026 +0000" }, "message": "refactor use of conditional compilation\n\nReplace preprocessor conditional compilation with if-constexpr.\n\nBenefit of this patch:\n\nNow everything goes through the parser (and possibly later stages),\nenabling greater code coverage in static analysis.\n\nTradeoff:\n\nThe possibility of minor performance loss due to extra class members.\n\nTested: mapper build and tests pass in both debug/release configuration\n\nTested: on Tyan S5549. Applications (e.g. EM) work normally and no\nnew errors in logs.\n\nChange-Id: I77679ea658fa7e4bd51b013150c4481fb004ed05\nSigned-off-by: Alexander Hansen \u003calexander.hansen@9elements.com\u003e\n" }, { "commit": "dce7a3286d37966ed5f60e2a206b035293e88283", "tree": "87732422c020dc57a882ef336f992afc5f04abc9", "parents": [ "6c0a8f0edaae3a242915423a3bd891057993c9e7" ], "author": { "name": "Alexander Hansen", "email": "alexander.hansen@9elements.com", "time": "Wed May 13 15:54:27 2026 +0200" }, "committer": { "name": "Alexander", "email": "alexander.hansen@9elements.com", "time": "Wed May 20 18:06:10 2026 +0000" }, "message": "fix includes for gcc 16\n\n`setw` requires header `\u003ciomanip\u003e`\n\nTested: code compiles with gcc (GCC) 16.1.1 20260430\n\nChange-Id: Ie404b55e4eadbfb25e2141fff2f375b831e291c4\nSigned-off-by: Alexander Hansen \u003calexander.hansen@9elements.com\u003e\n" }, { "commit": "6c0a8f0edaae3a242915423a3bd891057993c9e7", "tree": "ca91bbd2b443f03b87af9e6d83b9c7890765ed92", "parents": [ "8f47800917278f2976e06cc05863b05a354d1335" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Apr 28 09:57:48 2026 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Apr 28 09:59:01 2026 -0400" }, "message": "build: subproject: update boost\n\nUpdate boost to match the current version shipped with Yocto.\n\nChange-Id: I46b2c56e1bcc9f7d2f555ed8357ad806abf5c561\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "8f47800917278f2976e06cc05863b05a354d1335", "tree": "88817ef1a7593cdf5667d79381dd3d74fc710313", "parents": [ "9bca2860c8791e08260690a628267a3798c59dc4" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Apr 28 09:48:48 2026 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Apr 28 09:58:47 2026 -0400" }, "message": "build: subproject: update cli11\n\nRun meson wrap update cli11. In addition to the update, this lets us\nuse a more succinct dependency expression in meson.build.\n\nChange-Id: I7e6d93cd68975f041102057697af47d285910997\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "9bca2860c8791e08260690a628267a3798c59dc4", "tree": "77298ede27202d602880e70429c8c70ac3ca7d1b", "parents": [ "567350a0472ef36f009a2616386b64cb73919dfb" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Apr 28 09:43:47 2026 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Apr 28 09:58:34 2026 -0400" }, "message": "build: subprojects: fix cli11\n\nThe meson wrap command is case sensitive and the wrapdb subproject is\nlowercase cli11. Adjust the subproject accordingly.\n\nChange-Id: Id066b8762006ff0c5153172a28cb6c50e548fcdb\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "567350a0472ef36f009a2616386b64cb73919dfb", "tree": "c9c6d67645f8d04a6d9556fd59f8f281977bdd23", "parents": [ "ec91ce8f47074a93b46976bdaa4eabb6556fe76f" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Thu Apr 09 14:50:15 2026 -0400" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Thu Apr 09 14:50:15 2026 -0400" }, "message": "sdbusplus: use shorter type aliases\n\nThe sdbusplus headers provide shortened aliases for many types.\nSwitch to using them to provide better code clarity and shorter\nlines. Possible replacements are for:\n * bus_t\n * exception_t\n * manager_t\n * match_t\n * message_t\n * object_t\n * slot_t\n * object_path\n\nChange-Id: I316e4ab614d2380cc90f22ad4654d5c6a9d8492a\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "ec91ce8f47074a93b46976bdaa4eabb6556fe76f", "tree": "2b21c85874ae4eaa3a115de197dfec1f21630ba5", "parents": [ "b4767a54c067a1c082a61bc98a456f784124b091" ], "author": { "name": "Munawar Hussain", "email": "munawarhussain@google.com", "time": "Mon Mar 09 10:10:55 2026 +0000" }, "committer": { "name": "Munawar Hussain", "email": "munawarhussain@google.com", "time": "Wed Mar 11 08:12:53 2026 +0000" }, "message": "remove pending associations by owner\n\nChanges:\nMatch with the service while removing the pending association for an\nobject. Only the pending association introduced by the service which\nremoves the object/interface will be erased.\n\nIf Service A has an association on Object X with Object Y (which comes\nfrom Service C), and Service B also has an association definition for\nObject X, there is a scenario where the X-Y association will not be\ncreated, even though both objects are present in the D-Bus tree.\n\nSteps to Reproduce:\n1. Service A starts and creates Object X. It creates a pending\nassociation Y-X.\n2. Service B starts, registers properties for Object X, and creates\nother associations.\n3. Service B drops Object X and its associated definitions. As a\nresult, all pending associations for Object X are wiped out (including\nX-Y which is from A).\n4. Service B creates Object X again along with other associations\nrelated to Service B (but not the ones from Service A).\n5. Service C finally starts and creates Object Y, but no pending\nassociation is processed to complete the X-Y link.\n\nThis change resolves openbmc/phosphor-objmgr#24\n\nTested:\nUnit tested. Also tested on a machine with the above steps and all\nthe associations were still created.\n\nChange-Id: Ic9a59038040fd0b0af78aea0e35f44dffdfddffe\nSigned-off-by: Munawar Hussain \u003cmunawarhussain@google.com\u003e\n" }, { "commit": "b4767a54c067a1c082a61bc98a456f784124b091", "tree": "9268a32beb34903e3de00f9cf9ba7757d32860f1", "parents": [ "4a46c7116ce701191e7ed4f7903c4467b448e4c0" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Jan 23 02:38:35 2026 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Jan 23 02:38:35 2026 -0500" }, "message": "meson: reformat with meson formatter\n\nApply the `meson format` results.\n\nChange-Id: I4ad62142e19f763a981e292b0ab43da8f9082007\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "4a46c7116ce701191e7ed4f7903c4467b448e4c0", "tree": "cbe617cce5cbad0f986d95f1948598d2c66811db", "parents": [ "3622f519135a79cc10737196e61a3c8d6ea516a3" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Mon Dec 01 16:10:23 2025 -0500" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Mon Jan 05 13:25:57 2026 +0000" }, "message": "build: update to boost 1.88.0\n\nChange-Id: I8a75d98f11f00c38c8be716097b94af77111a6a5\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\nTested: Successful compilation and unit test run using subprojects.\n" }, { "commit": "3622f519135a79cc10737196e61a3c8d6ea516a3", "tree": "ef8c744a33b6af05fce34be32777692bdd685008", "parents": [ "d775f462046f6fefbb013cd2a51142a6960771de" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Mon Dec 01 17:14:18 2025 -0500" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Thu Dec 11 15:01:06 2025 +0000" }, "message": "build: update gtest to v1.17\n\nGenerated with \u0027meson wrap update\u0027\n\nChange-Id: I4a2150953e5929ea820a3db818fb02d38fdb87fb\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\nTested: Successful compilation and unit test run using subprojects.\n" }, { "commit": "d775f462046f6fefbb013cd2a51142a6960771de", "tree": "0a50b55b423e4e30560fed3a3fbd09e63aed1135", "parents": [ "85e956a9432577ae2a205f608cb18bc83311ce5c" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Mon Dec 01 17:20:58 2025 -0500" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Fri Dec 05 13:50:47 2025 +0000" }, "message": "build: update tinyxml2 to 11.0\n\nGenerated with \u0027meson wrap update --force\u0027\n\n--force needed just this one time because the prior wrapfile doesn\u0027t\nseem to have been generated with \u0027meson wrap update\u0027\n\nChange-Id: Ibc2927c27e983e639ded52621790b6449e2b9069\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\nTested: Successful compilation and unit test run using subprojects.\n" }, { "commit": "85e956a9432577ae2a205f608cb18bc83311ce5c", "tree": "683f3057d33a45f9fb0756c17179a6321cc47519", "parents": [ "d7926d0c7cb37c969de455ecfed44561f7f69aad" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Tue Nov 04 23:00:31 2025 -0500" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Wed Nov 05 19:04:42 2025 +0000" }, "message": "fail-monitor: remove is_method_error call\n\nAn `is_method_error` is not appropriate after an sdbus `call` since\n`call` will always throw an exception. Remove the pointless call\nand instead catch the exception.\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: I30147723917b0cbcc4b96e1881c8fb3e349e7cb1\n" }, { "commit": "d7926d0c7cb37c969de455ecfed44561f7f69aad", "tree": "e370b24e70b123e3e6434c67a2e592e181193a1b", "parents": [ "a46db63f4e87c0620a6f6a5545feadc12af7182e" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 14:52:18 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Wed Nov 05 03:15:57 2025 +0000" }, "message": "tidy: fix performance-unnecessary-value-param\n\nTested by building mapperx.\n\nChange-Id: Ib8c3f52e437a13de75f934bb17605dd8527bb15b\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "a46db63f4e87c0620a6f6a5545feadc12af7182e", "tree": "b4311a71b041347d0694397abd4d4a4d4eb6613c", "parents": [ "7dd2029eb1755a5ad59ac33bf462837d7e906dbf" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Feb 07 10:54:39 2025 -0500" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 12:14:51 2025 -0500" }, "message": "build: use allowed over enabled or not-disabled\n\nMeson feature options are typically in a tri-state of enabled, disabled,\nor auto. The enabled and disabled functions on an option (from\n`get_option`) no longer return true for auto features. Instead, the\nexpectation is to use `allowed()` which is true for both enabled and\nauto.\n\nSwitch all uses of `enabled` to `allowed`.\nSwitch all uses of `not disabled` to `allowed`.\n\nChange-Id: I29e5c0eba3337ed9a319f00ccb976e28233fb515\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "7dd2029eb1755a5ad59ac33bf462837d7e906dbf", "tree": "9bfdc7f0f3d0a209e0487a9b4810e482a727b5f0", "parents": [ "e5cb4aec678201ea0642c2f64151c8f3ddd54888" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 16:53:45 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 07:45:03 2025 -0500" }, "message": "build: fix -Wmissing-prototype\n\nThis helps make the author\u0027s original intent clear. Tested by building\nand running the unit tests with clang.\n\nChange-Id: If0231ec5634adad0b61e7fcff8ec1e5bcbbb92c1\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "e5cb4aec678201ea0642c2f64151c8f3ddd54888", "tree": "7c1230fe5dd18c344c5febeb0b2ca76519431236", "parents": [ "3d9fb99f5cfcb573260cba53439977cc6ede8861" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 15 18:18:03 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 07:43:06 2025 -0500" }, "message": "build: fix -Wmissing-variable-declarations\n\nThis helps make the author\u0027s original intent clear. Tested by building\nand running the unit tests with clang.\n\nChange-Id: Iaf0aca72b5a8cc931bcb84c27df1c55d2f438dda\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "3d9fb99f5cfcb573260cba53439977cc6ede8861", "tree": "9b7f0697ca27c3b651c06c0b375c0569c5c72c6e", "parents": [ "86436ac025f569f16512e71c979d4c2d4e514135" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 15 17:21:22 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 07:34:37 2025 -0500" }, "message": "fail-monitor: expand CLI11_PARSE macro\n\nThe expanded code helps readers unfamiliar with CLI11 understand what is\nhappening at a glance. It also sets up enabling more compiler warnings.\nTested by building the fail-monitor.\n\nChange-Id: I9cefff6cf7032cb262b07b005afb032d38a97867\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "86436ac025f569f16512e71c979d4c2d4e514135", "tree": "139c9de0415176664977960dcc57bf9b8e58c0e5", "parents": [ "18d3d4476455afaa127abbacdf688da5feda6ece" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 15 09:01:42 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 07:33:42 2025 -0500" }, "message": "build: fix -Wsuggest-override\n\nsuggest-override helps make the original author\u0027s intent clear for\nfuture readers. For false positives, use a different method name.\nTested by building and running the unit tests with clang.\n\nChange-Id: Icc657bf3c0269e681f26fb5e58c5198aa5aa4a19\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "18d3d4476455afaa127abbacdf688da5feda6ece", "tree": "8465928877c00e2ec8446ac79d029939784a42ff", "parents": [ "069470dca95d71e5b39afa90be2f1436211971aa" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 17:05:56 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 07:18:02 2025 -0500" }, "message": "build: remove unused headers\n\nRemove or reduce the scope of some unused headers. Tested by building\nthe code and running the unit tests.\n\nChange-Id: I7a2b311d90352f0a949e76e9387ba552e2a186ec\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "069470dca95d71e5b39afa90be2f1436211971aa", "tree": "cd887f8606b9ffe359cc9dcbb75fcc6b851ac947", "parents": [ "78585a778dbba4447a2f456c1db1c9f40abac1eb" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 16:23:08 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 06:50:55 2025 -0500" }, "message": "tidy: fix bugprone-unused-local-non-trivial-variable\n\nTested by building the code and running the unit tests.\n\nChange-Id: I2b50223f59fc53c4a73e3fd5527bc382e319609b\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "78585a778dbba4447a2f456c1db1c9f40abac1eb", "tree": "325d8c246cafefd25e6663585f15973ed999cecc", "parents": [ "7f900ff92c4a1bd7727361ff0fb25bdeb5ff0f97" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 15:27:08 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 06:50:55 2025 -0500" }, "message": "tidy: fix modernize-use-override\n\nTested by building the code.\n\nChange-Id: I57d44e54c67c7b6b3f5b758107a6d5988d1e47e3\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "7f900ff92c4a1bd7727361ff0fb25bdeb5ff0f97", "tree": "cdaa14a5f2f3ecf78fed6bc25a9ce3b56c1a4f69", "parents": [ "78b0486412e0e296fac119be2e4c20fc040d3e7a" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 15:18:44 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 06:50:54 2025 -0500" }, "message": "tidy: fix performance-avoid-endl\n\nTested by building the code.\n\nChange-Id: I3de027d34b3e91bc08923b47bda16b0b0f4b34b4\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "78b0486412e0e296fac119be2e4c20fc040d3e7a", "tree": "6751b337033ce78f3a74ec33d1a306f9a756ade3", "parents": [ "a725706d167d79169f411105e32c317026ed4410" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 08 16:00:37 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 06:50:54 2025 -0500" }, "message": "tidy: fix performance-inefficient-string-concatenation\n\nTested by building and running the unit tests.\n\nChange-Id: Ia7638172abcd737d6036bb5b2b96d03fb0a85562\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "a725706d167d79169f411105e32c317026ed4410", "tree": "2bf8ceab56fa6a2cb4ea5149f748f483627f13a6", "parents": [ "9d2c63445cae145ae0cb7baab3f6b65d5040dfe5" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 14:12:35 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 06:50:53 2025 -0500" }, "message": "tidy: fix readability-container-contains\n\nTested by building and running the unit tests.\n\nChange-Id: I146a62e54d37d8cb4b7f75f3ed8872f0c80bbb49\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "9d2c63445cae145ae0cb7baab3f6b65d5040dfe5", "tree": "55fc4ab0af874b6503700e9ffaaab8c4a936b45e", "parents": [ "9014d22b8389db78499448432e3457466e13f159" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Mon Jul 28 15:47:25 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 06:50:53 2025 -0500" }, "message": "lint: add clang-tidy configurations\n\nAdd a clang-tidy configuration for mapperx and fail-monitor.\n\nlibmapper gets its own configuration that just checks basic kernel\ncoding style.\n\nTested by running clang-tidy.\n\nChange-Id: Ifc97651e69b8a605e43a03cb63c79a23cc01274b\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "9014d22b8389db78499448432e3457466e13f159", "tree": "665f2fcaa6eec0023c06ff0e1c65734497b7fd1d", "parents": [ "aa9c24a6636dc6c89b289e7ae3457c489e152c37" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 15 16:16:35 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Nov 04 06:50:53 2025 -0500" }, "message": "meson: add clang configuration\n\nBoost requires a couple special flags to build with clang or run\nclang-tidy. sdbusplus does not build with clang-17, so we\u0027ll use\nclang-18 as our baseline for support. Tested by building and running\nthe unit tests with clang.\n\nChange-Id: Iaf4c81f4aeebcf2337691f88ce80daf03ce5e88b\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "aa9c24a6636dc6c89b289e7ae3457c489e152c37", "tree": "e46f84cbf3fb3f322e78b6fb1d52a0cbab9b630d", "parents": [ "5da44440e1b5502c605d72884e5f3a13abbdb18b" ], "author": { "name": "Alexander Hansen", "email": "alexander.hansen@9elements.com", "time": "Thu Oct 30 13:59:26 2025 +0100" }, "committer": { "name": "Alexander Hansen", "email": "alexander.hansen@9elements.com", "time": "Thu Oct 30 13:59:50 2025 +0100" }, "message": "Copyright: Move to SPDX-License-Identifier\n\nOriginal copyright holders have been preserved, this patch moves the\ncopyright notice to the compact SPDX notation.\n\nChange-Id: I7529524d48ed09eec487a7b156171ef9969aeb7d\nSigned-off-by: Alexander Hansen \u003calexander.hansen@9elements.com\u003e\n" }, { "commit": "5da44440e1b5502c605d72884e5f3a13abbdb18b", "tree": "4874eb88f4cb9f51f5c353b381b14703a8718b8d", "parents": [ "93d664fc1b21f1aad6a6759c33b913060fcf94ba" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Fri Oct 03 15:00:12 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Fri Oct 03 15:15:44 2025 -0400" }, "message": "build: fix cli11 subproject dependency\n\nAvoid the following error:\n\n```\nExecuting subproject phosphor-logging\nsubprojects/phosphor-logging/meson.build:1:0: ERROR: Multiple wrap files provide \u0027cli11\u0027 dependency: CLI11 and cli11\n```\n\nChange-Id: I67fd71c10e2fa0de48b5aaf39864af1860053f11\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "93d664fc1b21f1aad6a6759c33b913060fcf94ba", "tree": "540bf42fff3e2bf8bb7f8f5648d85bb8c5c5760c", "parents": [ "11c0cc3cacc6fd0bf194e65435e4b1e4a88f6a96" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 16:31:58 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Fri Oct 03 15:15:44 2025 -0400" }, "message": "codespell: run and fix\n\nFix the misspellings. Tested by building the code and running the unit\ntests.\n\nChange-Id: I3fb3f1f46fa41bb0c65157ff6e68bc4f1079f280\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "11c0cc3cacc6fd0bf194e65435e4b1e4a88f6a96", "tree": "505aab5bc3be1046afcb1e2a4fdf9b286dd1d2c3", "parents": [ "197de181eb41093c88dd00b9c7e314e7fc554980" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 08 16:04:25 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 12:25:59 2025 -0400" }, "message": "build: enable and fix -Wshadow\n\nShadow names can be prone to bugs. Enable the warning to avoid them in\nthe future. Tested by building and running the unit tests.\n\nChange-Id: Ic3227675f4ff40d266ae2a60c66c894f16e5888e\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "197de181eb41093c88dd00b9c7e314e7fc554980", "tree": "65c782ac8b66edf64e289922e72e6a911bbbda03", "parents": [ "4129bfa684f540ed385b67178a8403e3d1ca7bbd" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Wed Jul 09 14:41:37 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 12:25:58 2025 -0400" }, "message": "build: enable and fix -Wconversion,-Wsign-conversion\n\nFix instances of cppcoreguidelines-narrowing-conversions, and enable the\nwarnings to catch future cases. Tested by building and running the unit\ntests.\n\nChange-Id: I4047acd87a5ac6050ed7650e1efd32a921574041\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "4129bfa684f540ed385b67178a8403e3d1ca7bbd", "tree": "a9eb435f69bc5d140429b1134558e24f41423282", "parents": [ "ebff160a424e30f9c67fef187972f8d6ae66a7ce" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 15 13:43:16 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 12:25:58 2025 -0400" }, "message": "build: enable GCC warnings\n\nEnable a bunch of warnings that help keep the code robust, portable and\nmaintainable.\n\nNotably, the following desirable warnings cannot be enabled because of\npending issues in the code:\n\nWconversion\nWsign-conversion\nWnull-dereference\nWshadow\n\nTested by building and running the unit tests.\n\nChange-Id: I75627056ca9e8fd475aced172bf54983b6737bf4\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "ebff160a424e30f9c67fef187972f8d6ae66a7ce", "tree": "4146f3cd5c3b27da64214a38e4fe5b861c16a8be", "parents": [ "8f809dc572c24c505fa2fe80b7ff0fffda2cc272" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 15 13:38:34 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 12:25:58 2025 -0400" }, "message": "build: require GCC 13\n\nGCC 13 is widely available and pre-packaged. Require it as a baseline\nto make it obvious what is intended to work. Tested by building and\nrunning the unit tests with GCC 13 and GCC 14.\n\nChange-Id: Idf198bc0a7cc7ac506b6aaf5c18813df396f8c92\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "8f809dc572c24c505fa2fe80b7ff0fffda2cc272", "tree": "34ac6e4e0df33a63fcd8eb57499e318977534088", "parents": [ "e833e0b85fd983eefc3c6fd037818c931c8f288e" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 15 13:18:56 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 12:25:58 2025 -0400" }, "message": "meson: move c(xx)flags configuration\n\nGroup the c(xx)flags configuration nearer to the top, where it will be\ngrouped with additional compiler configuration in a subsequent patch.\nTested by building and running the unit tests.\n\nChange-Id: Ib2186e4756774b22ff507ce225c350e828b1e5c4\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "e833e0b85fd983eefc3c6fd037818c931c8f288e", "tree": "ca50e1623808854d75fd4d4c176a2acce7958335", "parents": [ "13cf45f18819f3df2609698099a775b83d0b0c29" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 15 09:51:44 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 12:25:50 2025 -0400" }, "message": "meson: update boost subproject\n\nUpdate the boost subproject version to match openbmc-build-scripts. The\nmeson wrapdb doesn\u0027t provide a boost wrap, so this wrap, and the\nassociated subproject build logic are derivatives of that found in\nbmcweb. Tested by building and running the unit tests using the boost\nsubproject.\n\nChange-Id: Ida616f70eabf2be5a0032f8fef22947b23e3a60e\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "13cf45f18819f3df2609698099a775b83d0b0c29", "tree": "6fb702e39221e82d0fa786fa33293017aebcf6fe", "parents": [ "0426f31bfd3adea5f53304692787fde1fdaf650b" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Mon Jul 14 16:35:13 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 09:18:17 2025 -0400" }, "message": "meson: remove implicit_include_directories\n\nWe don\u0027t generate any headers so skip looking for them in the build\ntree, which results in simpler compiler command lines and a minor\nefficiency improvement. Tested by building and running the unit tests.\n\nChange-Id: Ib666bf2b49922d9e495325948df10f1d06026b56\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "0426f31bfd3adea5f53304692787fde1fdaf650b", "tree": "b51678c97702ed075c6caa4b43fe4b4e5d2dace1", "parents": [ "5762276ab6ff7bd2edc77ad9e890594777f9113f" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Mon Jul 14 16:23:53 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 09:18:17 2025 -0400" }, "message": "meson: fail-monitor: add missing dependency\n\nTested by building fail-monitor.\n\nChange-Id: I776197d8448f9e3da106c5f895985c99ad437f74\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "5762276ab6ff7bd2edc77ad9e890594777f9113f", "tree": "ef6c9fcdb71eaed3a1f2a783debfd2a05406fc54", "parents": [ "1d7063710b7bab4cfaf2adb2c0c975fdb99e62c9" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Mon Jul 14 16:21:28 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 09:18:17 2025 -0400" }, "message": "meson: mapperx: remove unused dependency\n\nTested by building mapperx.\n\nChange-Id: I06bfcf7dfaac41807e64630fc684d481193229d3\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "1d7063710b7bab4cfaf2adb2c0c975fdb99e62c9", "tree": "b84ecd47d63a716701ec3b738a2abf974a31e267", "parents": [ "0db325c26b7cd60d5c82844ced0e96af2a42d418" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Wed Jul 09 16:39:15 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 09:18:13 2025 -0400" }, "message": "meson: refactor cli11 dependency\n\ncli11 has added support for being a meson subproject, so it can be\ndetected like any other dependency. Ensure it is used in system mode\n(-isystem...), to help static analysis tools which make use of that.\nTested by building and running the unit tests using the cli11\nsubproject.\n\nChange-Id: I8e3f4959a645eda34e995e736e1a05729fbc27f9\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "0db325c26b7cd60d5c82844ced0e96af2a42d418", "tree": "7f08dafc8541613710c66e6d8ab4b73be81b4185", "parents": [ "f3fe0db8e3fe87caf222a739d9635e9add7a477a" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Wed Jul 09 16:36:15 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 09:18:06 2025 -0400" }, "message": "meson: use cli11 from meson wrapdb\n\nThere\u0027s little point in introducing potential cli11 development\ninstability into our unit testing. Download cli11 from the meson wrapdb\nand use that verbatim. Use the same version as openbmc-build-scripts.\nTested by building and running the unit tests using the cli11\nsubproject.\n\nChange-Id: Ia61e24473ac6137d22543c802e22649396931946\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "f3fe0db8e3fe87caf222a739d9635e9add7a477a", "tree": "049c847c140ee0473ea8a05572f3a0e8c8c06958", "parents": [ "a3ab4d90d658752c48f1a74ef9003f624f66e81e" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Wed Jul 09 16:33:11 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 09:17:47 2025 -0400" }, "message": "meson: refactor gtest and gmock dependencies\n\ngtest has added first class support for being a meson subproject, so the\ngtest/gmock dependencies can be expressed with fewer conditions and more\nsuccinctly. Tested by building and running the unit tests using the\ngtest subproject.\n\nChange-Id: I57e43e36b866ebe0229b29f907b2c78acb55b7bc\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "a3ab4d90d658752c48f1a74ef9003f624f66e81e", "tree": "3431fdadad0826e1d6e9119ec8a2c4a9190048e6", "parents": [ "fcae52617d56810ab6083395bd2d11073b175b19" ], "author": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Wed Jul 09 16:24:23 2025 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradbish@qti.qualcomm.com", "time": "Tue Jul 29 09:17:42 2025 -0400" }, "message": "meson: use gtest from meson wrapdb\n\nThere\u0027s little point in introducing potential gtest development\ninstability into our unit testing. Download gtest from the meson wrapdb\nand use that verbatim. Use the same version as openbmc-build-scripts.\nTested by building and running the unit tests using the gtest\nsubproject.\n\nChange-Id: I1ecc28f233dd3dc22d3ee5923e9c8f0fabec2500\nSigned-off-by: Brad Bishop \u003cbradbish@qti.qualcomm.com\u003e\n" }, { "commit": "fcae52617d56810ab6083395bd2d11073b175b19", "tree": "5bc6c2224d8c5599f686ac755ef2f085ee991ba6", "parents": [ "1757675106e24990c43bb932120d2ff655a51a05" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Jul 09 11:27:59 2025 -0400" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Jul 09 11:27:59 2025 -0400" }, "message": "meson: use non-deprecated systemd packageconfig\n\nSystemd\u0027s packageconfig file has both `systemdsystemunitdir` and\n`systemd_system_unit_dir` defined. The non-underscore one appears\nto be a deprecated alias[1]. Move to the non-deprecated /\nunderscore-separated variant.\n\n[1]: https://github.com/systemd/systemd/commit/4908de44b0a0409f84a7cdc5641b114d6ce8ba03\n\nChange-Id: Ifa1dcc50959c2b2f5bda469c6a75a2e0be7534e5\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "1757675106e24990c43bb932120d2ff655a51a05", "tree": "3da700804093c885ca250f740c21116dfb077721", "parents": [ "1e3f7810c7e19008283937ab12a69fdce0d2df8c" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Thu Jun 12 12:26:19 2025 -0400" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Thu Jun 12 12:26:19 2025 -0400" }, "message": "markdownlint: minor fixes\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: I96bd138d6a3ead5a95763f7b98c085fd1adb09b4\n" }, { "commit": "1e3f7810c7e19008283937ab12a69fdce0d2df8c", "tree": "a271ee79ace0bfdb325dfd6f507ff9177fc1badf", "parents": [ "2a1ef0138c33036c82793104287e72d41f778108" ], "author": { "name": "Myung Bae", "email": "myungbae@us.ibm.com", "time": "Tue Mar 25 08:33:22 2025 -0700" }, "committer": { "name": "Myung Bae", "email": "myungbae@us.ibm.com", "time": "Tue Apr 01 07:31:05 2025 -0500" }, "message": "Fix SubTreePathsById when given ID is invalid\n\nThe commit 7a93d51 makes `getAssociatedSubTreePathsById()` to return\nempty paths if there are no associated subtrees. However, it also\nreturns empty paths as a success, even if the given id is not valid.\n\nThis is to check the invalid case and to return NotFound.\n\nFor example, currently, this gives the success\n- GET /redfish/v1/Chassis/INVALID/PowerSubsystem/PowerSupplies\n\nTested:\n- GET with the invalid id and check the return condition.\n\n```\nbusctl call -j \"xyz.openbmc_project.ObjectMapper\" \"/xyz/openbmc_project/object_mapper\" \\\n \"xyz.openbmc_project.ObjectMapper\" \"GetAssociatedSubTreePathsById\" ssassas \\\n \"InvalidChassis\" \\\n \"/xyz/openbmc_project/inventory\" \\\n 1 \"xyz.openbmc_project.Inventory.Item.Chassis\" \\\n \"powered_by\" \\\n 1 \"xyz.openbmc_project.Inventory.Item.PowerSupply\"\n\n--\u003e\nCall failed: The resource is not found.\n```\n\nChange-Id: If4f621e105bc1a5c443c1ec8b7762f57748e1d10\nSigned-off-by: Myung Bae \u003cmyungbae@us.ibm.com\u003e\n" }, { "commit": "2a1ef0138c33036c82793104287e72d41f778108", "tree": "c61b642ea60593d397d392092dbb1e2fb2a6736d", "parents": [ "41ad83808e8b1c5fbbf5c0e9fcb38fe8b23f7e4e" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Mon Mar 03 11:09:20 2025 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Mon Mar 03 11:09:20 2025 -0500" }, "message": "meson: reformat with meson formatter\n\nApply the `meson format` results.\n\nChange-Id: I5e218b58822e02ead2f37a069da4c0034c5005f5\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "41ad83808e8b1c5fbbf5c0e9fcb38fe8b23f7e4e", "tree": "7114483c77460124f6dbaaad51efe9d864ff5197", "parents": [ "d884cdf76957a2fce6c92d4c8bf0954c7a533998" ], "author": { "name": "Ed Tanous", "email": "ed@tanous.net", "time": "Thu Feb 13 17:00:14 2025 -0800" }, "committer": { "name": "Ed Tanous", "email": "ed@tanous.net", "time": "Thu Feb 13 17:00:53 2025 -0800" }, "message": "Fix deprecated asio usage\n\nAnd enable BOOST_ASIO_NO_DEPRECATED\n\nChange-Id: I83612501948e8c1c4a0df735aafc80e26af7f3cb\nSigned-off-by: Ed Tanous \u003ced@tanous.net\u003e\n" }, { "commit": "d884cdf76957a2fce6c92d4c8bf0954c7a533998", "tree": "1ae73483f41ec14861ad1763273e73cc0c6f795e", "parents": [ "7cc8d00aa97e915475ba74cde071c2db174d990f" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Sat Feb 01 08:23:38 2025 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Sat Feb 01 08:23:38 2025 -0500" }, "message": "clang-format: update latest spec and reformat\n\nCopy the latest format file from the docs repository and apply.\n\nChange-Id: I555df8382455157bff44671b6f284e3121d5c182\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "7cc8d00aa97e915475ba74cde071c2db174d990f", "tree": "9ff70a77f9e5bc5995589f053c6f8e9e86536949", "parents": [ "883a8929e3fba37460895d41435b9ae286c5406c" ], "author": { "name": "Lei YU", "email": "yulei.sh@bytedance.com", "time": "Tue Sep 24 02:33:16 2024 +0000" }, "committer": { "name": "Ed Tanous", "email": "ed@tanous.net", "time": "Fri Jan 31 20:30:07 2025 +0000" }, "message": "Remove default interfaces\n\nWhen association is used, mapper will create DBus objects on the leaf\nof the associated objects, and will \"assign\" the \"default interfaces\" to\nthe parent objects:\n* org.freedesktop.DBus.Introspectable\n* org.freedesktop.DBus.Peer\n* org.freedesktop.DBus.Properties\n\nFor example, software manager will create BMC version object and create\nassociation, and we could see below DBus objects created by mapper:\n```\n /xyz/openbmc_project/software/64876e4e/inventory\n /xyz/openbmc_project/software/64876e4e/software_version\n```\n\nAnd we could mapper\u0027s `GetObject` method will return the default\ninterfaces on the parent object:\n\n```json\n # busctl --json\u003dpretty call xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/object_mapper xyz.openbmc_project.ObjectMapper GetObject sas /xyz/openbmc_project/software/64876e4e 0\n {\n \"type\" : \"a{sas}\",\n \"data\" : [\n {\n \"xyz.openbmc_project.ObjectMapper\" : [\n \"org.freedesktop.DBus.Introspectable\",\n \"org.freedesktop.DBus.Peer\",\n \"org.freedesktop.DBus.Properties\"\n ],\n \"xyz.openbmc_project.Software.BMC.Updater\" : [\n \"org.freedesktop.DBus.Introspectable\",\n \"org.freedesktop.DBus.Peer\",\n \"org.freedesktop.DBus.Properties\",\n \"xyz.openbmc_project.Association.Definitions\",\n \"xyz.openbmc_project.Common.FilePath\",\n \"xyz.openbmc_project.Inventory.Decorator.Compatible\",\n \"xyz.openbmc_project.Software.Activation\",\n \"xyz.openbmc_project.Software.ExtendedVersion\",\n \"xyz.openbmc_project.Software.RedundancyPriority\",\n \"xyz.openbmc_project.Software.Version\"\n ]\n }\n ]\n }\n```\n\nThis patch removes registering the default interfaces (peer, properties,\nIntrospectable) when done as part of an ObjectManager event, because\nthey\u0027re generally not received as part of an InterfacesAdded call. These\ndon\u0027t generally get searched on, and don\u0027t make a ton of sense to have\nthe mapper implicitly add, but were done to make the introspect and\nobjectmapper results match.\n\nHowever, the default interfaces returned by introspect on the parents\ncould not be called, e.g.\n\n```\n # The leaf object is OK:\n busctl call xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/software/64876e4e/inventory org.freedesktop.DBus.Properties GetAll s org.freedesktop.DBus.Properties\n a{sv} 0\n\n # The parent object fails to call:\n busctl call xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/software/64876e4e org.freedesktop.DBus.Properties GetAll s org.freedesktop.DBus.Properties\n Call failed: Unknown object \u0027/xyz/openbmc_project/software/64876e4e\u0027.\n```\n\nThis probably means that the default interfaces returned by introspect\ncould not really be used, and thus we could remove then in mapper here.\n\nTested: With this patch, the parent object does not show default\ninterfaces:\n```json\n # busctl --json\u003dpretty call xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/object_mapper xyz.openbmc_project.ObjectMapper GetObject sas /xyz/openbmc_project/software/64876e4e 0\n {\n \"type\" : \"a{sas}\",\n \"data\" : [\n {\n \"xyz.openbmc_project.ObjectMapper\" : [],\n \"xyz.openbmc_project.Software.BMC.Updater\" : [\n \"org.freedesktop.DBus.Introspectable\",\n \"org.freedesktop.DBus.Peer\",\n \"org.freedesktop.DBus.Properties\",\n \"xyz.openbmc_project.Association.Definitions\",\n \"xyz.openbmc_project.Common.FilePath\",\n \"xyz.openbmc_project.Inventory.Decorator.Compatible\",\n \"xyz.openbmc_project.Software.Activation\",\n \"xyz.openbmc_project.Software.ExtendedVersion\",\n \"xyz.openbmc_project.Software.RedundancyPriority\",\n \"xyz.openbmc_project.Software.Version\"\n ]\n }\n ]\n }\n```\n\nNote: if this gets merged, bmcweb\u0027s related code in\nopenbmc_dbus_rest.hpp shall be updated to handle this case.\n\nSigned-off-by: Lei YU \u003cyulei.sh@bytedance.com\u003e\nChange-Id: I568f618141ac514f9720e83960a1b4e76f77eb54\n" }, { "commit": "883a8929e3fba37460895d41435b9ae286c5406c", "tree": "3beb26f82ced302b2a46014e31e1e601e024910a", "parents": [ "d01fca35bf709c179ba5ade01b127b2a31ebfbce" ], "author": { "name": "Benjamin Fair", "email": "benjaminfair@google.com", "time": "Wed Jan 22 22:07:23 2025 +0000" }, "committer": { "name": "Willy Tu", "email": "wltu@google.com", "time": "Sat Jan 25 01:14:36 2025 +0000" }, "message": "associations: fix leak in assocMaps.ifaces\n\nNew entries get added to ifaces whenever an object with associations is\ndetected, but the entries are never removed even if all the endpoints\nare gone, leading to a memory leak.\n\nChange-Id: Ide5439e4eaab7c70fb0331d59d59753905f45066\nSigned-off-by: Benjamin Fair \u003cbenjaminfair@google.com\u003e\nSigned-off-by: Willy Tu \u003cwltu@google.com\u003e\n" }, { "commit": "d01fca35bf709c179ba5ade01b127b2a31ebfbce", "tree": "b75ed623c0fe7bc686325cad78a5120968f005fe", "parents": [ "ea0e5d27a869a88ab40617675d2af4e6b9fbed92" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Dec 18 11:22:48 2024 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Dec 18 11:22:48 2024 -0500" }, "message": "clang-format: re-format for clang-19\n\nclang-format-19 isn\u0027t compatible with the clang-format-18 output, so we\nneed to reformat the code with the latest version. A few parameters\nin clang-tidy have been deprecated, so adjust the style file\naccordingly.\n\nSee Ie2f6eb3b043f2d655c9df806815afd7971fd0947 for updated style.\nSee I88192b41ab7a95599a90915013579608af7bc56f for clang-19 enablement.\n\nChange-Id: Id076d89feb8ae9b1a2053234f839affd8628600a\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "ea0e5d27a869a88ab40617675d2af4e6b9fbed92", "tree": "e7cc817ea227712c89b460f82e601e298ab39b75", "parents": [ "cf72403d7ef358974c18ff5257f5e763151952ec" ], "author": { "name": "Ed Tanous", "email": "etanous@nvidia.com", "time": "Mon Nov 11 10:00:56 2024 -0800" }, "committer": { "name": "Ed Tanous", "email": "ed@tanous.net", "time": "Mon Nov 11 10:07:00 2024 -0800" }, "message": "Fix compile error on boost 1.86\n\nBoost 1.86 shows a compiler error which is holding up the rebase here.\nThe mapper extends std::vector, because it results in smaller binary\nsizes compared to boost::vector. It\u0027s not clear if this is the problem,\nor if there\u0027s something else amiss within container. Triaging the real\nroot cause isn\u0027t important to this use case.\n\nThere are a few reported bugs and changes that might be related[1][2].\n\nIt\u0027s not clear which one would\u0027ve broken try_emplace in this case, nor\nis it worth the effort to resolve.\n\nThis patchset replaces try_emplace calls with emplace calls. In theory\nthese are slightly less efficient, but considering that map construction\nis a cold path, it\u0027s not terribly important that it be efficient.\n\n[1] https://github.com/boostorg/container/issues/292\n[2] https://github.com/boostorg/container/issues/280\n\nChange-Id: I30d176022d2cec45430a50dc02c84666541fb4d7\nSigned-off-by: Ed Tanous \u003cetanous@nvidia.com\u003e\n" }, { "commit": "cf72403d7ef358974c18ff5257f5e763151952ec", "tree": "86085847706f86f62e9ee7085bde0d631af70314", "parents": [ "7a93d516522f58b887a05cc93ff27a6674601e5f" ], "author": { "name": "George Liu", "email": "liuxiwei@ieisystem.com", "time": "Tue Oct 29 14:31:51 2024 +0800" }, "committer": { "name": "George Liu", "email": "liuxiwei@ieisystem.com", "time": "Thu Oct 31 10:11:49 2024 +0800" }, "message": "libmapper: format code using c++ clang-format\n\nRemove libmapper/.clang-fromat and format code using c++ clang-format\n\nSigned-off-by: George Liu \u003cliuxiwei@ieisystem.com\u003e\nChange-Id: I62781743b03fa1e806f51f4844c93aaf4cdfac4f\n" }, { "commit": "7a93d516522f58b887a05cc93ff27a6674601e5f", "tree": "ff6037cdf701a1ef1d16b0bffe7f823ff19d7d4c", "parents": [ "c363323e7ee9e42817922346b1f1ce6adc872d1b" ], "author": { "name": "Myung Bae", "email": "myungbae@us.ibm.com", "time": "Fri Sep 13 14:23:19 2024 -0400" }, "committer": { "name": "Myung Bae", "email": "myungbae@us.ibm.com", "time": "Sun Sep 15 16:28:56 2024 -0400" }, "message": "Return empty paths if no SubTreePathsById is found\n\n`getAssociatedSubTreePaths()` returns the empty paths if there are no\nassociated subtrees, and bmcweb uses it to differentiate from the error.\n\nHowever, `getAssociatedSubTreePathsById()` (which is introduced by [1])\nreturns NotFound error for the case. It would be more appropriate to\nmatch its behavior with `getAssociatedSubTreePaths()`.\n\nTested:\n- unit-test is succeeded for good and bad cases\n\n[1] https://gerrit.openbmc.org/c/openbmc/phosphor-objmgr/+/70699\n\nChange-Id: I84680b1b39ba8ade94ea6fdb7dae280505e5c050\nSigned-off-by: Myung Bae \u003cmyungbae@us.ibm.com\u003e\n" }, { "commit": "c363323e7ee9e42817922346b1f1ce6adc872d1b", "tree": "7f764ae61e929d4d3b5f5899d37b333aa4766016", "parents": [ "8168cafb17e45c2c7caab91a98c05bd72590ca13" ], "author": { "name": "Lakshmi Yadlapati", "email": "lakshmiy@us.ibm.com", "time": "Tue Apr 09 10:47:29 2024 -0500" }, "committer": { "name": "Myung Bae", "email": "myungbae@us.ibm.com", "time": "Thu Sep 05 15:54:19 2024 +0000" }, "message": "mapper: Add GetAssociatedSubTreePathsById and GetAssociatedSubTreeById\n\ndbus-interface change in:\nhttps://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/69999\n\nThis commit implements two new methods: GetAssociatedSubTreePathsById\nand GetAssociatedSubTreeById. These methods retrieve the paths of\nassociated endpoints corresponding to the provided identifier, filtering\nbased on their association with specified endpoint interfaces.\n\nGetAssociatedSubTreePathsById returns the D-Bus paths of associated\nendpoints, while GetAssociatedSubTreeById retrieves a dictionary of\nD-Bus paths of associated endpoints mapped to corresponding services\nassociated with the provided identifier.\n\nTested:\n\u0027\u0027\u0027\nbusctl call -j \"xyz.openbmc_project.ObjectMapper\" \"/xyz/openbmc_project/object_mapper\" \\\n \"xyz.openbmc_project.ObjectMapper\" \"GetAssociatedSubTreePathsById\" ssassas \\\n \"chassis\" \\\n \"/xyz/openbmc_project/inventory\" \\\n 1 \"xyz.openbmc_project.Inventory.Item.Chassis\" \\\n \"powered_by\" \\\n 1 \"xyz.openbmc_project.Inventory.Item.PowerSupply\"\n{\n \"type\" : \"as\",\n \"data\" : [\n [\n \"/xyz/openbmc_project/inventory/system/chassis/motherboard/powersupply0\",\n \"/xyz/openbmc_project/inventory/system/chassis/motherboard/powersupply1\",\n \"/xyz/openbmc_project/inventory/system/chassis/motherboard/powersupply2\",\n \"/xyz/openbmc_project/inventory/system/chassis/motherboard/powersupply3\"\n ]\n ]\n}\n\u0027\u0027\u0027\n\nAnother example.\n\n```\nbusctl call -j xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/object_mapper \\\n xyz.openbmc_project.ObjectMapper GetAssociatedSubTreePathsById ssassas \\\n disk_backplane0 \\\n /xyz/openbmc_project/inventory \\\n 1 xyz.openbmc_project.Inventory.Item.FabricAdapter \\\n connecting \\\n 1 xyz.openbmc_project.Inventory.Connector.Port\n{\n\t\"type\" : \"as\",\n\t\"data\" : [\n\t\t[\n\t\t\t\"/xyz/openbmc_project/inventory/system/chassis/motherboard/disk_backplane0/dp0_connector0\",\n\t\t\t\"/xyz/openbmc_project/inventory/system/chassis/motherboard/disk_backplane0/dp0_connector1\",\n\t\t\t\"/xyz/openbmc_project/inventory/system/chassis/motherboard/disk_backplane0/dp0_connector2\",\n\t\t\t\"/xyz/openbmc_project/inventory/system/chassis/motherboard/disk_backplane0/dp0_connector3\",\n\t\t\t\"/xyz/openbmc_project/inventory/system/chassis/motherboard/disk_backplane0/dp0_connector4\",\n\t\t\t\"/xyz/openbmc_project/inventory/system/chassis/motherboard/disk_backplane0/dp0_connector5\"\n\t\t]\n\t]\n}\n```\n\nChange-Id: Id55a9b41fe70f7204543d92b5396888f6914a1d4\nSigned-off-by: Lakshmi Yadlapati \u003clakshmiy@us.ibm.com\u003e\nSigned-off-by: Myung Bae \u003cmyungbae@us.ibm.com\u003e\n" }, { "commit": "8168cafb17e45c2c7caab91a98c05bd72590ca13", "tree": "3165c24b3fe1d4375a0ac0ab1c2976f32c06b4a7", "parents": [ "8c25006cb0667c390249d77b9feb77cb7a862094" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Tue Sep 03 15:58:03 2024 -0400" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Tue Sep 03 15:58:03 2024 -0400" }, "message": "meson: switch code dependency to libsystemd\n\nFor a while now systemd has packaged a separate `libsystemd`. For\nmeson dependency, `systemd` should be used when looking for systemd\nservice file paths and `libsystemd` should be used for `sd_*` functions.\n\nUpdate the dependency accordingly.\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: Iff3e5cc93a503e7391e4eea9624b04069135a42a\n" }, { "commit": "8c25006cb0667c390249d77b9feb77cb7a862094", "tree": "e25c544f23c4a9f2c7391de8e2ba669eee11b0e5", "parents": [ "9052ebd36b0e4a016f769983b774dff49a3ea423" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Aug 23 12:28:07 2024 -0400" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Aug 23 14:02:52 2024 -0400" }, "message": "handler: fix undefined behavior\n\nSaving a reference to a temporary returned from a function call is\nundefined behavior and subverts RVO. Remove the `\u0026` so these become\nlocal variables.\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: I8e7da30fddef808dfe7ee323d7f91a7c2a2cdeb6\n" }, { "commit": "9052ebd36b0e4a016f769983b774dff49a3ea423", "tree": "8d16926b2b94a288107c9e85a5570227bcf7503c", "parents": [ "fb853663a2e3ca3366bf96a21a0a045f69d7ed2a" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Aug 16 15:22:16 2024 -0400" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Aug 16 15:22:16 2024 -0400" }, "message": "clang-format: re-format for clang-18\n\nclang-format-18 isn\u0027t compatible with the clang-format-17 output, so we\nneed to reformat the code with the latest version. The way clang-18\nhandles lambda formatting also changed, so we have made changes to the\norganization default style format to better handle lambda formatting.\n\nSee I5e08687e696dd240402a2780158664b7113def0e for updated style.\nSee Iea0776aaa7edd483fa395e23de25ebf5a6288f71 for clang-18 enablement.\n\nChange-Id: Ic68b91e23738cafe198c50f40e46d4163bda02b6\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "fb853663a2e3ca3366bf96a21a0a045f69d7ed2a", "tree": "9a361efb550bb5609b6d56295ddb884003829694", "parents": [ "883d91d7d8f9a0f2bcf150990f75073cad21d94c" ], "author": { "name": "Andrew Jeffery", "email": "andrew@codeconstruct.com.au", "time": "Fri May 03 14:54:11 2024 +0930" }, "committer": { "name": "Andrew Jeffery", "email": "andrew@codeconstruct.com.au", "time": "Fri May 03 14:56:49 2024 +0930" }, "message": "meson: Declare a find_program() override for mapperx\n\nIf phosphor-objmgr is used as a subproject, allow discovery of its\nmapperx program for use in the parent project.\n\nChange-Id: I1cb448500cab015b8b309979ab26f7e51f03e8ea\nSigned-off-by: Andrew Jeffery \u003candrew@codeconstruct.com.au\u003e\n" }, { "commit": "883d91d7d8f9a0f2bcf150990f75073cad21d94c", "tree": "92019a67666ac95e71d405ab5950c6f1f249f254", "parents": [ "3e2eb6af4b112fcf7ceb5c2c502244211c0704aa" ], "author": { "name": "Konstantin Aladyshev", "email": "aladyshev22@gmail.com", "time": "Wed Apr 03 12:38:37 2024 +0300" }, "committer": { "name": "Konstantin Aladyshev", "email": "aladyshev22@gmail.com", "time": "Wed Apr 03 12:38:37 2024 +0300" }, "message": "meson: Disable tests build for tinyxml2 dependency\n\nCurrently local meson build breaks on tinyxml2 since the code from\nthis package produces a warning and all warnings are treated as errors:\n\"\"\"\n../subprojects/tinyxml2-9.0.0/xmltest.cpp:711:38: error: ignoring\nreturn value of ‘char* fgets(char*, int, FILE*)’ declared with\nattribute ‘warn_unused_result’ [-Werror\u003dunused-result]\n\"\"\"\nThe warning above comes from the compilation of the tinyxml2 unit tests.\nSince we don\u0027t really need them, disable \u0027tests\u0027 option for tinyxml2\ndependency to fix the meson build.\n\nTested:\nLocally executed \"meson setup build \u0026\u0026 cd build \u0026\u0026 meson compile\" now\nfinishes successfully.\n\nChange-Id: Ib35fef4e515df22c7b82a6f379689d4672059099\nSigned-off-by: Konstantin Aladyshev \u003caladyshev22@gmail.com\u003e\n" }, { "commit": "3e2eb6af4b112fcf7ceb5c2c502244211c0704aa", "tree": "bbb7f2681bea9b80d28505da4122fc5a45a41ed0", "parents": [ "ec87407cc1d69cbd11d9235a069a80efccf53e02" ], "author": { "name": "Konstantin Aladyshev", "email": "aladyshev22@gmail.com", "time": "Fri Mar 29 16:24:52 2024 +0300" }, "committer": { "name": "Konstantin Aladyshev", "email": "aladyshev22@gmail.com", "time": "Fri Mar 29 16:24:52 2024 +0300" }, "message": "build: Add wrap file for tinyxml2\n\nCurrently local build on the system without tinyxml2 can fail with\nthe following message:\n\"\"\"\nmeson.build:107:8: ERROR: Dependency \"tinyxml2\" not found, tried\npkgconfig and cmake\n\"\"\"\nProvide \"tinyxml2.wrap\" file to fix the issue.\nThe \"tinyxml2.wrap\" file is copied from the bmcweb repository.\n\nChange-Id: I652cae42192affd6513d1e148e4b4a177e00999d\nSigned-off-by: Konstantin Aladyshev \u003caladyshev22@gmail.com\u003e\n" }, { "commit": "ec87407cc1d69cbd11d9235a069a80efccf53e02", "tree": "cb20d1bdee267a3ee039dca4c6097e06cbe07cd9", "parents": [ "47b68cbc92a7e11f1e466be07135782be27501fe" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Nov 29 06:45:19 2023 -0600" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Nov 29 06:45:19 2023 -0600" }, "message": "build: use allowed over enabled\n\nMeson feature options are typically in a tri-state of enabled, disabled,\nor auto. The enabled and disabled functions on an option (from\n`get_option`) no longer return true for auto features. Instead, the\nexpectation is to use `allowed()` which is true for both enabled and auto.\n\nSwitch all uses of `enabled` to `allowed`.\n\nChange-Id: I2ce48ae6fe133c0b355672b5bb7d7c27bc6b3702\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "47b68cbc92a7e11f1e466be07135782be27501fe", "tree": "0ff44f96c3104d156d4124f3e26f3fde8b17de6e", "parents": [ "ef885e460dfe8fc533d5b97a80de178537e2162c" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Oct 20 11:20:07 2023 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Oct 20 11:20:07 2023 -0500" }, "message": "clang-format: copy latest and re-format\n\nclang-format-17 has some backwards incompatible changes that require\nadditional settings for best compatibility and re-running the formatter.\nCopy the latest .clang-format from the docs repository and reformat the\nrepository.\n\nChange-Id: Ibc0976e16acb6163431698832a461e9fc7335448\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "ef885e460dfe8fc533d5b97a80de178537e2162c", "tree": "58fea6d74104f722a35ee7aa9e7030a7bead712a", "parents": [ "26ed98050e64e76892be5d5c2416b8a0044e3d9d" ], "author": { "name": "George Liu", "email": "liuxiwei@inspur.com", "time": "Wed Aug 16 10:04:04 2023 +0800" }, "committer": { "name": "George Liu", "email": "liuxiwei@inspur.com", "time": "Wed Aug 16 10:04:04 2023 +0800" }, "message": "meson_options.txt: Support for reading options from meson.options\n\nSupport has been added for reading options from meson.options instead\nof meson_options.txt[1]. These are equivalent, but not using the .txt\nextension for a build file has a few advantages, chief among them\nmany tools and text editors expect a file with the .txt extension to\nbe plain text files, not build scripts.\n\n[1] https://mesonbuild.com/Release-notes-for-1-1-0.html#support-for-reading-options-from-mesonoptions\n\nSigned-off-by: George Liu \u003cliuxiwei@inspur.com\u003e\nChange-Id: I0714938f7ffd30c97c2f10bb45796103d29f4d12\n" }, { "commit": "26ed98050e64e76892be5d5c2416b8a0044e3d9d", "tree": "5c791484481ae6c83b20338bce9b8eea5fdade4f", "parents": [ "1a19e6aaf3663f7698fa17aeb74a887c8740ff93" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Jul 12 11:16:09 2023 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Jul 12 11:16:09 2023 -0500" }, "message": "build: upgrade to C++23\n\nMeson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of\nthe standard has been implemented. Upgrade the build to leverage it.\n\nChange-Id: Ifabec573b184316716aa692598caba26fb5857b3\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "1a19e6aaf3663f7698fa17aeb74a887c8740ff93", "tree": "aa938ce24f0a576bf40c3c92446cc2f990593106", "parents": [ "403027426d684b62459a2fe7225a7c60accb0e5c" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed May 10 07:51:32 2023 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed May 10 07:51:32 2023 -0500" }, "message": "clang-format: copy latest and re-format\n\nclang-format-16 has some backwards incompatible changes that require\nadditional settings for best compatibility and re-running the formatter.\nCopy the latest .clang-format from the docs repository and reformat the\nrepository.\n\nChange-Id: I9252bdd2509e5a9cff31fce9be31291f63b3a5cc\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "403027426d684b62459a2fe7225a7c60accb0e5c", "tree": "7c07f1fbe35e32205376ea32bbedbd5561f8ee4c", "parents": [ "bd441717aa78dc863c68012609f6096275c34648" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Apr 12 08:01:37 2023 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Apr 12 08:01:37 2023 -0500" }, "message": "meson: remove deprecated get_pkgconfig_variable\n\nSince meson 0.56, the `get_pkgconfig_variable` has been deprecated. In\nmeson 0.58 the `get_variable` was enhanced to no longer require the\n`pkgconfig` keyword argument. Ensure meson 0.58 is required and update\nthe usage of all `get_pkgconfig_variable` and `get_variable` to be the\nmodern variant.\n\nChange-Id: Idff7506a1f897993ba350fdc31ce5103c8a9e998\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "bd441717aa78dc863c68012609f6096275c34648", "tree": "538923faf6c0be02e7ca17d2d01619d38f84bdab", "parents": [ "670edd12179786c525d8624d983b95eef8f6914a" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Feb 15 14:49:05 2023 -0600" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Feb 15 15:08:23 2023 -0600" }, "message": "mapper-wait: add service dependency\n\nEven though ObjectMapper is dbus-activated, it takes long enough to\nstart up that the dbus client operation times out. This causes a\nwhole bunch of failed services on boot of the BMC. Add an explicit\ndependency back on the service in order to give the service time to\nstart up.\n\nTested: Applied on Bletchley and see no restarts of the various\nmapper-wait invocations now.\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: I473a8f251906a8267e2c44600b6802adfea1fd0e\n" }, { "commit": "670edd12179786c525d8624d983b95eef8f6914a", "tree": "eddadc799fb064e53a99de96f34514177ae847c6", "parents": [ "a99e109c5846d55b6df31c19410642b3520a17c8" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Feb 15 15:06:52 2023 -0600" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Wed Feb 15 15:08:23 2023 -0600" }, "message": "clang-format: update with latest\n\nCopy the latest clang-format and apply to the repository.\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: I10a3fe64d4ca77601e6855daf46954579312c7df\n" }, { "commit": "a99e109c5846d55b6df31c19410642b3520a17c8", "tree": "14b4ad052b6c06eb45062663c91089ea322f6818", "parents": [ "42e5aee2252cab246e0e9bedeab36aee335db816" ], "author": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Fri Feb 03 14:33:08 2023 -0500" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Tue Feb 14 20:15:20 2023 +0000" }, "message": "lint: update .clant-format and run\n\nChange-Id: If9cfc42ca96517a4df2c59c130c2e22a2a443467\nSigned-off-by: Brad Bishop \u003cbradleyb@fuzziesquirrel.com\u003e\n" }, { "commit": "42e5aee2252cab246e0e9bedeab36aee335db816", "tree": "c7509b1b4f771d103914271956fb6b996d2a36dc", "parents": [ "2ec9157fd431cad031123860796615ed4a16f6ba" ], "author": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Fri Feb 03 13:57:33 2023 -0500" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Tue Feb 14 20:15:20 2023 +0000" }, "message": "unit-fail-monitor: add meson option to disable\n\nAdd an option to disable this feature.\n\nChange-Id: Ifc953ad4bec492f7c6ca8190e4c6a0820b12630a\nSigned-off-by: Brad Bishop \u003cbradleyb@fuzziesquirrel.com\u003e\n" }, { "commit": "2ec9157fd431cad031123860796615ed4a16f6ba", "tree": "0195caf45748434ca9b0ade10acab561094fac0f", "parents": [ "58881d0b563ad6ea9e3d1c3fcb84496f41205f01" ], "author": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Fri Jun 03 08:52:11 2022 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Tue Feb 14 20:15:20 2023 +0000" }, "message": "mapper: install systemd unit, dbus conf files\n\nThe same systemd and dbus unit/configuration files can work for everyone\nnow, so ship them. Enable dbus activation, add unit file templates, and\nmove mapperx to libexec where it belongs.\n\nChange-Id: Ieb021889605767ffdb9d72faad5f5b856312b935\nSigned-off-by: Brad Bishop \u003cbradleyb@fuzziesquirrel.com\u003e\n" }, { "commit": "58881d0b563ad6ea9e3d1c3fcb84496f41205f01", "tree": "3d0ffa4baf856e0b444b03062148011dfbfdd2f0", "parents": [ "aba14d3dcbd28df946efa96500ebb5a8926e1abf" ], "author": { "name": "Willy Tu", "email": "wltu@google.com", "time": "Sun Oct 02 20:46:45 2022 +0000" }, "committer": { "name": "Willy Tu", "email": "wltu@google.com", "time": "Fri Feb 03 09:13:19 2023 -0800" }, "message": "mapper: Add Associated subtree method definition\n\ndbus-interface change in:\nhttps://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/57821\n\nCreate new mapper function to make subtree call and association call at\nthe same time. With the association definition, we will have more\nendpoints to fetch and verify that it is actually the resource that we\nare looking for. We need to verify that it has the expected interface to\nconfirm that it is right device. In the current workflow this will take\ntwo dbus calls to the mapper to get the association and the subtree.\n\nThis change aims to reduce that down to one by combining the two\noperations. It will not significant performance increase for the daemons,\nbut it can help reduce the load on the dbus-broker.service.\n\nTested:\n```\nNormal Call:\nbusctl call \"xyz.openbmc_project.ObjectMapper\" \\\n \"/xyz/openbmc_project/object_mapper\" \\\n \"xyz.openbmc_project.ObjectMapper\" \"GetSubTreePaths\" sias \\\n \"/xyz/openbmc_project/inventory\" 0 1 \\\n \"xyz.openbmc_project.Inventory.Item.Storage\"\nas 4 \"/xyz/openbmc_project/inventory/storage_0\" \"/xyz/openbmc_project/inventory/storage_1\"\n \"/xyz/openbmc_project/inventory/storage_4\" \"/xyz/openbmc_project/inventory/storage_3\"\n\nAssociated Call: (Only 0 and 1 is associated to the chassis)\nbusctl call \"xyz.openbmc_project.ObjectMapper\" \\\n \"/xyz/openbmc_project/object_mapper\" \\\n \"xyz.openbmc_project.ObjectMapper\" \"GetAssociatedSubTreePaths\" ssias \\\n \"/xyz/openbmc_project/inventory/chassis_0/storage\" \\\n \"/xyz/openbmc_project/inventory\" 0 1 \"xyz.openbmc_project.Inventory.Item.Storage\"\nas 2 \"/xyz/openbmc_project/inventory/storage_0\" \"/xyz/openbmc_project/inventory/storage_1\"\n```\n\nChange-Id: Ib532e660cf20b9efc83045fb1cbd9143d0c1841e\nSigned-off-by: Willy Tu \u003cwltu@google.com\u003e\n" }, { "commit": "aba14d3dcbd28df946efa96500ebb5a8926e1abf", "tree": "b4d10406e380c247ae3af028f51eea4c6a7281c2", "parents": [ "fffd34daac3d6d1882c58f46a2c5ac15bbc3d0a5" ], "author": { "name": "Willy Tu", "email": "wltu@google.com", "time": "Tue Jan 31 14:19:59 2023 -0800" }, "committer": { "name": "Willy Tu", "email": "wltu@google.com", "time": "Fri Feb 03 08:20:23 2023 -0800" }, "message": "test: Add dbus handler unit tests\n\nCreated unit tests for all of the object mapper dbus handlers.\n\nRefactor the handlers to handler.cpp to allow unit testing.\nAlso updated to use std::set_intersection.\n\nChange-Id: Ib87ab81f949d27f7dab55899d3e8d380e43bda04\nSigned-off-by: Willy Tu \u003cwltu@google.com\u003e\n" }, { "commit": "fffd34daac3d6d1882c58f46a2c5ac15bbc3d0a5", "tree": "c9651776c27cb9cfbeb9a1b620d8ff57cc65b34c", "parents": [ "60ffc244730b5dea126cfada6bb80429837318b5" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Thu Dec 08 06:13:26 2022 -0600" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Thu Dec 08 06:13:26 2022 -0600" }, "message": "prettier: re-format\n\nPrettier is enabled in openbmc-build-scripts on Markdown, JSON, and YAML\nfiles to have consistent formatting for these file types. Re-run the\nformatter on the whole repository.\n\nChange-Id: I604f9ce2677792518c9b124939bff0ac713cef34\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\n" }, { "commit": "60ffc244730b5dea126cfada6bb80429837318b5", "tree": "15b9744ff6de592938ddd7340382de2bf1818024", "parents": [ "5b4357da283bb924e6ce8e94f594c42b17eda39c" ], "author": { "name": "William A. Kennington III", "email": "wak@google.com", "time": "Fri Dec 02 19:36:01 2022 -0800" }, "committer": { "name": "William A. Kennington III", "email": "wak@google.com", "time": "Fri Dec 02 19:36:01 2022 -0800" }, "message": "mapperx: Use lambdas and move\n\nThis reduces the number of type-erased copies we have to store and makes\nthe type work with interface changes to the underlying stored callback\ntype.\n\nChange-Id: Ieccd0e7f182177e47e8d8d4ef836521109f1a2e7\nSigned-off-by: William A. Kennington III \u003cwak@google.com\u003e\n" }, { "commit": "5b4357da283bb924e6ce8e94f594c42b17eda39c", "tree": "18ed7d88cf7982d72b8c0f9bade660cd1f757171", "parents": [ "48248203c6cef3c90e8bcbd8e1c424fc0f0d973c" ], "author": { "name": "Kallas, Pawel", "email": "pawel.kallas@intel.com", "time": "Wed Oct 12 15:36:37 2022 +0200" }, "committer": { "name": "Kallas, Pawel", "email": "pawel.kallas@intel.com", "time": "Fri Nov 04 08:45:49 2022 +0100" }, "message": "add Associations endpoints change delay timer\n\nWhen multiple associations that point to the same interface are\ncreated, each change (adding or removing one) leads to updating\n\"endpoints\" property on dbus. This property update is time consuming\nwith many endpoints already present, because each update needs to send\nthe whole list of current elements plus/minus one. With a lot of\nchanges in short time it can cause the service to be unresponsive.\nThis change adds timer to delay updating dbus property. This way many\nassociations updates can be aggregated into single dbus property\nupdate.\n\nTested: Ran on hardware with dbus sensor tester. 4000 created sensors\nwith interfaces are processed within 10 seconds. Time before the change\nwas above 2 minutes.\n\nSigned-off-by: Kallas, Pawel \u003cpawel.kallas@intel.com\u003e\nChange-Id: I1083c027ab12238249cffc67fb29a8ffef6baf83\n" }, { "commit": "48248203c6cef3c90e8bcbd8e1c424fc0f0d973c", "tree": "8b0899bcae04a9ddb0bd28c76acba7dcf4e8ac78", "parents": [ "8c243629a9e6866ccac247d74f274b86bc1fad50" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Mon Oct 10 10:26:47 2022 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Mon Oct 10 10:26:57 2022 -0500" }, "message": "ignore unique-name dbus connections\n\nProcesses which only expose unique-name connections are dbus-clients\nand do not need to have their dbus tree interrogated. It is also\ncommon practice for a daemon to create a bus connection, create their\nobjects, and then claim a well-known name. By having objmgr look at\nthe unique-name connections from those daemons, objmgr is getting\ntoo early a look into the daemon.\n\nFilter out the unique-name connections from the dbus signal match.\n\nI ran this on Bletchley hardware and observed a small boot time\nperformance gain (\u003c1s) which was difficult to distinguish from\nnoise. On some machines with heavy script-driven `busctl` operations,\nthis was reported to have some noticeable performance benefit.\n\nTested: Ran on Bletchley and observed no change in behavior for\nmapper-wait, entity-manager, etc.\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: I93bde89409f049682fb7c67ab1fe45b7b9cf795a\n" }, { "commit": "8c243629a9e6866ccac247d74f274b86bc1fad50", "tree": "5342a9943cf5581025ab2d64b6e279a4cd706d87", "parents": [ "1e94e60b4fb6f6e1ebe42eeeeaf52a070ae11c30" ], "author": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Mon Jul 11 11:21:50 2022 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Wed Sep 14 15:06:01 2022 -0400" }, "message": "monitor: add cli11 argument parsing\n\nReplace our custom argument parser with one from CLIUtils.\n\nThis results in a number of minor behavioral differences.\n - The -? option is no longer recognized as an alias for -h/--help.\n - The original parser returned EXIT_FAILURE in all cases, but CLIUtils\n uses different non-zero error codes in different situations.\n - Minor changes to help text descriptions:\n----new----\nUsage: build/phosphor-unit-failure-monitor.new [OPTIONS]\n\nOptions:\n -h,--help Print this help message and exit\n -s,--source TEXT REQUIRED\n -t,--target TEXT REQUIRED\n -a,--action ENUM:value in {start-\u003e0,stop-\u003e1} OR {0,1} REQUIRED\n\n----old----\nUsage: build/phosphor-unit-failure-monitor [options]\nOptions:\n --help Print this menu\n --source\u003d\u003csource\u003e The source unit to monitor\n --target\u003d\u003ctarget\u003e The target unit to start or stop\n --action\u003d\u003caction\u003e Target unit action - start or stop\n\n - The original parser displays the full help text on error, CLIUtils\n does not:\n\n----new----\n$ phosphor-unit-failure-monitor\n--source is required\nRun with --help for more information.\n\n----old----\n$ phosphor-unit-failure-monitor\nERROR: Source not specified\nUsage: build/phosphor-unit-failure-monitor [options]\nOptions:\n --help Print this menu\n --source\u003d\u003csource\u003e The source unit to monitor\n --target\u003d\u003ctarget\u003e The target unit to start or stop\n --action\u003d\u003caction\u003e Target unit action - start or stop\n\nChange-Id: I2417d9c857c6d8fc04807fe4729d2fa154e746a3\nSigned-off-by: Brad Bishop \u003cbradleyb@fuzziesquirrel.com\u003e\n" }, { "commit": "1e94e60b4fb6f6e1ebe42eeeeaf52a070ae11c30", "tree": "7cd5b948fcaeaa2b75e52303a11ca6161e4bee1e", "parents": [ "ff46420f4c983726e3bba3a2afb1120206c3725c" ], "author": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Thu Jun 02 19:47:53 2022 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Wed Sep 14 17:40:09 2022 +0000" }, "message": "mapper: remove configurable service allowlists\n\nRemove the ability to specify an allow list at runtime. Instead,\nintrospect any DBus service except those that start with\norg.freedesktop. This removes the need for argument parsing code and\ncomplex bitbake metadata.\n\nSkip org.freedesktop because applications (OpenBMC or otherwise) should\naccess org.freedesktop services directly and not indirectly via the\nOpenBMC mapper.\n\nChange-Id: I83038a121580dcde2a2b3b1f994b3066cc9d955f\nSigned-off-by: Brad Bishop \u003cbradleyb@fuzziesquirrel.com\u003e\n" }, { "commit": "ff46420f4c983726e3bba3a2afb1120206c3725c", "tree": "f0fb46f06c8e77ada5ca6c7b164eee03917e806e", "parents": [ "86d2880ef1ec78bd224875791b75a0704478e765" ], "author": { "name": "Matt Spinler", "email": "spinler@us.ibm.com", "time": "Wed Aug 17 16:58:38 2022 -0500" }, "committer": { "name": "Matt Spinler", "email": "spinler@us.ibm.com", "time": "Wed Aug 17 17:02:06 2022 -0500" }, "message": "Update README\n\nPoint to some mapper documentation I wrote awhile ago.\n\nSigned-off-by: Matt Spinler \u003cspinler@us.ibm.com\u003e\nChange-Id: Ib3f89d5f3991c75647c4b45855a12c2e0eb81607\n" }, { "commit": "86d2880ef1ec78bd224875791b75a0704478e765", "tree": "a7ad3cd9b87d7d3a43a5915d5feb3bc77eedab32", "parents": [ "f6ebfc73cd3cf1e59ecd3dfc9c1729d17bf30f51" ], "author": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Mon Jul 11 15:49:31 2022 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Wed Aug 17 08:35:18 2022 -0400" }, "message": "treewide: prefer std::starts_with to boost\n\nReplace boost versions of starts_with and ends_with with the std\nversions provided with c++20.\n\nSigned-off-by: Brad Bishop \u003cbradleyb@fuzziesquirrel.com\u003e\nChange-Id: I97218c607ff04aedad490b3af428071b0fa75615\n" }, { "commit": "f6ebfc73cd3cf1e59ecd3dfc9c1729d17bf30f51", "tree": "37f71570c675589b60d0868311c46b9d9e6a3f3a", "parents": [ "3714dd3058bd383ea80cb06d0a45ea66ed4d9a50" ], "author": { "name": "Brandon Kim", "email": "brandonkim@google.com", "time": "Thu Jul 07 12:53:44 2022 -0700" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Wed Aug 17 12:30:50 2022 +0000" }, "message": "main: Fix getSubTree \u0026 getSubTreePaths partial match\n\nDue to us popping out the trailing \"/\" in our \"reqPath\", partial matches\nsatisfied the \"boost::starts_with(\" searches for paths - leading to\n\"core1X\" for example, matching a reqPath of \"core1\" or \"core1/\".\n\nSince we do not want to return the base reqPath anyway, add a trailing\n\"/\" in the reqPath for the match to ensure complete match.\n\nTested:\ndbus-send --system --print-reply --dest\u003dxyz.openbmc_project.ObjectMapper /xyz/openbmc_project/object_mapper xyz.openbmc_project.ObjectMapper.GetSubTreePaths string:\"/xyz/openbmc_oject/inventory/system/chassis/motherboard/cpu0/core1\" int32:0 array:string:\"xyz.openbmc_project.Inventory.Item.CpuThread\"\n\nWithout the fix:\n array [\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core1/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core11/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core12/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core13/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core14/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core15/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core16/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core17/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core18/thread0\"\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core19/thread0\"\n ]\n\nWith the fix:\n array [\n string \"/xyz/openbmc_project/inventory/system/chassis/motherboard/cpu0/core1/thread0\"\n ]\n\nSimilar results with GetSubTree, results not pasted as it\u0027s too long:\ndbus-send --system --print-reply --dest\u003dxyz.openbmc_project.ObjectMapper /xyz/openbmc_project/object_mapper xyz.openbmc_project.ObjectMapper.GetSubTree string:\"/xyz/openbmc_proje/inventory/system/chassis/motherboard/cpu0/core1\" int32:0 array:string:\"xyz.openbmc_project.Inventory.Item.CpuThread\"\n\n-------------\nAlso verified that \"depth\" remained the same using print outs:\n\nBefore change:\nmapperx[12580]: thisPath: /xyz/openbmc_project/sensors/power\nmapperx[12580]: reqPath:\nmapperx[12580]: thisDepth: 4\n\nmapperx[12580]: thisPath: /xyz/openbmc_project/sensors/temperature\nmapperx[12580]: reqPath: /xyz/openbmc_project\nmapperx[12580]: thisDepth: 2\n\nAfter:\nmapperx[30959]: thisPath: /xyz/openbmc_project/network\nmapperx[30959]: reqPath: /\nmapperx[30959]: thisDepth: 3\n\nmapperx[30959]: thisPath: /xyz/openbmc_project/inventory/system/board\nmapperx[30959]: reqPath: /xyz/openbmc_project/inventory/system/\nmapperx[30959]: thisDepth: 1\n\nSigned-off-by: Brandon Kim \u003cbrandonkim@google.com\u003e\nChange-Id: If488402a3839de21c50b4585e51fa156016e768d\n" }, { "commit": "3714dd3058bd383ea80cb06d0a45ea66ed4d9a50", "tree": "9cabf1b2eca9345fa4add8034e53a63fe9b60627", "parents": [ "cc8070bad869fc871bcbd3675d6ba6b32716c7f8" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Thu Aug 04 11:40:12 2022 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Thu Aug 04 11:40:12 2022 -0500" }, "message": "MAINTAINERS: remove file\n\nThe MAINTAINERS file is deprecated in favor of OWNERS.\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: I5bfc6a2f6e7209e00e26ef1b09d610af95e989dc\n" }, { "commit": "cc8070bad869fc871bcbd3675d6ba6b32716c7f8", "tree": "0b9e493e39bfc35d1e93e8bcebfde224cd006afe", "parents": [ "2bd2cc7286e75d2d1637e176af2cc95a1dbbdb44" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Jul 22 19:26:55 2022 -0500" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Mon Jul 25 18:08:20 2022 +0000" }, "message": "sdbusplus: use shorter type aliases\n\nThe sdbusplus headers provide shortened aliases for many types.\nSwitch to using them to provide better code clarity and shorter\nlines. Possible replacements are for:\n * bus_t\n * exception_t\n * manager_t\n * match_t\n * message_t\n * object_t\n * slot_t\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: I1c3acea60ea58b8a0f9a76fd32b5d0d007bf1a16\n" }, { "commit": "2bd2cc7286e75d2d1637e176af2cc95a1dbbdb44", "tree": "5b39b4d5d1de0518d19b456a669877024b972b69", "parents": [ "26ed4a1ee54f77e5476cb8cfa1c1543eb66e51e7" ], "author": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Jul 22 15:58:36 2022 -0500" }, "committer": { "name": "Patrick Williams", "email": "patrick@stwcx.xyz", "time": "Fri Jul 22 15:58:36 2022 -0500" }, "message": "OWNERS: switch \u0027matches\u0027 to \u0027matchers\u0027\n\nThe original OWNERS template had a mistake which used \u0027matches\u0027 instead\nof the field supported by the Gerrit plugin \u0027matchers\u0027. Update the\nOWNERS file to have the correct field.\n\nSigned-off-by: Patrick Williams \u003cpatrick@stwcx.xyz\u003e\nChange-Id: Ie44d548701c9271520a7dc6354c8d8bc8e49b612\n" }, { "commit": "26ed4a1ee54f77e5476cb8cfa1c1543eb66e51e7", "tree": "06ad521def56ce8cb36c50835dc6fc3f02f6021a", "parents": [ "82720f07ac561af6c6ceb29e323f3c59d8effd13" ], "author": { "name": "Ed Tanous", "email": "edtanous@google.com", "time": "Wed Jul 13 13:46:08 2022 -0700" }, "committer": { "name": "Ed Tanous", "email": "edtanous@google.com", "time": "Wed Jul 13 14:04:39 2022 -0700" }, "message": "Add boost wrap file\n\nRecent changes make us incompatible with older versions of boost (1.74\nin my testing). Like we do elsewhere, add a wrapfile for boost.\n\nAt the same time, the tests were missing an explicit #include for boost,\nso even when the above patch was applied, it still doesn\u0027t build. Add\nan explicit dependency.\n\nTested:\nmeson buildlocal --buildtype\u003ddebug \u0026\u0026 ninja -C buildlocal\n\nSucceeds. Should have no impact to yocto builds.\n\nSigned-off-by: Ed Tanous \u003cedtanous@google.com\u003e\nChange-Id: I20543fd184491dc138a2e145945eb71051c4cd52\n" }, { "commit": "82720f07ac561af6c6ceb29e323f3c59d8effd13", "tree": "23eb990bb98e099201d36738b0bd54456c18ba8c", "parents": [ "964681ca5738ec760b9428e5cce5ef3494b10a25" ], "author": { "name": "Brandon Kim", "email": "brandonkim@google.com", "time": "Fri Jul 08 09:57:22 2022 -0700" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Mon Jul 11 20:08:47 2022 +0000" }, "message": "main: Match getSubTrees* to GetAncestors pattern\n\ngetSubTrees, getSubTreePaths and GetAncestors have a code snippet that\u0027s\nvery similar. Unify them so that they are easier to read.\n\nFor the ret vector being declared further down, it\u0027s better practice to\ndeclare variables closer to where it\u0027s going to be used since\nwe wouldn\u0027t have needed ret if we error out early.\n\nTested: The code is only syntatically different. Tested with the\nfollowing commit in more depth.\n\nSigned-off-by: Brandon Kim \u003cbrandonkim@google.com\u003e\nChange-Id: Ib449ab4ffab502c801040cf6bce28b30507d8e0b\n" }, { "commit": "964681ca5738ec760b9428e5cce5ef3494b10a25", "tree": "d79372ff167b9f2ac95e151211ade15fccdf0dc2", "parents": [ "f9d8c18787fafdca34ef5bd625a93f31072fb311" ], "author": { "name": "Ed Tanous", "email": "edtanous@google.com", "time": "Fri Jul 08 12:47:24 2022 -0700" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Mon Jul 11 20:08:40 2022 +0000" }, "message": "Modernize mapper core types\n\nThere\u0027s a number of best practices that have evolved in our use of\nflat_maps and flat_sets since this code was originally written.\n\nFirst, add std::less\u003c\u003e to the Compare template argument. The default\nfor this is std::less\u003cKey\u003e which limits find() calls (and any lookup for\nthat matter) to only supporting std::string. Using std::less\u003c\u003e allows\nlookup by std::string_view, which can prevent some copies in some cases.\n\nNext, add std::vector\u003c...\u003e to the types under the AllocatorOrContainer\ntemplate arg. Per our coding standard, this overrides the default of\nusing boost::vector, and replaces it with std::vector, which, although\nit has a very similar interface, tends to optimize better, and be better\nsupported overall.\n\nThe rest of this patch updates a couple places where the various types\nwere specifically hardcoded, so it moves to using the various using\ndeclarations.\n\nTested: Unit tests; Next patch tests this more fully.\n\nChange-Id: I11e8ecb669f31193c55dda344b25d3fa5d191502\nSigned-off-by: Ed Tanous \u003cedtanous@google.com\u003e\n" }, { "commit": "f9d8c18787fafdca34ef5bd625a93f31072fb311", "tree": "764742c279b00b0aaa6faf9b3f903c29359e0783", "parents": [ "6770a89b6f6af78cdc7b43357fcedc92e8b32612" ], "author": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Fri Jun 03 09:26:47 2022 -0400" }, "committer": { "name": "Matt Spinler", "email": "spinler@us.ibm.com", "time": "Fri Jul 08 21:04:07 2022 +0000" }, "message": "Add myself as a maintainer\n\nI\u0027d like to help out.\n\nSigned-off-by: Brad Bishop \u003cbradleyb@fuzziesquirrel.com\u003e\nChange-Id: Icabec5f97b55cc3bcad4e0fcd271e91120e71c4e\n" }, { "commit": "6770a89b6f6af78cdc7b43357fcedc92e8b32612", "tree": "435a83f99b10820d15c6925db035d44ce1ed1162", "parents": [ "d488775791cd0bb8d47ce5b36ea4090fb7cafca8" ], "author": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Thu Jun 02 21:33:41 2022 -0400" }, "committer": { "name": "Brad Bishop", "email": "bradleyb@fuzziesquirrel.com", "time": "Wed Jun 29 16:24:37 2022 +0000" }, "message": "mapper: meson: minor formatting change\n\nUse a multiline format to facilitate longer descriptions as future\noptions are defined.\n\nSigned-off-by: Brad Bishop \u003cbradleyb@fuzziesquirrel.com\u003e\nChange-Id: I06040a302d291829eed0d514c74ce221de06803f\n" } ], "next": "d488775791cd0bb8d47ce5b36ea4090fb7cafca8" }