Question on save/restore mfn canonicalization

[Andres Lagar-Cavilla <andreslc@cs.toronto.edu>]

While exploring the paravirt save/restore code path, I noted a 
(seemingly) lack of consistency between where/who canonicalizes 
something and who un-canonicalizes. For example, the guest kernel 
canonicalizes the store and console mfn's in pre_suspend, but 
xc_domain_restore uncanonicalizes them before scheduling back the 
restored guest.
The question is if there is a mandatory reason for this, or is just the 
way the code was written. Can I, e.g, fill the 
pfn_to_mfn_frame_list(_list) entries from "outside" and remove that code 
from post_suspend, or will something break?

Thanks!
Andres