Project

General

Profile

Actions

Feature #2836

closed

Remove check for broken sendfile implementations

Added by stbuehler over 6 years ago. Updated over 6 years ago.

Status:
Fixed
Priority:
Normal
Category:
build_autotools
Target version:
ASK QUESTIONS IN Forums:

Description

Since svn|r1 autobuild (back then source:svn|configure.in@1#L301, now configure.ac) checks for broken sendfile implementations (HAVE_SENDFILE_BROKEN), and assumes sendfile is always broken when cross compiling.

Imho it's probably safe to drop the check completely, but we could at least assume working sendfile when cross compiling. Users can manually select a server.network-backend in the config anyway.

Actions #1

Updated by gstrauss over 6 years ago

Makes sense. If sendfile() fails due to not being implemented, we fall back to other mechanisms (writev() or write()).

However, if sendfile is present but broken, then server.network-backend must be explicitly specified in config file to something else, since sendfile is the default if available and server.network-backend is not specified.

Actions #2

Updated by gstrauss over 6 years ago

  • Status changed from New to Patch Pending
  • Target version changed from 1.4.x to 1.4.48
Actions #3

Updated by stbuehler over 6 years ago

  • Description updated (diff)
Actions #4

Updated by gstrauss over 6 years ago

  • Status changed from Patch Pending to Fixed
  • % Done changed from 0 to 100
Actions

Also available in: Atom