{"push_id":1421627963,"size":1,"distinct_size":1,"ref":"refs/heads/nebula5e_20161123-eas","head":"5c15e18d9493693503924da6164a0625ca46dc91","before":"ff866280d6a265878d9c0f33eb80eb19e8ba3f05","commits":[{"sha":"5c15e18d9493693503924da6164a0625ca46dc91","author":{"name":"Eliminater74","email":"d6699dcc31967de8edd2c704b097a2c4ed828622@gmail.com"},"message":"kthread: kthread worker API cleanup #2 (Missed from @58d04bbb31937b43cf625a2a3f2546322fc82b2e )\nA good practice is to prefix the names of functions by the name\nof the subsystem.\n\nThe kthread worker API is a mix of classic kthreads and workqueues.  Each\nworker has a dedicated kthread.  It runs a generic function that process\nqueued works.  It is implemented as part of the kthread subsystem.\n\nThis patch renames the existing kthread worker API to use\nthe corresponding name from the workqueues API prefixed by\nkthread_:\n\n__init_kthread_worker()\t\t-> __kthread_init_worker()\ninit_kthread_worker()\t\t-> kthread_init_worker()\ninit_kthread_work()\t\t-> kthread_init_work()\ninsert_kthread_work()\t\t-> kthread_insert_work()\nqueue_kthread_work()\t\t-> kthread_queue_work()\nflush_kthread_work()\t\t-> kthread_flush_work()\nflush_kthread_worker()\t\t-> kthread_flush_worker()\n\nNote that the names of DEFINE_KTHREAD_WORK*() macros stay\nas they are. It is common that the \"DEFINE_\" prefix has\nprecedence over the subsystem names.\n\nNote that INIT() macros and init() functions use different\nnaming scheme. There is no good solution. There are several\nreasons for this solution:\n\n  + \"init\" in the function names stands for the verb \"initialize\"\n    aka \"initialize worker\". While \"INIT\" in the macro names\n    stands for the noun \"INITIALIZER\" aka \"worker initializer\".\n\n  + INIT() macros are used only in DEFINE() macros\n\n  + init() functions are used close to the other kthread()\n    functions. It looks much better if all the functions\n    use the same scheme.\n\n  + There will be also kthread_destroy_worker() that will\n    be used close to kthread_cancel_work(). It is related\n    to the init() function. Again it looks better if all\n    functions use the same naming scheme.\n\n  + there are several precedents for such init() function\n    names, e.g. amd_iommu_init_device(), free_area_init_node(),\n    jump_label_init_type(),  regmap_init_mmio_clk(),\n\n  + It is not an argument but it was inconsistent even before.\n\nSigned-off-by: Eliminater74 <eliminater74@gmail.com>","distinct":true,"url":"https://api.github.com/repos/DevHTC10/HTC10_pme_msm8996_Nebula_kernel/commits/5c15e18d9493693503924da6164a0625ca46dc91"}]}
{"actor":{"display_login":"Eliminater74"}}