[PATCH libdrm] amdgpu: add the function to get the marketing name (v3)

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

 



On 7 November 2016 at 11:43, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 7 November 2016 at 09:09, Michel Dänzer <michel at daenzer.net> wrote:
>
>> +static struct amdgpu_asic_id_table_t {
>> +       uint32_t did;
>> +       uint32_t rid;
>> +       char marketing_name[64];
> Using a char * here might be a better. From a quick look [64] is quite wasteful.
>
By "quite wasteful" I meant ~doubling the binary .text size.
Some sample numbers which I should have mentioned earlier :-\

  text    data     bss     dec     hex filename
 10587     552       8   11147    2b8b ./a.out.fixed-size
  5227     552       8    5787    169b ./a.out.char-star

Regards,
Emil


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

  Powered by Linux