Re: [PATCH 1/1] memory: Handle struct slab changes in linux-next

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

 



Hi Kazu,


HAGIO KAZUHITO(萩尾 一仁)	<k-hagio-ab@xxxxxxx> writes:

> Hi Alex,
>
> -----Original Message-----
>> Since linux-next commit fe1e19081321 ("mm: Split slab into its own type"),
>> the struct slab is used for both SLAB and SLUB. Therefore, don't depend
>> on the non-presence of the struct slab to decide whether SLAB implementation
>> should be chosen and use the member variable "cpu_slab" of the struct
>> kmem_cache instead, it should be present only in SLUB.
>> 
>> Signed-off-by: Alexander Egorenkov <egorenar@xxxxxxxxxxxxx>
>
> Thanks for the early fix as always.  It looks good to me.
> We'd like to add the error message to the commit log:
> ---
> Without the patch, crash fails to start with the error message:
>
>   crash: invalid structure member offset: kmem_cache_s_num
>              FILE: memory.c  LINE: 9619  FUNCTION: kmem_cache_init()
> ---
>
> Acked-by: Kazuhito Hagio <k-hagio-ab@xxxxxxx>
>

Thanks! Looks good.


> btw, kmem -s does not work well with the current linux-next.
> I've not looked into it yet, any patches would be helpful.
>

Thanks for the hint, i'll look into it when i have time, was not aware
of it.

Regards
Alex


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




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

 

Powered by Linux