Oh, I'm sorry. That's just to silence the compiler because of the unused variable addr. We only need the paddr output value in that context.
On Thu, Apr 8, 2021, 12:07 AM Shawn Webb <shawn.w...@hardenedbsd.org> wrote: > No, just the one line: (void)addr; > > On Wed, Apr 07, 2021 at 11:58:45PM +0200, Vincenzo Maffione wrote: > > Hi, > > Do you mean the whole commit? This adds vtnet support for "offsets" in > > the netmap buffers. Main use case is that applications can easily > zerocopy > > push and pop protocol headers from packets. > > > > Cheers, > > Vincenzo > > > > On Wed, Apr 7, 2021, 11:46 PM Shawn Webb <shawn.w...@hardenedbsd.org> > wrote: > > > > > Hey Vincenzo, > > > > > > On Wed, Apr 07, 2021 at 09:42:53PM +0000, Vincenzo Maffione wrote: > > > > The branch main has been updated by vmaffione: > > > > > > > > URL: > > > > https://cgit.FreeBSD.org/src/commit/?id=15dc713ceb57d0d61c1dc54b1d550da42d250730 > > > > > > > > commit 15dc713ceb57d0d61c1dc54b1d550da42d250730 > > > > Author: Vincenzo Maffione <vmaffi...@freebsd.org> > > > > AuthorDate: 2021-04-07 21:32:20 +0000 > > > > Commit: Vincenzo Maffione <vmaffi...@freebsd.org> > > > > CommitDate: 2021-04-07 21:32:20 +0000 > > > > > > > > netmap: vtnet: add support for netmap offsets > > > > > > > > Follow-up change to a6d768d845c173823785c71bb18b40074e7a8998. > > > > This change adds support for netmap offsets. > > > > --- > > > > sys/dev/netmap/if_vtnet_netmap.h | 16 ++++++++++------ > > > > 1 file changed, 10 insertions(+), 6 deletions(-) > > > > > > > > diff --git a/sys/dev/netmap/if_vtnet_netmap.h > > > b/sys/dev/netmap/if_vtnet_netmap.h > > > > index a05781255218..a423e71331be 100644 > > > > --- a/sys/dev/netmap/if_vtnet_netmap.h > > > > +++ b/sys/dev/netmap/if_vtnet_netmap.h > > > > @@ -84,12 +84,14 @@ vtnet_netmap_txsync(struct netmap_kring *kring, > int > > > flags) > > > > for (; nm_i != head; nm_i = nm_next(nm_i, lim)) { > > > > /* we use an empty header here */ > > > > struct netmap_slot *slot = &ring->slot[nm_i]; > > > > + uint64_t offset = nm_get_offset(kring, slot); > > > > u_int len = slot->len; > > > > uint64_t paddr; > > > > void *addr = PNMB(na, slot, &paddr); > > > > int err; > > > > > > > > - NM_CHECK_ADDR_LEN(na, addr, len); > > > > + (void)addr; > > > > > > What is this change for? > > > > > > Thanks, > > > > > > -- > > > Shawn Webb > > > Cofounder / Security Engineer > > > HardenedBSD > > > > > > > > > > https://git.hardenedbsd.org/hardenedbsd/pubkeys/-/raw/master/Shawn_Webb/03A4CBEBB82EA5A67D9F3853FF2E67A277F8E1FA.pub.asc > > > > > -- > Shawn Webb > Cofounder / Security Engineer > HardenedBSD > > > https://git.hardenedbsd.org/hardenedbsd/pubkeys/-/raw/master/Shawn_Webb/03A4CBEBB82EA5A67D9F3853FF2E67A277F8E1FA.pub.asc > _______________________________________________ dev-commits-src-main@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main To unsubscribe, send any mail to "dev-commits-src-main-unsubscr...@freebsd.org"
