From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id A9555431A6 for ; Thu, 19 Oct 2023 09:24:21 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9854A40E2D; Thu, 19 Oct 2023 09:24:21 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mails.dpdk.org (Postfix) with ESMTP id 903E540E25 for ; Thu, 19 Oct 2023 09:24:20 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1697700260; 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: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=PPSXinha02H3WixS4BX/SJY1AkoZcLQAW2eKHdGHvZ8=; b=G9Yb2cQGBGaIS2biEN+1d1ABc1E+2pf6JP73uVtqgEQSqaTQclUncJ6/zy1G7e4XdCumVx ZteEY8NVTrfyZTQniBHKKVk32hTmM4hQcXl3VBlT2IrtkhJrLE2vqYrg4wVzonDfzmbjAx RjSqrpF6ckKelkhhAbNVKWzrSWnP8ME= Received: from mail-lj1-f200.google.com (mail-lj1-f200.google.com [209.85.208.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-539-3RvwzZXiPDWJ3Xr-8x99dw-1; Thu, 19 Oct 2023 03:24:16 -0400 X-MC-Unique: 3RvwzZXiPDWJ3Xr-8x99dw-1 Received: by mail-lj1-f200.google.com with SMTP id 38308e7fff4ca-2ba1949656bso71391521fa.0 for ; Thu, 19 Oct 2023 00:24:16 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697700255; x=1698305055; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=PPSXinha02H3WixS4BX/SJY1AkoZcLQAW2eKHdGHvZ8=; b=gvpe+hpuYHnmqk4a4rM2wqhrHUnH/nqmTPgOIbKAU7TgbxjbVmg0eYBpb9LXhJdypl FJoN65ineiQRSfMZhHMolutem4nfeK8gSgi1CTLaQCZN4Ig41aQ10buYXUdh7TXtC1xI Iye1a5ShvwG8oJ7b8JEdtyAiNcpbEhfBE2HWnz7jwyPKm+kOWIrYMjcYw2YtuUYdMbP3 mdcFNIt15/BRVQYpwSfCnSBQP8a/OfQGFx5A1sjjpWhke2zY/DjV75Dy7LleTDTC/bxT HzbjKQfmr1izqfkzlnlkBmospTIFG1QhlLsjp4LoBzRpCSKj+0V6T3fiTyR4GY15DINL DZTg== X-Gm-Message-State: AOJu0YyQ3aJ8iSJ5cMO/yfzA/kFTQa8ZGbKxpIada2BCPskubtoHI1i4 aR0Qbnty1gwtSSnVwsmfHNpJGygd0sE2q2VyVW2TVniLIjoumaJKfhgwZQ0PboXp7odCKkRNwWM f1R/lCoOXNnbeOqv/oHTay48= X-Received: by 2002:a2e:7401:0:b0:2be:3ca8:bdc9 with SMTP id p1-20020a2e7401000000b002be3ca8bdc9mr741534ljc.49.1697700254983; Thu, 19 Oct 2023 00:24:14 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEARH3XCE+kj5pmImug+H/oRW06xL/0Y+8sS+upKa5o0L++5IppaqKsqSifzUiN8K+TSbcG3V9IwdU23S0EtcI= X-Received: by 2002:a2e:7401:0:b0:2be:3ca8:bdc9 with SMTP id p1-20020a2e7401000000b002be3ca8bdc9mr741523ljc.49.1697700254664; Thu, 19 Oct 2023 00:24:14 -0700 (PDT) MIME-Version: 1.0 References: <20230925163610.3307750-1-maxime.coquelin@redhat.com> <20230925163610.3307750-2-maxime.coquelin@redhat.com> In-Reply-To: <20230925163610.3307750-2-maxime.coquelin@redhat.com> From: David Marchand Date: Thu, 19 Oct 2023 09:24:03 +0200 Message-ID: Subject: Re: [PATCH 1/7] vhost: fix missing vring call check on virtqueue access To: Maxime Coquelin Cc: dev@dpdk.org, chenbo.xia@intel.com, mb@smartsharesystems.com, stable@dpdk.org, Li Feng X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org On Mon, Sep 25, 2023 at 6:36=E2=80=AFPM Maxime Coquelin wrote: > > Acquiring the access lock is not enough to ensure > virtqueue's metadata such as vring pointers are valid. > > The access status must also be checked. Even if adding the lock was not enough, I would flag Fixes: 6c299bb7322f ("vhost: introduce vring call API") > > Fixes: c5736998305d ("vhost: fix missing virtqueue lock protection") > Fixes: 830f7e790732 ("vhost: add non-blocking API for posting interrupt") > Cc: stable@dpdk.org > > Reported-by: Li Feng > Signed-off-by: Maxime Coquelin --=20 David Marchand