From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758107Ab0JSJe0 (ORCPT ); Tue, 19 Oct 2010 05:34:26 -0400 Received: from t111.niisi.ras.ru ([193.232.173.111]:60120 "EHLO t111.niisi.ras.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752030Ab0JSJeZ (ORCPT ); Tue, 19 Oct 2010 05:34:25 -0400 X-Greylist: delayed 3271 seconds by postgrey-1.27 at vger.kernel.org; Tue, 19 Oct 2010 05:34:24 EDT Message-ID: <4CBD5CC9.8070706@niisi.msk.ru> Date: Tue, 19 Oct 2010 12:54:33 +0400 From: "Gleb O. Raiko" Organization: NIISI RAN User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.5) Gecko/20091204 Thunderbird/3.0 MIME-Version: 1.0 To: Kevin Cernekee CC: Ralf Baechle , Shinya Kuribayashi , linux-mips@linux-mips.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH resend 5/9] MIPS: sync after cacheflush References: <17ebecce124618ddf83ec6fe8e526f93@localhost> <17d8d27a2356640a4359f1a7dcbb3b42@localhost> <4CBC4F4E.5010305@pobox.com> <20101018191936.GH27377@linux-mips.org> In-Reply-To: Content-Type: text/plain; charset=KOI8-R; format=flowed Content-Transfer-Encoding: 7bit X-Antivirus: Dr.Web (R) for Mail Servers on t111.niisi.ras.ru host X-Antivirus-Code: 100000 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 18.10.2010 23:41, Kevin Cernekee wrote: > I have not been able to find any official statement from MIPS that > says that CACHE + SYNC should be used, but that seems like the most > intuitive way to implement things on the hardware side. Indeed, both Architecture for Programmers in Vol. 2 describing instruction sets not so clearly say that sync is needed after cache. For example, documents with rev. 2.62, p. 92 (for MIPS32 ISA) or p. 96 (for MIPS64). Considering whether just sync enough I'd like to note some boxes may implement dma master and slave blocks to be unsynchronized. Also,there may be write buffers somewhere in the path between cpu, memory, and even a dma master. BTW, we have plat_extra_sync_for_device which has appropriate name but invented to do things before cache flush. :-) It seems we need another one which will do something after. Gleb.