From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Subject: Re: [PATCH] ucc_geth: Rework the TX logic. Date: Mon, 30 Mar 2009 12:22:03 -0500 Message-ID: <49D0FFBB.7000009@freescale.com> References: <1238089445-28396-1-git-send-email-Joakim.Tjernlund@transmode.se> <2a27d3730903270245k6e8633eehfb5cd3fcebd36240@mail.gmail.com> <49CCD3E8.9040007@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Cc: linuxppc-dev@ozlabs.org, Li Yang , pku.leo@gmail.com, netdev@vger.kernel.org To: Joakim Tjernlund Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane.org@ozlabs.org Errors-To: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane.org@ozlabs.org List-Id: netdev.vger.kernel.org Joakim Tjernlund wrote: > gianfar does not seem to use in_/out_ functions for the BDs. Works just > fine that too it seems. It does now that it has explicit barriers in a few places. Before they were added, it would sometimes fail under load. That was due to a compiler reordering, but CPU reordering was possible as well. -Scott