On Mon, 29 Nov 2004, Gerrit Huizenga wrote:

> +#ifdef CONFIG_ACCEPT_QUEUES
> +     tp->class_index = 0;
> +     for (i=0; i < NUM_ACCEPT_QUEUES; i++) {
> +             tp->acceptq[i].aq_tail = NULL;
> +             tp->acceptq[i].aq_head = NULL;
> +             tp->acceptq[i].aq_wait_time = 0; 
> +             tp->acceptq[i].aq_qcount = 0; 
> +             tp->acceptq[i].aq_count = 0; 
> +             if (i == 0) {
> +                     tp->acceptq[i].aq_ratio = 1; 
> +             }
> +             else {
> +                     tp->acceptq[i].aq_ratio = 0; 
> +             }
> +     }
> +#endif

Do not litter the networking code with #ifdef sections.  Use functions
which are defined to nothing via static inline if disabled.


- James
-- 
James Morris
<[EMAIL PROTECTED]>




-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. 
http://productguide.itmanagersjournal.com/
_______________________________________________
ckrm-tech mailing list
https://lists.sourceforge.net/lists/listinfo/ckrm-tech

Reply via email to