Re: [PATCH] libxl: Use xenbus to communicate with xenstore if the socket fails

[Vincent Hanquez <vincent.hanquez@eu.citrix.com>]

On 09/12/10 09:17, Ian Campbell wrote:
> On Wed, 2010-12-08 at 20:47 +0000, Mihir Nanavati wrote:
>> Adds an open xenstore connection function which tries to use the
>> xenbus interface (xs_domain_open) when the socket interface
>> (xs_daemon_opn) fails.
>
> I like the concept of this patch.
>
> I can't think of any reason why the callers of libxenstore should need
> to care about how which communication channel gets used so there's no
> reason for the library to defer that choice to the caller. So perhaps we
> should go one step further push this behaviour down into libxenstore
> itself? i.e. add "xs_open(int ro)" and make
> xs_{daemon,domain}_open{,_readonly} compatibility aliases to that
> function.

Unfortunately, there is practical difference between the socket and the 
page interface. Which is the reason why every xs connection in xapi are 
sockets; IIRC polling for events on the page interface didn't really 
work; I think there was at least 1 other practical differences, but it 
seems to have eluded me.

-- 
Vincent