linux-next: manual merge of the drm tree with the origin tree

--Ps6CxN/a09x6fbXU
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi all,

Today's linux-next merge of the drm tree got a conflict in:

  drivers/gpu/drm/xe/xe_pci.c

between commit:

  130910bac905a ("drm/xe/pf: Disable display in admin only PF mode")

=66rom the origin tree and commits:

  7ef55ae582eba ("drm/xe/pf: Disable display in admin only PF mode")
  b623bd790db04 ("drm/xe: Drop 'force_execlist' module parameter")

=66rom the drm tree.

I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging.  You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.

Again, please consider not doing cherry picking as a matter of such
routine.

diff --combined drivers/gpu/drm/xe/xe_pci.c
index 9cd873708136c,5f2a0b19839d1..0000000000000
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@@ -22,10 -22,12 +22,12 @@@
  #include "xe_device.h"
  #include "xe_drv.h"
  #include "xe_gt.h"
+ #include "xe_gt_printk.h"
  #include "xe_gt_sriov_vf.h"
  #include "xe_guc.h"
  #include "xe_mmio.h"
  #include "xe_module.h"
+ #include "xe_pci_error.h"
  #include "xe_pci_rebar.h"
  #include "xe_pci_sriov.h"
  #include "xe_pci_types.h"
@@@ -35,6 -37,7 +37,7 @@@
  #include "xe_step.h"
  #include "xe_survivability_mode.h"
  #include "xe_tile.h"
+ #include "xe_tile_printk.h"
 =20
  enum toggle_d3cold {
  	D3COLD_DISABLE,
@@@ -119,6 -122,7 +122,7 @@@ static const struct xe_graphics_desc gr
  static const struct xe_graphics_desc graphics_xe3p_lpg =3D {
  	XE2_GFX_FEATURES,
  	.has_indirect_ring_state =3D 1,
+ 	.has_uncorrectable_error_reporting =3D 1,
  	.multi_queue_engine_class_mask =3D BIT(XE_ENGINE_CLASS_COPY) | BIT(XE_EN=
GINE_CLASS_COMPUTE),
  	.num_geometry_xecore_fuse_regs =3D 3,
  	.num_compute_xecore_fuse_regs =3D 3,
@@@ -128,6 -132,7 +132,7 @@@ static const struct xe_graphics_desc gr
  	XE2_GFX_FEATURES,
  	.has_access_counter =3D 0,
  	.has_indirect_ring_state =3D 1,
+ 	.has_uncorrectable_error_reporting =3D 1,
  	.hw_engine_mask =3D
  		GENMASK(XE_HW_ENGINE_BCS8, XE_HW_ENGINE_BCS1) |
  		GENMASK(XE_HW_ENGINE_CCS3, XE_HW_ENGINE_CCS0),
@@@ -150,6 -155,14 +155,14 @@@ static const struct xe_media_desc media
  		BIT(XE_HW_ENGINE_GSCCS0)
  };
 =20
+ static const struct xe_media_desc media_xe3p_hpm =3D {
+ 	.has_uncorrectable_error_reporting =3D 1,
+ 	.hw_engine_mask =3D
+ 		GENMASK(XE_HW_ENGINE_VCS7, XE_HW_ENGINE_VCS0) |
+ 		GENMASK(XE_HW_ENGINE_VECS3, XE_HW_ENGINE_VECS0) |
+ 		BIT(XE_HW_ENGINE_GSCCS0)
+ };
+=20
  /* Pre-GMDID Graphics IPs */
  static const struct xe_ip graphics_ip_xelp =3D { 1200, "Xe_LP", &graphics=
_xelp };
  static const struct xe_ip graphics_ip_xelpp =3D { 1210, "Xe_LP+", &graphi=
cs_xelp };
@@@ -185,7 -198,7 +198,7 @@@ static const struct xe_ip media_ips[]=20
  	{ 3000, "Xe3_LPM", &media_xelpmp },
  	{ 3002, "Xe3_LPM", &media_xelpmp },
  	{ 3500, "Xe3p_LPM", &media_xelpmp },
- 	{ 3503, "Xe3p_HPM", &media_xelpmp },
+ 	{ 3503, "Xe3p_HPM", &media_xe3p_hpm },
  };
 =20
  #define MULTI_LRC_MASK \
@@@ -355,6 -368,7 +368,7 @@@ static const __maybe_unused struct xe_d
  	PLATFORM(PVC),
  	.dma_mask_size =3D 52,
  	.has_display =3D false,
