From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754961AbbINKaD (ORCPT ); Mon, 14 Sep 2015 06:30:03 -0400 Received: from mail-pa0-f42.google.com ([209.85.220.42]:33871 "EHLO mail-pa0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752227AbbINKaB (ORCPT ); Mon, 14 Sep 2015 06:30:01 -0400 Date: Mon, 14 Sep 2015 15:59:51 +0530 From: Sudip Mukherjee To: Dave Airlie Cc: Ingo Molnar , architt@codeaurora.org, David Airlie , Daniel Vetter , LKML , dri-devel , Archit Taneja Subject: Re: [PATCH] drm/mgag200: fix memory leak Message-ID: <20150914102951.GA23251@sudip-pc> References: <1441627110-13783-1-git-send-email-sudipm.mukherjee@gmail.com> <20150913093607.GA6074@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 14, 2015 at 08:05:37PM +1000, Dave Airlie wrote: > (this time with correct email address). > > On 14 September 2015 at 20:04, Dave Airlie wrote: > >> > >> > >> There's a new regression: v4.3-rc1 crashes on bootup on non-supported hardware, if > >> CONFIG_DRM_MGAG200=y (built into the kernel). > > > > Archit, I'm guessing this is some fallout from the fbdev changes. > > > > There is no reason we should need CONFIG_FB_LITTLE_ENDIAN I don't think. I am assuming v4.2 has worked for Ingo. So in that case I don't see any change in fbdev/core/fbmem.c between 4.2 and 4.3-rc1. But in drm (I almost know nothing about drm) commit e829d7ef9f17 changes few codes related to the card revision. Can this help? regards sudip diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c b/drivers/gpu/drm/mgag200/mgag200_mode.c index c99d3fe..5cdfa53 100644 --- a/drivers/gpu/drm/mgag200/mgag200_mode.c +++ b/drivers/gpu/drm/mgag200/mgag200_mode.c @@ -1602,7 +1602,7 @@ static int mga_vga_mode_valid(struct drm_connector *connector, if (mga_vga_calculate_mode_bandwidth(mode, bpp) > (24400 * 1024)) return MODE_BANDWIDTH; - } else if (mdev->unique_rev_id == 0x02) { + } else if (mdev->unique_rev_id >= 0x02) { if (mode->hdisplay > 1920) return MODE_VIRTUAL_X; if (mode->vdisplay > 1200)