[ovs-git] [openvswitch/ovs] 7a3608: ovsdb-idl: Remove 'uuid' member of struct ovsdb_id...

GitHub noreply at github.com
Mon Dec 11 22:45:35 UTC 2017


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 7a3608245c0b68e4e449fe05ac7ebb774911235b
      https://github.com/openvswitch/ovs/commit/7a3608245c0b68e4e449fe05ac7ebb774911235b
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-12-11 (Mon, 11 Dec 2017)

  Changed paths:
    M lib/ovsdb-idl.c

  Log Message:
  -----------
  ovsdb-idl: Remove 'uuid' member of struct ovsdb_idl.

This was used to uniquely identify the monitor, but there's no need for
that.  A fixed monitor name works fine.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Alin Gabriel Serdean <aserdean at ovn.org>


  Commit: 25540a777ff6c81ff71ace04ffabfd0df93e5e2d
      https://github.com/openvswitch/ovs/commit/25540a777ff6c81ff71ace04ffabfd0df93e5e2d
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-12-11 (Mon, 11 Dec 2017)

  Changed paths:
    M lib/ovsdb-idl-provider.h
    M lib/ovsdb-idl.c
    M ovsdb/ovsdb-idlc.in

  Log Message:
  -----------
  ovsdb-idl: Tolerate initialization races for singleton tables.

By verifying that singleton tables (that is, tables that should have exactly
one row) are empty when they emit transactions that insert into them,
ovs-vsctl and similar tools tolerate initialization races, where more than one
client at a time tries to initialize a singleton table.

The upshot is that if you create a database and then run multiple ovs-vsctl
(etc.) commands against it in parallel (without first initializing it
serially), then without this patch sometimes you will sometimes get failures
but this patch avoids them.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Alin Gabriel Serdean <aserdean at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/fca1b42abe3e...25540a777ff6


More information about the git mailing list