Re: [Xen-devel] [PATCH 03/16] tmem: cleanup: rm unused tmem_op

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2013-11-25 at 11:37 -0500, Konrad Rzeszutek Wilk wrote:
> On Mon, Nov 25, 2013 at 09:52:16AM +0000, Ian Campbell wrote:
> > On Mon, 2013-11-25 at 09:43 +0000, Jan Beulich wrote:
> > > >>> On 22.11.13 at 18:38, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> 
> > > >>> wrote:
> > > > On Wed, Nov 20, 2013 at 04:46:12PM +0800, Bob Liu wrote:
> > > >> TMEM_READ/TMEM_WRITE/TMEM_XCHG/TMEM_NEW_PAGE are never be used.
> > > > 
> > > > You also need to delete them in include/public/tmem.h and rev up the
> > > > TMEM_SPEC_VERSION I think?
> > > 
> > > I said this elsewhere already - you can't simply delete things
> > > from the public interface.
> > 
> > Should 006a687ba4de74d7933c09b43872abc19f126c63 be partially reverted
> > then, to reinstate the typedef under a suitable #ifdef
> > __XEN_INTERFACE_VERSION__ ?
> 
> I think that is OK - the typedef was of 'tmem_cli_mfn_t' which
> was 'xen_pfn_t' based. And the struct that got changed is now
> using 'xen_pfn_t' instead of 'tmem_cli_mfn_t'.

It's not OK for any existing code using the heads and the old name
though, since it will fail to build.

We normally try and care about such things. As TMEM maintainer are you
telling us not to bother this time?

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel