pan/bi: use os_read_file-helper

We already have a more robust helper for this, so let's use it rather
than open-coding the same.

While we're at it, return early on error for readability here. There's
no need to continue the logic in those cases.

CID: 1444074
Reviewed-by: Eric R. Smith <eric.smith@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/36724>
This commit is contained in:
Erik Faye-Lund 2025-08-11 16:16:19 +02:00 committed by Marge Bot
parent 046710ce95
commit 49183bfb79

View file

@ -31,6 +31,8 @@
#include "valhall/disassemble.h" #include "valhall/disassemble.h"
#include "panfrost/lib/pan_props.h" #include "panfrost/lib/pan_props.h"
#include "util/os_file.h"
unsigned gpu_id = 0x72120000; unsigned gpu_id = 0x72120000;
int verbose = 0; int verbose = 0;
@ -44,18 +46,13 @@ disassemble(const char *filename)
FILE *fp = fopen(filename, "rb"); FILE *fp = fopen(filename, "rb");
assert(fp); assert(fp);
fseek(fp, 0, SEEK_END); size_t filesize = 0;
unsigned filesize = ftell(fp); uint32_t *code = (uint32_t *)os_read_file(filename, &filesize);
rewind(fp); if (!code) {
fprintf(stderr, "Couldn't read full file\n");
uint32_t *code = malloc(filesize); return;
unsigned res = fread(code, 1, filesize, fp);
if (res != filesize) {
printf("Couldn't read full file\n");
} }
fclose(fp);
void *entrypoint = code; void *entrypoint = code;
if (filesize && code[0] == BI_FOURCC('M', 'B', 'S', '2')) { if (filesize && code[0] == BI_FOURCC('M', 'B', 'S', '2')) {