[dpdk-dev] [PATCH v2 5/9] eal: add additional function overrides in windows header files
Pallavi Kadam
pallavi.kadam at intel.com
Tue Oct 1 01:11:15 CEST 2019
Hi Jerin,
On 9/27/2019 1:06 AM, Jerin Jacob wrote:
> On Fri, Sep 27, 2019 at 2:25 AM Pallavi Kadam <pallavi.kadam at intel.com> wrote:
>> Adding additional function definitions for pthread, cpuset
>> implementation, asprintf implementation, in order to support
>> common code.
>>
>> Signed-off-by: Pallavi Kadam <pallavi.kadam at intel.com>
>> Signed-off-by: Antara Ganesh Kolar <antara.ganesh.kolar at intel.com>
>> Reviewed-by: Ranjit Menon <ranjit.menon at intel.com>
>> Reviewed-by: Keith Wiles <keith.wiles at intel.com>
>> ---
>> lib/librte_eal/common/include/rte_lcore.h | 5 ++
>> lib/librte_eal/windows/eal/include/pthread.h | 66 +++++++++++++++++++
>> lib/librte_eal/windows/eal/include/rte_os.h | 28 ++++++++
>> lib/librte_eal/windows/eal/include/sched.h | 58 ++++++++++++++--
>> .../windows/eal/include/sys/queue.h | 8 +++
>> 5 files changed, 159 insertions(+), 6 deletions(-)
>>
>> diff --git a/lib/librte_eal/common/include/rte_lcore.h b/lib/librte_eal/common/include/rte_lcore.h
>> index c86f72eb1..d5e7e3e33 100644
>> --- a/lib/librte_eal/common/include/rte_lcore.h
>> +++ b/lib/librte_eal/common/include/rte_lcore.h
>> @@ -64,6 +64,11 @@ typedef cpuset_t rte_cpuset_t;
>> CPU_NAND(&tmp, src); \
>> CPU_COPY(&tmp, dst); \
>> } while (0)
>> +#elif defined(_WIN64)
>> +#define RTE_CPU_AND(dst, src1, src2) CPU_AND(dst, src1, src2)
>> +#define RTE_CPU_OR(dst, src1, src2) CPU_OR(dst, src1, src2)
>> +#define RTE_CPU_FILL(set) CPU_FILL(set)
>> +#define RTE_CPU_NOT(dst, src) CPU_NOT(dst, src)
> RTE_CPU* definitions are OS-specific. Instead of adding new #elif and
> make the common
> code less readable. Please create eal abstraction for the same and
> move Linux, FreeBSD, Windows to the specific
> directory to avoid #ifdef clutter in lib/librte_eal/common/include/rte_lcore.h.
Thanks for the comments.
We have submitted a patch to move RTE_CPU* definitions to the
OS-specific directory.
Please ACK when possible. https://patches.dpdk.org/patch/60244/.
Thanks,
>
>> #endif
>>
More information about the dev
mailing list