From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Cooper Subject: Re: [PATCH] tools/many: Fix Generation ID restore interface. Date: Tue, 26 Nov 2013 10:51:28 +0000 Message-ID: <52947D30.3090100@citrix.com> References: <1385412035-6938-1-git-send-email-andrew.cooper3@citrix.com> <1385460681.20703.21.camel@kazak.uk.xensource.com> <52947AF1.5070801@citrix.com> <1385462748.23112.16.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1385462748.23112.16.camel@kazak.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: George Dunlap , Ian Jackson , Xen-devel List-Id: xen-devel@lists.xenproject.org On 26/11/13 10:45, Ian Campbell wrote: > On Tue, 2013-11-26 at 10:41 +0000, Andrew Cooper wrote: >> On 26/11/13 10:11, Ian Campbell wrote: >>> On Mon, 2013-11-25 at 20:40 +0000, Andrew Cooper wrote: >>>> The original Generation ID code was submitted before the specification had >>>> been finalised, and changed completely between the final draft and formal >>>> release. >>>> >>>> Most notably, the size of the Generation ID has doubled to 128 bits, and >>>> changing it now involves writing a new cryptographically random number in the >>>> appropriate location, rather than simply incrementing it. >>>> >>>> The xc_domain_save() side of the code is fine, but the xc_domain_restore() >>>> needs substantial changes to be usable. >>>> >>>> This patch replaces the old xc_domain_restore() parameters with a new optional >>>> restore_callback. If the callback is provided, and an appropriate hunk is >>>> found in the migration stream, the appropriate piece of guest memory is mapped >>>> and provided to the callback. >>>> >>>> This patch also fixes all the in-tree callers of xc_domain_restore(). All >>>> callers had the functionality disabled, and never exposed in a public way. >>>> >>>> Signed-off-by: Andrew Cooper >>>> CC: Ian Campbell >>>> CC: Ian Jackson >>>> CC: George Dunlap >>>> >>>> --- >>>> >>>> George: >>>> >>>> Despite this being a feature, I am requesting a freeze exemption. It is >>>> functionality which was not used (or indeed useful) before, and remains that >>>> way as far as in-tree consumers are concerned. >>> If it is both wrong and neither used nor useful why aren't we ripping >>> it out? >> The implementation in the tree currently is neither useful nor used. It >> is hardwired to disabled for each intree caller, which is why this >> change wont affect any current functionality in tree. > That doesn't answer my question. Well I am ripping out the old interface. It is at the same time as providing a new sane interface to be used. Just because there are no intree users doesn't mean there are no users. Part of moving Xapi to libxl will involve plumbing all this stuff into libxl. ~Andrew