{"action":"closed","issue":{"url":"https://api.github.com/repos/pscedu/slash2/issues/331","labels_url":"https://api.github.com/repos/pscedu/slash2/issues/331/labels{/name}","comments_url":"https://api.github.com/repos/pscedu/slash2/issues/331/comments","events_url":"https://api.github.com/repos/pscedu/slash2/issues/331/events","html_url":"https://github.com/pscedu/slash2/issues/331","id":89864263,"number":331,"title":"IOD: bim_updateseq spin","user":{"login":"slovichon","id":6945653,"avatar_url":"https://avatars.githubusercontent.com/u/6945653?v=3","gravatar_id":"","url":"https://api.github.com/users/slovichon","html_url":"https://github.com/slovichon","followers_url":"https://api.github.com/users/slovichon/followers","following_url":"https://api.github.com/users/slovichon/following{/other_user}","gists_url":"https://api.github.com/users/slovichon/gists{/gist_id}","starred_url":"https://api.github.com/users/slovichon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/slovichon/subscriptions","organizations_url":"https://api.github.com/users/slovichon/orgs","repos_url":"https://api.github.com/users/slovichon/repos","events_url":"https://api.github.com/users/slovichon/events{/privacy}","received_events_url":"https://api.github.com/users/slovichon/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/pscedu/slash2/labels/bug","name":"bug","color":"fc2929"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-06-21T04:43:57Z","updated_at":"2015-06-21T04:43:57Z","closed_at":"2015-06-21T04:43:57Z","body":"\nOn 2013-06-25 12:14:40 -0400, Jared Yanovich wrote:\n\n> ```\nThe IOS can get stuck in a tight loop in bim_updateseq() in certain cases when his minseq is > what the MDS thinks.  I am not sure why this can happen but it can.  Perhaps the MDS restarts, resetting the low watermark, and the client has no leases, so the next (first) lease issued has a seqno < what the IOS has??\n> \n> [1372176709:690807 sliricthr26:2985:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:690819 sliricthr26:2985:def bim_getcurseq bmap_iod.c 142] failed to get bmap seqno rc=1\n> [1372176709:691086 sliricthr05:2964:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:691095 sliricthr05:2964:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:691106 sliricthr05:2964:def bim_getcurseq bmap_iod.c 142] failed to get bmap seqno rc=1\n> [1372176709:691385 sliricthr05:2964:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:691395 sliricthr05:2964:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:691407 sliricthr05:2964:def bim_getcurseq bmap_iod.c 142] failed to get bmap seqno rc=1\n> [1372176709:691660 sliricthr26:2985:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:691670 sliricthr26:2985:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:691683 sliricthr26:2985:def bim_getcurseq bmap_iod.c 142] failed to get bmap seqno rc=1\n> [1372176709:691995 sliricthr26:2985:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:692004 sliricthr26:2985:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:692017 sliricthr26:2985:def bim_getcurseq bmap_iod.c 142] failed to get bmap seqno rc=1\n> [1372176709:692302 sliricthr05:2964:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=467)\n> [1372176709:692312 sliricthr05:2964:def bim_updateseq bmap_iod.c 80] seq 10 is invalid (bim_minseq=46\n```\nOn 2013-07-24 19:49:37 -0400, Jared Yanovich wrote:\n\n> ```\nI think Zhihui has fixed this with a recent patch.\n```\nOn 2013-10-23 18:20:43 -0400, Jared Yanovich wrote:\n\n> ```\nZhihui, can you confirm that your patch has resolved this in theory?  I have no encountered it in practice recently.\n```\nOn 2013-10-24 11:24:17 -0400, Zhihui Zhang wrote:\n\n> ```\nYeah, the flawed logic has been reworked.\n```\n"}}