---- Original Message ----
> Add a command queue for the main thread to pass commands to the
> sessions thread.
>
> Defined an implemented a set of IPC messages for creating,
> closing and managing jsonrpc sessions from the main process to jsonrpc
> sessions threads.
>
> Add a doorbell mechanism for a jsonrpc session to inform other sessions
> of possible OVSDB changes.
>
> Although all supporting functions for running a multithreading OVSDB
> server have been implemented, multithread is still not turned on,
> because 'n_max_threads' is still hard coded to zero. Future patches
> will remove this restriction.
I think there is a small typo in the first line of the second para:
Did you mean "Defined and implemented"?
[snip...]
> @@ -115,13 +116,112 @@ static struct json
*ovsdb_jsonrpc_monitor_compose_update(
> struct sessions_thread {
> pthread_t thread;
>
> + struct ovs_list all_sessions;
> +
This is a nit, but I'd prefer if this were "thread_sessions" or
"thread_all_sessions" to avoid potential confusion when reading
the code...
Otherwise...
Acked-by: Ryan Moats <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
