Received: by 2002:a6b:500f:0:0:0:0:0 with SMTP id e15csp1085445iob; Thu, 28 Apr 2022 18:55:57 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxJtVnfHIK/S0c9v+1PmEMqekSv24sT6vkjSuPOKSW7YE2JifDDT4z6wjfCpJqYUHcgTdcF X-Received: by 2002:a17:902:7296:b0:151:62b1:e2b0 with SMTP id d22-20020a170902729600b0015162b1e2b0mr36086002pll.165.1651197357589; Thu, 28 Apr 2022 18:55:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1651197357; cv=none; d=google.com; s=arc-20160816; b=eIE59tKzH6fqdugWPmfzmpqPMXvdTIb3kFWWHDm34/0GoUhZ8UfF86jnn9rkhBLzZ4 FfpZt47+dheQXO4SaDq380+w/ulffLyGGQ8w7K9pqjtJkRF+YJkN9MDG3F20pXWBRbHs 5Ge0dy5W9vWylDEUHjvWZh4qP/5utLG9esQ9sR/tpKAqzIxXIjwFflYAktjQcZZc5H20 pzz0jL+ycekOXlRzsVEokzdB6g5gk0Sh6YBr02A8q1/a9aVwP78L9rxl2MnoPNfgdQbX b3PdS5So5GBRu7I04rNXc5iPWXiJKLkpaea6X4VjRVyRiFOJicj34Q8Qw3yASULwZSpA IWMQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:content-transfer-encoding :content-id:content-language:accept-language:in-reply-to:references :message-id:date:thread-index:thread-topic:subject:cc:to:from :dkim-signature; bh=Osnnz3SGQ3g/VbQYJdclIP+f6CEdAQIARqwCtbhZApE=; b=LiQ09PYVEtuPU3orWB8G0rxKJItg6U4W3YCeU//IqOwau8OkGr/9CfiypOeaPrmiWw TuEXA0MOyFsIH9Vqyu0KEW8OCfVs6WgpmYwRH5kx3a87EaGvdDnTGpbwAcMDsWpNM/Bq 3oV0/w9qyIdJe60mfRUcJ9WGhEC5owiyWIUllxS51Vsf915q+SDWVbxL7lVGWdJ+pkUm DCI1foNGrTTrTQFgZXHYVw0mK5+0zAxqlTHUyyuU0kYnOwls5egVmZtHulWAd5Ovunb5 RUsdXs6+z3gblYUfRCBeXirzpt7wK/jmKwF1vkZ2u6g1/1yAkICe/LS6hlmTlqG5IK/z YHww== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sberdevices.ru header.s=mail header.b=oNpfdTYi; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=sberdevices.ru Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c20-20020a6566d4000000b003a0771d526esi6449870pgw.705.2022.04.28.18.55.31; Thu, 28 Apr 2022 18:55:57 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@sberdevices.ru header.s=mail header.b=oNpfdTYi; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=sberdevices.ru Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1351426AbiD1UEh (ORCPT + 99 others); Thu, 28 Apr 2022 16:04:37 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37120 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231396AbiD1UEg (ORCPT ); Thu, 28 Apr 2022 16:04:36 -0400 Received: from mail.sberdevices.ru (mail.sberdevices.ru [45.89.227.171]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3973774868; Thu, 28 Apr 2022 13:01:19 -0700 (PDT) Received: from s-lin-edge02.sberdevices.ru (localhost [127.0.0.1]) by mail.sberdevices.ru (Postfix) with ESMTP id 65EF75FD03; Thu, 28 Apr 2022 23:01:16 +0300 (MSK) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sberdevices.ru; s=mail; t=1651176076; bh=Osnnz3SGQ3g/VbQYJdclIP+f6CEdAQIARqwCtbhZApE=; h=From:To:Subject:Date:Message-ID:Content-Type:MIME-Version; b=oNpfdTYiFBwtPAF/VzIe5X6+HaR/Mw4kAZ1M/9iS0hjgbUSNuFcK0lAoHFrODtCuC DHWuX/XEU89eNzO3cOTHMhE3GQGu9z6k8At5ELDc0k/VpZgHfaubEdYinkjaxxU0Bf oQC1Q6Mmw7rqYfi4sx3UVrwEzrw2KgACEO+f7RROqfwPT1Oef3qmJfGm0HWuy3zgY/ RM3OuPBuujIc47wqa0dZmajNUbZWxOHuvEUFIfYOTfd+fish+23fodcDI6ZzY8UEVl od3JTlXTQy4Mr9eGRUacBciLz498j71YQlu9jVdKChOD/1LLIYF2Wb3+GVhaxgHtd1 /DnM21t9+zlIg== Received: from S-MS-EXCH02.sberdevices.ru (S-MS-EXCH02.sberdevices.ru [172.16.1.5]) by mail.sberdevices.ru (Postfix) with ESMTP; Thu, 28 Apr 2022 23:01:14 +0300 (MSK) From: Aleksey Romanov To: Sergey Senozhatsky , "minchan@kernel.org" CC: "minchan@kernel.org" , "ngupta@vflare.org" , "linux-block@vger.kernel.org" , "axboe@chromium.org" , kernel , "linux-kernel@vger.kernel.org" , "mnitenko@gmail.com" , Dmitry Rokosov Subject: Re: [PATCH v2] zram: remove double compression logic Thread-Topic: [PATCH v2] zram: remove double compression logic Thread-Index: AQHYWh39fkxoPWqFH028+oB3r+sQB60Dbc4AgAIhEIA= Date: Thu, 28 Apr 2022 20:01:12 +0000 Message-ID: <20220428200107.hz65qvxf2aoz27q2@cab-wsm-0029881> References: <20220427100345.29461-1-avromanov@sberdevices.ru> In-Reply-To: Accept-Language: ru-RU, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.16.1.12] Content-Type: text/plain; charset="us-ascii" Content-ID: <93E992117724AD41B9856520CDF1E0BD@sberdevices.ru> Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-KSMG-Rule-ID: 4 X-KSMG-Message-Action: clean X-KSMG-AntiSpam-Status: not scanned, disabled by settings X-KSMG-AntiSpam-Interceptor-Info: not scanned X-KSMG-AntiPhishing: not scanned, disabled by settings X-KSMG-AntiVirus: Kaspersky Secure Mail Gateway, version 1.1.2.30, bases: 2022/04/28 14:34:00 #19338612 X-KSMG-AntiVirus-Status: Clean, skipped X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Thanks for the reply! On Wed, Apr 27, 2022 at 08:30:16PM +0900, Sergey Senozhatsky wrote: > On (22/04/27 13:03), Alexey Romanov wrote: > > diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_dr= v.c > > index cb253d80d72b..4be6caf43b1d 100644 > > --- a/drivers/block/zram/zram_drv.c > > +++ b/drivers/block/zram/zram_drv.c > > @@ -1153,9 +1153,8 @@ static ssize_t debug_stat_show(struct device *dev= , > > =20 > > down_read(&zram->init_lock); > > ret =3D scnprintf(buf, PAGE_SIZE, > > - "version: %d\n%8llu %8llu\n", > > + "version: %d\n%8llu\n", > > version, > > - (u64)atomic64_read(&zram->stats.writestall), > > (u64)atomic64_read(&zram->stats.miss_free)); > > up_read(&zram->init_lock); >=20 > I think this also has to bump `version` to 2, since format of the > file has changed. Yes, I'll do that in the next patch. Minchan, do you have any suggestions on this patch?=20 I want to fix Sergey suggestion and sumbit next patch.=