[Beignet] [PATCH v2] remove lspci, gbe_bin_genenrater would generator llvm binary by default.

[Yichao Yu]

On Tue, Jul 8, 2014 at 10:56 PM, Andi Kleen <andi at firstfloor.org> wrote:
> xionghu.luo at intel.com writes:
>
>> From: Luo <xionghu.luo at intel.com>
>>
>> driver can get chipset id by ioctl instead of calling lspci in cmake;
>> user could generator gen binary by configuring cmake option
>> -DGEN_PCI_ID=xxxx or calling the gbe_bin_generater with option -t
>> GEN_PCI_ID.
>
> But what would a distribution do which wants to run on all supported
> hardware? Build beignet multiple times with different options?
>
> I think you need an option to generate binaries for all supported
> targets.

I think the point of generating llvm binary (by default) is making it
architecture independent. Therefore, the distribution should just
build the package WITHOUT setting GEN_PCI_ID. There's no need to add
an option to generate all hardware-specific binaries (unless using
llvm binaries have significant overhead.)

>
> -Andi
>
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet