Bug 9231 - NT_STATUS_IO_TIMEOUT during slow import of printers into registry
Summary: NT_STATUS_IO_TIMEOUT during slow import of printers into registry
Status: RESOLVED FIXED
Alias: None
Product: Samba 3.6
Classification: Unclassified
Component: Printing (show other bugs)
Version: 3.6.8
Hardware: All All
: P5 normal
Target Milestone: ---
Assignee: Karolin Seeger
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-01 11:46 UTC by Andreas Schneider
Modified: 2012-10-02 06:59 UTC (History)
0 users

See Also:


Attachments
v3-6-test patch (4.93 KB, patch)
2012-10-01 11:46 UTC, Andreas Schneider
jra: review+
asn: review? (ddiss)
Details
v4-0-test patch (1.73 KB, patch)
2012-10-01 11:48 UTC, Andreas Schneider
no flags Details
v4-0-test patch (1.74 KB, patch)
2012-10-01 18:38 UTC, Andreas Schneider
jra: review+
asn: review? (ddiss)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Schneider 2012-10-01 11:46:15 UTC
Importing 200 printers from cups to samba takes lots of time. In the meantime if you try to do a smbclient connection it end with "protocol negotiation failed: NT_STATUS_IO_TIMEOUT"

1) add 200 printers using lpadmin
2) wait for printcap cache to expire
3) try to list the new printers using e.g. smbclient -L
Comment 1 Andreas Schneider 2012-10-01 11:46:54 UTC
Created attachment 7971 [details]
v3-6-test patch
Comment 2 Andreas Schneider 2012-10-01 11:48:58 UTC
Created attachment 7972 [details]
v4-0-test patch
Comment 3 David Disseldorp 2012-10-01 15:25:52 UTC
Thanks for working on this Andreas, the changes look good.

It should also be possible to move the other remaining SIGHUP / MSG_SMB_CONF_UPDATED handler pcap_cache_reload() calls out of smbd and into the background printer process.

Patch to follow...
Comment 4 Andreas Schneider 2012-10-01 16:59:41 UTC
Comment on attachment 7972 [details]
v4-0-test patch

Patch has a bug. Fix will follow soon.
Comment 5 Andreas Schneider 2012-10-01 18:38:45 UTC
Created attachment 7974 [details]
v4-0-test patch
Comment 6 Jeremy Allison 2012-10-01 20:07:57 UTC
Comment on attachment 7971 [details]
v3-6-test patch

Looks good to me.
Jeremy.
Comment 7 Jeremy Allison 2012-10-01 20:12:34 UTC
Comment on attachment 7974 [details]
v4-0-test patch

Looks good to me.
Comment 8 Jeremy Allison 2012-10-01 20:13:08 UTC
Re-assigning to Karolin for inclusion in 4.0.0rc2 and 3.6.next.
Jeremy.
Comment 9 Karolin Seeger 2012-10-02 06:59:38 UTC
Pushed to autobuild-v4-0-test and v3-6-test.
Closing out bug report.

Thanks!