> Is it really necessary to have AFS_GLOCK()/GUNLOCK() wrapped with an
> #ifdef RX_ENABLE_LOCKS everywhere in the source? It appears that the
> definitions of those macros are #ifdef'd already. Seems like it would
> clean up the code quite a bit to get rid of all the spurious ifdef's. 

There's RX_AFS_GLOCK() and RX_AFS_GUNLOCK() in the mainline (which
I snuck in with the dcache locking changes), which are defined to
be AFS_GLOCK/AFS_GUNLOCK when RX_ENABLE_LOCKS is set, and blank
otherwise.  So, you can replace the #ifdef/AFS_GLOCK/#endif combo
with RX_AFS_GLOCK, etc, if you want.

-- kolya
_______________________________________________
OpenAFS-devel mailing list
[EMAIL PROTECTED]
https://lists.openafs.org/mailman/listinfo/openafs-devel

Reply via email to