On Thu, Jan 28, 2021, 5:09 AM Souptick Joarder <jrdr.linux@gmail.com> wrote:
On Thu, Jan 21, 2021 at 3:27 PM Tian Tao <tiantao6@hisilicon.com> wrote:
>
> the entry has not been initialized when it is used, so allocate PAGE_SIZE

Can you please add the original error msg in change logs ?

I second that. Another option would be to come up with an update of the original patch which would incorporate all the fixes. I believe it should be possible, since the initial patch has not been merged into Linus tree.

Best regards,
   Vitaly


>
> Signed-off-by: Tian Tao <tiantao6@hisilicon.com>
> Reported-by: kernel test robot <lkp@intel.com>
> ---
>  mm/zswap.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/mm/zswap.c b/mm/zswap.c
> index 6e0bb61..4b7b6ad 100644
> --- a/mm/zswap.c
> +++ b/mm/zswap.c
> @@ -944,7 +944,7 @@ static int zswap_writeback_entry(struct zpool *pool, unsigned long handle)
>
>         if (!zpool_can_sleep_mapped(pool)) {
>
> -               tmp = kmalloc(entry->length, GFP_ATOMIC);
> +               tmp = kmalloc(PAGE_SIZE, GFP_ATOMIC);
>                 if (!tmp)
>                         return -ENOMEM;
>         }
> --
> 2.7.4
>
>