On Tue, 28 Dec 2021 15:01:20 +0800 Yanling Song <son...@ramaxel.com> wrote:
> On Fri, 24 Dec 2021 09:44:57 -0800 > Stephen Hemminger <step...@networkplumber.org> wrote: > > > On Fri, 24 Dec 2021 16:32:18 +0800 > > Yanling Song <son...@ramaxel.com> wrote: > > > > > The patchsets introduce SPNIC driver for Ramaxel's SPNxx serial NIC > > > cards into DPDK 22.03. Ramaxel Memory Technology is a company which > > > supply a lot of electric products: storage, communication, PCB... > > > SPNxxx is a serial PCIE interface NIC cards: > > > SPN110: 2 PORTs *25G > > > SPN120: 4 PORTs *25G > > > SPN130: 2 PORTs *100G > > > > > > The following is main features of our SPNIC: > > > - TSO > > > - LRO > > > - Flow control > > > - SR-IOV(Partially supported) > > > - VLAN offload > > > - VLAN filter > > > - CRC offload > > > - Promiscuous mode > > > - RSS > > > > > > v2->v3: > > > 1. Fix clang compiling failure. > > > > > > v1->v2: > > > 1. Fix coding style issues and compiling failures; > > > 2. Only support linux in meson.build; > > > 3. Use CLOCK_MONOTONIC_COARSE instead of > > > CLOCK_MONOTONIC/CLOCK_MONOTONIC_RAW; 4. Fix time_before(); > > > 5. Remove redundant checks in spnic_dev_configure(); > > > > Overall looks good. > > > > Please use a consistent prefix to all globally visible symbols to > > avoid any possible name clashes when statically linking. > > > > $ nm ./build/drivers/librte_net_spnic.a | grep ' t ' | grep -v > > spnic_ | grep -v rte_ 00000000000006f0 t remove_aeq > > 0000000000000040 t fault_event_handler > > 0000000000000060 t ffm_event_msg_handler > > 00000000000016c0 t alloc_mbox_info > > 0000000000000ca0 t send_mbox_to_func > > 0000000000000890 t send_tlp_mbox_to_func > > 00000000000000c0 t pciinitfn_net_spnic > > 0000000000000890 t clean_queue_offload_ctxt > > > > > Good point. There are some external functions which have no spnic_ > prefix. and there are also some static functions which have no > spnic_ prefix. > Since static functions only works in the file, it doesn't matter to add > prefix or not. My plan is to add spnic_ prefix to those external > functions. Is it ok to you? > > Yes, that is what this script was looking for, external functions with no prefix
