Re: [PATCH] drm/i915: bump platform_mask to u64

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 14, 2019 at 5:19 PM Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
>
> Quoting Lucas De Marchi (2019-03-15 00:13:31)
> > With Elkhart Lake being added to the platforms we are almost on the edge
> > of platforms that fits on this 32 bits mask. So bump it to 64 bits to
> > have room for new ones.
> >
> > Cc: José Roberto de Souza <jose.souza@xxxxxxxxx>
> > Signed-off-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
>
> Since platform_mask exists as an optimisation for code size, it'll be
> useful to track the impact of changes.

code size? Moving to u64 for this mask would make almost no
difference. Just checking here:

$ size drivers/gpu/drm/i915/i915.o*
   text       data        bss        dec        hex    filename
1836757      40454       4176    1881387     1cb52b
drivers/gpu/drm/i915/i915.o
1836533      40454       4176    1881163     1cb44b
drivers/gpu/drm/i915/i915.o.old

So... around 200 bytes.

Lucas De Marchi

> -Chris
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Lucas De Marchi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux