21/07/2021 02:17, Long Li: > From: Stephen Hemminger <[email protected]> > > > > Looks good, minor comment. You don't have to check for NULL before calling > > rte_free(). > > Rte_free(NULL) is a NOP like free(NULL). > > > > Sorry for top posting; but if you send to my Microsoft account you are stuck > > with what Outlook can do... > > > From: jerb <[email protected]> > > > > Have secondary processes construct their own copy of primary channel with > > own mappings. > > > > Remove vmbus_channel primary ptr from struct mapped_vmbus_resource as > > its not used. > > > > Populate virtual memory address "addr" in struct rte_mem_resource for > > secondary processes as netvsc will attempt to reference it thus causing a > > crash. > > It was initialized for primary processes but not for secondary. > > > > Signed-off-by: jerb <[email protected]>
Please give your complete name. > Looks good. > > This should also go to stable. Please send a v2 with Fixes and Cc: [email protected] lines as documented in the contributor's guide. If you don't know how to do, the maintainers can help. Thank you
