Re: [Devel] [PATCH 3/3] hooks_ct: fix ub limits setting for upstream containers

2013-05-13 Thread Glauber Costa
On 05/11/2013 05:07 AM, Igor M Podlesny wrote: On 30 April 2013 11:46, Glauber Costa glom...@openvz.org wrote: Currently, vps_setup_res() have an explicit test for state != VPS_STARTING before applying beancounter limits. This means that we can set limits without further problems when the

Re: [Devel] [PATCH 3/3] hooks_ct: fix ub limits setting for upstream containers

2013-05-13 Thread Glauber Costa
On 05/13/2013 12:11 PM, Igor M Podlesny wrote: On 13 May 2013 16:11, Glauber Costa glom...@parallels.com wrote: On 05/13/2013 12:08 PM, Igor M Podlesny wrote: On 13 May 2013 15:50, Glauber Costa glom...@parallels.com wrote: [...] Aren't macroses supposed to be UPPER CASE named? Yes, and

Re: [Devel] [PATCH 3/3] hooks_ct: fix ub limits setting for upstream containers

2013-05-13 Thread Igor M Podlesny
On 13 May 2013 16:16, Glauber Costa glom...@parallels.com wrote: On 05/13/2013 12:11 PM, Igor M Podlesny wrote: On 13 May 2013 16:11, Glauber Costa glom...@parallels.com wrote: On 05/13/2013 12:08 PM, Igor M Podlesny wrote: On 13 May 2013 15:50, Glauber Costa glom...@parallels.com wrote:

[Devel] [PATCH 3/3] hooks_ct: fix ub limits setting for upstream containers

2013-05-10 Thread Glauber Costa
Currently, vps_setup_res() have an explicit test for state != VPS_STARTING before applying beancounter limits. This means that we can set limits without further problems when the container is running, but will fail to do so when the container is starting. In fact, hooks_vz have an explicit call

Re: [Devel] [PATCH 3/3] hooks_ct: fix ub limits setting for upstream containers

2013-05-10 Thread Igor M Podlesny
On 30 April 2013 11:46, Glauber Costa glom...@openvz.org wrote: Currently, vps_setup_res() have an explicit test for state != VPS_STARTING before applying beancounter limits. This means that we can set limits without further problems when the container is running, but will fail to do so when