Re: [Xen-devel] [PATCH v3] tools/xenconsoled: Increase file descriptor limit

2015-02-24 Thread Ian Jackson
Andrew Cooper writes ("Re: [PATCH v3] tools/xenconsoled: Increase file descriptor limit"): > On 19/02/15 16:30, Ian Jackson wrote: > > Why is this only a stopgap solution ? > > It is yet another place with an arbitrary limit, which is one more > moving part to go wrong. OIC. I meant `unlimited'

Re: [Xen-devel] [PATCH v3] tools/xenconsoled: Increase file descriptor limit

2015-02-19 Thread David Vrabel
On 19/02/2015 17:56, Andrew Cooper wrote: [1] along with a todo note suggesting that opening evtchn for each domain is inefficient. The evtchn device has per-fd locks so using only a single fd for all events may scale badly. David ___ Xen-devel

Re: [Xen-devel] [PATCH v3] tools/xenconsoled: Increase file descriptor limit

2015-02-19 Thread Andrew Cooper
On 19/02/15 16:30, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH v3] tools/xenconsoled: Increase file descriptor > limit"): >> On Tue, Feb 17, 2015 at 05:55:52PM +, Andrew Cooper wrote: >>> XenServer's VM density testing uncovered a regression when moving from >>> sysvinit to systemd where

Re: [Xen-devel] [PATCH v3] tools/xenconsoled: Increase file descriptor limit

2015-02-19 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v3] tools/xenconsoled: Increase file descriptor limit"): > On Tue, Feb 17, 2015 at 05:55:52PM +, Andrew Cooper wrote: > > XenServer's VM density testing uncovered a regression when moving from > > sysvinit to systemd where the file descriptor limit dropped from 4096

Re: [Xen-devel] [PATCH v3] tools/xenconsoled: Increase file descriptor limit

2015-02-19 Thread Ian Campbell
On Tue, 2015-02-17 at 17:55 +, Andrew Cooper wrote: > + /* > + * In the likely case that we are a root process with > + * CAP_SYS_RESOURCE, attempt to up our hard limit. > + */ (Nit: also the soft limit...) Acked-by: Ian Campbell

Re: [Xen-devel] [PATCH v3] tools/xenconsoled: Increase file descriptor limit

2015-02-19 Thread Wei Liu
On Tue, Feb 17, 2015 at 05:55:52PM +, Andrew Cooper wrote: > XenServer's VM density testing uncovered a regression when moving from > sysvinit to systemd where the file descriptor limit dropped from 4096 to > 1024. (XenServer had previously inserted a ulimit statement into its > initscripts.) >

Re: [Xen-devel] [PATCH v3] tools/xenconsoled: Increase file descriptor limit

2015-02-17 Thread Don Slutz
On 02/17/15 12:55, Andrew Cooper wrote: > XenServer's VM density testing uncovered a regression when moving from > sysvinit to systemd where the file descriptor limit dropped from 4096 to > 1024. (XenServer had previously inserted a ulimit statement into its > initscripts.) > > One solution is to

[Xen-devel] [PATCH v3] tools/xenconsoled: Increase file descriptor limit

2015-02-17 Thread Andrew Cooper
XenServer's VM density testing uncovered a regression when moving from sysvinit to systemd where the file descriptor limit dropped from 4096 to 1024. (XenServer had previously inserted a ulimit statement into its initscripts.) One solution is to use LimitNOFILE=4096 in xenconsoled.service to match