Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1162032AbbKEPac (ORCPT ); Thu, 5 Nov 2015 10:30:32 -0500 Received: from mezzanine.sirena.org.uk ([106.187.55.193]:48609 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754933AbbKEPa3 (ORCPT ); Thu, 5 Nov 2015 10:30:29 -0500 Date: Thu, 5 Nov 2015 15:29:43 +0000 From: Mark Brown To: Vignesh R Cc: Tony Lindgren , Michal Suchanek , Russell King , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mtd@lists.infradead.org, linux-spi@vger.kernel.org Message-ID: <20151105152943.GP18409@sirena.org.uk> References: <1446545174-14193-1-git-send-email-vigneshr@ti.com> <1446545174-14193-2-git-send-email-vigneshr@ti.com> <20151104143915.GI1717@sirena.org.uk> <563B4BB0.9020206@ti.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="1LW0Rr0Uq98qh6Rv" Content-Disposition: inline In-Reply-To: <563B4BB0.9020206@ti.com> X-Cookie: Put your trust in those who are worthy. User-Agent: Mutt/1.5.24 (2015-08-30) X-SA-Exim-Connect-IP: 94.197.121.29 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH v2 1/5] spi: introduce mmap read support for spi flash devices X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on mezzanine.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2030 Lines: 47 --1LW0Rr0Uq98qh6Rv Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Nov 05, 2015 at 05:59:36PM +0530, Vignesh R wrote: > On 11/04/2015 08:09 PM, Mark Brown wrote: > > It's a bit worrying that this doesn't sync with the message queue except > > via the mutex: this means that we might be out of order with respect to > > any asynchronous transfers that are happening on the device. I'm not > > sure that this is a practical problem, though there is some risk of > > unfair scheduling that would have to be under extreme load and it might > > make sense to prioritise reads anyway. > Since mmap interface is used only by mtd flash drivers and since almost > all mtd flash devices use synchronous transfers (spi_sync()), IMO, there > wont be out of order problem wrt mtd flashes. > But mmap read might delay transfers queued for non mtd flash devices. It > is difficult to wait for all transfers already queued to be pumped out > by __spi_pump_messages() and then do the mmap transfer. Do you have any > thoughts on how to sync with message queue? Like I say I think it probably doesn't matter. The main issue would be with other devices sharing the bus with a flash chip. --1LW0Rr0Uq98qh6Rv Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJWO3XmAAoJECTWi3JdVIfQ8H4H/3upAPiXLF/N9hkvlFj/e0Da K8r42KaKguWp+Y3ERjZgOx+Y/KBHfR3yj/ugXa6pi+yqPx48AxB3m0bxQp9dENK8 qTzW9/paC7WWZCM9ShGZQoMvhLvqQZbzjxDkofQz43WBv4P+SC+6rJSSJrENe7KK zg8rZCRtGpSQcJp7W113ugTkr81cri9DtCv9mYDUADHgfmJFA2k7Oa6QExN5I91d duuFw09PJV6ajNwY2lA7LF+2OJLyTkE+WGoQq5fet9+7HDi0ronHcVBk9GnCCnu/ EJ4zsJ7PplNUAuTKwRGM8HfEzLee1UVRx40BDyz8LKXGAs9NygPpuobv8K3jTAY= =UrfS -----END PGP SIGNATURE----- --1LW0Rr0Uq98qh6Rv-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/