drm/xe: Move suballocator init to after display init
authorMaarten Lankhorst <dev@lankhorst.se>
Tue, 10 Dec 2024 08:31:03 +0000 (09:31 +0100)
committerMaarten Lankhorst <dev@lankhorst.se>
Tue, 21 Jan 2025 13:59:38 +0000 (14:59 +0100)
No allocations should be done before we have had a chance to preserve
the display fb.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241210083111.230484-4-dev@lankhorst.se
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
drivers/gpu/drm/xe/xe_device.c
drivers/gpu/drm/xe/xe_tile.c
drivers/gpu/drm/xe/xe_tile.h

index 6ecbf7d..af9b2bb 100644 (file)
@@ -840,6 +840,12 @@ int xe_device_probe(struct xe_device *xe)
        if (err)
                goto err;
 
+       for_each_tile(tile, xe, id) {
+               err = xe_tile_init(tile);
+               if (err)
+                       goto err;
+       }
+
        for_each_gt(gt, xe, id) {
                last_gt = id;
 
index 07cf7cf..2825553 100644 (file)
@@ -170,10 +170,6 @@ int xe_tile_init_noalloc(struct xe_tile *tile)
        if (err)
                return err;
 
-       tile->mem.kernel_bb_pool = xe_sa_bo_manager_init(tile, SZ_1M, 16);
-       if (IS_ERR(tile->mem.kernel_bb_pool))
-               return PTR_ERR(tile->mem.kernel_bb_pool);
-
        xe_wa_apply_tile_workarounds(tile);
 
        err = xe_tile_sysfs_init(tile);
@@ -181,6 +177,14 @@ int xe_tile_init_noalloc(struct xe_tile *tile)
        return 0;
 }
 
+int xe_tile_init(struct xe_tile *tile)
+{
+       tile->mem.kernel_bb_pool = xe_sa_bo_manager_init(tile, SZ_1M, 16);
+       if (IS_ERR(tile->mem.kernel_bb_pool))
+               return PTR_ERR(tile->mem.kernel_bb_pool);
+
+       return 0;
+}
 void xe_tile_migrate_wait(struct xe_tile *tile)
 {
        xe_migrate_wait(tile->migrate);
index 1c9e42a..eb93931 100644 (file)
@@ -12,6 +12,7 @@ struct xe_tile;
 
 int xe_tile_init_early(struct xe_tile *tile, struct xe_device *xe, u8 id);
 int xe_tile_init_noalloc(struct xe_tile *tile);
+int xe_tile_init(struct xe_tile *tile);
 
 void xe_tile_migrate_wait(struct xe_tile *tile);