+ 	.has_drm_ras =3D true,
  	.has_gsc_nvm =3D 1,
  	.has_heci_gscfi =3D 1,
  	.max_gt_per_tile =3D 1,
@@@ -447,7 -461,6 +461,6 @@@ static const struct xe_device_desc nvls
  	.has_sriov =3D true,
  	.max_gt_per_tile =3D 2,
  	MULTI_LRC_MASK,
- 	.require_force_probe =3D true,
  	.va_bits =3D 48,
  	.vm_max_level =3D 4,
  };
@@@ -457,6 -470,7 +470,7 @@@ static const struct xe_device_desc cri_
  	PLATFORM(CRESCENTISLAND),
  	.dma_mask_size =3D 52,
  	.has_display =3D false,
+ 	.has_drm_ras =3D true,
  	.has_flat_ccs =3D false,
  	.has_gsc_nvm =3D 1,
  	.has_i2c =3D true,
@@@ -599,8 -613,6 +613,6 @@@ static int read_gmdid(struct xe_device=20
  	struct xe_reg gmdid_reg =3D GMD_ID;
  	u32 val;
 =20
- 	KUNIT_STATIC_STUB_REDIRECT(read_gmdid, xe, type, ver, revid);
-=20
  	if (IS_SRIOV_VF(xe)) {
  		/*
  		 * To get the value of the GMDID register, VFs must obtain it
@@@ -726,14 -738,30 +738,30 @@@ static int handle_gmdid(struct xe_devic
  	return 0;
  }
 =20
- static void init_devid(struct xe_device *xe)
+ struct xe_probed_info {
+ 	u16 devid;
+ 	u8 revid;
+ 	u8 tile_count;
+ 	struct xe_step_info step;
+ 	const struct xe_ip *graphics_ip;
+ 	const struct xe_ip *media_ip;
+ };
+=20
+ /*
+  * Probe from the hardware the info required by xe_info_init_early().
+  */
+ static int xe_probe_info_early(struct xe_device *xe,
+ 			       const struct xe_device_desc *desc,
+ 			       struct xe_probed_info *probed_info)
  {
  	struct pci_dev *pdev =3D to_pci_dev(xe->drm.dev);
 =20
- 	KUNIT_STATIC_STUB_REDIRECT(init_devid, xe);
+ 	probed_info->devid =3D pdev->device;
+ 	probed_info->revid =3D pdev->revision;
 =20
- 	xe->info.devid =3D pdev->device;
- 	xe->info.revid =3D pdev->revision;
+ 	xe_step_platform_get(desc->platform, probed_info->revid, &probed_info->s=
tep);
+=20
+ 	return 0;
  }
 =20
  /*
@@@ -742,17 -770,20 +770,20 @@@
   */
  static int xe_info_init_early(struct xe_device *xe,
  			      const struct xe_device_desc *desc,
- 			      const struct xe_subplatform_desc *subplatform_desc)
+ 			      const struct xe_subplatform_desc *subplatform_desc,
+ 			      struct xe_probed_info *probed_info)
  {
  	int err;
 =20
+ 	xe->info.devid =3D probed_info->devid;
+ 	xe->info.revid =3D probed_info->revid;
+ 	xe->info.step.platform =3D probed_info->step.platform;
+=20
  	xe->info.platform_name =3D desc->platform_name;
  	xe->info.platform =3D desc->platform;
  	xe->info.subplatform =3D subplatform_desc ?
  		subplatform_desc->subplatform : XE_SUBPLATFORM_NONE;
 =20
- 	init_devid(xe);
-=20
  	xe->info.dma_mask_size =3D desc->dma_mask_size;
  	xe->info.va_bits =3D desc->va_bits;
  	xe->info.vm_max_level =3D desc->vm_max_level;
@@@ -760,6 -791,7 +791,7 @@@
 =20
  	xe->info.is_dgfx =3D desc->is_dgfx;
  	xe->info.has_cached_pt =3D desc->has_cached_pt;
+ 	xe->info.has_drm_ras =3D desc->has_drm_ras;
  	xe->info.has_fan_control =3D desc->has_fan_control;
  	/* runtime fusing may force flat_ccs to disabled later */
  	xe->info.has_flat_ccs =3D desc->has_flat_ccs;
@@@ -790,14 -822,10 +822,10 @@@
  				 xe_modparam.probe_display &&
  				 desc->has_display &&
  				 !xe_device_is_admin_only(xe);
- 	xe->info.force_execlist =3D xe_modparam.force_execlist;
 =20
  	xe_assert(xe, desc->max_gt_per_tile > 0);
  	xe_assert(xe, desc->max_gt_per_tile <=3D XE_MAX_GT_PER_TILE);
  	xe->info.max_gt_per_tile =3D desc->max_gt_per_tile;
- 	xe->info.tile_count =3D 1 + desc->max_remote_tiles;
-=20
- 	xe_step_platform_get(xe);
 =20
  	err =3D xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
  	if (err)
@@@ -806,22 -834,21 +834,21 @@@
  	return 0;
  }
 =20
- /*
-  * Possibly override number of tile based on configuration register.
-  */
- static void xe_info_probe_tile_count(struct xe_device *xe)
+ static void xe_probe_tile_count(struct xe_device *xe,
+ 				const struct xe_device_desc *desc,
+ 				struct xe_probed_info *probed_info)
  {
  	struct xe_mmio *mmio;
  	u8 tile_count;
  	u32 mtcfg;
 =20
- 	KUNIT_STATIC_STUB_REDIRECT(xe_info_probe_tile_count, xe);
+ 	probed_info->tile_count =3D 1 + desc->max_remote_tiles;
 =20
  	/*
  	 * Probe for tile count only for platforms that support multiple
  	 * tiles.
  	 */
- 	if (xe->info.tile_count =3D=3D 1)
+ 	if (probed_info->tile_count =3D=3D 1)
  		return;
 =20
  	mmio =3D xe_root_tile_mmio(xe);
@@@ -834,10 -861,10 +861,10 @@@
  	mtcfg =3D xe_mmio_read32(mmio, XEHP_MTCFG_ADDR);
  	tile_count =3D REG_FIELD_GET(TILE_COUNT, mtcfg) + 1;
 =20
- 	if (tile_count < xe->info.tile_count) {
+ 	if (tile_count < probed_info->tile_count) {
  		drm_info(&xe->drm, "tile_count: %d, reduced_tile_count %d\n",
- 			 xe->info.tile_count, tile_count);
- 		xe->info.tile_count =3D tile_count;
+ 			 probed_info->tile_count, tile_count);
+ 		probed_info->tile_count =3D tile_count;
  	}
  }
 =20
@@@ -849,7 -876,7 +876,7 @@@ static struct xe_gt *alloc_primary_gt(s
  	struct xe_gt *gt;
 =20
  	if (!xe_configfs_primary_gt_allowed(to_pci_dev(xe->drm.dev))) {
- 		xe_info(xe, "Primary GT disabled via configfs\n");
+ 		xe_tile_info(tile, "Primary GT disabled via configfs\n");
  		return NULL;
  	}
 =20
@@@ -860,7 -887,13 +887,13 @@@
  	gt->info.type =3D XE_GT_TYPE_MAIN;
  	gt->info.id =3D tile->id * xe->info.max_gt_per_tile;
  	gt->info.has_indirect_ring_state =3D graphics_desc->has_indirect_ring_st=
ate;
+ 	gt->info.has_uncorrectable_error_reporting =3D
+ 		graphics_desc->has_uncorrectable_error_reporting;
  	gt->info.multi_queue_engine_class_mask =3D graphics_desc->multi_queue_en=
gine_class_mask;
+ 	if (!xe_configfs_get_enable_multi_queue(to_pci_dev(xe->drm.dev))) {
+ 		xe_gt_info(gt, "Multi-queue disabled via configfs\n");
+ 		gt->info.multi_queue_engine_class_mask =3D 0;
+ 	}
  	gt->info.engine_mask =3D graphics_desc->hw_engine_mask;
  	gt->info.num_geometry_xecore_fuse_regs =3D graphics_desc->num_geometry_x=
ecore_fuse_regs;
  	gt->info.num_compute_xecore_fuse_regs =3D graphics_desc->num_compute_xec=
ore_fuse_regs;
@@@ -891,7 -924,7 +924,7 @@@ static struct xe_gt *alloc_media_gt(str
  	struct xe_gt *gt;
 =20
  	if (!xe_configfs_media_gt_allowed(to_pci_dev(xe->drm.dev))) {
- 		xe_info(xe, "Media GT disabled via configfs\n");
+ 		xe_tile_info(tile, "Media GT disabled via configfs\n");
  		return NULL;
  	}
 =20
@@@ -905,11 -938,70 +938,70 @@@
  	gt->info.type =3D XE_GT_TYPE_MEDIA;
  	gt->info.id =3D tile->id * xe->info.max_gt_per_tile + 1;
  	gt->info.has_indirect_ring_state =3D media_desc->has_indirect_ring_state;
+ 	gt->info.has_uncorrectable_error_reporting =3D media_desc->has_uncorrect=
able_error_reporting;
  	gt->info.engine_mask =3D media_desc->hw_engine_mask;
 =20
  	return gt;
  }
 =20
+ static int xe_probe_ips(struct xe_device *xe,
+ 			const struct xe_device_desc *desc,
+ 			struct xe_probed_info *probed_info)
+ {
+ 	/*
+ 	 * If this platform supports GMD_ID, we'll detect the proper IP
+ 	 * descriptor to use from hardware registers.
+ 	 * desc->pre_gmdid_graphics_ip will only ever be set at this point for
+ 	 * platforms before GMD_ID. In that case the IP descriptions and
+ 	 * versions are simply derived from that.
+ 	 */
+ 	if (desc->pre_gmdid_graphics_ip) {
+ 		probed_info->graphics_ip =3D desc->pre_gmdid_graphics_ip;
+ 		probed_info->media_ip =3D desc->pre_gmdid_media_ip;
+ 		xe_step_pre_gmdid_get(xe, &probed_info->step);
+ 	} else {
+ 		int err;
+ 		u32 graphics_revid, media_revid;
+=20
+ 		xe_assert(xe, !desc->pre_gmdid_media_ip);
+=20
+ 		err =3D handle_gmdid(xe, &probed_info->graphics_ip, &probed_info->media=
_ip,
+ 				   &graphics_revid, &media_revid);
+ 		if (err)
+ 			return err;
+=20
+ 		xe_step_gmdid_get(xe, graphics_revid, media_revid, &probed_info->step);
+ 	}
+=20
+ 	/*
+ 	 * If we couldn't detect the graphics IP, that's considered a fatal
+ 	 * error and we should abort driver load.  Failing to detect media
+ 	 * IP is non-fatal; we'll just proceed without enabling media support.
+ 	 */
+ 	if (!probed_info->graphics_ip)
+ 		return -ENODEV;
+=20
+ 	return 0;
+ }
+=20
+ /*
+  * Probe from the hardware the info required by xe_info_init().
+  */
+ static int xe_probe_info(struct xe_device *xe,
+ 			 const struct xe_device_desc *desc,
+ 			 struct xe_probed_info *probed_info)
+ {
+ 	int err;
+=20
+ 	xe_probe_tile_count(xe, desc, probed_info);
+=20
+ 	err =3D xe_probe_ips(xe, desc, probed_info);
+ 	if (err)
+ 		return err;
+=20
+ 	return 0;
+ }
+=20
  /*
   * Initialize device info content that does require knowledge about
   * graphics / media IP version.
@@@ -917,46 -1009,24 +1009,24 @@@
   * present in device info.
   */
  static int xe_info_init(struct xe_device *xe,
- 			const struct xe_device_desc *desc)
+ 			const struct xe_device_desc *desc,
+ 			struct xe_probed_info *probed_info)
  {
- 	u32 graphics_gmdid_revid =3D 0, media_gmdid_revid =3D 0;
  	const struct xe_ip *graphics_ip;
  	const struct xe_ip *media_ip;
  	const struct xe_graphics_desc *graphics_desc;
  	const struct xe_media_desc *media_desc;
  	struct xe_tile *tile;
  	struct xe_gt *gt;
- 	int ret;
  	u8 id;
 =20
- 	/*
- 	 * If this platform supports GMD_ID, we'll detect the proper IP
- 	 * descriptor to use from hardware registers.
- 	 * desc->pre_gmdid_graphics_ip will only ever be set at this point for
- 	 * platforms before GMD_ID. In that case the IP descriptions and
- 	 * versions are simply derived from that.
- 	 */
- 	if (desc->pre_gmdid_graphics_ip) {
- 		graphics_ip =3D desc->pre_gmdid_graphics_ip;
- 		media_ip =3D desc->pre_gmdid_media_ip;
- 		xe_step_pre_gmdid_get(xe);
- 	} else {
- 		xe_assert(xe, !desc->pre_gmdid_media_ip);
- 		ret =3D handle_gmdid(xe, &graphics_ip, &media_ip,
- 				   &graphics_gmdid_revid, &media_gmdid_revid);
- 		if (ret)
- 			return ret;
+ 	graphics_ip =3D probed_info->graphics_ip;
+ 	media_ip =3D probed_info->media_ip;
 =20
- 		xe_step_gmdid_get(xe, graphics_gmdid_revid, media_gmdid_revid);
- 	}
-=20
- 	/*
- 	 * If we couldn't detect the graphics IP, that's considered a fatal
- 	 * error and we should abort driver load.  Failing to detect media
- 	 * IP is non-fatal; we'll just proceed without enabling media support.
- 	 */
- 	if (!graphics_ip)
- 		return -ENODEV;
+ 	xe->info.tile_count =3D probed_info->tile_count;
+ 	xe->info.step.basedie =3D probed_info->step.basedie;
+ 	xe->info.step.graphics =3D probed_info->step.graphics;
+ 	xe->info.step.media =3D probed_info->step.media;
 =20
  	xe->info.graphics_verx100 =3D graphics_ip->verx100;
  	xe->info.graphics_name =3D graphics_ip->name;
@@@ -989,8 -1059,6 +1059,6 @@@
  		xe->info.has_soc_remapper_telem =3D 0;
  	}
 =20
- 	xe_info_probe_tile_count(xe);
-=20
  	for_each_remote_tile(tile, xe, id) {
  		int err;
 =20
@@@ -1079,9 -1147,11 +1147,11 @@@ static void xe_pci_remove(struct pci_de
   */
  static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id =
*ent)
  {
+ 	struct xe_probed_info probed_info =3D {};
  	const struct xe_device_desc *desc =3D (const void *)ent->driver_data;
  	const struct xe_subplatform_desc *subplatform_desc;
  	struct xe_device *xe;
+ 	void *group;
  	int err;
 =20
  	subplatform_desc =3D find_subplatform(desc, pdev->device);
@@@ -1109,6 -1179,11 +1179,11 @@@
  	if (xe_display_driver_probe_defer(pdev))
  		return -EPROBE_DEFER;
 =20
+ 	/* Group all devres so xe_pci_error_slot_reset() can release them as a u=
nit. */
+ 	group =3D devres_open_group(&pdev->dev, NULL, GFP_KERNEL);
+ 	if (!group)
+ 		return -ENOMEM;
+=20
  	err =3D pcim_enable_device(pdev);
  	if (err)
  		return err;
@@@ -1117,13 -1192,19 +1192,19 @@@
  	if (IS_ERR(xe))
  		return PTR_ERR(xe);
 =20
+ 	xe->devres_group =3D group;
+=20
  	pci_set_drvdata(pdev, &xe->drm);
 =20
  	xe_pm_assert_unbounded_bridge(xe);
 =20
  	pci_set_master(pdev);
 =20
- 	err =3D xe_info_init_early(xe, desc, subplatform_desc);
+ 	err =3D xe_probe_info_early(xe, desc, &probed_info);
+ 	if (err)
+ 		return err;
+=20
+ 	err =3D xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
  	if (err)
  		return err;
 =20
@@@ -1142,7 -1223,11 +1223,11 @@@
  	if (err)
  		return err;
 =20
- 	err =3D xe_info_init(xe, desc);
+ 	err =3D xe_probe_info(xe, desc, &probed_info);
+ 	if (err)
+ 		return err;
+=20
+ 	err =3D xe_info_init(xe, desc, &probed_info);
  	if (err)
  		return err;
 =20
@@@ -1355,6 -1440,7 +1440,7 @@@ static struct pci_driver xe_pci_driver=20
  	.remove =3D xe_pci_remove,
  	.shutdown =3D xe_pci_shutdown,
  	.sriov_configure =3D xe_pci_sriov_configure,
+ 	.err_handler =3D &xe_pci_error_handlers,
  #ifdef CONFIG_PM_SLEEP
  	.driver.pm =3D &xe_pm_ops,
  #endif

--Ps6CxN/a09x6fbXU
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmpwpj4ACgkQJNaLcl1U
h9Dtfgf/WAVuunz7sfKMagGC82HM6xjblczU19JVhvVxgKa03YtWqx4cJxaYJQhc
IVEl+5G1p+e2IcTp+3M071it2aeGgivqOWW6ug4Bhb2YPuhjp6vqF7t/hBho0id4
2z69QSWmm2NIFnI6rZJk+RVd4GmWh3pum8RrXD69T7MC6Wqjcn4D0M4ziWjVL7bC
GcafK90wjBmVLhfDHpdaiyOFiyxRnxCt14MZnRIkRrxZxjDfWTVN5iwEfLqSb25b
51ASBYOIM3Oz9w40Uw01zymG3MKRHP94hFh9pIzqwhk1bbauUBImjzEp6iM9KhAR
OIiLtrOlisKoBuOvC9CdC/gWcpBRyg==
=Ajtn
-----END PGP SIGNATURE-----

--Ps6CxN/a09x6fbXU--