OpenBMC Phosphor IPMI daemon module loader sources https://git.raptorcs.com/git/phosphor-ipmi-blobs/atom?h=master 2019-11-25T18:06:06+00:00 2019-11-25T18:06:06+00:00 Kun Yi kunyi731@gmail.com 2019-11-22T22:59:40+00:00 urn:sha1:cd833aa11075ac1041531270060e1fb3ba8aa5ac The old code deletes session from the session map before doing a lookup of the path. Fix the order and add a unit test to catch the error case. (The unit test would fail without this fix). Signed-off-by: Kun Yi <kunyi731@gmail.com> Change-Id: If043d6aad7bbaafa19ae3b63a6d9cc7a836d25b5 2019-11-21T18:06:48+00:00 Kun Yi kunyi731@gmail.com 2019-11-14T21:12:42+00:00 urn:sha1:aa17bdaf3e34ce3a876dfb828ace2ec2c635e5e5 Add simple tests to ensure the session expire method is called when the timeout is set to zero. Signed-off-by: Kun Yi <kunyi731@gmail.com> Change-Id: Ic6e973ede760a6a6f15726d5b6c93a892d858c8b 2019-06-17T18:51:19+00:00 Patrick Venture venture@google.com 2019-06-04T14:20:24+00:00 urn:sha1:8bc117792fbf118dd74d015546c22612961ccc26 Because this structure is not anything special, simply use the cpp idiomatic way of referencing the object. Consistently, structures that are packed still have "struct" used in declarations. However, this distinction is only clear through its usage and nothing in the language. Perhaps a better approach would be notational naming struct PackedXYZ {};. However, that can get out of control quickly. Signed-off-by: Patrick Venture <venture@google.com> Change-Id: I3b7e48e4b6687ef2e15e9d07c0eeba96eb5d2552 2019-05-20T20:39:15+00:00 Patrick Venture venture@google.com 2019-05-18T02:21:35+00:00 urn:sha1:00d5f0de56f3e453be37c2ffc45418fb406396a2 Signed-off-by: Patrick Venture <venture@google.com> Change-Id: Ic5aa3e38b4df33a2cfa29c575f17097e38d06605 2019-03-19T15:27:16+00:00 Patrick Venture venture@google.com 2019-03-19T14:36:44+00:00 urn:sha1:924bc380c47f33689a1243b2d79cb619674c86fa Split the object into two pieces so we can pass part of it to the unit-tests. Tested: Verified the coverage output processed the files. Change-Id: Idf1bdf9d21eb1324395eb9a2c09ff8f1b4ecad6b Signed-off-by: Patrick Venture <venture@google.com> 2019-03-19T14:29:29+00:00 William A. Kennington III wak@google.com 2019-03-15T21:14:07+00:00 urn:sha1:374350d7446aa56c57bc9f92c58fd946bc12da5b The code coverage macros from the archive changed in a backward incompatible way. This adds a workaround to autodetect either version and do the right thing. Change-Id: I09b88e6b4792c99140a73a4ceb11d2896ee13562 Signed-off-by: William A. Kennington III <wak@google.com> 2019-03-08T21:10:01+00:00 Patrick Venture venture@google.com 2019-03-07T20:48:32+00:00 urn:sha1:de8a16e2e85f49e142461b7e6fe901d3f8740fdf Drop all code that is now handled by the ipmiblob library provided by the new ipmi-blob-tool. This is a library that can be included on the BMC if necessary, but relies on nothing that is strictly meant for the BMC. Change-Id: I2b02ae0d432e84c08e598d27eef85b57c06a70fc Signed-off-by: Patrick Venture <venture@google.com> 2019-03-08T21:09:37+00:00 Patrick Venture venture@google.com 2019-03-08T21:08:42+00:00 urn:sha1:48eb4029694c228ee8bb09202efb9d6c8b741d96 Use filesystem now that it should be available instead of experimental::filesystem. Change-Id: Ia9b17f33ced70fe2f1f12f11d8708d15ff422bfa Signed-off-by: Patrick Venture <venture@google.com> 2019-03-07T15:38:28+00:00 Patrick Venture venture@google.com 2019-03-07T15:38:28+00:00 urn:sha1:fd4cdea449f01b910b654b78f2076cf76540619c Enable valgrind tests. Change-Id: Ib5ec22a19c043ef45f5274e4716e2abbae9c19da Signed-off-by: Patrick Venture <venture@google.com> 2019-03-07T15:33:09+00:00 Patrick Venture venture@google.com 2019-03-06T21:23:42+00:00 urn:sha1:a2efc76b21b6a0349850c9ce3a0e25f1dc4d9c0d Enable code coverage for the project. Valgrind not enabled in this step. Change-Id: I439d58b23d2bc29d6f602600319074c8fd8c04d6 Signed-off-by: Patrick Venture <venture@google.com>