Re: [v6 15/15] mm: debug for raw alloctor

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

 



However, now thinking about it, I will change it to CONFIG_MEMBLOCK_DEBUG,
and let users decide what other debugging configs need to be enabled, as
this is also OK.

Actually the more I think about it the more I am convinced that a kernel
boot parameter would be better because it doesn't need the kernel to be
recompiled and it is a single branch in not so hot path.

The main reason I do not like kernel parameter is that automated test suits for every platform would need to be updated to include this new parameter in order to test it.

Yet, I think it is important at least initially to test it on every platform unconditionally when certain debug configs are enabled.

This patch series allows boot allocator to return uninitialized memory, this behavior Linux never had before, but way too often firmware explicitly zero all the memory before starting OS. Therefore, it would be hard to debug issues that might be only seen during kinit type of reboots.

In the future, when memory sizes will increase so that this memset will become unacceptable even on debug kernels, it can always be removed, but at least at that time we will know that the code has been tested for many years.
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux