From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Thu, 23 Aug 2012 16:31:33 -0400 Subject: [U-Boot] [PATCH 04/12] tools/env: Reduce the impact on real-time processes In-Reply-To: References: <20100622222932.DCDB71524EE@gemini.denx.de> <201208222330.55978.vapier@gentoo.org> Message-ID: <201208231631.34653.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thursday 23 August 2012 12:26:08 Joe Hershberger wrote: > On Wed, Aug 22, 2012 at 10:30 PM, Mike Frysinger wrote: > > On Friday 17 August 2012 16:49:38 Joe Hershberger wrote: > >> + * Break reads up into very small chunks so fw_printenv doesn't > >> + * block the kernel long enough to starve other kernel tasks. > > > > err, this sounds like a bug in your kernel driver. why should userspace > > be working around buggy drivers ? > > The problem is something to do with the mtd driver blocking some > resource on large reads that starves USB transfers. We never fully > investigated it. It's true that this is a work-around. Is that so > bad? Maybe so. i think it's a bad precedent that we shouldn't encourage. you add your wonky hardware workaround, and then the next guy will want to add his hack for their broken driver, and we're left with an ugly unmaintainable pile. -mike -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: This is a digitally signed message part. URL: