From: Blaisorblade <bla...@ya...> - 2005-01-24 19:55:58
|
On Monday 24 January 2005 20:43, Marcus Better wrote: > Hi all, > > I applied Blaisorblade's host-skas3-2.6.9-v7.patch, and it breaks the > fglrx graphics driver from ATI[1]. This is with Debian kernel 2.6.9, but > I believe it applies to vanilla kernels also. > > The problem is the following declaration added to include/asm-i386/desc.h: > > ---------------------------------------------------------- > extern int modify_ldt(struct mm_struct * mm, int func, void __user *ptr, > unsigned long bytecount); > ---------------------------------------------------------- > > The fglrx driver contains the following macro in a source file: > > _syscall3( int, modify_ldt, int, func, void *, ptr, unsigned long, > bytecount ) > > Here the compiler chokes with the following error: > --------------------------------------------------------- > /usr/src/modules/fglrx-4.3.0-3.14.6/firegl_public.c:181: error: > conflicting types for `modify_ldt' > include/asm/desc.h:144: error: previous declaration of `modify_ldt' > --------------------------------------------------------- > > I don't understand the details of the kernel code, but it seems that the > problem is that the SKAS patch defines the function modify_ldt which > conflicts with the syscall macros. > I renamed the new function to _modify_ldt instead (and changed the two > or three references to it in ldt.c), and this seems to fix the problem. I.e. the compilation problem is not in the kernel module, right? In this case, the problem is that you have /usr/include/asm symlinked to your kernel's include/asm-i386 directory, which is the wrong setup. However your solution should not give any problem. If this is wrong, instead, then you are surely right... but then the solution you explain would be unlikely to fix anything... and the code would also be different, I guess. It's just a quick shoot however, so feel free to correct me. > Please CC replies to me, I'm not subscribed to the list. > Thanks for your work! Thanks to you for using UML! > Marcus B. > > [1] http://www.ati.com/support/drivers/linux/radeon-linux.html -- Paolo Giarrusso, aka Blaisorblade Linux registered user n. 292729 http://www.user-mode-linux.org/~blaisorblade |