On Fri, Apr 18, 2014 at 9:55 PM, David Herrmann <dh.herrm...@gmail.com> wrote:
> On Fri, Apr 18, 2014 at 3:16 AM, Djalal Harouni <tix...@opendz.org> wrote:
>> Take the write lock in kdbus_name_release() instead of
>> kdbus_cmd_name_release() in order to reduce the lock hold time.
>>
>> This change permits to convert the kdbus_bus_find_conn_by_id() call to
>> kdbus_conn_find_peer() since the bus lock will also be taken later in
>> kdbus_name_release().
>>
>> Another advantage is that now kdbus_cmd_name_release() and
>> kdbus_name_release() have the same semantic of kdbus_cmd_name_acquire()
>> and kdbus_name_acquire()
>>
>> Signed-off-by: Djalal Harouni <tix...@opendz.org>
>
> Looks good to me.

Applied.

Thanks,
Kay
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to