• P
    IB/fmr_pool: Convert the cleanup thread into kthread worker API · 50b6778c
    Petr Mladek 提交于
    Kthreads are currently implemented as an infinite loop. Each
    has its own variant of checks for terminating, freezing,
    awakening. In many cases it is unclear to say in which state
    it is and sometimes it is done a wrong way.
    
    The plan is to convert kthreads into kthread_worker or workqueues
    API. It allows to split the functionality into separate operations.
    It helps to make a better structure. Also it defines a clean state
    where no locks are taken, IRQs blocked, the kthread might sleep
    or even be safely migrated.
    
    The kthread worker API is useful when we want to have a dedicated
    single thread for the work. It helps to make sure that it is
    available when needed. Also it allows a better control, e.g.
    define a scheduling priority.
    
    This patch converts the frm_pool kthread into the kthread worker
    API because I am not sure how busy the thread is. It is well
    possible that it does not need a dedicated kthread and workqueues
    would be perfectly fine. Well, the conversion between kthread
    worker API and workqueues is pretty trivial.
    
    The patch moves one iteration from the kthread into the work function.
    It is queued only when there is a pending work. Therefore we do not
    need to compare flush_ser and req_ser at the beginning. On the contrary,
    the same work could be queued only once at a time. Therefore it has to
    re-queue itself if some requests are pending.
    
    Otherwise, wake_up_process() is replaced by queuing the work.
    
    Important: The change is only compile tested. I did not find an easy
    way how to check it in a real life.
    Signed-off-by: NPetr Mladek <pmladek@suse.com>
    TO: Doug Ledford <dledford@redhat.com>
    CC: Sean Hefty <sean.hefty@intel.com>
    CC: Hal Rosenstock <hal.rosenstock@gmail.com>
    CC: linux-rdma@vger.kernel.org
    Signed-off-by: NDoug Ledford <dledford@redhat.com>
    50b6778c
fmr_pool.c 13.4 KB