4 Oct
2017
4 Oct
'17
12:15 p.m.
On Wed, Oct 04, 2017 at 11:43:23AM +0100, Chris Wilson wrote:
In hindsight, having both the per-fence and global variable be called info was a mistake. Certainly made this diff a little harder to parse than required!
I agree, I had to re-read the code a few times to find & fix the bug.
Personally I would have set info.status directly rather than add a new fences_status. But that's immaterial to the bug fix,
I can clean that up if there is a need for a v2.
Reviewed-by: Chris Wilson chris@chris-wilson.co.uk
Thanks!
John