Ticket #116 (new defect)

Opened 15 years ago

Last modified 10 years ago

savannah: infinite loop reading large directories via fish

Reported by: mlo Owned by:
Priority: major Milestone: Future Releases
Component: mc-vfs Version: master
Keywords: Cc: zaytsev
Blocked By: Blocking:
Branch state: no branch Votes for changeset:

Description (last modified by ossi) (diff)

Original: http://savannah.gnu.org/bugs/?15801

Submitted by:Mario Lorenz <mlo>Submitted on:Sun 19 Feb 2006 12:15:18 PM UTC
Category:VFSSeverity:3 - Normal
Status:In ProgressPrivacy:Public
Assigned to:Pavel Tsekov <ptsekov>Open/Closed:Open
Release:4.6.1Operating System:GNU/Linux

Original submission:

Reading large remote directories via fish (shell link)
over slow network links causes an infinite or at least very long
loop when mc tries to read the directory multiple times.

This is due to the fish directory timeout being hardcoded to 10 seconds,
whereas reading a 15000 entry directory via a 64kbit/s link will take
two minutes (way longer if not using compression). This means the
directory objects will be marked obsolete before the directory is even
loaded, causing an immediate reload once finished, with this pattern
sometimes repeating even more often.

That timeout should be tied to the (user settable) ftp directory 
timeout, or be given its own user settable value; at the very least it
should be set to a sane value (that is, >> 10 seconds).

Comment 1 by Pavel Tsekov <ptsekov> at Thu 23 Feb 2006 03:38:12 PM UTC:

This problem has been bugging me for a while. I've just commited a patch 
which exposes a new user configurable option:

fish_directory_timeout

It contains the lifetime of a directory cache entry measured in seconds. 
I've adjusted the default value to 900 seconds (same as in ftpfs).

This option is not configurable through the user interface, yet - one 
can change it only by directly editing MC's ini file. I plan to fix this
soon.

To test the new code you need to fetch MC from the cvs repository or 
grab a snapshot.

Change History

comment:1 Changed 15 years ago by styx

  • Milestone set to future releases

comment:2 Changed 14 years ago by zaytsev

  • severity set to no branch
  • Description modified (diff)

comment:3 Changed 14 years ago by zaytsev

  • Description modified (diff)

comment:4 Changed 14 years ago by zaytsev

  • Cc zaytsev added
  • Version changed from 4.6.1 to master
  • Milestone changed from Future Releases to 4.7

Now this patch is in the mainline and it's set to 15 minutes by default. What is this ticket all about then? An option exposed into the configuration dialog?

comment:5 Changed 12 years ago by andrew_b

  • Branch state set to no branch
  • Milestone changed from 4.7 to Future Releases

comment:6 Changed 10 years ago by ossi

  • Description modified (diff)
  • Reporter changed from slavazanko to mlo
Note: See TracTickets for help on using tickets.