Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-11825

Remove LU-8964/pio feature & supporting framework

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Lustre 2.13.0, Lustre 2.12.1
    • Labels:
      None
    • Severity:
      3
    • Rank (Obsolete):
      9223372036854775807

      Description

      As discussed recently on the lustre-devel mailing list:

      http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2018-November/008230.html

      http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2018-November/008231.html

      http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2018-November/008233.html

      http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2018-November/008234.html

      I think the pio feature should be removed.

      Quoting my email to the list:
      "

      elieve that long term it’s headed out of Lustre. It only improves performance in a limited way in certain circumstances, and harms it in various others. So it’s off by default, and, I suspect, remains completely unused. A while back I noticed its test framework test didn’t activate it correctly, and once fixed, it sometimes deadlocks (race with truncate). There’s a patch to fix that, but a problem was found in it and it has since languished.

      I would still suggest you take it, Neil, as othewise you’ll complicate a bunch of potentially nasty porting working in the CLIO stack, as you apply the years of patches written with it there. Instead, I’d suggest we pull it in the open sfs branch (Sorry! It was a promising idea but it hasn’t panned out, and the current parallel readahead work isn’t going to use it.) and then eventually you could pick that up."

      Patches forthcoming.  I have also asked the lustre-discuss mailing list if anyone is using it, and we can change course if someone pipes up.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                paf Patrick Farrell (Inactive)
                Reporter:
                paf Patrick Farrell (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: