From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH 1/2] drm: fix drm_vm for NOMMU builds Date: Wed, 11 Jan 2017 17:33:21 +0100 Message-ID: <2059511.haYOJvAx2D@wuerfel> References: <20170111133357.3664191-1-arnd@arndb.de> <20170111162713.veyyu56sbjaym3fe@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.126.187]) by gabe.freedesktop.org (Postfix) with ESMTPS id 696B56E978 for ; Wed, 11 Jan 2017 16:33:45 +0000 (UTC) In-Reply-To: <20170111162713.veyyu56sbjaym3fe@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: Daniel Vetter , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Daniel Vetter List-Id: dri-devel@lists.freedesktop.org T24gV2VkbmVzZGF5LCBKYW51YXJ5IDExLCAyMDE3IDU6Mjc6MTMgUE0gQ0VUIERhbmllbCBWZXR0 ZXIgd3JvdGU6Cj4gT24gV2VkLCBKYW4gMTEsIDIwMTcgYXQgMDI6MzM6MzRQTSArMDEwMCwgQXJu ZCBCZXJnbWFubiB3cm90ZToKPiA+IFdoZW4gYnVpbGRpbmcgRFJNIHdpdGhvdXQgYW4gTU1VLCB3 ZSBydW4gaW50byBhIGNvbXBpbGUtdGltZSBlcnJvciBiZWNhdXNlCj4gPiBwdGVfd3Jwcm90ZWN0 KCkgaXMgbm90IGRlZmluZWQ6Cj4gPiAKPiA+IGRyaXZlcnMvZ3B1L2RybS9kcm1fdm0uYzogSW4g ZnVuY3Rpb24gJ2RybV9tbWFwX2RtYSc6Cj4gPiBkcml2ZXJzL2dwdS9kcm0vZHJtX3ZtLmM6NDk2 Ojk6IGVycm9yOiBpbXBsaWNpdCBkZWNsYXJhdGlvbiBvZiBmdW5jdGlvbiAncHRlX3dycHJvdGVj dCcgWy1XZXJyb3I9aW1wbGljaXQtZnVuY3Rpb24tZGVjbGFyYXRpb25dCj4gPiAKPiA+IFRoZSBs aW5lIGlzIG5vdCBtZWFuaW5nZnVsIGhlcmUsIHNvIHdlIGNhbiBzaW1wbHkgYWRkIGFub3RoZXIK PiA+IGNvbXBpbGUtdGltZSBjaGVjayBhcm91bmQgaXQuCj4gPiAKPiA+IEZpeGVzOiA2MmEwZDk4 YTE4OGMgKCJkcm06IGFsbG93IHRvIHVzZSBtbXVsZXNzIFNvQyIpCj4gPiBTaWduZWQtb2ZmLWJ5 OiBBcm5kIEJlcmdtYW5uIDxhcm5kQGFybmRiLmRlPgo+IAo+IFdlIGRvbid0IG5lZWQgZHJtX3Zt LmMgb24gbW9kZXJuIGRyaXZlcnMsIGFuZCB0aGUgaWRlYSB3YXMgdG8gc2ltcGx5IG5vdAo+IGNv bXBpbGUgaXQgd2hlbiBub3QgbmVlZGVkLiBTZWU6Cj4gCj4gY29tbWl0IDk5YzQ4ZTFlMzhmMGFl YWExMDdhZDY3YzhkOTFmNmM5ZDlkNTY3YTkKPiBBdXRob3I6IEJlbmphbWluIEdhaWduYXJkIDxi ZW5qYW1pbi5nYWlnbmFyZEBsaW5hcm8ub3JnPgo+IERhdGU6ICAgV2VkIEphbiA0IDEwOjEyOjU2 IDIwMTcgKzAxMDAKPiAKPiAgICAgZHJtOiBjb21waWxlIGRybV92bS5jIG9ubHkgd2hlbiBuZWVk ZWQKPiAKPiAKPiBIb3cgZGlkIHlvdSBtYW5hZ2UgdG8gZW5hYmxlIHRoaXMgc3R1ZmY/Cj4gLURh bmllbAoKVGhpcyB3YXMgYSByYW5kY29uZmlnIGJ1aWxkLCB0aGUgRFJNIHNwZWNpZmljIHN5bWJv bHMgaGVyZSBhcmUKCkNPTkZJR19EUk09eQpDT05GSUdfRFJNX01JUElfRFNJPXkKIyBDT05GSUdf RFJNX0RQX0FVWF9DSEFSREVWIGlzIG5vdCBzZXQKIyBDT05GSUdfRFJNX0RFQlVHX01NIGlzIG5v dCBzZXQKIyBDT05GSUdfRFJNX0RFQlVHX01NX1NFTEZURVNUIGlzIG5vdCBzZXQKQ09ORklHX0RS TV9LTVNfSEVMUEVSPXkKQ09ORklHX0RSTV9LTVNfRkJfSEVMUEVSPXkKIyBDT05GSUdfRFJNX0ZC REVWX0VNVUxBVElPTiBpcyBub3Qgc2V0CkNPTkZJR19EUk1fTE9BRF9FRElEX0ZJUk1XQVJFPXkK Q09ORklHX0RSTV9HRU1fQ01BX0hFTFBFUj15CkNPTkZJR19EUk1fS01TX0NNQV9IRUxQRVI9eQpD T05GSUdfRFJNX1ZNPXkKCiMKIyBJMkMgZW5jb2RlciBvciBoZWxwZXIgY2hpcHMKIwojIENPTkZJ R19EUk1fSTJDX0NINzAwNiBpcyBub3Qgc2V0CkNPTkZJR19EUk1fSTJDX1NJTDE2ND15CiMgQ09O RklHX0RSTV9JMkNfTlhQX1REQTk5OFggaXMgbm90IHNldApDT05GSUdfRFJNX0FSTT15CkNPTkZJ R19EUk1fSERMQ0Q9eQpDT05GSUdfRFJNX0hETENEX1NIT1dfVU5ERVJSVU49eQojIENPTkZJR19E Uk1fTUFMSV9ESVNQTEFZIGlzIG5vdCBzZXQKCiMKIyBBQ1AgKEF1ZGlvIENvUHJvY2Vzc29yKSBD b25maWd1cmF0aW9uCiMKQ09ORklHX0RSTV9WR0VNPXkKQ09ORklHX0RSTV9ST0NLQ0hJUD15CkNP TkZJR19ST0NLQ0hJUF9BTkFMT0dJWF9EUD15CkNPTkZJR19ST0NLQ0hJUF9EV19IRE1JPW0KQ09O RklHX1JPQ0tDSElQX0RXX01JUElfRFNJPXkKQ09ORklHX1JPQ0tDSElQX0lOTk9fSERNST1tCiMg Q09ORklHX0RSTV9VREwgaXMgbm90IHNldApDT05GSUdfRFJNX0FSTUFEQT15CkNPTkZJR19EUk1f QVRNRUxfSExDREM9bQpDT05GSUdfRFJNX1JDQVJfRFU9eQojIENPTkZJR19EUk1fUkNBUl9IRE1J IGlzIG5vdCBzZXQKQ09ORklHX0RSTV9SQ0FSX0xWRFM9eQojIENPTkZJR19EUk1fU0hNT0JJTEUg aXMgbm90IHNldApDT05GSUdfRFJNX1NVTjRJPXkKQ09ORklHX0RSTV9USUxDREM9bQpDT05GSUdf RFJNX1RJTENEQ19TTEFWRV9DT01QQVQ9eQpDT05GSUdfRFJNX01TTT15CkNPTkZJR19EUk1fTVNN X1JFR0lTVEVSX0xPR0dJTkc9eQpDT05GSUdfRFJNX01TTV9IRE1JX0hEQ1A9eQpDT05GSUdfRFJN X01TTV9EU0k9eQojIENPTkZJR19EUk1fTVNNX0RTSV9QTEwgaXMgbm90IHNldAojIENPTkZJR19E Uk1fTVNNX0RTSV8yOE5NX1BIWSBpcyBub3Qgc2V0CiMgQ09ORklHX0RSTV9NU01fRFNJXzIwTk1f UEhZIGlzIG5vdCBzZXQKQ09ORklHX0RSTV9NU01fRFNJXzI4Tk1fODk2MF9QSFk9eQojIENPTkZJ R19EUk1fRlNMX0RDVSBpcyBub3Qgc2V0CiMgQ09ORklHX0RSTV9URUdSQSBpcyBub3Qgc2V0CkNP TkZJR19EUk1fUEFORUw9eQoKIwojIERpc3BsYXkgUGFuZWxzCiMKQ09ORklHX0RSTV9QQU5FTF9T SU1QTEU9bQpDT05GSUdfRFJNX1BBTkVMX0pESV9MVDA3ME1FMDUwMDA9bQpDT05GSUdfRFJNX1BB TkVMX1BBTkFTT05JQ19WVlgxMEYwMzROMDA9bQojIENPTkZJR19EUk1fUEFORUxfU0FNU1VOR19T NkU4QUEwIGlzIG5vdCBzZXQKIyBDT05GSUdfRFJNX1BBTkVMX1NIQVJQX0xRMTAxUjFTWDAxIGlz IG5vdCBzZXQKIyBDT05GSUdfRFJNX1BBTkVMX1NIQVJQX0xTMDQzVDFMRTAxIGlzIG5vdCBzZXQK Q09ORklHX0RSTV9CUklER0U9eQoKIwojIERpc3BsYXkgSW50ZXJmYWNlIEJyaWRnZXMKIwpDT05G SUdfRFJNX0FOQUxPR0lYX0FOWDc4WFg9eQpDT05GSUdfRFJNX0RVTUJfVkdBX0RBQz1tCkNPTkZJ R19EUk1fRFdfSERNST1tCkNPTkZJR19EUk1fRFdfSERNSV9BSEJfQVVESU89bQojIENPTkZJR19E Uk1fTlhQX1BUTjM0NjAgaXMgbm90IHNldApDT05GSUdfRFJNX1BBUkFERV9QUzg2MjI9bQojIENP TkZJR19EUk1fU0lMX1NJSTg2MjAgaXMgbm90IHNldAojIENPTkZJR19EUk1fU0lJOTAyWCBpcyBu b3Qgc2V0CiMgQ09ORklHX0RSTV9UT1NISUJBX1RDMzU4NzY3IGlzIG5vdCBzZXQKQ09ORklHX0RS TV9USV9URlA0MTA9eQpDT05GSUdfRFJNX0FOQUxPR0lYX0RQPXkKIyBDT05GSUdfRFJNX0kyQ19B RFY3NTExIGlzIG5vdCBzZXQKQ09ORklHX0RSTV9WQzQ9bQpDT05GSUdfRFJNX0VUTkFWSVY9bQoj IENPTkZJR19EUk1fRVROQVZJVl9SRUdJU1RFUl9MT0dHSU5HIGlzIG5vdCBzZXQKQ09ORklHX0RS TV9BUkNQR1U9bQpDT05GSUdfRFJNX01YUz15CkNPTkZJR19EUk1fTVhTRkI9bQpDT05GSUdfRFJN X01FU09OPXkKQ09ORklHX0RSTV9MRUdBQ1k9eQojIENPTkZJR19EUk1fTElCX1JBTkRPTSBpcyBu b3Qgc2V0CgpObyBpZGVhIHdoYXQgZGlkIGl0IGluIHRoZSBlbmQuCgpJIGFsc28gaGF2ZSBzb21l IG9sZGVyIGJ1aWxkIGZpeGVzIGFwcGxpZWQsIHNvIGl0IG1heSBiZSBzb21ldGhpbmcgdGhhdApo YXBwZW5zIG9uIG15IHRyZWUgYnV0IG5vdCBvbiBwbGFpbiBsaW51eC1uZXh0LgoKCUFybmQKX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1h aWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMu ZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1766015AbdAKQek (ORCPT ); Wed, 11 Jan 2017 11:34:40 -0500 Received: from mout.kundenserver.de ([212.227.126.187]:56183 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1765839AbdAKQd6 (ORCPT ); Wed, 11 Jan 2017 11:33:58 -0500 From: Arnd Bergmann To: Daniel Vetter Cc: Daniel Vetter , Daniel Vetter , Benjamin Gaignard , Jani Nikula , Sean Paul , David Airlie , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] drm: fix drm_vm for NOMMU builds Date: Wed, 11 Jan 2017 17:33:21 +0100 Message-ID: <2059511.haYOJvAx2D@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <20170111162713.veyyu56sbjaym3fe@phenom.ffwll.local> References: <20170111133357.3664191-1-arnd@arndb.de> <20170111162713.veyyu56sbjaym3fe@phenom.ffwll.local> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:FpzclL7GmVVTWtgWUK0IPbcxCkk0bCnwZRTapqnQU0US+AWYyB2 PtKtc1nvVQqgY2r0D3IaAvfsMGVByffR3o1k+RYzK9+VeM07r32VLUta9l3T6ikXjOXiDFl chbkhV1Lsk7WUDS5q+Z2FWEkdlheC7pTxjr1mHJCO9AnNPQgztxcevapVBmumgLimTGL+eB 84OpcZ62ITOJVHGtrp7uA== X-UI-Out-Filterresults: notjunk:1;V01:K0:pEckzH1hLA4=:0aKBI2FjMQ1ZuMp/Kb2zV/ L5Lj7gd0aWznJ0iXtm7yWwzQJKiqJqWf3kPfQvJOpWrSujxX+7fJIdGgzvlOeBdeIFApUkWAN E3Jg/Cq7ubNuh42lbmI4Yn6q0561slcxOudzUJ/4kyqsAGWt7e0u5emUaH2XzbrtUnSJDgJjV 7A0QNruJXoYd81OzNowu9TdM3rQdB+j9IQ34dWWEbUUWP0QtvCQWi37IP8TtA5B1qEZ+GxyFK z5Tr9rnHwTwaqZByU9GCLBXyZ/jr/UTdUnQbGRp/pSXytTFc5v/tI5GYtip+NgkDl9ghLIdFp YOzGrlWEvrxKvTuZU7O6nem7WXv92Ac5GDltI9L9aq7cdSfQdKXRmN1CNEUYgIrrbUzqx0Ss/ zRY3YGPOSNNGb5gFWVPh5DSODRHl1S+WfhQW8h7PZpFTFXQEpzFbY+UWLFpElBej8xwyVpjRk DKHo0DWCms9GODNutgzjICFSvHRnxKSL6+Fb/RWkHHigvpAT4THbUx18uZysuWHE0yNox10p1 ZkRzswrDYLPmJavMhWn8ILOF1l028IUE5DspejYuL0ekPrhl3lYXirPW3BSA9YzrfhHUCxLfO FfampZA41yA4BR4jN+qxkVkzr0x3LPu6pNl4+W2DP9qUi4KnIaso91hHE9rTf6HlodFGlF3Hg YUcKHbZC9Ni5JvJx13eDxoxzOQxFjDgBzW9g6JYF7JjKi9RZB9YqYMBuIl3oYZr4JgSJjOgHG gXSsU3ZqxeLlqUnM Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday, January 11, 2017 5:27:13 PM CET Daniel Vetter wrote: > On Wed, Jan 11, 2017 at 02:33:34PM +0100, Arnd Bergmann wrote: > > When building DRM without an MMU, we run into a compile-time error because > > pte_wrprotect() is not defined: > > > > drivers/gpu/drm/drm_vm.c: In function 'drm_mmap_dma': > > drivers/gpu/drm/drm_vm.c:496:9: error: implicit declaration of function 'pte_wrprotect' [-Werror=implicit-function-declaration] > > > > The line is not meaningful here, so we can simply add another > > compile-time check around it. > > > > Fixes: 62a0d98a188c ("drm: allow to use mmuless SoC") > > Signed-off-by: Arnd Bergmann > > We don't need drm_vm.c on modern drivers, and the idea was to simply not > compile it when not needed. See: > > commit 99c48e1e38f0aeaa107ad67c8d91f6c9d9d567a9 > Author: Benjamin Gaignard > Date: Wed Jan 4 10:12:56 2017 +0100 > > drm: compile drm_vm.c only when needed > > > How did you manage to enable this stuff? > -Daniel This was a randconfig build, the DRM specific symbols here are CONFIG_DRM=y CONFIG_DRM_MIPI_DSI=y # CONFIG_DRM_DP_AUX_CHARDEV is not set # CONFIG_DRM_DEBUG_MM is not set # CONFIG_DRM_DEBUG_MM_SELFTEST is not set CONFIG_DRM_KMS_HELPER=y CONFIG_DRM_KMS_FB_HELPER=y # CONFIG_DRM_FBDEV_EMULATION is not set CONFIG_DRM_LOAD_EDID_FIRMWARE=y CONFIG_DRM_GEM_CMA_HELPER=y CONFIG_DRM_KMS_CMA_HELPER=y CONFIG_DRM_VM=y # # I2C encoder or helper chips # # CONFIG_DRM_I2C_CH7006 is not set CONFIG_DRM_I2C_SIL164=y # CONFIG_DRM_I2C_NXP_TDA998X is not set CONFIG_DRM_ARM=y CONFIG_DRM_HDLCD=y CONFIG_DRM_HDLCD_SHOW_UNDERRUN=y # CONFIG_DRM_MALI_DISPLAY is not set # # ACP (Audio CoProcessor) Configuration # CONFIG_DRM_VGEM=y CONFIG_DRM_ROCKCHIP=y CONFIG_ROCKCHIP_ANALOGIX_DP=y CONFIG_ROCKCHIP_DW_HDMI=m CONFIG_ROCKCHIP_DW_MIPI_DSI=y CONFIG_ROCKCHIP_INNO_HDMI=m # CONFIG_DRM_UDL is not set CONFIG_DRM_ARMADA=y CONFIG_DRM_ATMEL_HLCDC=m CONFIG_DRM_RCAR_DU=y # CONFIG_DRM_RCAR_HDMI is not set CONFIG_DRM_RCAR_LVDS=y # CONFIG_DRM_SHMOBILE is not set CONFIG_DRM_SUN4I=y CONFIG_DRM_TILCDC=m CONFIG_DRM_TILCDC_SLAVE_COMPAT=y CONFIG_DRM_MSM=y CONFIG_DRM_MSM_REGISTER_LOGGING=y CONFIG_DRM_MSM_HDMI_HDCP=y CONFIG_DRM_MSM_DSI=y # CONFIG_DRM_MSM_DSI_PLL is not set # CONFIG_DRM_MSM_DSI_28NM_PHY is not set # CONFIG_DRM_MSM_DSI_20NM_PHY is not set CONFIG_DRM_MSM_DSI_28NM_8960_PHY=y # CONFIG_DRM_FSL_DCU is not set # CONFIG_DRM_TEGRA is not set CONFIG_DRM_PANEL=y # # Display Panels # CONFIG_DRM_PANEL_SIMPLE=m CONFIG_DRM_PANEL_JDI_LT070ME05000=m CONFIG_DRM_PANEL_PANASONIC_VVX10F034N00=m # CONFIG_DRM_PANEL_SAMSUNG_S6E8AA0 is not set # CONFIG_DRM_PANEL_SHARP_LQ101R1SX01 is not set # CONFIG_DRM_PANEL_SHARP_LS043T1LE01 is not set CONFIG_DRM_BRIDGE=y # # Display Interface Bridges # CONFIG_DRM_ANALOGIX_ANX78XX=y CONFIG_DRM_DUMB_VGA_DAC=m CONFIG_DRM_DW_HDMI=m CONFIG_DRM_DW_HDMI_AHB_AUDIO=m # CONFIG_DRM_NXP_PTN3460 is not set CONFIG_DRM_PARADE_PS8622=m # CONFIG_DRM_SIL_SII8620 is not set # CONFIG_DRM_SII902X is not set # CONFIG_DRM_TOSHIBA_TC358767 is not set CONFIG_DRM_TI_TFP410=y CONFIG_DRM_ANALOGIX_DP=y # CONFIG_DRM_I2C_ADV7511 is not set CONFIG_DRM_VC4=m CONFIG_DRM_ETNAVIV=m # CONFIG_DRM_ETNAVIV_REGISTER_LOGGING is not set CONFIG_DRM_ARCPGU=m CONFIG_DRM_MXS=y CONFIG_DRM_MXSFB=m CONFIG_DRM_MESON=y CONFIG_DRM_LEGACY=y # CONFIG_DRM_LIB_RANDOM is not set No idea what did it in the end. I also have some older build fixes applied, so it may be something that happens on my tree but not on plain linux-next. Arnd