Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Andrew Cooper
On 20/07/16 14:29, George Dunlap wrote: > On 20/07/16 14:23, Ian Jackson wrote: >> But I think you need to be more careful with your use of language. >> `Disingenuous' implies dishonesty. > The way I've heard it used implies a level of pretense, not necessarily > dishonesty -- i.e., pretending that

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions"): > However, calling what previously exists WRT xenstored stubdomains as > "supported" is laughable. The lack of integration meant that anyone > trying to

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread George Dunlap
On 20/07/16 14:23, Ian Jackson wrote: > But I think you need to be more careful with your use of language. > `Disingenuous' implies dishonesty. The way I've heard it used implies a level of pretense, not necessarily dishonesty -- i.e., pretending that stubdom xenstored was a usable option when it

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Juergen Gross
On 20/07/16 14:32, Andrew Cooper wrote: > On 20/07/16 13:08, Ian Jackson wrote: >> Andrew Cooper writes ("Re: [Xen-devel] [PATCH 1/2] tools: remove systemd >> xenstore socket definitions"): >>> On 20/07/16 12:12, Juergen Gross wrote: >>>> To be clear:

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Juergen Gross
On 20/07/16 14:11, Ian Jackson wrote: > Ross Lagerwall writes ("Re: [Xen-devel] [PATCH 1/2] tools: remove systemd > xenstore socket definitions"): >> Nevertheless, I feel that this patch series makes the implementation >> worse for users of xenstored as a daemon: >

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Andrew Cooper
On 20/07/16 13:08, Ian Jackson wrote: > Andrew Cooper writes ("Re: [Xen-devel] [PATCH 1/2] tools: remove systemd > xenstore socket definitions"): >> On 20/07/16 12:12, Juergen Gross wrote: >>> To be clear: I don't want to avoid systemd by any means. I just

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Juergen Gross
On 20/07/16 14:08, Ian Jackson wrote: > Andrew Cooper writes ("Re: [Xen-devel] [PATCH 1/2] tools: remove systemd > xenstore socket definitions"): >> On 20/07/16 12:12, Juergen Gross wrote: >>> To be clear: I don't want to avoid systemd by any means. I just

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Ian Jackson
Ross Lagerwall writes ("Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions"): > Nevertheless, I feel that this patch series makes the implementation > worse for users of xenstored as a daemon: > > - Because Type=oneshot is not intended to be use

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions"): > On 20/07/16 12:12, Juergen Gross wrote: > > To be clear: I don't want to avoid systemd by any means. I just don't > > want to have a complex and ugly

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Juergen Gross
On 20/07/16 13:21, Andrew Cooper wrote: > On 20/07/16 12:12, Juergen Gross wrote: >> On 20/07/16 12:52, George Dunlap wrote: >>> On Wed, Jul 20, 2016 at 10:58 AM, Juergen Gross wrote: On 20/07/16 11:02, Ross Lagerwall wrote: > On 06/29/2016 02:44 PM, Juergen Gross wrote: >> On 29/06/1

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Andrew Cooper
On 20/07/16 12:12, Juergen Gross wrote: > On 20/07/16 12:52, George Dunlap wrote: >> On Wed, Jul 20, 2016 at 10:58 AM, Juergen Gross wrote: >>> On 20/07/16 11:02, Ross Lagerwall wrote: On 06/29/2016 02:44 PM, Juergen Gross wrote: > On 29/06/16 15:31, Ross Lagerwall wrote: >> On 06/29/

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Juergen Gross
On 20/07/16 12:52, George Dunlap wrote: > On Wed, Jul 20, 2016 at 10:58 AM, Juergen Gross wrote: >> On 20/07/16 11:02, Ross Lagerwall wrote: >>> On 06/29/2016 02:44 PM, Juergen Gross wrote: On 29/06/16 15:31, Ross Lagerwall wrote: > On 06/29/2016 02:00 PM, Juergen Gross wrote: >> On 2

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread George Dunlap
On Wed, Jul 20, 2016 at 10:58 AM, Juergen Gross wrote: > On 20/07/16 11:02, Ross Lagerwall wrote: >> On 06/29/2016 02:44 PM, Juergen Gross wrote: >>> On 29/06/16 15:31, Ross Lagerwall wrote: On 06/29/2016 02:00 PM, Juergen Gross wrote: > On 29/06/16 14:52, Andrew Cooper wrote: >> On 2

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Juergen Gross
On 20/07/16 11:02, Ross Lagerwall wrote: > On 06/29/2016 02:44 PM, Juergen Gross wrote: >> On 29/06/16 15:31, Ross Lagerwall wrote: >>> On 06/29/2016 02:00 PM, Juergen Gross wrote: On 29/06/16 14:52, Andrew Cooper wrote: > On 29/06/16 13:44, Juergen Gross wrote: >> @@ -2068,13 +1964,6

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-20 Thread Ross Lagerwall
On 06/29/2016 02:44 PM, Juergen Gross wrote: On 29/06/16 15:31, Ross Lagerwall wrote: On 06/29/2016 02:00 PM, Juergen Gross wrote: On 29/06/16 14:52, Andrew Cooper wrote: On 29/06/16 13:44, Juergen Gross wrote: @@ -2068,13 +1964,6 @@ int main(int argc, char *argv[]) /* Tell the kernel we

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-13 Thread Juergen Gross
On 29/06/16 15:44, Juergen Gross wrote: > On 29/06/16 15:31, Ross Lagerwall wrote: >> On 06/29/2016 02:00 PM, Juergen Gross wrote: >>> On 29/06/16 14:52, Andrew Cooper wrote: On 29/06/16 13:44, Juergen Gross wrote: > @@ -2068,13 +1964,6 @@ int main(int argc, char *argv[]) > /* Tel

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-08 Thread Wei Liu
On Fri, Jul 08, 2016 at 02:32:31PM +0200, Juergen Gross wrote: > On 08/07/16 14:15, Wei Liu wrote: > > On Wed, Jun 29, 2016 at 03:00:41PM +0200, Juergen Gross wrote: > >> On 29/06/16 14:52, Andrew Cooper wrote: > >>> On 29/06/16 13:44, Juergen Gross wrote: > @@ -2068,13 +1964,6 @@ int main(int

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-08 Thread Juergen Gross
On 08/07/16 14:15, Wei Liu wrote: > On Wed, Jun 29, 2016 at 03:00:41PM +0200, Juergen Gross wrote: >> On 29/06/16 14:52, Andrew Cooper wrote: >>> On 29/06/16 13:44, Juergen Gross wrote: @@ -2068,13 +1964,6 @@ int main(int argc, char *argv[]) /* Tell the kernel we're up and running. */

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-07-08 Thread Wei Liu
On Wed, Jun 29, 2016 at 03:00:41PM +0200, Juergen Gross wrote: > On 29/06/16 14:52, Andrew Cooper wrote: > > On 29/06/16 13:44, Juergen Gross wrote: > >> @@ -2068,13 +1964,6 @@ int main(int argc, char *argv[]) > >>/* Tell the kernel we're up and running. */ > >>xenbus_notify_running(); > >>

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-06-29 Thread Juergen Gross
On 29/06/16 15:31, Ross Lagerwall wrote: > On 06/29/2016 02:00 PM, Juergen Gross wrote: >> On 29/06/16 14:52, Andrew Cooper wrote: >>> On 29/06/16 13:44, Juergen Gross wrote: @@ -2068,13 +1964,6 @@ int main(int argc, char *argv[]) /* Tell the kernel we're up and running. */

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-06-29 Thread Ross Lagerwall
On 06/29/2016 02:00 PM, Juergen Gross wrote: On 29/06/16 14:52, Andrew Cooper wrote: On 29/06/16 13:44, Juergen Gross wrote: @@ -2068,13 +1964,6 @@ int main(int argc, char *argv[]) /* Tell the kernel we're up and running. */ xenbus_notify_running(); -#if defined(XEN_SYSTEMD_ENA

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-06-29 Thread Juergen Gross
On 29/06/16 14:52, Andrew Cooper wrote: > On 29/06/16 13:44, Juergen Gross wrote: >> @@ -2068,13 +1964,6 @@ int main(int argc, char *argv[]) >> /* Tell the kernel we're up and running. */ >> xenbus_notify_running(); >> >> -#if defined(XEN_SYSTEMD_ENABLED) >> -if (systemd) { >> -

Re: [Xen-devel] [PATCH 1/2] tools: remove systemd xenstore socket definitions

2016-06-29 Thread Andrew Cooper
On 29/06/16 13:44, Juergen Gross wrote: > @@ -2068,13 +1964,6 @@ int main(int argc, char *argv[]) > /* Tell the kernel we're up and running. */ > xenbus_notify_running(); > > -#if defined(XEN_SYSTEMD_ENABLED) > - if (systemd) { > - sd_notify(1, "READY=1"); > -