Re: [PATCH v4 1/2] panic: add option to dump task maps info in panic_print

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

 



On Wed, Sep 25, 2024 at 02:19:58PM +0200, Oleg Nesterov wrote:
> On 09/25, chenqiwu wrote:
> >
> > On Tue, Sep 24, 2024 at 01:33:54PM +0200, Oleg Nesterov wrote:
> > >
> > > > +		if (path) {
> > > > +			name = d_path(path, name_buf, sizeof(name_buf));
> > > > +			name = IS_ERR(name) ? "?" : name;
> > >
> > I think this is an easier way to get file path name which deals with IS_ERR(name) case.
> > > perhaps this needs mangle_path() ...
> 
> Sorry, I don't understand your reply...
> 
I means we can dump the file path name by d_path() directly without escaping "\n"
by mangle_path(). I try to test the patch in arm64 qemu, it can show the same file
path name as /proc/$pid/maps.

Thanks
Qiwu




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux