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 867CB42C12; Fri, 2 Jun 2023 21:45:53 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D516D42D4A; Fri, 2 Jun 2023 21:45:27 +0200 (CEST) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2051.outbound.protection.outlook.com [40.107.220.51]) by mails.dpdk.org (Postfix) with ESMTP id EEA7C42D48 for ; Fri, 2 Jun 2023 21:45:26 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=HPy3+Sh50UmZAaR4kFs4CFmgfCDnMkFgXblLyAo2VWnqvyC1g7/qdqHJuDE9Xkd6AyEcJIQiFnONi31auXUbt2LgFY2LQIgJFQx4NOrJBkljDKkRvoFjawf/sTXtDuAWW2XuSXRJQbn1wDm+K/yzZnn/NZbsSrqT9oD7VAhBulhycZqJlUMuVPvL7+TiokpBC5BjOtAE0pCv0Gi+iHhls3j9lbQy3UvHHiX7m9syrbWsXaJto25m49lTO4GcOflFZ3qt2jJBkblLAVRYmGTDQqKGaX8PrEukbcelEOsdZzK32A8IWdoDCD12ErEZfVzi28dQRXJl2okHvP23ls0miw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=1EyLRCoEdDWJiZyYz49gn4cicY57QWHieNa2xb/fZK8=; b=Rsmtio/QC5qyTGOVHN1mrw1pBRqs5b++ZQ18K2YrA8dctCz98VxNprPa2eOmL6FkAF8wnCnSGKZTGhmpHWjwXrjsy2mnHS11K2yRJ33bRmvLix6pMmGyM4w8DawRsKEfhDC0rfDuPeH24HUDXs9Me/lrZ3dG5DKRorqGiUS5J4mos8qQyadfWep/acPlGnn6KWrUqWt9JtSgBQOkafzJZwZc22H4lI4b5Aw2Yfkcf4O+NNP2K3/ONNXCNn+od4X51uJvzl6FKqrhR8EErxfIF3ae5cJH2B/C6iLg3u9uUUSRFRj7sf7/prvj/9VaqqrBG4Rw7FRs/VM42mDmPan3/A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=1EyLRCoEdDWJiZyYz49gn4cicY57QWHieNa2xb/fZK8=; b=WgjF62RC3wrkDLWT5Os4hxai2juKwniaXdZdGTCrGQpWBw1eiaxuKQkt75aSZN2Ybqv52aXcRdkrTWEPzPXIFdUzu1g6O1x5zF6oXdcoPjCQl8Ve2acL/VpJ5lru/hFqAwJUoT9r3Zo7u3wBhtu44rb00VLG7+/GeKBYkjaJ7jE= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com; Received: from CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) by MW4PR12MB6975.namprd12.prod.outlook.com (2603:10b6:303:20a::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6455.23; Fri, 2 Jun 2023 19:45:24 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::7957:641d:6aba:3f9a]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::7957:641d:6aba:3f9a%4]) with mapi id 15.20.6433.022; Fri, 2 Jun 2023 19:45:24 +0000 Message-ID: <26ac3085-7b67-3b25-37a0-f4a852682ab5@amd.com> Date: Fri, 2 Jun 2023 20:45:18 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.11.2 Subject: Re: [RFC v3 2/3] examples/ptpclient: refine application Content-Language: en-US To: Simei Su , thomas@monjalon.net, andrew.rybchenko@oktetlabs.ru, kirill.rybalchenko@intel.com, qi.z.zhang@intel.com Cc: dev@dpdk.org, wenjun1.wu@intel.com References: <20230403092248.81551-1-simei.su@intel.com> <20230522132332.102030-1-simei.su@intel.com> <20230522132332.102030-3-simei.su@intel.com> From: Ferruh Yigit In-Reply-To: <20230522132332.102030-3-simei.su@intel.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: LO6P265CA0009.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:339::14) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|MW4PR12MB6975:EE_ X-MS-Office365-Filtering-Correlation-Id: ac8264dd-174e-42d8-31dd-08db63a1e84f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: wpsW3kl2kAA2v1vQME4HuLh+7QXf3K4A/3nPSkkArPDEfqVZSnWLcj6dNrq8vDrRH5sQ1LAviOBaKs16F2qbEXhFk4+5CfOdS9jnwYe/cc6v+0zmKK5pn/J3jl2B6n/1tU5p6+2kP9gcABXkWV2CNWCveeSZLxSeNRqvJO3swBp5Bmwt3TQkJAkLZz2akMbuK45Y/1Q58OpU7jRkC4k1rxaFh/juwPX1ZwAfN35XLknZXuKr236SP5mNO+M7VFKzHeLpOzHN1NPQBS81EWFBOcmTSHpQ+AaJddxRcIX07aipUzJ3J0u8lCNHtm6PMzcUD/JtWZec5aJs2X7CgBrFvewueWsO7PGvzoVKusU0jvP4CjZVSBodVQ7U841vhZbsvg0oYjlB8kLmQ4eXoRq6xgIo6yJKoxIRgrshb5HK9y1q66wPz6b1waUy94TcCbTxAj0Ia93sE1Qyu0AIB1MQtIrNsF/cncnFmFek84Eib6KZuMxuXOvDmEdnT3PDR6PnEvBKM5YzMe7J3uBzFEB0qxDmPpRUCcQTSQjCxwHlWMvIl1mvWw12wcySLvNrA/U64DQpkymCQs52IdnaZUIZYuP06weHZCxnJZ6uvXh1n355WzMpsmhY/oYcNN/iYD1xbtGik8E/Sysz2Mettltw3w== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CH2PR12MB4294.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(396003)(136003)(376002)(39860400002)(346002)(366004)(451199021)(83380400001)(186003)(4744005)(2616005)(6666004)(4326008)(6486002)(66476007)(478600001)(66556008)(316002)(66946007)(2906002)(26005)(6512007)(6506007)(44832011)(5660300002)(53546011)(8676002)(41300700001)(8936002)(38100700002)(36756003)(31696002)(86362001)(31686004)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UTQvaGRlcDRCWEM5NmFjeTYzV3Bxemdadjc2am9Xa001U0NRYlhON3pnNHRY?= =?utf-8?B?cWZRY0ZOeVZoYnVSVUpsTy9FaU9lNFFzMEt0TUxtckFIMWREalFnRlNmVnla?= =?utf-8?B?dm5ycjh1VXR4ejNraGpqMUp5b2cxb1hsSzdXWGJXM1RSWHEwUmRGazdPQ3ZP?= =?utf-8?B?aDgyZDVxV2RtMWxjT1h6YWdrc0FBMFRlVDEwTDF1ekxTM3VkcVp6ODg4UFJs?= =?utf-8?B?WWtpck5BVTlwUW1FU0ZON3dITGNVMWpkSEdZdURGd0dsVUhJaStIRXRQNkov?= =?utf-8?B?cFlsU0JyaXNBSXlETjhuNThTY1JDRWEvMG9vTzhZMWEyTzlQUnhCNFdKQXRM?= =?utf-8?B?OEt1eE0ra05VSnBoMzhKQjQxUExhVTgzZ3pWWTFTWE9LUEtpU2t3TXNlcy9X?= =?utf-8?B?OHoxMUJidWgwSVJGS3ZacDhQRGx6blZjbVBNcFRPVlRXL1VralJRQUNic0tL?= =?utf-8?B?Z2NKQnJvejFRNUJpUjFmRGlHMVY1bytCSHBhbkw4Mzh1OXk1OEpNWG12ekVO?= =?utf-8?B?RTE0YkFFbXV0WWFja1pQbjlLbU1reDg0dHJvWnh5MTk4Mkc1ckFHNHlpZ0Nr?= =?utf-8?B?VTNnRTYwSk1sNjZad0V3cldQWlpmNW4yU1IwWE1hVTg0N2I2STY1UUNLakJy?= =?utf-8?B?QUFhMnNIUXREZFBPKzA3SzBhSzZBb1p3SU1GMDdabC9iT0FuWTRzSk9wVXFY?= =?utf-8?B?REZUVSsxMkFDWitncFM0d0VNYjQyUXlRWGRiZVZBQWxnTjk3WWVzWUMydWY1?= =?utf-8?B?eTJBVWp6KzFCQUhOY0t1SDJDUjNQUVdFb2M0bFNaRnpKTHREYjRLOEJtNkFh?= =?utf-8?B?YmRiOFRHMUs2ZHp1b2dCRWN3RlZxNk95U0paNHg3aVh6MTdlMTN0cDhiQXBB?= =?utf-8?B?bHhFSzIvR3hJRmp2VlN6dmNNUmZYMUs1ZzMxTm9GOXo3dW9kaFE1TlBVNmFr?= =?utf-8?B?WnN6ckQrWG1YSDJJVVg3cFJvOW9CcTlzOHFvSHRDcW0xSHZQKzQ2Q0hlelpK?= =?utf-8?B?VVFPb08yR0gvRGpEQXdvQXJXQkJnSFpIeXI4VkROb2dKS1MzT05Oby96RnRX?= =?utf-8?B?S01LTW43cUpoVG4wbGpaa3FHQ2k3c1JxQi9UV3NCblRIaGpTVUxWOHMyaktV?= =?utf-8?B?eG5sVko4NHVncFBKaG5KZU1UdHpnd3dzWXVheDdNWU11NUhSMVgwZk1nb2dN?= =?utf-8?B?enlJdkUxQzBRcHppRkZQdi8yRTIweUJYRUNmVGtxWHNKM0FuUjEyWGJPNFk2?= =?utf-8?B?TVUvcTFYZlVWdXg4Vmp3TEZpay9HVTBONU9WeE9ZL1VZZW9BOWFsT2tlenI2?= =?utf-8?B?ZHUyQ3NBRGxVZ29WdExudXNkdy9NU2U1aTJaK3N5Y3NYNzZ6NksxTlkrT2Fo?= =?utf-8?B?Ty9MMm5wem10Zkd0UTZMOEpoTXdZSUJtZnI0UTRWSXYra3l6MmNhOUZuZThn?= =?utf-8?B?SUd6ZitPMDVyQVcvRzdOajBHNDRuYjdSVzFtWXl5Q0Jnb1B4clliNy9MMzlB?= =?utf-8?B?bHpGSG5SM3VsSjA1ajg4WHF6Qk1PSUQrTk5LVWJrczFpUFBRZGN3R282R1o2?= =?utf-8?B?TjhPU3habERVemtvcExveVo3RU5XT1BqZ3BKRDNQWkdvMFBvZGtPVkdxZ0la?= =?utf-8?B?SEpEdy9NQzg0bUNEOXBZWjNjOStkYlJ2bjRPVGQySFp2Z3AvYUY0WTZzeVl2?= =?utf-8?B?dTdDOUxEdjYvVjVkVDZWUGJzOGVpMlZ5YmU0ejBSaG1qRXRYeFkvY1NjN0gv?= =?utf-8?B?TE1xdGJaMzE1c2pOY1J6SWFrcktkZlYyQ0xaS1FiZ0UrR0p4MGsxWHdVWUR3?= =?utf-8?B?eHg2M00zbW1DankrTFo0NFhRZ3dKYklVYXQ1SnQ3MTA5MVN5UmRKcTBjQ1ZX?= =?utf-8?B?TFlrZ3ZmUk1kcmY0Vk9GM0ppSlZZNllJVm5nMUxFODE1TXlVSzRHaVJMK2kr?= =?utf-8?B?ZTVsaWlwOCtmU3dCQyt2SGd6L2svVzJmclRhN0pla0h6eHhrY055NThiaTJt?= =?utf-8?B?VEhHV2xzQmg5TmVmSU5KSzVZd2xhMXRFU3Mza2RjKzdFMzlMMFV4SEt0Q1NK?= =?utf-8?B?VlZvSkpmbElWZG1VSjBWRWVRbU5xZm5UVU9XTEdzVDlDREdjaWluSTg3MHd5?= =?utf-8?Q?l5GRY7qLFrZhjgrblzeudjqnX?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: ac8264dd-174e-42d8-31dd-08db63a1e84f X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jun 2023 19:45:24.5409 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: YYgUhgL2S2x3M2c7k2GyHtCqvMsB+v4K5vjukGGKHGzQ8b2QDId7fjFCALiGs3W7 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB6975 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On 5/22/2023 2:23 PM, Simei Su wrote: > This patch reworks code to split delay request message parsing > from follow up message parsing which doesn't break original logic. > > Signed-off-by: Simei Su > Signed-off-by: Wenjun Wu > Refactoring looks good to me.