> Nit: It seems like there's a redundant call to frame_id_eq in frame_find_by_id.

Ooops, you're right. Thanks for catching this!  Here is a new patch.
I used "frame_stash" to avoid the confusion with the "cache" terminology
used in that unit.

Currently testing on x86_64-linux...

-- 
Joel