On Fri, Sep 22, 2023 at 11:38:35AM -0700, Nuno Das Neves wrote: > +static int __init mshv_vtl_init(void) > +{ > + int ret; > + > + tasklet_init(&msg_dpc, mshv_vtl_sint_on_msg_dpc, 0); > + init_waitqueue_head(&fd_wait_queue); > + > + if (mshv_vtl_get_vsm_regs()) { > + pr_emerg("%s: Unable to get VSM capabilities !!\n", __func__); > + BUG(); > + } So you crash the whole kernel if someone loads this module on a non-mshv system? That seems quite excessive and hostile :( Or am I somehow reading this incorrectly? thanks, greg k-h