On Tue, Apr 26, 2016 at 10:24:44AM +0200, Alexander Gordeev wrote: > On Fri, Apr 22, 2016 at 05:54:12PM +0200, Andrew Jones wrote: > > > -static inline unsigned long virt_to_phys(const void *virt) > > > -{ > > > - return (unsigned long)virt; > > > -} > > > - > > > -static inline void *phys_to_virt(unsigned long phys) > > > -{ > > > - return (void *)phys; > > > -} > > > - > > > > Uh oh. You've now changed x86's virt_to_phys/phys_to_virt to using > > phys_addr_t. The unit tests may not be happy! > > What might go wrong with phys_addr_t? Compilation errors. > > > thanks, > > drew > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html