public inbox for netdev@vger.kernel.org 
 help / color / mirror / Atom feed
From: Jan Ceuleers <jan.ceuleers@computer•org>
To: Ben Hutchings <bhutchings@solarflare•com>
Cc: Krishna Gudipati <kgudipat@Brocade•com>,
	"'davem@davemloft•net'" <davem@davemloft•net>,
	"'netdev@vger•kernel.org'" <netdev@vger•kernel.org>,
	Adapter Linux Open SRC Team
	<adapter_linux_open_src_team@Brocade•com>,
	Rasesh Mody <rmody@Brocade•com>
Subject: Re: [PATCH 1/2] bna: Added flash sub-module and ethtool eeprom entry points.
Date: Mon, 26 Dec 2011 13:02:27 +0100	[thread overview]
Message-ID: <4EF86253.3010108@computer.org> (raw)
In-Reply-To: <1324721314.3860.11.camel@deadeye>

On 12/24/2011 11:08 AM, Ben Hutchings wrote:
> I asked how long a single call would take, because the RTNL lock will be
> held for this time.  Presumably it doesn't take very long to write a 4K
> chunk, so this shouldn't be a problem.
>

461K is around 115 4K blocks. If these take around 3 minutes to flash 
then that's about 1.5s per 4K block.

Is it OK to hold the RTNL lock for that length of time?

  reply	other threads:[~2011-12-26 12:07 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-22 23:29 [PATCH 1/2] bna: Added flash sub-module and ethtool eeprom entry points kgudipat
2011-12-23  9:44 ` Ben Hutchings
2011-12-23 15:09   ` Krishna Gudipati
2011-12-24 10:08     ` Ben Hutchings
2011-12-26 12:02       ` Jan Ceuleers [this message]
2011-12-26 15:10         ` David Miller
2011-12-26 15:46           ` Jan Ceuleers

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=4EF86253.3010108@computer.org \
    --to=jan.ceuleers@computer$(echo .)org \
    --cc=adapter_linux_open_src_team@Brocade$(echo .)com \
    --cc=bhutchings@solarflare$(echo .)com \
    --cc=davem@davemloft$(echo .)net \
    --cc=kgudipat@Brocade$(echo .)com \
    --cc=netdev@vger$(echo .)kernel.org \
    --cc=rmody@Brocade$(echo .)com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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