Thanks for suggestion. Will clean up code in next revision Thanks, Satha
-----Original Message----- From: Sunil Kovvuri [mailto:sunil.kovv...@gmail.com] Sent: Wednesday, December 21, 2016 4:44 AM To: Koteshwar Rao, Satha Cc: LKML; Goutham, Sunil; Robert Richter; David S. Miller; Daney, David; Vatsavayi, Raghu; Chickles, Derek; Romanov, Philip; Linux Netdev List; LAKML Subject: Re: [RFC PATCH 5/7] Multiple VF's grouped together under single physical port called PF group PF Group maintainance API's On Wed, Dec 21, 2016 at 2:16 PM, Satha Koteswara Rao <satha....@caviumnetworks.com> wrote: > +struct tns_global_st { > + u64 magic; > + char version[16]; > + u64 reg_cnt; > + struct table_static_s tbl_info[TNS_MAX_TABLE]; }; > + > +#define PF_COUNT 3 > +#define PF_1 0 > +#define PF_2 64 > +#define PF_3 96 > +#define PF_END 128 Some comments please ... what is 0, 64, 96 ?? You can read PCI_SRIOV_TOTAL_VF from PCI config space instead of defining PF_END with 128.