On 08/07/2012 04:44 PM, Alexander Graf wrote: > >> >> Is this the correct place? Who says the caller of hva_to_pfn() is going >> to map it? > > I don't think anyone is. However, we need the struct page, and all the > generic kvm mm code tries hard to hide it from its users. The alternative > would be to expose all those details, and I'm not sure that's a good idea. > > Essentially, we don't care if we're overly cautious. Clearing one page too > much is way better than clearing one too few.
Are you sure everyone uses hva_to_pfn()? x86 uses gfn_to_hva_many(), in one place. -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
