> -----Original Message----- > From: Ferruh Yigit <ferruh.yi...@intel.com> > Sent: Wednesday, October 10, 2018 10:48 PM > To: Phil Yang (Arm Technology China) <phil.y...@arm.com>; dev@dpdk.org > Cc: jerin.ja...@caviumnetworks.com; Gavin Hu (Arm Technology China) > <gavin...@arm.com>; Honnappa Nagarahalli > <honnappa.nagaraha...@arm.com>; Ola Liljedahl <ola.liljed...@arm.com> > Subject: Re: [PATCH v3 1/4] config: use one single config option for C11 > memory > model > > On 10/8/2018 10:11 AM, phil.y...@arm.com wrote: > > Keep only single config option RTE_USE_C11_MEM_MODEL for C11 memory > > model, so all modules can leverage C11 atomic extension by enable this > > option. > > > > Signed-off-by: Phil Yang <phil.y...@arm.com> > > Reviewed-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com> > > Reviewed-by: Gavin Hu <gavin...@arm.com> > > Acked-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> > > For series, > Reviewed-by: Ferruh Yigit <ferruh.yi...@intel.com> > > Are you planning to send a new version to include the clean up, or get this > one > first an do the cleanup on top of it? I think both are OK but please let us > know. > And if there will be a new version with cleanup, please keep the review tag.
Hi Ferruh, Thanks for your remind. I think it is better to merge this series first. Because the cleanup patch won't block any functionality. I will do it on top of this series. Thanks Phil Yang IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
