On 2023/11/17 18:10, Johan.Erlandsson(a)sony.com wrote:
>> Hi
>> Sharing 3 changes for zram regarding swap cache handling. Please have a look.
>>
>> Subject: [PATCH 1/3] zram, swap cache missing page tree offset
>> Subject: [PATCH 2/3] zram, swap cache entries are pointer to struct page
>> Subject: [PATCH 3/3] zram, exclude shadow entries from swap cache lookup
>
> Thank you for the patches.
>
> > /* this already exists in maple_tree.h add to defs.h ? */
>
> Is it ok to add '#include maple_tree.h' ?
Yes, that should work perfectly for 'xa_is_value'.
Thanks for the reply.
I'd like to squash the patches into a patch and add our signed-off-by
tags. Please let me know if there is any trouble with the attached patch.
One more thing, do you have any error message without the patch? I'd
like to add it to the commit log, if possible.
Thanks,
Kazu
>
> and the warning below is emitted, this can be fixed when merging.
>
> diskdump.c: In function 'lookup_swap_cache':
> diskdump.c:2890:29: warning: unused variable 'page' [-Wunused-variable]
> ulong swp_type, swp_space, page;
> ^~~~
Sorry, I missed that one. Thanks for reviewing.
Johan