[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-ia64-devel] name conflict(map_domain_page)



Hi, Yamahata and all

I think good sugesstion.
Should map_new_domain0_page()  also be renamed?
How are alloc_xxxx?
(I'm also bad at naming...)

Best Regards,

Akio Takebe

>
>I found that name map_domain_page is defined in two places.
>one is defined in xen/include/xen/domain_page.h.
>another is defined in xen/arch/ia64/xen/domain.c.
>Luckily this name conflict doesn't cause problem for now.
>However this is a bad practice, so it should be fixed.
>The one in xen/include/xen/domain_page.h is used in xen commonly,
>so the one defined in xen/arch/ia64/xen/domain.c should be renamed.
>
>I propose following name changes.
>(Only the name map_domain_page() is conflicted, 
>For consistency others should also be changed.)
>
>map_new_domain_page()  -> assign_new_domain_page()
>map_domain_page()      -> assign_domain_page()
>map_domain_io_page()   -> assign_domain_io_page()
>
>
>I am bad at naming, I hope someone propose better names.
>Any suggenstions?
>Once new names are decided, I will make a patch.
>Or if no objections, I will go with above name.
>
>-- 
>yamahata
>
>_______________________________________________
>Xen-ia64-devel mailing list
>Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
>http://lists.xensource.com/xen-ia64-devel


_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.