Hi Tyler, On 11/01/2019 20:53, Tyler Baicar wrote: > On Fri, Jan 11, 2019 at 1:09 PM James Morse <james.morse@xxxxxxx> wrote: >> We can return on ENOENT out earlier, as nothing needs doing in that case. Its >> what the GHES_TO_CLEAR spaghetti is for, we can probably move the ack thing into >> ghes_clear_estatus(), that way that thing means 'I'm done with this memory'. >> >> Something like: >> ------------------------- >> rc = ghes_read_estatus(); >> if (rc == -ENOENT) >> return 0; > > We still should be returning at least the -ENOENT from ghes_read_estatus(). > That is being used by the SEA handling to determine if an SEA was properly > reported/handled by the host kernel in the KVM SEA case. Sorry, my terrible example code. You'll be glad to know I would have caught this when testing it! Thanks, James
