From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2AF1126B080 for ; Mon, 23 Jun 2025 12:02:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.9 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750680140; cv=none; b=S/y3sqLkr2E7QB5ql0MZZOkjrMezMeojoaX0DzUQSzLJN9kCZydicvzNXr+z1cTtu5N6Q1/Kgph3TD4iE21O/z/wSverY2EyDmQGlzM7z8cUnW3XjT98YcxDQbHirvR82ASolhMG00NjpeOa4+KRybeeJk4qQmV2ymDmQOctT+4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750680140; c=relaxed/simple; bh=Y5hDoV8L/G+XG/Mk2UP5nwzN7exrsEd+MJZCvR+m6uA=; h=Message-ID:Date:MIME-Version:Subject:To:References:From: In-Reply-To:Content-Type; b=uNSnELXTioMlvUzJUcYO6yfXgs0ifqkCaj+LVyv/z7lTWF13/jZ8/Wk41jKoen7WHb4GSqLMn5DOykc3OtWCC2MGzDkHD8n5xkK6ZSnI2HubPjz6T0kZxJwsu25vYtkG2/qJSfksOJ4F62MevELbqVznfjRxmd6YgjwWyChuJi0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=none smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=Sv0HvcbA; arc=none smtp.client-ip=192.198.163.9 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="Sv0HvcbA" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1750680138; x=1782216138; h=message-id:date:mime-version:subject:to:references:from: in-reply-to:content-transfer-encoding; bh=Y5hDoV8L/G+XG/Mk2UP5nwzN7exrsEd+MJZCvR+m6uA=; b=Sv0HvcbAfS5jN+j7EWt8FX3n878gAmAyChOdGitu/7FVAEaXZIYALz5z SIh2jLXC7w8GcjTj2qs9DSh2ITI36xNESLR+YtC+UjTrbvxw/6rmx0s8t TJS/b52p3xrfptpFmRcMuPgVVFXojkniUe+0o5MmZUe1konX7hDrJTd2b aDKdzzuHSeaBXdpJUY43n35Y2FBOO4OSJt3NzzoLfseioR7RlIg7UuwMq HJx98wTSmoChpFdRUL0bnj5ICq0ClxfNV7OWAYYnKaCfI/VJQuy2SgSiJ fKoOFBLi79CttNXDCWxPYex3xglkXow58AtPY4ViyZw4R05JKOWgJuYyW Q==; X-CSE-ConnectionGUID: jd8XUFCkTQm6+BNYsyhuIw== X-CSE-MsgGUID: P2uLmSvtRoOXZKG+oHwWmw== X-IronPort-AV: E=McAfee;i="6800,10657,11473"; a="63575049" X-IronPort-AV: E=Sophos;i="6.16,258,1744095600"; d="scan'208";a="63575049" Received: from fmviesa006.fm.intel.com ([10.60.135.146]) by fmvoesa103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Jun 2025 05:02:18 -0700 X-CSE-ConnectionGUID: x9EzZMHTRbWoIMVvnxW2wA== X-CSE-MsgGUID: I4ScjVvrRLGRQ8lgvOs7Gw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.16,258,1744095600"; d="scan'208";a="151732162" Received: from mjarzebo-mobl1.ger.corp.intel.com (HELO [10.245.244.111]) ([10.245.244.111]) by fmviesa006-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Jun 2025 05:02:13 -0700 Message-ID: Date: Mon, 23 Jun 2025 14:02:11 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v10 00/10] drm/i915: Add drm_panic support To: Jocelyn Falempe , Jani Nikula , Rodrigo Vivi , Joonas Lahtinen , Tvrtko Ursulin , =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= , David Airlie , Simona Vetter , Christian Koenig , Huang Rui , Matthew Auld , Matthew Brost , Maxime Ripard , Thomas Zimmermann , intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org References: <20250618094011.238154-1-jfalempe@redhat.com> <4d81cf64-7bf1-4a7d-8682-fc817d74c373@redhat.com> Content-Language: en-US From: Maarten Lankhorst In-Reply-To: <4d81cf64-7bf1-4a7d-8682-fc817d74c373@redhat.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Hey, On 2025-06-23 12:10, Jocelyn Falempe wrote: > On 23/06/2025 09:40, Maarten Lankhorst wrote: >> Hey, >> >> Thanks for the series. I didn't see you on irc so I wanted to ask if you are planning to send a v11 with >> the changes from void * to struct intel_panic_data and adding the VRAM support? > > Yes, I'm preparing a v11, and I'm considering to do something like this, to allocate the panic data with the struct intel_framebuffer: > > struct xe_framebuffer { >     struct intel_framebuffer base; >     struct xe_panic_data panic; > }; > > struct intel_framebuffer *intel_bo_alloc_framebuffer(void) > { >     struct xe_framebuffer *xe_fb; > >     xe_fb = kmalloc(sizeof(struct xe_framebuffer), GFP_KERNEL); >     return &xe_fb->base; > } > > (And the same for i915). > That should allow you to add battlemage support. > >> >> >> Other than that, I think the series looks good and I'll be able to test it on my battlemage. >> A private member is fine, but if you can get signoff from the i915 people for the xe_fb slpit, I would really like that for other things too. :) Kind regards, ~Maarten