>>> On 16.02.16 at 12:02, <paul.durr...@citrix.com> wrote: >> From: Ian Campbell [mailto:ian.campb...@citrix.com] >> Sent: 16 February 2016 10:23 >> On Mon, 2016-02-15 at 11:14 +0000, Paul Durrant wrote: >> > + */ >> > +#ifdef NETIF_DEFINE_TOEPLITZ >> >> If we go with this then this should have an addtional XEN_ on the >> front. > > The header is inconsistent at the moment. Some things are prefixed with XEN_ > some are not so if you want this prefixed then I think it's best I add > another patch before this to change all unqualified netif/NETIF occurrences > to xen_netif/XEN_NETIF... it will also mean less post-processing when I > re-import the header into Linux.
You'd need to be rather careful here: Any such identifiers which were there already in 4.6 (or any other release) would have to remain unchanged. For any new ones adding prefixes would indeed seem very desirable. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel