From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 11040C433F5 for ; Tue, 2 Nov 2021 13:04:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id E5F4360C41 for ; Tue, 2 Nov 2021 13:04:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231176AbhKBNGh (ORCPT ); Tue, 2 Nov 2021 09:06:37 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:26631 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230282AbhKBNGg (ORCPT ); Tue, 2 Nov 2021 09:06:36 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1635858241; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=F5yWkmOg5/0evPimUvtS3b9rTndZ2SZlpCnwbNELc7c=; b=PMSbfmEWo7tP53+HkkNAuX2kzBCQLd/667tNRIE/HL+q+fVQFXb2X4BTy2sn7F514JJpxR 4Ws0cPJelUd5VNuEYJdTB9Ma9zT+msZDxJz/qtxlIw/oy9K3bG0i8jky/EvnpvD6g5A7RY dIurL4+UDJdhQXe+Qkf//U8AC5A+EuA= Received: from mail-wm1-f72.google.com (mail-wm1-f72.google.com [209.85.128.72]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-526-dc1ejtd5M0OPGSzQ8xstUg-1; Tue, 02 Nov 2021 09:04:00 -0400 X-MC-Unique: dc1ejtd5M0OPGSzQ8xstUg-1 Received: by mail-wm1-f72.google.com with SMTP id u14-20020a05600c19ce00b0030d8549d49aso682414wmq.0 for ; Tue, 02 Nov 2021 06:04:00 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=F5yWkmOg5/0evPimUvtS3b9rTndZ2SZlpCnwbNELc7c=; b=NDhKzdMYBV2YDog5w3+5lw39mCj6SX5KGYOFnHxSiyfrM4cFb+zBZmwSo/P7m/rQ/H HEOOGETniRYBIKhxrdvsP1Js/Xw+jg5t3RfrtXGReGjYxCFWN+q5tZ5mVI4MTR8ehB4i Bc7GgqYB7nm/TYCG+bIhbeVxZ4jBIev9v1Q18xyelUr+yWjGhx/Vc3EvyBtPBCyGZukL a10cYDHI7j6pBWnVvTn4a4xr3fYTt3e6SB4045WMLaOKtESN0m8nMSrq84n+DqM7/W4F 9U0VR3mp0BYJ0NXOXYJZS/QI6kktZnaWahIcGlT7Wb8cDNbwcAkYYSrVx3dS8YeeX1/p Igbg== X-Gm-Message-State: AOAM5335PFD4kw5YraZA/CP5Ee9pqs+6YcJ3GwgGsLkQj6gYoThy+Qy6 XhVIaSqs7twuWNVCcbnj7+uIHLu/ApipI4sOnp/JiDVsUsqvAvT++q5ieiTHKEnU4C0rXzw2j0w KynNrV2i4YB4pMqX/+EVJTutq X-Received: by 2002:adf:d1e3:: with SMTP id g3mr46970725wrd.63.1635858238979; Tue, 02 Nov 2021 06:03:58 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzuDILBX8p9T4euEWEY2xWP7A4Nb7oizJIhNiY6eGE/hNzVLg2a0xhHfdf5bmVh6MTgKK0GBQ== X-Received: by 2002:adf:d1e3:: with SMTP id g3mr46970686wrd.63.1635858238747; Tue, 02 Nov 2021 06:03:58 -0700 (PDT) Received: from vitty.brq.redhat.com (g-server-2.ign.cz. [91.219.240.2]) by smtp.gmail.com with ESMTPSA id k37sm2389496wms.21.2021.11.02.06.03.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 02 Nov 2021 06:03:58 -0700 (PDT) From: Vitaly Kuznetsov To: Boqun Feng , linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org Cc: "K. Y. Srinivasan" , Haiyang Zhang , Stephen Hemminger , Wei Liu , Dexuan Cui , Boqun Feng , stable@vger.kernel.org, Baihua Lu Subject: Re: [PATCH] Drivers: hv: balloon: Use VMBUS_RING_SIZE() wrapper for dm_ring_size In-Reply-To: <20211101150026.736124-1-boqun.feng@gmail.com> References: <20211101150026.736124-1-boqun.feng@gmail.com> Date: Tue, 02 Nov 2021 14:03:57 +0100 Message-ID: <87h7cuk8ua.fsf@vitty.brq.redhat.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-hyperv@vger.kernel.org Boqun Feng writes: > Baihua reported an error when boot an ARM64 guest with PAGE_SIZE=64k and > BALLOON is enabled: > > hv_vmbus: registering driver hv_balloon > hv_vmbus: probe failed for device 1eccfd72-4b41-45ef-b73a-4a6e44c12924 (-22) > > The cause of this is that the ringbuffer size for hv_balloon is not > adjusted with VMBUS_RING_SIZE(), which makes the size not large enough > for ringbuffers on guest with PAGE_SIZE=64k. Therefore use > VMBUS_RING_SIZE() to calculate the ringbuffer size. Note that the old > size (20 * 1024) counts a 4k header in the total size, while > VMBUS_RING_SIZE() expects the parameter as the payload size, so use > 16 * 1024. > > Cc: # 5.15.x > Reported-by: Baihua Lu > Signed-off-by: Boqun Feng > --- > drivers/hv/hv_balloon.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/hv/hv_balloon.c b/drivers/hv/hv_balloon.c > index 7f11ea07d698..ca873a3b98db 100644 > --- a/drivers/hv/hv_balloon.c > +++ b/drivers/hv/hv_balloon.c > @@ -480,7 +480,7 @@ module_param(pressure_report_delay, uint, (S_IRUGO | S_IWUSR)); > MODULE_PARM_DESC(pressure_report_delay, "Delay in secs in reporting pressure"); > static atomic_t trans_id = ATOMIC_INIT(0); > > -static int dm_ring_size = 20 * 1024; > +static int dm_ring_size = VMBUS_RING_SIZE(16 * 1024); > > /* > * Driver specific state. Tested-by: Vitaly Kuznetsov Thanks! -- Vitaly