printing: traverse_read the printer list for share updates
commit83f448d7f3e6cce42a4b3070f45c84b85ec23ec0
authorDavid Disseldorp <ddiss@samba.org>
Wed, 9 Jul 2014 22:18:10 +0000 (10 00:18 +0200)
committerKarolin Seeger <kseeger@samba.org>
Mon, 1 Sep 2014 19:34:11 +0000 (1 21:34 +0200)
tree4d5f941ddeb7cab8e13fd8efdae0c200308a99e2
parentb95dbbeab9c03b354d9c8cff9ebb9f66411af851
printing: traverse_read the printer list for share updates

The printcap update procedure involves the background printer process
obtaining the printcap information from the printing backend, writing
this to printer_list.tdb, and then notifying all smbd processes of the
new list. The processes then all attempt to simultaneously traverse
printer_list.tdb, in order to update their local share lists.

With a large number of printers, and a large number of per-client smbd
processes, this traversal results in significant lock contention, mostly
due to the fact that the traversal is unnecessarily done with an
exclusive (write) lock on the printer_list.tdb database.

This commit changes the share update code path to perform a read-only
traversal.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10652

Reported-by: Alex K <korobkin+samba@gmail.com>
Reported-by: Franz Pförtsch <franz.pfoertsch@brose.com>
Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
(cherry picked from commit 1e83435eac2cef03fccb4cf69ef5e0bfbd710410)
source3/printing/load.c
source3/printing/pcap.c
source3/printing/pcap.h
source3/printing/printer_list.c
source3/printing/printer_list.h