Re: [PATCH][ATM] assorted he driver cleanup

chas williams (chas@cmf.nrl.navy.mil)
Sun, 01 Jun 2003 22:34:24 -0400


In message <20030601.184254.71111683.davem@redhat.com>,"David S. Miller" writes:
>1) Aparently the bug only needs to be worked around when
> multiple cpus can access the card at the same time.
> Therefore on uniprocessor the bug isn't relevant.

right. which is why i optimized the HE_SPIN_LOCK away on
!CONFIG_SMP. actually, its probably not a problem on certain
smp machines either. i dont believe the i386 will reorder
read/writes from multiple cpus so in theory it would be safe to
do away with this lock on smp i386's. the only arch that
can reorder (and actually has posted read/writes) is the ia64/sn2.

>I personally don't think it's worth all the maintainence cost
>to special case all of this junk for uniprocessor.

well it wasnt much cost but i essentially decided this some
time ago and just defined HE_SPIN_LOCK to be spin_lock_irqsave.
the lock probably needs to be more fine-grained though in the
tasklet handler though.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/