linux-kernel - Re: [PATCH 1/7] [RFC PATCH 1/7] osdump: Add one new os minidump module

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date: Mon, 8 Jan 2024 13:02:25 +0530
From: Mukesh Ojha <quic_mojha@...cinc.com>
To: ruipeng qi <ruipengqi7@...il.com>,
        Krzysztof Kozlowski
	<krzysztof.kozlowski@...aro.org>
CC: <catalin.marinas@....com>, <will@...nel.org>, <bhe@...hat.com>,
        <vgoyal@...hat.com>, <dyoung@...hat.com>,
        <linux-kernel@...r.kernel.org>, <kexec@...ts.infradead.org>,
        <zalbassam@...gle.com>, <dianders@...omium.org>,
        <mark.rutland@....com>, <lecopzer.chen@...iatek.com>, <maz@...nel.org>,
        <arnd@...db.de>, <yury.norov@...il.com>, <brauner@...nel.org>,
        <mcgrof@...nel.org>, <maninder1.s@...sung.com>,
        <michael.christie@...cle.com>, <samitolvanen@...gle.com>,
        <linux-arm-kernel@...ts.infradead.org>, <qiruipeng@...iang.com>
Subject: Re: [PATCH 1/7] [RFC PATCH 1/7] osdump: Add one new os minidump
 module



On 12/23/2023 9:00 PM, ruipeng qi wrote:
> On Sat, Dec 23, 2023 at 11:13 PM ruipeng qi <ruipengqi7@...il.com> wrote:
> 
>>> I don't see a problem why minidump cannot do it.
>> Qcom minidump allows one kernel client to register its region to dump, and then
>> driver will create and maintain one elf head for it.
>> But osdump will dump thousands of non-continuous regions. If you register one
>> by one, and add an elf head for each, it will cause heavy overhead. So it store
>> data in binary, and reassemble a standard elf format file offline for debug.
>>
> Osdump will only dump useful data, which may have been allocated before but
> freed now. In a word, useful data are changing all the time. I think this is not
> suitable for Qcom minidump's framework。


Hey @ruipeng,

I like the idea, However, as i already said here

https://lore.kernel.org/lkml/520e377d-e990-c185-4a20-07806873e506@quicinc.com/

This looks more close to pstore ramoops based on Ram backend, as your 
entire thing is based on RAM content to be preserved across boot and
that way it aligns to pstore ramoops concept.

Also most of the stuff like compression is already available in pstore.
So, let me know if you think that way, will be happy to collaborate and 
test this.

-Mukesh

> 
> Best Regards
> Ruipeng
>