git: if_tun: Use pointer to ifnet in tun_softc struct
Aaron LI
aly at aaronly.me
Thu Jul 19 21:13:56 PDT 2018
OK, thanks.
I'll keep looking at the net code, and can improve it later.
On Fri, 20 Jul 2018 12:07:00 +0800
Sepherosa Ziehau <sepherosa at gmail.com> wrote:
> Well, I don't think if_alloc makes sense for Dfly. But if you have
> already changed some code, then let it be.
>
> On Fri, Jul 20, 2018 at 11:19 AM, Aaron LI <aly at aaronly.me> wrote:
> > On Fri, 20 Jul 2018 10:43:16 +0800
> > Sepherosa Ziehau <sepherosa at gmail.com> wrote:
> >
> >> On Fri, Jul 20, 2018 at 10:19 AM, Aaron LI <aly at aaronly.me> wrote:
> >> > On Fri, 20 Jul 2018 09:40:21 +0800
> >> > Sepherosa Ziehau <sepherosa at gmail.com> wrote:
> >> >
> >> >> I am not quite sure why we want to use if_alloc() in the first
> >> >> place?
> >> >
> >> > Since our if_alloc() is very simple, so there is no big benefit to use
> >> > it over the direct kmalloc(). But if_alloc() is easier to use and more
> >> > intuitive.
> >>
> >> I mean why allocate in the first place? What's wrong w/ embedding?
> >
> > There is nothing wrong with embedding ifnet inside tun_softc.
> >
> > So do you think is it better to embed or not embed the ifnet? I'll follow
> > your advice. Thanks.
> >
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.dragonflybsd.org/pipermail/commits/attachments/20180720/a3216bbe/attachment-0002.bin>
More information about the Commits
mailing list