Page 1 of 3 123 LastLast
Results 1 to 10 of 29

Thread: patch: vanilla fglrx for 2.6.29

  1. #1
    Join Date
    Apr 2007
    Posts
    99

    Default patch: vanilla fglrx for 2.6.29

    Someone released a giant patch for fglrx that makes it compatible with 2.6.29.

    Well, I really don't know how they do it, but for some reason their patches have so many offset errors and doesn't work out of the box. They even copied the kernel headers from their kernel and put them into the patch... WTF, hahahaha. So the Mandriva guys fixed it (thanks!) and i modified the patch to work stand alone - as in, you extract the fglrx .run file and do patch -p1 < ../patch.file.

    Please, the next person who makes a patch... it would be really nice if you could explain to us how you always seem to have a different source than the rest of us as the patches never seem work the first time.

    It's as if the patches are designed to infuriate users or try to spread FUD over fglrx. Whatever the deal, it works, so keep it up if that's your objective!

    Anyway, fixed the patch, gg noobs:

    Code:
    diff -Nparu a/common/lib/modules/fglrx/build_mod/firegl_public.c b/common/lib/modules/fglrx/build_mod/firegl_public.c
    --- a/common/lib/modules/fglrx/build_mod/firegl_public.c	2009-01-23 20:00:26.000000000 -0200
    +++ b/common/lib/modules/fglrx/build_mod/firegl_public.c	2009-02-13 15:25:00.000000000 -0200
    @@ -1460,7 +1460,11 @@ KCL_TYPE_Pid ATI_API_CALL KCL_GetTgid(vo
      */
     KCL_TYPE_Uid ATI_API_CALL KCL_GetEffectiveUid(void)
     {
    +#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,28)
    +    return current->cred->euid;
    +#else
         return current->euid;
    +#endif
     }
     
     /** /brief Delay execution for the specified number of microseconds
    @@ -1832,15 +1836,30 @@ int ATI_API_CALL KCL_PosixSecurityCapChe
      */
     void ATI_API_CALL KCL_PosixSecurityCapSetIPCLock(unsigned int lock)
     {
    +
    +#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,28)
    +    struct cred *new = prepare_creds();
    +    if (!new) {
    +	    printk(KERN_ERR "fglrx: could not allocate memory\n");
    +	    return;
    +    }
    +#else
    +#define new current
    +#endif
         if (lock == 0 )
         {
    -        cap_lower(current->cap_effective, CAP_IPC_LOCK);
    +        cap_lower(new->cap_effective, CAP_IPC_LOCK);
         }
         else
         {
    -        cap_raise(current->cap_effective, CAP_IPC_LOCK);
    +        cap_raise(new->cap_effective, CAP_IPC_LOCK);
         }    
    -    return; 
    +
    +#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,28)
    +    commit_creds(new);
    +#else
    +#undef new
    +#endif
     }
     
     /** \brief Get number of available RAM pages
    diff -Nparu a/common/lib/modules/fglrx/build_mod/firegl_public.h b/common/lib/modules/fglrx/build_mod/firegl_public.h
    --- a/common/lib/modules/fglrx/build_mod/firegl_public.h	2009-01-23 20:00:26.000000000 -0200
    +++ b/common/lib/modules/fglrx/build_mod/firegl_public.h	2009-02-16 14:02:50.000000000 -0300
    @@ -18,6 +18,7 @@
     #define _FIREGL_PUBLIC_H_
     
     #include <stdarg.h>
    +#include <asm/pgtable.h>
     #include "kcl_pci.h"
     #include "kcl_io.h"
     
    @@ -590,6 +591,11 @@ extern unsigned long        KCL_SYSINFO_
     #define cpu_has_pge test_bit(X86_FEATURE_PGE, &boot_cpu_data.x86_capability)
     #endif
     
    +#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,28)
    +#undef pgprot_writecombine
    +#undef pgprot_noncached
    +#endif
    +
     #ifndef pgprot_writecombine
     #define pgprot_writecombine(prot) __pgprot((pgprot_val(prot) & ~(_PAGE_PCD)) | _PAGE_PWT)
     #endif
    @@ -598,6 +604,7 @@ extern unsigned long        KCL_SYSINFO_
     #define pgprot_noncached(prot) __pgprot(pgprot_val(prot) | _PAGE_PCD | _PAGE_PWT)
     #endif
     
    +
     #endif //FIREGL_USWC_SUPPORT
     
     
    diff -Nparu a/common/lib/modules/fglrx/build_mod/kcl_acpi.c b/common/lib/modules/fglrx/build_mod/kcl_acpi.c
    --- a/common/lib/modules/fglrx/build_mod/kcl_acpi.c	2009-01-23 20:00:26.000000000 -0200
    +++ b/common/lib/modules/fglrx/build_mod/kcl_acpi.c	2009-02-13 15:25:00.000000000 -0200
    @@ -18,6 +18,12 @@
     #include <linux/autoconf.h>
     #include <linux/acpi.h>
     
    +#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,28)
    +#include <acpi/acpica/acconfig.h>
    +#include <acpi/acpica/aclocal.h>
    +#include <acpi/acpica/acobject.h>
    +#endif
    +
     #include "kcl_config.h"
     #include "kcl_type.h"
     #include "kcl_acpi.h"
     #include "kcl_acpi.h"
    PS, add an empty return line to this patch otherwise you'll get a "patch ended unexpectedly" error or related.
    Last edited by damentz; 03-24-2009 at 03:57 PM.

  2. #2
    Join Date
    Aug 2007
    Posts
    6,598

    Default

    Looks better, then you provide the url to that patch. Must be from a dkms package I guess.

  3. #3
    Join Date
    Mar 2007
    Posts
    36

    Default

    patch runs, but module build on 32 bit Debian Sid fails at: /lib/modules/fglrx/build_mod/2.6.x/kcl_acpi.o, ati module builder exits with error, X fails to start though it doesn't totally fail, there is something there, but not usable.

    ctrl alt f1 works to return to console.

    Update: kcl_acpi.o module builder requires the missing files acpi/acpica/acconf.h, aclocal.h, and acobject.h

    This error is shown in the ati installer error logs, so that's the problem, it's why that acpi part fails. So those things need to go somewhere or other, and not be removed.
    Last edited by gfxdrone; 03-24-2009 at 05:21 PM.

  4. #4
    Join Date
    Apr 2007
    Posts
    99

    Default

    Kano pointed out earlier:

    +#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,28)
    +#include <acpi/acpica/acconfig.h>
    +#include <acpi/acpica/aclocal.h>
    +#include <acpi/acpica/acobject.h>
    +#endif
    These headers don't actually exist, so they were copied from the kernel source - that's why the previous patch was so large.

  5. #5
    Join Date
    Aug 2008
    Posts
    22

    Default

    Perhaps you should have a look at: /usr/src/linux/drivers/acpi/acpica

  6. #6

    Default

    Quote Originally Posted by damentz View Post
    Kano pointed out earlier:



    These headers don't actually exist, so they were copied from the kernel source - that's why the previous patch was so large.
    The problem is, most of the code fglrx uses was made private to the kernel in 2.6.29.

  7. #7
    Join Date
    Apr 2007
    Posts
    99

    Default

    Ok, new patch -> http://liquorix.net/patches/

    It's a modification of Kano's, same procedure as the original patch.

  8. #8
    Join Date
    Aug 2007
    Posts
    6,598

    Default

    The problem is that Debian/Ubuntu header packages are just wrong and do not include the new acpi dir.

  9. #9
    Join Date
    Mar 2009
    Posts
    141

    Default

    1) Is this safe?
    2) Will ati be releasing their own fix for this anytime soon?

  10. #10
    Join Date
    Aug 2007
    Posts
    437

    Default

    But does it work on X-Server 1.6?

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •