From owner-freebsd-hackers@FreeBSD.ORG Fri Jan 2 00:59:10 2015 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id E0B27C99 for ; Fri, 2 Jan 2015 00:59:10 +0000 (UTC) Received: from mail-ig0-x232.google.com (mail-ig0-x232.google.com [IPv6:2607:f8b0:4001:c05::232]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 9DF2E64C98 for ; Fri, 2 Jan 2015 00:59:10 +0000 (UTC) Received: by mail-ig0-f178.google.com with SMTP id a13so4296215igq.17 for ; Thu, 01 Jan 2015 16:59:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc:content-type; bh=xoZ1ItJ0qlT+wk2g3RnTfsnUBTnMLHfybyyWyiXmFXU=; b=zT/jd6zJ6LE2Gy5kODYQkcLuxW4B0AuQhM42Gj5PeBDcTz7pSZfY6grZ97tJMpwH7r 180lnc0ZfT2lMO+2GhS52p8MGCz1K0gQJAYBH3eapI4KOjNnjeLnjb3j1xoe5LwbHV1J 8xmifDQwo1lxM+mzZ88RslYzeaOJwDKdUYzE76i0V67VIlBSX8FibD+UqXJgbIQSiLgN UoKqTeBA0OENSi4Y1pFBY8gXvJQ7ucttGx2r5soBi80aLoQ2TwR5ElZHnb7giKeg1o+E fvUW76L8CvZiv/89zExRZjDV/rFDWyb7wZovgKk4cJPsE1NTdDhdndcopjGcxG1iQ7IP UsNg== X-Received: by 10.42.107.211 with SMTP id e19mr54028633icp.27.1420160349851; Thu, 01 Jan 2015 16:59:09 -0800 (PST) MIME-Version: 1.0 Sender: carpeddiem@gmail.com Received: by 10.107.0.85 with HTTP; Thu, 1 Jan 2015 16:58:49 -0800 (PST) In-Reply-To: <20150101192219.GA46601@voyager> References: <20150101192219.GA46601@voyager> From: Ed Maste Date: Thu, 1 Jan 2015 19:58:49 -0500 X-Google-Sender-Auth: MPY-Xc3W7ubc-5dxT_llutJxQ1I Message-ID: Subject: Re: [PATCH] Fixing panic in vt_fb_blank() if fb_size is not a multiple of fb_stride To: Andre Albsmeier Content-Type: text/plain; charset=UTF-8 Cc: "freebsd-hackers@freebsd.org" X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 02 Jan 2015 00:59:11 -0000 On 1 January 2015 at 14:22, Andre Albsmeier wrote: > [Crossposting to -hackers and -x11 as this is vt and i915kms related] > > I can reliably crash an older notebook (Fujitsu E8310) with Intel > graphics (GM965) by loading i915kms after having booted but only > if vt(4) is used instead of old syscons. > > Reason for the crash is a page fault in vt_fb_blank() which is > in /sys/dev/vt/hw/fb/vt_fb.c: > ... > > in vt_fb_blank() we find: > > for (o = info->fb_stride; o < info->fb_size; o += info->fb_stride) { > info->copy(info, o, 0, info->fb_stride); > } Thank you for the report and investigation. This is fixed in HEAD by r268771 and r268796, which rework vt_fb_blank. There are a number of other vt(4) changes that are not in stable/9; I will see if most/all of it may be feasibly merged back, or otherwise commit a change for this problem.