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

OSC page & grant simplification/batching

Details

    • Bug
    • Resolution: Won't Do
    • Minor
    • None
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      This is a ticket to capture what I expect will be a different OSC grant code simplifications/efficiency improvements that don't really need separate LUs for each.

      The basic goal is to do certain operations less often - Various grant (and related page update) operations are done per page, and for some of them this is either totally unnecessary or can be avoided with some care.

      The goal is to do at least a bit less work for each page submission in the write path, and potentially prepare some of the ground for write containers (aggressively simplified write path dealing with pages in large batches).

      Attachments

        Issue Links

          Activity

            [LU-11942] OSC page & grant simplification/batching

            The specific work intended for this ticket wound up not being of benefit, and there are several other tickets where related work was done, so no reason to leave this open.

            pfarrell Patrick Farrell (Inactive) added a comment - The specific work intended for this ticket wound up not being of benefit, and there are several other tickets where related work was done, so no reason to leave this open.

            Also Li is work on similar improvements.

            simmonsja James A Simmons added a comment - Also Li is work on similar improvements.

            Peter isn't on this ticket, but looking at LU-4423, I would say use that.  Forgot we had it.

            pfarrell Patrick Farrell (Inactive) added a comment - Peter isn't on this ticket, but looking at LU-4423 , I would say use that.  Forgot we had it.

            Peter do you want a new ticket or should LU-4423 be reused?

            simmonsja James A Simmons added a comment - Peter do you want a new ticket or should LU-4423 be reused?

            If you're ready, that would be great.  My first patch (which I didn't push yet because of a Gerrit issue, I'll push it soon) is like, a one liner, so it won't matter.  But I am contemplating some bigger changes.

            I think a different LU for Neil's stuff would be good though.

            pfarrell Patrick Farrell (Inactive) added a comment - If you're ready, that would be great.  My first patch (which I didn't push yet because of a Gerrit issue, I'll push it soon) is like, a one liner, so it won't matter.  But I am contemplating some bigger changes. I think a different LU for Neil's stuff would be good though.

            Yes I can merge them in. The question is should I do it right away to prevent collisions with you?

            simmonsja James A Simmons added a comment - Yes I can merge them in. The question is should I do it right away to prevent collisions with you?

            Ah, no, it will not.  I wasn't aware of those.  This is rather different - It's more performance oriented, though there may be some other stuff.

            Are you planning to merge those in?

            pfarrell Patrick Farrell (Inactive) added a comment - Ah, no, it will not.  I wasn't aware of those.  This is rather different - It's more performance oriented, though there may be some other stuff. Are you planning to merge those in?
            simmonsja James A Simmons added a comment - https://patchwork.kernel.org/project/lustre-devel/list/?series=63793

            Dunno, what did Neil do upstream?  Can you shoot me a link or two?

            pfarrell Patrick Farrell (Inactive) added a comment - Dunno, what did Neil do upstream?  Can you shoot me a link or two?

            Will this cover the changes Neil did upstream?

            simmonsja James A Simmons added a comment - Will this cover the changes Neil did upstream?

            People

              pfarrell Patrick Farrell (Inactive)
              pfarrell Patrick Farrell (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: