From: Stefan Richter <stefanr@s5r6.in-berlin.de>
To: "Kristian Høgsberg" <krh@redhat.com>
Cc: linux-kernel@vger.kernel.org, linux1394-devel@lists.sourceforge.net
Subject: Re: [PATCH 0/4] New firewire stack - updated patches
Date: Wed, 20 Dec 2006 22:52:30 +0100 [thread overview]
Message-ID: <4589B09E.40503@s5r6.in-berlin.de> (raw)
In-Reply-To: <458997B5.3040607@redhat.com>
Kristian Høgsberg wrote:
> And as for gap count optimization, I just added that to my git repo.
What can I say.
...
> and the optimization is definitely noticable. This is a setup
> where the box and the disk are both connected to a hub so the max hops
> is 2, so we're using gap count 4:
...
> Though I see that Mac OS X uses a more conservative setting for a
> similiar topology, so maybe we need to add a bit or "margin" to the
> numbers from the table from 1394.
The table in IEEE 1394-1995 is not entirely safe. Use the one from IEEE
1394a-2000:
Hops GC Hops GC Hops GC Hops GC Hops GC
1 5 6 16 11 29 16 43 21 57
2 7 7 18 12 32 17 46 22 59
3 8 8 21 13 35 18 48 23 62
4 10 9 24 14 37 19 51
5 13 10 26 15 40 20 54
(It's certainly not necessary to optimize for more than ~8 hops.)
But note, this does not work anymore as soon as there is an 1394b node
in the mix --- at least with one or more 1394b repeater node, I'm not
sure about 1394b leaf nodes. Because of the potentially much larger
repeater delays of 1394b PHYs, the only suitable method to determine a
working least gap count of such setups is round-trip delay measurement
with ping packets. But a good compromise would be to run table-based gap
count optimization for 1394a environments and no optimization for 1394b
or mixed environments. (Even though the latter would also clearly
benefit from it.)
--
Stefan Richter
-=====-=-==- ==-- =-=--
http://arcgraph.de/sr/
next prev parent reply other threads:[~2006-12-20 21:52 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-12-20 0:58 [PATCH 0/4] New firewire stack - updated patches Kristian Høgsberg
2006-12-20 10:42 ` Stefan Richter
2006-12-20 17:35 ` Kristian Høgsberg
2006-12-20 18:57 ` Stefan Richter
2006-12-20 20:06 ` Kristian Høgsberg
2006-12-20 21:52 ` Stefan Richter [this message]
2006-12-20 23:01 ` Stefan Richter
2006-12-20 20:34 ` Stefan Richter
2006-12-20 15:29 ` Pieter Palmers
2006-12-20 18:39 ` Kristian Høgsberg
2006-12-21 23:03 ` Stefan Richter
-- strict thread matches above, loose matches on Subject: below --
2006-12-21 11:47 Duncan Beadnell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4589B09E.40503@s5r6.in-berlin.de \
--to=stefanr@s5r6.in-berlin.de \
--cc=krh@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux1394-devel@lists.sourceforge.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox