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

Re: [Xen-devel] [PATCH RFC 1/7] xen/x86: Add mbd.h header file

On Wed, Sep 03, 2014 at 03:11:18PM +0100, Ian Campbell wrote:
> On Mon, 2014-08-11 at 17:16 +0100, Stefano Stabellini wrote:
> > Also in the interest of code sharing across multiple arches, it seems to
> > me that these struct should be common code.
> I don't intend to block the existing arm64 UEFI work for not using this
> new infrastructure, so any code sharing common stuff would have to come
> later.
> Whether it makes sense to preemptively make this new stuff common I'm
> not sure. As it stands the mem_* and mmap(_size) fields aren't really
> architecture agnostic.

I thought about sharing xbi_t instead of mbd_t (please look into patch 2 in
this series). xbi_t could contain arch member (I am not sure about that yet;
Currently my patches use idea similar to page_info struct - I prefer this
solution) which would be used to store arch specific stuff. Fortunately as
I can see only arch specific code touches arch specific members. So it looks
that xbi_t with common members among archs should work on any architecture.


Xen-devel mailing list



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