Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-20300

[TCPIP, TDI, ...] Sort out NDIS40, and NDIS50, define usages

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Unresolved
    • Major
    • None
    • Networking
    • None

    Description

      With regard to
      https://git.reactos.org/?p=reactos.git;a=blob;f=sdk/include/ddk/ndis.h;hb=HEAD#l156

       157 #if defined(NDIS30)
       161 #if defined(NDIS620)
       166 #elif defined(NDIS61)
       171 #elif defined(NDIS60)
       176 #elif defined(NDIS51)
       179 #elif defined(NDIS50)
       182 #elif defined(NDIS40)
      

      https://git.reactos.org/?p=reactos.git&a=search&h=HEAD&st=grep&s=NDIS30
      Not actually used. Fine.

      https://git.reactos.org/?p=reactos.git&a=search&h=HEAD&st=grep&s=NDIS40
      Used by: tcpip, tdi.
      Upgrade these "core" modules to NDIS50, then/or NDIS51,
      or add a code comment to explain why they can/must remain as NDIS40.

      https://git.reactos.org/?p=reactos.git&a=search&h=HEAD&st=grep&s=NDIS50
      Used by: e1000, pcnet, rtl8139, ndisuio.
      Upgrade these ("core") modules to NDIS51,
      or add a code comment to explain why they can/must remain as NDIS50.

      https://git.reactos.org/?p=reactos.git&a=search&h=HEAD&st=grep&s=NDIS51
      Used by: dc21x4, ne2000, netkvm, nvnet, pcnet, ndis.
      I assume this is the wanted value, for ROS (as WXP/WS2003).

      Attachments

        Activity

          People

            Unassigned Unassigned
            Serge Gautherie Serge Gautherie
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated: