From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1FC5E481A4 for ; Wed, 27 Dec 2023 20:58:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux-foundation.org header.i=@linux-foundation.org header.b="MjY/HB9f" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 661C2C433C8; Wed, 27 Dec 2023 20:58:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linux-foundation.org; s=korg; t=1703710734; bh=nVIXdXTdlWdkqE/2/J9dc3xAOUa+ZEXhxjCS1ovFzRU=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=MjY/HB9fglm3RR9yjFHrErDDhjl2mpNchHTuMZPLSRlKBY802XFzhXarLC0b589ez cNjeuOJlejNZ+h2vXKbu1g4+gz0NxqaJxehHi5ln0zPn7PF5Xuqb4KWxcTW3Omf1tu Z6UvQ41VMPxOfmtoM7MwEpeAZr/hEWbWIg/CvOFM= Date: Wed, 27 Dec 2023 12:58:53 -0800 From: Andrew Morton To: Chengming Zhou Cc: Barry Song <21cnbao@gmail.com>, Seth Jennings , Johannes Weiner , Vitaly Wool , Nhat Pham , Chris Li , Yosry Ahmed , Dan Streetman , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Chris Li Subject: Re: [PATCH v4 1/6] mm/zswap: change dstmem size to one page Message-Id: <20231227125853.2dd07cd61f8a8df95042677c@linux-foundation.org> In-Reply-To: References: <20231213-zswap-dstmem-v4-0-f228b059dd89@bytedance.com> <20231213-zswap-dstmem-v4-1-f228b059dd89@bytedance.com> X-Mailer: Sylpheed 3.8.0beta1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Wed, 27 Dec 2023 14:11:06 +0800 Chengming Zhou wrote: > > i remember there was an over-compression case, that means the compressed > > data can be bigger than the source data. the similar thing is also done in zram > > drivers/block/zram/zcomp.c > > Right, there is a buffer overflow report[1] that I just +to you. What does "[1]" refer to? Is there a bug report about this series?