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
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

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!