Re: [PATCH]: gcore extension, anonymous union in inode struct

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

 



From: Dave Anderson <anderson@xxxxxxxxxx>
Subject: Re:  [PATCH]: gcore extension, anonymous union in inode struct
Date: Tue, 11 Sep 2012 08:50:32 -0400

> 
> 
> ----- Original Message -----
>> From: Per Fransson <per.fransson.ml@xxxxxxxxx>
>> Subject:  [PATCH]: gcore extension, anonymous union in
>> inode struct
>> Date: Wed, 5 Sep 2012 12:29:43 +0200
>> 
>> > Hi Crash people,
>> > 
>> > The gcore extension fails on the 3.4 kernel I'm using. It attempts to
>> > find the offset of a member within the inode struct, but the member  is
>> > part of an anonymous union. This patch fixes the problem for me.
>> > 
>> > Regards,
>> > Per
>> 
>> Hello Per,
>> 
>> Thanks for reporting that. According to git repository, this was
>> changed by the following commit at the v3.1 period.
>> 
>> $ git describe a78ef704a8dd430225955f0709b22d4a6ba21deb
>> v3.1-8569-ga78ef70
>> 
>> $ git log -1 a78ef704a8dd430225955f0709b22d4a6ba21deb
>> commit a78ef704a8dd430225955f0709b22d4a6ba21deb
>> Author: Miklos Szeredi <mszeredi@xxxxxxx>
>> Date:   Fri Oct 28 14:13:30 2011 +0200
>> 
>>     vfs: protect i_nlink
>> 
>>     Prevent direct modification of i_nlink by making it const and adding a
>>     non-const __i_nlink alias.
>> 
>>     Signed-off-by: Miklos Szeredi <mszeredi@xxxxxxx>
>>     Tested-by: Toshiyuki Okajima <toshi.okajima@xxxxxxxxxxxxxx>
>>     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
>> 
>> The patch appears fine to me so I'll apply it.
>> 
>> Thanks.
>> HATAYAMA, Daisuke
> 
> Hi Daisuke,
> 
> Will you be updating the crash-gcore-command tar.gz package?
> 

Hello Dave,

I have another gcore test plan soon. I'm going to update this change
together with it. Please wait for a few weeks.

Thanks.
HATAYAMA, Daisuke

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility


[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux