Modify

Opened 8 years ago

Closed 3 years ago

#7556 closed enhancement (moved_to_github)

transmission-daemon: open file limit config option

Reported by: dirtyfreebooter <openwrt@…> Owned by: developers
Priority: normal Milestone: Barrier Breaker 14.07
Component: packages Version: Trunk
Keywords: transmission, transmission-daemon Cc:

Description

Transmission has a option open_file_limit but if you want to up that limit to something higher than the default system limit (usually 1024), it will cause the daemon to exit due to it going over the open file limit.

This patch adds a config option to call ulimit before starting the daemon to set the open file limit.

Also, I wanted to normalize these daemon-based options, so I renamed the one just committed in r22065, they all start with `daemon_' now.

For example,

config transmission
	option daemon_user 'transmission'
	option daemon_nofiles 2048
	...

Attachments (2)

transmission-nofiles-config.patch (910 bytes) - added by dirtyfreebooter <openwrt@…> 8 years ago.
transmission-daemon_user_openfiles.patch (1.0 KB) - added by dirtyfreebooter <openwrt@…> 8 years ago.

Download all attachments as: .zip

Change History (10)

Changed 8 years ago by dirtyfreebooter <openwrt@…>

comment:1 Changed 8 years ago by dirtyfreebooter <openwrt@…>

Sorry for extra work. I should have realize this open file limit issue before submitting my earlier ticket.

comment:2 Changed 8 years ago by jow

"nofiles" sounds strange :) What about "numfiles" ?

comment:3 Changed 8 years ago by dirtyfreebooter <openwrt@…>

i was following the ulimit output, ulimit -a uses the text "nofiles", if we are not going to use that, why not "daemon_max_openfiles" or something just as descriptive that says exactly what we are setting?

comment:4 Changed 8 years ago by dirtyfreebooter <openwrt@…>

Also, I am not sure this working as noted. I need to verify the max open file limit set by the init.d script is being carried over to the process started by start-stop-daemon. So please hold off on this until I can verify. Thanks.

comment:5 Changed 8 years ago by dirtyfreebooter <openwrt@…>

I am just confirming that this patch does indeed work as advertised. The problem I was seeing seemed to be related to my USB bus being reset. Feel free to commit and choose the wording that best suits you. I really just wanted to normalize these daemon specific options with the "daemon_" prefix.

comment:6 Changed 8 years ago by dirtyfreebooter <openwrt@…>

Attaching patch to use daemon_user' and daemon_max_openfiles' options in transmission.init

Changed 8 years ago by dirtyfreebooter <openwrt@…>

comment:7 Changed 4 years ago by jow

  • Milestone changed from Attitude Adjustment 12.09 to Barrier Breaker 14.07

Milestone Attitude Adjustment 12.09 deleted

comment:8 Changed 3 years ago by jogo

  • Resolution set to moved_to_github
  • Status changed from new to closed

Transmission is now maintained here: https://github.com/openwrt/packages.

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.