On Wed, Sep 25, 2019 at 08:16:16AM -0700, Jerry Snitselaar wrote: > On Wed Sep 25 19, Jarkko Sakkinen wrote: > > On Wed, Sep 25, 2019 at 12:25:05PM +0200, Ard Biesheuvel wrote: > > > On Wed, 25 Sep 2019 at 12:16, Jarkko Sakkinen > > > <[email protected]> wrote: > > > > > > > > From: Peter Jones <[email protected]> > > > > > > > > Some machines generate a lot of event log entries. When we're > > > > iterating over them, the code removes the old mapping and adds a > > > > new one, so once we cross the page boundary we're unmapping the page > > > > with the count on it. Hilarity ensues. > > > > > > > > This patch keeps the info from the header in local variables so we don't > > > > need to access that page again or keep track of if it's mapped. > > > > > > > > Fixes: 44038bc514a2 ("tpm: Abstract crypto agile event size > > > > calculations") > > > > Cc: [email protected] > > > > Cc: [email protected] > > > > Cc: [email protected] > > > > Signed-off-by: Peter Jones <[email protected]> > > > > Tested-by: Lyude Paul <[email protected]> > > > > Reviewed-by: Jarkko Sakkinen <[email protected]> > > > > Acked-by: Matthew Garrett <[email protected]> > > > > Acked-by: Ard Biesheuvel <[email protected]> > > > > Signed-off-by: Jarkko Sakkinen <[email protected]> > > > > > > Thanks Jarkko. > > > > > > Shall I take these through the EFI tree? > > > > Would be great, if you could because I already sent one PR with fixes for > > v5.4-rc1 yesterday. > > > > /Jarkko > > My patch collides with this, so I will submit a v3 that applies on top of > these once I've run a test with all 3 applied on this t480s.
Great, thanks. /Jarkko
