Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.1.0
    • Fix Version/s: 3.1.1
    • Component/s: Network
    • Labels:
      None

      Description

      Add support for accept filters (Solaris) triggered by proxy.config.net.defer_accept

      1. accept_filter.patch
        2 kB
        Theo Schlossnagle

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        13s 1 Theo Schlossnagle 03/Sep/11 03:10
        Patch Available Patch Available Resolved Resolved
        32d 13h 30m 1 Theo Schlossnagle 05/Oct/11 16:40
        Resolved Resolved Closed Closed
        248d 9h 54m 1 Leif Hedstrom 10/Jun/12 02:35
        Leif Hedstrom made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Theo Schlossnagle made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Theo Schlossnagle made changes -
        Description Add support for accept filters (Solaris and FreeBSD) triggered by proxy.config.net.defer_accept Add support for accept filters (Solaris) triggered by proxy.config.net.defer_accept
        Hide
        Theo Schlossnagle added a comment -

        We need to create a new ticket for adding FreeBSD support. Only Solaris is supported in trunk now.


        Theo Schlossnagle
        http://lethargy.org/~jesus/

        Show
        Theo Schlossnagle added a comment - We need to create a new ticket for adding FreeBSD support. Only Solaris is supported in trunk now. – Theo Schlossnagle http://lethargy.org/~jesus/
        Hide
        Leif Hedstrom added a comment -

        Can this be closed ?

        Show
        Leif Hedstrom added a comment - Can this be closed ?
        Leif Hedstrom made changes -
        Fix Version/s 3.1.1 [ 12316496 ]
        Hide
        Leif Hedstrom added a comment -

        Looks good to me. The only minor thing would be that it could be nice to unify the code that fetches the config value into one place, instead of two with this patch. I'm talking about

        IOCORE_ReadConfigInteger(should_filter_int, "proxy.config.net.defer_accept");

        Which is also used in the section where we support defer accept for Linux. Not a huge deal.

        Show
        Leif Hedstrom added a comment - Looks good to me. The only minor thing would be that it could be nice to unify the code that fetches the config value into one place, instead of two with this patch. I'm talking about IOCORE_ReadConfigInteger(should_filter_int, "proxy.config.net.defer_accept"); Which is also used in the section where we support defer accept for Linux. Not a huge deal.
        Theo Schlossnagle made changes -
        Attachment accept_filter.patch [ 12492867 ]
        Theo Schlossnagle made changes -
        Field Original Value New Value
        Status Open [ 1 ] Patch Available [ 10002 ]
        Theo Schlossnagle created issue -

          People

          • Assignee:
            Theo Schlossnagle
            Reporter:
            Theo Schlossnagle
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 2h
              2h
              Remaining:
              Remaining Estimate - 2h
              2h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development