From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id B23A1439B4;
	Wed, 24 Jan 2024 10:38:48 +0100 (CET)
Received: from mails.dpdk.org (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 81C9A402AB;
	Wed, 24 Jan 2024 10:38:48 +0100 (CET)
Received: from inbox.dpdk.org (inbox.dpdk.org [95.142.172.178])
 by mails.dpdk.org (Postfix) with ESMTP id 2DFB74026F
 for <dev@dpdk.org>; Wed, 24 Jan 2024 10:38:47 +0100 (CET)
Received: by inbox.dpdk.org (Postfix, from userid 33)
 id 1DC7E439B5; Wed, 24 Jan 2024 10:38:47 +0100 (CET)
From: bugzilla@dpdk.org
To: dev@dpdk.org
Subject: =?UTF-8?B?W0RQREsgQnVnIDEzMzhdIFtkcGRrLTIyLjExLjRdIHZob3N0X3Zp?=
 =?UTF-8?B?cnRpb191c2VyX2ludGVycnVwdF93aXRoX3Bvd2VyX21vbml0b3IvcGVyZl93?=
 =?UTF-8?B?YWtlX3VwX3NwbGl0X3Jpbmdfdmhvc3RfdXNlcl9jb3JlX3dpdGhfbDNmd2Rf?=
 =?UTF-8?B?cG93ZXJfc2FtcGxl77yadmlydGlvLXVzZXIgY2FuJ3QgcmVjZWl2ZSBwYWNr?=
 =?UTF-8?B?ZXQ=?=
Date: Wed, 24 Jan 2024 09:38:46 +0000
X-Bugzilla-Reason: AssignedTo
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: DPDK
X-Bugzilla-Component: vhost/virtio
X-Bugzilla-Version: 22.11
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: normal
X-Bugzilla-Who: linglix.chen@intel.com
X-Bugzilla-Status: RESOLVED
X-Bugzilla-Resolution: FIXED
X-Bugzilla-Priority: Normal
X-Bugzilla-Assigned-To: dev@dpdk.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: bug_status resolution
Message-ID: <bug-1338-3-6lx6ZO1tM3@http.bugs.dpdk.org/>
In-Reply-To: <bug-1338-3@http.bugs.dpdk.org/>
References: <bug-1338-3@http.bugs.dpdk.org/>
Content-Type: multipart/alternative; boundary=17060891260.3366Ccf33.755514
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://bugs.dpdk.org/
Auto-Submitted: auto-generated
X-Auto-Response-Suppress: All
MIME-Version: 1.0
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org


--17060891260.3366Ccf33.755514
Date: Wed, 24 Jan 2024 10:38:46 +0100
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://bugs.dpdk.org/
Auto-Submitted: auto-generated
X-Auto-Response-Suppress: All

https://bugs.dpdk.org/show_bug.cgi?id=3D1338

linglix.chen@intel.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from linglix.chen@intel.com ---
fix commit dc9c799c7d vhost: fix missing spinlock unlock
Two regressions were introduced when backporting below
patch:
b4c4e5675c85 ("vhost: fix missing lock protection in power monitor API")

First, rte_vhost_get_monitor_addr did not release the lock
in the success case. Then, rte_rwlock_read_lock() was
converted to rte_spinlock_trylock() instead of
rte_spinlock_lock().

This patch addresses both of these issues.

Fixes: a07736eb68da ("vhost: fix missing lock protection in power monitor A=
PI")
Cc: stable@dpdk.org

Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>

--=20
You are receiving this mail because:
You are the assignee for the bug.=

--17060891260.3366Ccf33.755514
Date: Wed, 24 Jan 2024 10:38:46 +0100
MIME-Version: 1.0
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://bugs.dpdk.org/
Auto-Submitted: auto-generated
X-Auto-Response-Suppress: All

<html>
    <head>
      <base href=3D"https://bugs.dpdk.org/">
    </head>
    <body><span class=3D"vcard"><a class=3D"email" href=3D"mailto:linglix.c=
hen&#64;intel.com" title=3D"linglix.chen&#64;intel.com">linglix.chen&#64;in=
tel.com</a>
</span> changed
          <a class=3D"bz_bug_link=20
          bz_status_RESOLVED  bz_closed"
   title=3D"RESOLVED FIXED - [dpdk-22.11.4] vhost_virtio_user_interrupt_wit=
h_power_monitor/perf_wake_up_split_ring_vhost_user_core_with_l3fwd_power_sa=
mple=EF=BC=9Avirtio-user can't receive packet"
   href=3D"https://bugs.dpdk.org/show_bug.cgi?id=3D1338">bug 1338</a>
          <br>
             <table border=3D"1" cellspacing=3D"0" cellpadding=3D"8" class=
=3D"bz_diff_table">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style=3D"text-align:right;">Status</td>
           <td>UNCONFIRMED
           </td>
           <td>RESOLVED
           </td>
         </tr>

         <tr>
           <td style=3D"text-align:right;">Resolution</td>
           <td>---
           </td>
           <td>FIXED
           </td>
         </tr></table>
      <p>
        <div class=3D"bz_comment_block">
            <b class=3D"bz_comment_head"><a class=3D"bz_bug_link=20
          bz_status_RESOLVED  bz_closed"
   title=3D"RESOLVED FIXED - [dpdk-22.11.4] vhost_virtio_user_interrupt_wit=
h_power_monitor/perf_wake_up_split_ring_vhost_user_core_with_l3fwd_power_sa=
mple=EF=BC=9Avirtio-user can't receive packet"
   href=3D"https://bugs.dpdk.org/show_bug.cgi?id=3D1338#c2">Comment # 2</a>
              on <a class=3D"bz_bug_link=20
          bz_status_RESOLVED  bz_closed"
   title=3D"RESOLVED FIXED - [dpdk-22.11.4] vhost_virtio_user_interrupt_wit=
h_power_monitor/perf_wake_up_split_ring_vhost_user_core_with_l3fwd_power_sa=
mple=EF=BC=9Avirtio-user can't receive packet"
   href=3D"https://bugs.dpdk.org/show_bug.cgi?id=3D1338">bug 1338</a>
              from <span class=3D"vcard"><a class=3D"email" href=3D"mailto:=
linglix.chen&#64;intel.com" title=3D"linglix.chen&#64;intel.com">linglix.ch=
en&#64;intel.com</a>
</span>
            </b>
          <pre class=3D"bz_comment_text">fix commit dc9c799c7d vhost: fix m=
issing spinlock unlock
Two regressions were introduced when backporting below
patch:
b4c4e5675c85 (&quot;vhost: fix missing lock protection in power monitor API=
&quot;)

First, rte_vhost_get_monitor_addr did not release the lock
in the success case. Then, rte_rwlock_read_lock() was
converted to rte_spinlock_trylock() instead of
rte_spinlock_lock().

This patch addresses both of these issues.

Fixes: a07736eb68da (&quot;vhost: fix missing lock protection in power moni=
tor API&quot;)
Cc: <a href=3D"mailto:stable&#64;dpdk.org">stable&#64;dpdk.org</a>

Signed-off-by: Maxime Coquelin &lt;<a href=3D"mailto:maxime.coquelin&#64;re=
dhat.com">maxime.coquelin&#64;redhat.com</a>&gt;
          </pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
      <div itemscope itemtype=3D"http://schema.org/EmailMessage">
        <div itemprop=3D"action" itemscope itemtype=3D"http://schema.org/Vi=
ewAction">
=20=20=20=20=20=20=20=20=20=20
          <link itemprop=3D"url" href=3D"https://bugs.dpdk.org/show_bug.cgi=
?id=3D1338#c2">
          <meta itemprop=3D"name" content=3D"View bug">
        </div>
        <meta itemprop=3D"description" content=3D"Bugzilla bug update notif=
ication">
      </div>
    </body>
</html>=

--17060891260.3366Ccf33.755514--