Seokmann Ju <sju@xxxxxxxx> writes: > This patch fixes incorrect irq data type in the driver which led driver > initialization failure in some cases. > The problem was reported by Eric @. Biederman <ebiederm@xxxxxxxxxxxx>. We have a race going on, I read your first message then this one pops up. But I will reply again for wider distribution. Nacked-by: Eric Biederman. The problem is that struct mcontroller is exported to user space and is a packed structure so it has no free bytes. Therefore you will break user space when you add bytes in the middle of your structure. We can't do that. You either need to add an extra field at the end (while keeping the structure size the same) or you need to add a new ioctl that gets this information correct, or you need to give up entirely on the idea of exporting the irq number to user space this way. What you need not to do is break the kernel<->user space ABI. That is just confusing and couples the kernel and user space in unpleasant ways. I believe struct mraid_hba_info has the same problem but I could not see where it was exported to user space. Eric - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
