[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.02.1312111618120.7093@kaball.uk.xensource.com>
Date: Wed, 11 Dec 2013 16:18:36 +0000
From: Stefano Stabellini <stefano.stabellini@...citrix.com>
To: Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>
CC: Ian Campbell <Ian.Campbell@...rix.com>,
David Vrabel <david.vrabel@...rix.com>,
Stefano Stabellini <stefano.stabellini@...citrix.com>,
Julien Grall <julien.grall@...aro.org>,
<linux-kernel@...r.kernel.org>, <xen-devel@...ts.xenproject.org>,
"Boris Ostrovsky" <boris.ostrovsky@...cle.com>,
Roger Pau Monne <roger.pau@...rix.com>
Subject: Re: [Xen-devel] [PATCH RFC] xen-block: correctly define structures
in public headers
On Tue, 3 Dec 2013, Konrad Rzeszutek Wilk wrote:
> > > If Konrad and Boris agree that breaking the kernel's ABI in this way is
> > > acceptable in this specific case, I'll defer to them.
> >
> > My opinion as Xen on ARM hypervisor maintainer is that this is the right
> > thing to do in this case.
>
> Heh. If somebody can guarantee me that (by testing the right variants and
> mentioning this in the git commit) that this does not break x86, then
> I am fine.
>
> And by 'break x86' I mean that this combination works:
> 32-bit domU on 64-bit dom0
> 64-bit domU on 32-bit dom0
>
> And perhaps also the obvious:
> 64-bit domU on 64-bit dom0
> 32-bit domU on 32-bit dom0
>
> Since the xen-blkback has its own version of the structs there is no
> need to change change newer and older version of it.
>
> As long as that works I am OK sticking it in.
>
> I think from the ARM perspective it is still in 'experimental' phase
> so anything goes to make it work under ARM.
Roger, can you please test this patch on x86 as suggested by Konrad and
confirm that it doesn't break anything?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists