On 30.03.20 08:51, Anshuman Khandual wrote: > Enumerate all existing bad page reasons which can be used in bad_page() for > reporting via __dump_page(). Unfortunately __dump_page() cannot be changed. > __dump_page() is called from dump_page() that accepts a raw string and is > also an exported symbol that is currently being used from various generic > memory functions and other drivers. This reduces code duplication while > reporting bad pages. Yeah sounds nice, but "56 insertions(+), 20 deletions(-)" does not sound so nice ... and the "code duplication" is actually "repeating strings". ... I don't think we want/need this. -- Thanks, David / dhildenb