From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2E486A04B0 for ; Tue, 22 Sep 2020 14:06:07 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 919D31DC12; Tue, 22 Sep 2020 14:06:05 +0200 (CEST) Received: from huawei.com (szxga08-in.huawei.com [45.249.212.255]) by dpdk.org (Postfix) with ESMTP id BF9251DB07; Tue, 22 Sep 2020 14:06:02 +0200 (CEST) Received: from DGGEMM402-HUB.china.huawei.com (unknown [172.30.72.56]) by Forcepoint Email with ESMTP id CB291E4F00A1A85A729B; Tue, 22 Sep 2020 20:06:00 +0800 (CST) Received: from DGGEMM533-MBX.china.huawei.com ([169.254.5.102]) by DGGEMM402-HUB.china.huawei.com ([10.3.20.210]) with mapi id 14.03.0487.000; Tue, 22 Sep 2020 20:05:50 +0800 From: wangyunjian To: Harman Kalra CC: "dev@dpdk.org" , "david.marchand@redhat.com" , "ferruh.yigit@intel.com" , Thomas Monjalon , "Lilijun (Jerry)" , xudingke , "stable@dpdk.org" Thread-Topic: [EXT] RE: [dpdk-dev] [PATCH 1/1] eal/linux: fix event set error for rte_epoll_event Thread-Index: AQHWWlJiiz2rd9JhOEuRRtYaP5NCAalKd5rwgCW2oBCABC4lgIAAn2iw Date: Tue, 22 Sep 2020 12:05:50 +0000 Message-ID: <34EFBCA9F01B0748BEB6B629CE643AE60DA6F3D5@DGGEMM533-MBX.china.huawei.com> References: <2dc88362a7dc347547eb24e5d0570db11a59598e.1594780487.git.wangyunjian@huawei.com> <34EFBCA9F01B0748BEB6B629CE643AE60DA5CEA2@dggemm513-mbx.china.huawei.com> <20200922103141.GA54896@outlook.office365.com> In-Reply-To: <20200922103141.GA54896@outlook.office365.com> Accept-Language: en-US Content-Language: zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.174.185.168] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-CFilter-Loop: Reflected Subject: Re: [dpdk-stable] [EXT] RE: [dpdk-dev] [PATCH 1/1] eal/linux: fix event set error for rte_epoll_event X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 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 Sender: "stable" > -----Original Message----- > From: Harman Kalra [mailto:hkalra@marvell.com] > Sent: Tuesday, September 22, 2020 6:32 PM > To: wangyunjian > Cc: dev@dpdk.org; david.marchand@redhat.com; ferruh.yigit@intel.com; > Thomas Monjalon ; Lilijun (Jerry) > ; xudingke ; > stable@dpdk.org > Subject: Re: [EXT] RE: [dpdk-dev] [PATCH 1/1] eal/linux: fix event set er= ror for > rte_epoll_event >=20 > On Sat, Sep 19, 2020 at 10:42:30AM +0000, wangyunjian wrote: > > External Email > > > > ---------------------------------------------------------------------- > > Friendly ping. >=20 > Hi Wang, >=20 > I am sorry for the delay in review, I will surely complete the review > by end of this week. > Can you please elaborate on the specific case/scenario which caused > failure i.e. which lead you to this change. EPOLLERR event will be triggered when an exception occurs. This event will = be set in 'evs.events', we should use 'evs.events' instead of 'rev->epdata.event' = to assign to 'events.epdata.event'. Otherwise, the EPOLLERR event will lost. Thanks, Yunjian >=20 > Thanks > Harman >=20 > > > > > -----Original Message----- > > > From: wangyunjian > > > Sent: Wednesday, August 26, 2020 6:56 PM > > > To: dev@dpdk.org; hkalra@marvell.com; david.marchand@redhat.com > > > Cc: 'ferruh.yigit@intel.com' ; 'Thomas Monjal= on' > > > ; Lilijun (Jerry) ; > > > xudingke ; stable@dpdk.org > > > Subject: RE: [dpdk-dev] [PATCH 1/1] eal/linux: fix event set error > > > for rte_epoll_event > > > > > > Ping for review, please. > > > > > > > -----Original Message----- > > > > From: wangyunjian > > > > Sent: Wednesday, July 15, 2020 10:40 AM > > > > To: dev@dpdk.org; hkalra@marvell.com > > > > Cc: Lilijun (Jerry) ; xudingke > > > > ; wangyunjian ; > > > > stable@dpdk.org > > > > Subject: [dpdk-dev] [PATCH 1/1] eal/linux: fix event set error for > > > > rte_epoll_event > > > > > > > > From: Yunjian Wang > > > > > > > > The "rev->epdata.event" assigned to "events.epdata.event" > > > > directly, which was wrong in case of epoll events. It should be set= to the > "evs.events". > > > > > > > > Fixes: 9efe9c6cdcac ("eal/linux: add epoll wrappers") > > > > Cc: stable@dpdk.org > > > > > > > > Signed-off-by: Yunjian Wang > > > > --- > > > > lib/librte_eal/linux/eal_interrupts.c | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/lib/librte_eal/linux/eal_interrupts.c > > > > b/lib/librte_eal/linux/eal_interrupts.c > > > > index 13db5c4e8..8a3c95cec 100644 > > > > --- a/lib/librte_eal/linux/eal_interrupts.c > > > > +++ b/lib/librte_eal/linux/eal_interrupts.c > > > > @@ -1237,7 +1237,7 @@ eal_epoll_process_event(struct epoll_event > > > > *evs, unsigned int n, > > > > events[count].status =3D RTE_EPOLL_VALID; > > > > events[count].fd =3D rev->fd; > > > > events[count].epfd =3D rev->epfd; > > > > - events[count].epdata.event =3D rev->epdata.event; > > > > + events[count].epdata.event =3D evs[i].events; > > > > events[count].epdata.data =3D rev->epdata.data; > > > > if (rev->epdata.cb_fun) > > > > rev->epdata.cb_fun(rev->fd, > > > > -- > > > > 2.23.0 > > > > > >