Hi Jason, Thanks for your comment. > I wonder if vq_num and max_size_host_cfg is not about to be changed by > the upper layer, any reason not fail the psnet_read_cfg()? You're right, I can catch this error earlier in psnet_read_cfg. > So during reset, the irqs were freed, but I don't see how they are > allocated here. The IRQs are allocated in probe and are freed in reset. This assumes that the device won't be revived after reset. I will change it and request the IRQs in snet_set_status. > Any reason for this reset in the remove()? We must send a SNET_MSG_DESTROY message to the DPU before removing the device. This remove() makes sure that if somehow remove was called before/without reset, the DPU will receive this message. Alvaro _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization