Missed to respond to one of your comments in my previous response. On 6/3/25 2:07 PM, Ming Lei wrote: >> -static struct elevator_type *elevator_find_get(const char *name) >> +struct elevator_type *elevator_find_get(const char *name) >> { >> struct elevator_type *e; >> >> @@ -128,6 +117,7 @@ static struct elevator_type *elevator_find_get(const char *name) >> spin_unlock(&elv_list_lock); >> return e; >> } >> +EXPORT_SYMBOL(elevator_find_get); > > `elevator_find_get()` is block core internal helper, why do you export it? > Yes we don't need to. My bad, we just need to make it non static so that it could be invoked from other file. I will fix this in next patch. Thanks, --Nilay