On Mon, 18 Apr 2005, Chris Feist wrote:
I've attached the redhat patch which should fix this problem as well. (And we've tested it and it seems to work pretty well).
Thanks, Chris
[EMAIL PROTECTED] wrote:On Sun, 17 Apr 2005, Daniel Andre Vaquero wrote:
> Hi,
> > My editor broke the indentation style in the patch; I'm resending it as
> an attached file.
Will check out the patch.
Thanks for the feedback.
Hi guys,
I've reviewed the two patches submitted regarding this problem.
Thanks for your efforts Daniel but the patch that Chris has submitted covers the issue that you reported and addresses a couple of other things as well.
Thanks Chris, I'll merge the patch and copy it up to kernel.org.
There was however one thing that Daniel pointed out that I added to your patch (hunk below). That is the selection of the first multi-mount entry if all else fails. I know that this was changed due to excessive mount delays but as your patch avoids the ping altogether for single entries and the doco says it behaves this way I expect it will be OK. If not I'll revert it.
Ian
--- autofs-4.1.4/modules/mount_nfs.c.select-first 2005-04-24 16:51:04.000000000 +0800 +++ autofs-4.1.4/modules/mount_nfs.c 2005-04-24 16:52:01.000000000 +0800 @@ -341,11 +341,9 @@ int get_best_mount(char *what, const cha } }
- /* No winner found so bail */ - if (!winner) { - *what = '\0'; - return 0; - } + /* No winner found so return first */ + if (!winner) + winner = what;
/* * We now have our winner, copy it to the front of the string,
_______________________________________________ autofs mailing list autofs@linux.kernel.org http://linux.kernel.org/mailman/listinfo/autofs