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

fault_in_iov_iter_readable() with a spinlock held

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • Medium
    • None
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      in LU-13805 and LU-18006 the following code was introduced:

             if (rw == WRITE) {
                     spin_lock(&sdio->csd_lock);
      ...
             /* fault in the entire userspace iovec */
             if (rw == WRITE) {
                     if (unlikely(ll_iov_iter_fault_in_readable(iter, count)))
      

      I think it's a bad idea to repopulate userspace pages holding a spinlock - this can involve page allocation and I/O.

      Attachments

        Issue Links

          Activity

            People

              paf0186 Patrick Farrell
              bzzz Alex Zhuravlev
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated: