All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter.Chubb at data61.csiro.au <Peter.Chubb@data61.csiro.au>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3] net: Fix cache misalignment message after network load operations
Date: Wed, 14 Sep 2016 04:48:23 +0000	[thread overview]
Message-ID: <84bmzrf5ns.wl-Peter.Chubb@data61.csiro.au> (raw)
In-Reply-To: <57D8D083.3060901@denx.de>

>>>>> "Heiko" == Heiko Schocher <hs@denx.de> writes:

Heiko> Hello Peter, Am 14.09.2016 um 05:49 schrieb
Heiko> Peter.Chubb at data61.csiro.au:
>> After any operation that downloads a file (e.g., pxe get, or dhcp),
>> the buffer containing the downloaded data is flushed.  This is
>> unnecessary and annoying.  Unnecessary, because the network driver
>> should already have fliushed the cache for the DMAed area, and
>> annoying because it generates a cache misalignment message.
>> 
>> Signed-off-by: Peter Chubb <peter.chubb@data61.csiro.au>
>> ---
>> cmd/net.c | 3 --- 1 file changed, 3 deletions(-)

Heiko> I posted a fix for this here:
Heiko> http://patchwork.ozlabs.org/patch/663489/

Heiko> but I did not remove the flush operation ... can we really
Heiko> remove it?
 I believe so -- removing it was suggested by Joe Hershberger.

Heiko> If so, you can add my Acked-by: Heiko Schocher <hs@denx.de>

OK. thanks.
-- 
Dr Peter Chubb         Tel: +61 2 9490 5852      http://ts.data61.csiro.au/
Trustworthy Systems Group                           Data61 (formerly NICTA)

  reply	other threads:[~2016-09-14  4:48 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-14  3:49 [U-Boot] [PATCH v3] net: Fix cache misalignment message after network load operations Peter.Chubb at data61.csiro.au
2016-09-14  4:22 ` Heiko Schocher
2016-09-14  4:48   ` Peter.Chubb at data61.csiro.au [this message]
2016-09-23 20:53 ` Joe Hershberger
2016-10-06  0:14 ` [U-Boot] [U-Boot, " Tom Rini
2016-10-11 20:50   ` Joe Hershberger
2016-10-13 17:39 ` [U-Boot] " Joe Hershberger

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=84bmzrf5ns.wl-Peter.Chubb@data61.csiro.au \
    --to=peter.chubb@data61.csiro.au \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.