[ovs-dev] [v2 2/2] ovsdb: Raise the jsonrpc server session limit

Andy Zhou azhou at nicira.com
Tue Feb 24 18:47:10 UTC 2015


Pushed,  added comment about 330 being a random choice.

On Tue, Feb 24, 2015 at 10:12 AM, Andy Zhou <azhou at nicira.com> wrote:
> I missed your comment on v1.  I agree. Will add a comment.
>
> On Tue, Feb 24, 2015 at 10:10 AM, Justin Pettit <jpettit at nicira.com> wrote:
>> No, but since it's a random number, I think it would be good to explain why it was chosen in the source code; otherwise, it looks meaningful.
>>
>> --Justin
>>
>>
>>> On Feb 24, 2015, at 10:08 AM, Andy Zhou <azhou at nicira.com> wrote:
>>>
>>> It is a random number.  Since the goal is to scale from tens of
>>> connections to hundreds of connection, 330 seems like a good first
>>> step.  Is there another number you'd prefer?
>>>
>>> On Tue, Feb 24, 2015 at 10:01 AM, Justin Pettit <jpettit at nicira.com> wrote:
>>>> Can you comment on why 330 was chosen?
>>>>
>>>> --Justin
>>>>
>>>>
>>>>> On Feb 24, 2015, at 9:58 AM, Andy Zhou <azhou at nicira.com> wrote:
>>>>>
>>>>> Raise the connection limit to allow larger number of concurrent
>>>>> ovsdb-server connections. Note, ovsdb-server may not perform well
>>>>> at the new limit. It is rather a prelude to further scaling tests and
>>>>> optimizations.
>>>>>
>>>>> Signed-off-by: Andy Zhou <azhou at nicira.com>
>>>>>
>>>>> ---
>>>>> This limit is currently hard coded. Should we make it a parameter
>>>>> for the caller to set?
>>>>> ---
>>>>> ovsdb/jsonrpc-server.c | 2 +-
>>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/ovsdb/jsonrpc-server.c b/ovsdb/jsonrpc-server.c
>>>>> index 1092ffa..91e24ee 100644
>>>>> --- a/ovsdb/jsonrpc-server.c
>>>>> +++ b/ovsdb/jsonrpc-server.c
>>>>> @@ -121,7 +121,7 @@ ovsdb_jsonrpc_server_create(void)
>>>>> {
>>>>>    struct ovsdb_jsonrpc_server *server = xzalloc(sizeof *server);
>>>>>    ovsdb_server_init(&server->up);
>>>>> -    server->max_sessions = 64;
>>>>> +    server->max_sessions = 330;
>>>>>    shash_init(&server->remotes);
>>>>>    return server;
>>>>> }
>>>>> --
>>>>> 1.9.1
>>>>>
>>>>> _______________________________________________
>>>>> dev mailing list
>>>>> dev at openvswitch.org
>>>>> http://openvswitch.org/mailman/listinfo/dev
>>>>
>>



More information about the dev mailing list