From owner-freebsd-sparc64@FreeBSD.ORG Fri Apr 15 03:09:38 2005 Return-Path: Delivered-To: freebsd-sparc64@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5D05416A4CE for ; Fri, 15 Apr 2005 03:09:38 +0000 (GMT) Received: from mail21.sea5.speakeasy.net (mail21.sea5.speakeasy.net [69.17.117.23]) by mx1.FreeBSD.org (Postfix) with ESMTP id E93AD43D2F for ; Fri, 15 Apr 2005 03:09:37 +0000 (GMT) (envelope-from jmg@hydrogen.funkthat.com) Received: (qmail 7942 invoked from network); 15 Apr 2005 03:09:37 -0000 Received: from gate.funkthat.com (HELO hydrogen.funkthat.com) ([69.17.45.168]) (envelope-sender ) by mail21.sea5.speakeasy.net (qmail-ldap-1.03) with SMTP for ; 15 Apr 2005 03:09:37 -0000 Received: from hydrogen.funkthat.com (bmtuhk@localhost.funkthat.com [127.0.0.1])j3F39ZGH082113; Thu, 14 Apr 2005 20:09:36 -0700 (PDT) (envelope-from jmg@hydrogen.funkthat.com) Received: (from jmg@localhost) by hydrogen.funkthat.com (8.12.10/8.12.10/Submit) id j3F39ZJE082112; Thu, 14 Apr 2005 20:09:35 -0700 (PDT) Date: Thu, 14 Apr 2005 20:09:35 -0700 From: John-Mark Gurney To: Pyun YongHyeon Message-ID: <20050415030935.GQ56487@funkthat.com> Mail-Followup-To: Pyun YongHyeon , freebsd-sparc64@freebsd.org References: <20050414092608.GB2855@michelle.rndsoft.co.kr> <20050414192749.GO56487@funkthat.com> <20050415023902.GC7393@michelle.rndsoft.co.kr> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050415023902.GC7393@michelle.rndsoft.co.kr> User-Agent: Mutt/1.4.1i X-Operating-System: FreeBSD 4.2-RELEASE i386 X-PGP-Fingerprint: B7 EC EF F8 AE ED A7 31 96 7A 22 B3 D8 56 36 F4 X-Files: The truth is out there X-URL: http://resnet.uoregon.edu/~gurney_j/ X-Resume: http://resnet.uoregon.edu/~gurney_j/resume.html cc: freebsd-sparc64@freebsd.org Subject: Re: em(4) patch X-BeenThere: freebsd-sparc64@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list Reply-To: John-Mark Gurney List-Id: Porting FreeBSD to the Sparc List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Apr 2005 03:09:38 -0000 Pyun YongHyeon wrote this message on Fri, Apr 15, 2005 at 11:39 +0900: > On Thu, Apr 14, 2005 at 12:27:49PM -0700, John-Mark Gurney wrote: > > > > . always honor IFF_OACTIVE flag > > > > I made this mistake before myself.. Make sure you read the comment > > describing OACTIVE.. you only set OACTIVE when the TX queue is full, > > and clear it when it is not full.. not always set it when the board > > is transmitting: > > define IFF_OACTIVE 0x400 /* tx hardware queue is full */ > > > > Do you mean IFF_OACTIVE should be set regardless of TX queue state > when h/w is transmitting frames? Would you elaborate on an IFF_OACTIVE? IFF_OACTIVE should be set when the hardward cannot queue anymore packets, and should be cleared when the hardware can accept additional packets.. It basicly controls calling an interface's if_start routine.. If IFF_OACTIVE is set, then it is assumed no additional packets can be processed, and if_start will not be called when a packet is queued for the interface... If you look at net/if_var.h for the define IFQ_HANDOFF_ADJ, you'll see where it is tested... > > > . TX/RX descriptor alignment is 16 not PAGE_SIZE > > > . TX/RX descriptor length should be multiple of 128 not 4096. > > > > You should probably use a define for this... > > > > Yes. But AFAIK if_em_hw.h has no defined constants for these purposes. Then add them.. :) > There are another hardcoded constants for processing 64bit IO BAR. I > belive these constants should go pcireg.h. Hmmm. which ones are these? > > > I have tested 82545GM chipset based card on Ultra60(SMP). VLAN and > > > POLLING were not tested at all so it may have bugs. JUMBO frame support > > > needs more coding and cleanups since sparc64 does not allow non-aligned > > > memory access. So don't try to use JUMBO frame at this time. > > > If you want to test em(4) on SMP with ofw_console you need additional > > > patch. Alternatively you can use sc(4) or serial console. > > > I'd like to hear success/failure. > > > > I think you should put back the while(!IFQ_DRV_IS_EMPTY, loop.. It > > prevents an unnessary lock/unlock if both queues are empty... > > > > Yes, you're right. > > > Also, I have work in progress to make non-aligned packets better... > > I committed part of it as the m_copyup change... On a -current, you > > should just be able to apply the following change: > > http://perforce.freebsd.org/fileDiffView.cgi?FSPC=//depot/user/jmg/arm/src/sys/netinet/ip%5finput.c&REV=3&ACT=edit > > > > This looks promising. Some miserable hardwares that use sf(4)/rl(4) > driver could take advatnage of it. BTW what is IP_HDR_ALIGNED_P macro? Look at: http://perforce.freebsd.org/fileDiffView.cgi?FSPC=//depot/user/jmg/arm/src/sys/netinet/ip%5fvar.h&REV=2&ACT=edit and __NO_STRICT_ALIGNMENT should be defined somewhere for i386 and amd64.. I sent an email to bde asking about where to define this, but he didn't respond... > > I'll try the patch when I get home... > > Thanks. np... -- John-Mark Gurney Voice: +1 415 225 5579 "All that I will do, has been done, All that I have, has not."