Re: 2.6.24 breaks nvidia and amd/ati binary drivers, by exporting paravirt symbols as GPL

On Nov 1 2007 19:36, Tobias Powalowski wrote:
>Hi
>commit to .24 tree:
>http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=93b1eab3d29e7ea32ee583de3362da84db06ded8
>
>introduces:
>+EXPORT_SYMBOL_GPL(pv_mmu_ops);
>+EXPORT_SYMBOL_GPL(pv_cpu_ops);
>
>pv_cpu_ops is for nvidia
>pv_mmu_ops' is for amd(ati)
>

Speaking of which, the patch introduced a strict-aliasing warning (that 
is, common userspace program compilation would warn about such) at:

+       return *((void **)&tmpl + type);

&tmpl is of type struct paravirt_patch_template **, and cross-casting 
this to void ** usually gives a strict-aliasing warning.
A workaround is to pre-cast to void *,
	return *((void **)(void *)&tmpl + type);
is that ok?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • 2.6.24 breaks nvidia and amd/ati binary drivers, by exporting paravirt symbols as GPL
    • From: Tobias Powalowski <[email protected]>

• Prev by Date: Re: [PATCH] x86: show cpuinfo only for online CPUs
• Next by Date: Re: [PATCH] e1000, e1000e valid-addr fixes
• Previous by thread: 2.6.24 breaks nvidia and amd/ati binary drivers, by exporting paravirt symbols as GPL
• Next by thread: build #337 failed for 2.6.24-rc1-gb1d08ac In function `usbnet_set_settings':
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]