From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: xenbus and the message of doom Date: Mon, 2 Jan 2012 18:16:31 +0100 Message-ID: <20120102171631.GA13603@aepfle.de> References: <4EEA4877.8010307@canonical.com> <20111215193942.GA7640@andromeda.dapyr.net> <20111216113300.GA4854@aepfle.de> <1324375910.23729.31.camel@zakaz.uk.xensource.com> <20111220131533.GA7800@aepfle.de> <20111220141612.GA25139@konrad-lan> <20208.50657.799146.57017@mariner.uk.xensource.com> <1324412394.8252.163.camel@dagon.hellion.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1324412394.8252.163.camel@dagon.hellion.org.uk> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Campbell Cc: Konrad Rzeszutek Wilk , "xen-devel@lists.xensource.com" , Ian Jackson , Stefan Bader , Konrad Rzeszutek Wilk List-Id: xen-devel@lists.xenproject.org On Tue, Dec 20, Ian Campbell wrote: > On Tue, 2011-12-20 at 17:29 +0000, Ian Jackson wrote: > > Konrad Rzeszutek Wilk writes ("Re: [Xen-devel] xenbus and the message of doom"): > > > Sorry Olaf, have to revert that commit. > > > > I agree. When we introduced it we weren't aware that most existing > > implementations of xenstored simply ignore unknown commands rather > > than replying with an error. If we had known this we would not have > > approved Olaf's patch. > > > > That they ignore unknown commands is of course a bug but expecting > > everyone to update is no good. Really the best approach would be some > > kind of discovery mechanism. > > > > Maybe we should have a special path @xenstore/fail_unknown_commands > > which you could read, or something. But this time we should try it > > against old implementations. > > I was sure I'd seen some precedent (and therefore an existing path) for > this sort of thing at some point but I can't for the life of me find it. If you could come up with a patch to present xenstored (or other dom0/toolstack) features, that would be great. Olaf