{"action":"created","issue":{"url":"https://api.github.com/repos/openbmc/openbmc/issues/1677","repository_url":"https://api.github.com/repos/openbmc/openbmc","labels_url":"https://api.github.com/repos/openbmc/openbmc/issues/1677/labels{/name}","comments_url":"https://api.github.com/repos/openbmc/openbmc/issues/1677/comments","events_url":"https://api.github.com/repos/openbmc/openbmc/issues/1677/events","html_url":"https://github.com/openbmc/openbmc/issues/1677","id":231436140,"number":1677,"title":"Boot count not getting correctly read and updated by hostboot","user":{"login":"geissonator","id":1612261,"avatar_url":"https://avatars2.githubusercontent.com/u/1612261?v=4","gravatar_id":"","url":"https://api.github.com/users/geissonator","html_url":"https://github.com/geissonator","followers_url":"https://api.github.com/users/geissonator/followers","following_url":"https://api.github.com/users/geissonator/following{/other_user}","gists_url":"https://api.github.com/users/geissonator/gists{/gist_id}","starred_url":"https://api.github.com/users/geissonator/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geissonator/subscriptions","organizations_url":"https://api.github.com/users/geissonator/orgs","repos_url":"https://api.github.com/users/geissonator/repos","events_url":"https://api.github.com/users/geissonator/events{/privacy}","received_events_url":"https://api.github.com/users/geissonator/received_events","type":"User","site_admin":false},"labels":[{"id":260827636,"url":"https://api.github.com/repos/openbmc/openbmc/labels/bug","name":"bug","color":"fc2929","default":true},{"id":650604358,"url":"https://api.github.com/repos/openbmc/openbmc/labels/Phase%205","name":"Phase 5","color":"5319e7","default":false}],"state":"open","locked":false,"assignee":{"login":"tomjoseph83","id":15232150,"avatar_url":"https://avatars3.githubusercontent.com/u/15232150?v=4","gravatar_id":"","url":"https://api.github.com/users/tomjoseph83","html_url":"https://github.com/tomjoseph83","followers_url":"https://api.github.com/users/tomjoseph83/followers","following_url":"https://api.github.com/users/tomjoseph83/following{/other_user}","gists_url":"https://api.github.com/users/tomjoseph83/gists{/gist_id}","starred_url":"https://api.github.com/users/tomjoseph83/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomjoseph83/subscriptions","organizations_url":"https://api.github.com/users/tomjoseph83/orgs","repos_url":"https://api.github.com/users/tomjoseph83/repos","events_url":"https://api.github.com/users/tomjoseph83/events{/privacy}","received_events_url":"https://api.github.com/users/tomjoseph83/received_events","type":"User","site_admin":false},"assignees":[{"login":"tomjoseph83","id":15232150,"avatar_url":"https://avatars3.githubusercontent.com/u/15232150?v=4","gravatar_id":"","url":"https://api.github.com/users/tomjoseph83","html_url":"https://github.com/tomjoseph83","followers_url":"https://api.github.com/users/tomjoseph83/followers","following_url":"https://api.github.com/users/tomjoseph83/following{/other_user}","gists_url":"https://api.github.com/users/tomjoseph83/gists{/gist_id}","starred_url":"https://api.github.com/users/tomjoseph83/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomjoseph83/subscriptions","organizations_url":"https://api.github.com/users/tomjoseph83/orgs","repos_url":"https://api.github.com/users/tomjoseph83/repos","events_url":"https://api.github.com/users/tomjoseph83/events{/privacy}","received_events_url":"https://api.github.com/users/tomjoseph83/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/openbmc/openbmc/milestones/2","html_url":"https://github.com/openbmc/openbmc/milestone/2","labels_url":"https://api.github.com/repos/openbmc/openbmc/milestones/2/labels","id":1678661,"number":2,"title":"openBMC v2.0 Backlog","description":"Backlog for v2.0.","creator":{"login":"williamspatrick","id":105009,"avatar_url":"https://avatars0.githubusercontent.com/u/105009?v=4","gravatar_id":"","url":"https://api.github.com/users/williamspatrick","html_url":"https://github.com/williamspatrick","followers_url":"https://api.github.com/users/williamspatrick/followers","following_url":"https://api.github.com/users/williamspatrick/following{/other_user}","gists_url":"https://api.github.com/users/williamspatrick/gists{/gist_id}","starred_url":"https://api.github.com/users/williamspatrick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/williamspatrick/subscriptions","organizations_url":"https://api.github.com/users/williamspatrick/orgs","repos_url":"https://api.github.com/users/williamspatrick/repos","events_url":"https://api.github.com/users/williamspatrick/events{/privacy}","received_events_url":"https://api.github.com/users/williamspatrick/received_events","type":"User","site_admin":false},"open_issues":246,"closed_issues":642,"state":"open","created_at":"2016-04-01T19:04:51Z","updated_at":"2017-08-07T19:28:39Z","due_on":"2017-10-25T07:00:00Z","closed_at":null},"comments":2,"created_at":"2017-05-25T19:32:14Z","updated_at":"2017-08-07T19:32:04Z","closed_at":null,"body":"I've been doing some SBE seeprom redundant side testing and I found a bug of some sort in the path where hostboot does an SBE update and requests a reboot.\r\n\r\nTheir design is to read the BootCount sensor, increment it by 1, and then set that sensor with the new value.  For some reason, they always get \"0\" back for this sensor when reading from obmc, even when I know for sure it's at 2 or 3.  \r\n\r\nI added some trace on our side to see when it gets set.  This is all we get when they read and set it\r\n```\r\nApr 20 20:13:08 witherspoon systemd[1]: Started Start host0 SBE.\r\nApr 20 20:13:08 witherspoon openpower-proc-control[6572]: rebootObj = :1.156\r\nApr 20 20:13:08 witherspoon openpower-proc-control[6572]: rebootCount=2\r\n…\r\nApr 20 20:16:15 witherspoon ipmid[861]: Bus: :1.156, Path: /org/openbmc/sensors/host/BootCount, Interface: org.openbmc.SensorValue\r\nApr 20 20:16:15 witherspoon sensor_manager2.py[2237]: Setting boot count to 1\r\n```\r\n\r\nHere's the relevant hostboot trace:\r\n```\r\n84.37492|SBE|I>updateProcessorSbeSeeproms(): Restart Needed (1). Calling preReIplCheck()\r\n84.40026|SBE|>>sbeDoReboot\r\n84.50229|IPMI|rp: queuing sync 10:2d\r\n84.50348|IPMI|dd: I>write ok 10:2d seq 9e len 1\r\n84.69492|IPMI|dd: I>read b2h ok 14:2d seq 9e len 4 cc 0\r\n84.69516|SBE|I>sbeDoReboot: Writing Reboot Sensor Count=1\r\n84.69577|IPMI|rp: queuing async 10:30\r\n84.69578|sbe|System Rebooting To Perform SBE Update\r\n```\r\n\r\nSince hostboot sets the correct sensor, it makes me think they must read it correctly which means there's a bug on the obmc side with providing them this sensor value."},"comment":{"url":"https://api.github.com/repos/openbmc/openbmc/issues/comments/320758242","html_url":"https://github.com/openbmc/openbmc/issues/1677#issuecomment-320758242","issue_url":"https://api.github.com/repos/openbmc/openbmc/issues/1677","id":320758242,"user":{"login":"geissonator","id":1612261,"avatar_url":"https://avatars2.githubusercontent.com/u/1612261?v=4","gravatar_id":"","url":"https://api.github.com/users/geissonator","html_url":"https://github.com/geissonator","followers_url":"https://api.github.com/users/geissonator/followers","following_url":"https://api.github.com/users/geissonator/following{/other_user}","gists_url":"https://api.github.com/users/geissonator/gists{/gist_id}","starred_url":"https://api.github.com/users/geissonator/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geissonator/subscriptions","organizations_url":"https://api.github.com/users/geissonator/orgs","repos_url":"https://api.github.com/users/geissonator/repos","events_url":"https://api.github.com/users/geissonator/events{/privacy}","received_events_url":"https://api.github.com/users/geissonator/received_events","type":"User","site_admin":false},"created_at":"2017-08-07T19:32:04Z","updated_at":"2017-08-07T19:32:04Z","body":"New boot count:\r\n```\r\n[witherspoon]# busctl get-property xyz.openbmc_project.State.Host /xyz/openbmc_project/state/host0 xyz.openbmc_project.Control.Boot.RebootAttempts AttemptsLeft\r\nu 3\r\n```"}}
{"actor":{"display_login":"geissonator"}}