Re: AoE for 4.x

From: Julian Elischer (julian_at_elischer.org)
Date: 09/22/04

  • Next message: Sam: "Re: AoE for 4.x" 
    Date: Tue, 21 Sep 2004 15:50:39 -0700
To: Sam <sah@softcardsystems.com>

    Sam wrote:

    >>
    >>
    >> I guess we are used to it.
    >>
    >> I can't recommend enough the utility of having a cvs mirror and using
    >> cvs to keep your
    >> work trees sync'd against FreeBSD. It is also a goodd way of making
    >> sure that you don't
    >> forget parts of the diff (e.g. Makefiles etc.) as CVS will pick up
    >> all the changed files.
    >>
    >
    > And I can't thank you enough. I forgot about a few other (rather)
    > important file changes this picked up.
    >
    > A patchfile for aoe against today's 4.10 source tree
    > is at:
    > http://www.coraid.com/support/freebsd/aoe.patch
    >
    > A few notes:
    >
    > The file sys/dev/aoe/aoe4bsd.dd is basic documentation for
    > the driver.
    >
    > All references to chr major 200 will have to be changed to whatever
    > gets assigned, obviously.
    >
    > The netisr selected in sys/net/netisr.h was picked at random. Is it
    > OK?
    >
    > The label typeunknown in sys/net/if_ethersubr.c is added to give
    > netgraph a chance to get the packets if AoE is not loaded. Appropriate?
    >
    > I dinna know how to fill out the copyright/license comment in the
    > headers,
    > but i gave it a shot. Thoughts here would be appreciated.
    >
    > TIA to all the curious eyes that skim the driver for glaring flaws.

    haven't read it yet, just skimmed..

    Very CLEAN looking code.. style(9) complient to a large extent..
    (you do know about 'man 9 style' right?)

    now that it is written, this is the time to
    go through it and write down everything that comes to mind in comments..
    Everything that a person trying to decode the driver might want to know.
    Including comments about the standards, tricky bits of code,
    reasons you decided to do things in certain ways. etc.

    >
    >
    > Cheers,
    >
    > Sam

    _______________________________________________
    freebsd-arch@freebsd.org mailing list
    http://lists.freebsd.org/mailman/listinfo/freebsd-arch
    To unsubscribe, send any mail to "freebsd-arch-unsubscribe@freebsd.org"

  • Next message: Sam: "Re: AoE for 4.x"

    Relevant Pages

    • Re: AoE for 4.x
      ... > Sam wrote: ... >> I decided to write a 4.x driver thinking it would a) be easier to port ... > I can't recommend enough the utility of having a cvs mirror and using cvs to ... netgraph a chance to get the packets if AoE is not loaded. ...
      (freebsd-arch)
    • Re: AoE for 4.x
      ... > Sam wrote: ... >> important file changes this picked up. ... >> netgraph a chance to get the packets if AoE is not loaded. ... >> TIA to all the curious eyes that skim the driver for glaring flaws. ...
      (freebsd-arch)
    • Re: pscript5 customization
      ... I downloaded your rleased code from cvs (which installs ok) but throws ... exceptions in a few places once i tried playing with it. ... To get some direction, checkout my driver, well it won't ...
      (microsoft.public.development.device.drivers)
    • ANN: DBXOODBC 3.025
      ... DbxOODBC 3.025 - Kylix/Delphi Open Source ODBC dbExpress driver. ... + ReAllocation of memory at reading BLOBS is excluded. ... Project Sources (Browse CVS): ... echo - When prompted for a password for anonymous, ...
      (borland.public.delphi.thirdpartytools.general)
    • Re: USB WiFi
      ... > Is anyone using USB WiFi devices with Fedora? ... > But I can't find an orinoco-usb CVS. ... where is the orinoco-usb CVS? ... > the building and installation of the driver. ...
      (Fedora)