Does this patch about module loading make sense?

[Julian LaGattuta]

Hi,
I'm new to the linux kernel and have never made a patch. I just made a
modification, on my own system, to linux that cancels the loading of a
module if the module crashes during its init call.
This has the benefit of allowing modules to be unloaded after they
have crashed in the init, and allow callers of the init_module syscall
to get a response saying the module crashed.

Normally, the init call happens in the same thread as whoever wants to
load the module.
The way this is implemented is that I moved the init call to a
separate kthread and I detect if the kthread exited normally. If the
kthread failed, load_module returns -ECANCELED.

My questions are:
Is this a feature worth submitting a patch for?
Is there a reason this hasn't been done?
Are there fundamental flaws in my approach?
Is there other feedback I should have?

I don't know other people with experience with the linux kernel so
anyone's feedback would  be helpful.

Sincerely,
Julian