Re: [PATCH] Fix recursion loop in mod_count_all_dependencies()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 5, 2014 at 7:16 AM, Matwey V. Kornilov
<matwey.kornilov@xxxxxxxxx> wrote:
> 2014-05-05 14:11 GMT+04:00 Rusty Russell <rusty@xxxxxxxxxxxxxxx>:
>>  A provides symbol S1.
>>  B provides symbol S1, needs S2.
>
> I think this should be reported too. This case is the most probably a
> bug in the kernel or/and in its configuration. Can depmod detect two
> modules providing the same symbol?

Currently it doesn't because the kernel modpost already warned about
it. However it's very easy to add. Just replacing hash_add() with
hash_add_unique() while adding the symbol and treating the return
would do the job.

Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux