Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 20 Apr 2004 19:29:28 -0700 (PDT)
From:      Bill Paul <wpaul@FreeBSD.org>
To:        src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   cvs commit: src/sys/dev/if_ndis if_ndis.c
Message-ID:  <200404210229.i3L2TSes070256@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
wpaul       2004/04/20 19:29:28 PDT

  FreeBSD src repository

  Modified files:
    sys/dev/if_ndis      if_ndis.c 
  Log:
  Fix the problems people have been having with the Intel 2100B Centrino
  wireless ever since I added the new spinlock code. Previously, I added
  a special ndis_rxeof_serial() function to insure that when we receive
  a packet, we never end up calling the MiniportReturnPacket() routine
  until after the receive handler has finished. I set things up so that
  ndis_rxeof_serial() would only be used for serialized miniports since
  they depend on this property. Well, it turns out deserialized miniports
  depend on a similar property: you can't let MiniportReturnPacket() be
  called from the same context as the receive handler at all. The 2100B
  driver happens to use a single spinlock for all of its synchronization,
  and it tries to acquire it both while in MiniportHandleInterrupt() and
  in MiniportReturnPacket(), so if we call MiniportReturnPacket() from
  the MiniportHandleInterrupt() context, we will end up trying to acquire
  the spinlock recursively, which you can't do.
  
  To fix this, I made the ndis_rxeof_serial() handler the default. An
  alternate solution would be to make ndis_return_packet() submit
  the call to MiniportReturnPacket() to the NDIS task queue thread.
  I may do that in the future, after I've tested things a bit more.
  
  Revision  Changes    Path
  1.54      +19 -107   src/sys/dev/if_ndis/if_ndis.c



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200404210229.i3L2TSes070256>