[GURU KERNEL] wd1100.c: Can't register sysctl

Recherche :

Mot : Pseudo : Filtrer
Page : 1 2 Page Suivante Page Précédente Bas de page
Auteur	Sujet : [GURU KERNEL] wd1100.c: Can't register sysctl

M300A

:hello:

Je m'amuse un peu sur de l'embarqué (WRAP de chez pc-engines) et je bloque sur le dernier truc.

La carte est dotée d'une watchdog wd1100. J'ai trouvé un patch porté sur le 2.6 par voyage-linux. Ca patch et ca build bien.
Cependant une fois que j'insère le module je me mange un "wd1100.c: Can't register sysctl"

Qu'est ce que ca signifie ?

Voici le code qui arrive à cette erreur :

ret = misc_register(&sc1x00wd_miscdev);
if (ret)
printk(KERN_ERR "wd1100.c: Can't register device.\n" );
else
{
wd_table_header = register_sysctl_table(wd_root_table, 1);
if (wd_table_header == NULL)
printk(KERN_ERR "wd1100.c: Can't register sysctl.\n" );
}

Pourriez vous m'expliquer ce qui se passe ?

Merci :jap:

Publicité

M300A

Code :

/*
* National Semiconductor SC1x00 CPU watchdog driver
* Copyright (c) Inprimis Technologies 2002
*
* by Mark Grosberg <markg@inprimis.com>
* and Rolando Goldman <rolandog@inprimis.com>
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version
* 2 of the License, or (at your option) any later version.
*
* 09/10/2003
* - Module parameter support
* - Modern processor support
* added by Erich Titl erich.titl@think.ch
*
* 09/01/2005
* - port to 2.6 kernel
* added by Punky Tse punkytse@yahoo.com
*
*/
#include <linux/module.h>
#include <linux/types.h>
#include <linux/kernel.h>
#include <linux/fs.h>
#include <linux/mm.h>
#include <linux/miscdevice.h>
#include <linux/watchdog.h>
#include <linux/spinlock.h>
#include <linux/sysctl.h>
#include <linux/pci.h>
/*
* Since the SC1100 is an x86 clone, we don't even bother with
* allowing other architectures to compile us.
*/
#ifndef CONFIG_X86
# error Sorry this driver is only for x86.
#endif
#include <asm/system.h>
#include <asm/io.h>
#include <asm/uaccess.h>
#include <asm/processor.h>
/*
#define DEBUG_WD1100
*/
static int proc_wd_timeout(ctl_table *ctl,
int write,
struct file *file,
void *buffer,
size_t *lenp,
loff_t *ppos);
static int proc_wd_graceful(ctl_table *ctl,
int write,
struct file *file,
void *buffer,
size_t *lenp,
loff_t *ppos);
/* Register definitions */
#define SC1100_F5_VENDOR_ID 0x100B
#define SC1100_F5_DEVICE_ID 0x0515
#define CPU_WDTO_REG 0x00 /* watchdog time out, 16 bit register */
#define CPU_WDCNFG_REG 0x02 /* watchdog config , 16 bit register */
#define CPU_WDSTS_REG 0x04 /* watchdog status , 8 bit register */
/* Default timeout: 4 seconds (changeable via sysctl) */
static unsigned int sysctl_wd_timeout = 4;
static unsigned int sysctl_wd_graceful = 1;
static int in_use = 0;
static unsigned short cpu_base;
static struct semaphore wd_semaphore;
/**************************************************************************/
#define GCB_LOCATION 0x1234 // just a dummy value
int gcb = GCB_LOCATION;
MODULE_PARM (gcb, "i" );
MODULE_PARM_DESC (gcb, "The GCB location (default is to look into the scratch pad)" );
MODULE_PARM (sysctl_wd_graceful, "i" );
MODULE_PARM_DESC (sysctl_wd_graceful, "Graceful shutdown on close" );
MODULE_PARM (sysctl_wd_timeout, "i" );
MODULE_PARM_DESC (sysctl_wd_timeout, "Watchdog timeout value in seconds" );
/**************************************************************************/
/* XXX To-do: DEV_WATCHDOG must be in include/linux/sysctl.h */
enum
{ DEV_WATCHDOG = 6 };
enum
{
DEV_WD_TIMEOUT = 1,
DEV_WD_GRACEFUL = 2
};
static struct ctl_table_header *wd_table_header;
static ctl_table wd_table[] = {
{
DEV_WD_TIMEOUT, "timeout",
&sysctl_wd_timeout, sizeof(int), 0644, NULL, &proc_wd_timeout
},
{
DEV_WD_GRACEFUL, "graceful",
&sysctl_wd_graceful, sizeof(int), 0644, NULL, &proc_wd_graceful
},
{0}
};
static ctl_table wd_dir_table[] = {
{DEV_WATCHDOG, "wd", NULL, 0, 0555, wd_table},
{0}
};
static ctl_table wd_root_table[] = {
{CTL_DEV, "dev", NULL, 0, 0555, wd_dir_table},
{0}
};
static int proc_wd_timeout(ctl_table *ctl,
int write,
struct file *file,
void *buffer,
size_t *lenp,
loff_t *ppos)
{
int rc;
rc = proc_dointvec(ctl, write, file, buffer, lenp, ppos);
if (write && (rc == 0))
{
/* Clamp to limits. */
if (sysctl_wd_timeout < 1)
sysctl_wd_timeout = 1;
else if (sysctl_wd_timeout > 65535)
sysctl_wd_timeout = 65535;
}
return (rc);
}
static int proc_wd_graceful(ctl_table *ctl,
int write,
struct file *file,
void *buffer,
size_t *lenp,
loff_t *ppos)
{
int rc;
rc = proc_dointvec(ctl, write, file, buffer, lenp, ppos);
if (write && (rc == 0))
{
/* Clamp to true/false. */
if (sysctl_wd_graceful)
sysctl_wd_graceful = 1;
}
return (rc);
}
/**************************************************************************/
static __inline__ void reset_wd(void)
{
outw(sysctl_wd_timeout * 8, cpu_base + CPU_WDTO_REG);
}
static int reboot_reason(void)
{
unsigned static char sr;
static int fetched = 0;
if (!fetched)
{
sr = inb(cpu_base + CPU_WDSTS_REG);
outb(sr | 1, cpu_base + CPU_WDSTS_REG);
fetched = 1;
}
return ((sr & 0x01) ? 1 : 0);
}
static struct watchdog_info wd_info =
{
0, /* Options */
0, /* Firmware version */
"NSC SC1x00 WD"
};
static int wd_ioctl(struct inode *inode,
struct file *file,
unsigned int cmd,
unsigned long arg)
{
int i;
switch (cmd)
{
default:
return (-ENOTTY);
case WDIOC_GETSUPPORT:
i = access_ok(VERIFY_WRITE, (void *)arg, sizeof(struct watchdog_info));
if (i)
return (i);
else
return copy_to_user((struct watchdog_info *)arg,
&wd_info,
sizeof(wd_info));
break;
case WDIOC_KEEPALIVE:
reset_wd();
return (0);
case WDIOC_GETBOOTSTATUS:
i = reboot_reason();
return (put_user(i, (int *)arg));
case WDIOC_GETSTATUS:
i = inw(cpu_base + CPU_WDTO_REG) / 8;
return (put_user(i, (int *)arg));
}
}
static int wd_open(struct inode *inode,
struct file *file)
{
int i;
if (down_trylock(&wd_semaphore))
return -EBUSY;
/*
* Configure the chip to do a reset if the timer goes to 0.
* Set the clock divisor to 4096.
*/
i = inw(cpu_base + CPU_WDCNFG_REG);
// keep the high order 7 bits of the config register
// clear bit 8 and write 0xfc to the low order 8 bits
outw((i & 0xfe00) | 0x00fc, cpu_base + CPU_WDCNFG_REG);
/* Start the watchdog: It won't run until we write the TO reg. */
reset_wd();
return nonseekable_open(inode, file);
}
static int wd_release(struct inode *inode,
struct file *file)
{
in_use = 0;
/*
* If graceful shutdown is not set, then don't bother to stop the
* watchdog timer. This handles the scenario where the user process
* that is poking the watchdog gets terminated due to some error
* (say a SEGV or some VM condition).
*
* In that case, the kernel would happily close the descriptor for
* us and leave us in a state where we aren't watching the dog...
*
* To work around this, the "graceful" sysctl prevents reset of the
* watchdog on close.
*/
#ifdef DEBUG_WD1100
printk(KERN_INFO "wd1100.c: releasing wd1100 sysctl_wd_graceful = %x\n",sysctl_wd_graceful);
#endif
if (sysctl_wd_graceful)
outw(0, cpu_base + CPU_WDCNFG_REG);
up(&wd_semaphore);
return (0);
}
static ssize_t wd_write(struct file *file,
const char *data,
size_t len,
loff_t *ppos)
{
if (len > 0)
reset_wd();
return (len);
}
static struct file_operations wd_fops=
{
owner: THIS_MODULE,
write: wd_write,
ioctl: wd_ioctl,
open: wd_open,
release: wd_release,
};
static struct miscdevice sc1x00wd_miscdev=
{
WATCHDOG_MINOR,
"watchdog",
&wd_fops
};
static int __init wd_init(void)
{
int ret;
struct pci_dev *dev;
unsigned int cw;
if (strcmp(boot_cpu_data.x86_vendor_id, "Geode by NSC" ) != 0)
{
printk(KERN_ERR "wd1100.c: This is not an SC1100 processor!\n" );
return (0);
}
switch (boot_cpu_data.x86_model)
{
case 4: // those are the models we know
case 9:
break;
default:
printk(KERN_WARNING "wd1100.c: unknown model value %d, trying anyway\n",boot_cpu_data.x86_model);
break;
}
if (gcb == GCB_LOCATION) // this is the assigned dummy value
{
#ifdef DEBUG_WD1100
printk(KERN_INFO "wd1100.c: fetching the value for gcb from scratch pad\n" );
#endif
/* get the CONFIG BLOCK ADDRESS from scratch pad register */
dev = pci_find_device(SC1100_F5_VENDOR_ID,SC1100_F5_DEVICE_ID,0);
if (dev == NULL)
{
printk(KERN_ERR "wd1100.c: Can not find bridge device.\n" );
return (0);
}
// read a double word at offset 0x64 into cw
pci_read_config_dword(dev, 0x64, &cw);
cpu_base = (unsigned short )cw;
}
else // it is not the default value, just use it
{
#ifdef DEBUG_WD1100
printk(KERN_INFO "wd1100.c: using the assigned value for gcb \n" );
#endif
cpu_base = (unsigned short) gcb;
}
#ifdef DEBUG_WD1100
printk(KERN_INFO "wd1100.c: CPU base = 0x%X\n", (unsigned int )cpu_base);
#endif
printk(KERN_INFO "SC1x00 Watchdog driver by Inprimis Technolgies.\n" );
printk(KERN_INFO "wd1100.c: a few hacks by erich.titl@think.ch\n" );
/*
* We must call reboot_reason() to reset the flag in the WD.
*
* Even though it is available as an ioctl(), we call it during
* module initialization to perform the clear. You can take out
* the printk(), but don't take out the call to reboot_reason().
*/
if (reboot_reason())
printk(KERN_INFO "Last reboot was by watchdog!\n" );
sema_init(&wd_semaphore, 1);
ret = misc_register(&sc1x00wd_miscdev);
if (ret)
printk(KERN_ERR "wd1100.c: Can't register device.\n" );
else
{
wd_table_header = register_sysctl_table(wd_root_table, 1);
if (wd_table_header == NULL)
printk(KERN_ERR "wd1100.c: Can't register sysctl.\n" );
}
return 0;
}
static void __exit wd_exit(void)
{
if (wd_table_header != NULL)
unregister_sysctl_table(wd_table_header);
misc_deregister(&sc1x00wd_miscdev);
}
module_init(wd_init);
module_exit(wd_exit);
MODULE_LICENSE("GPL" );
MODULE_AUTHOR("Mark Grosberg <markg@inprimis.com> and Rolando Goldman <rolandog@inprimis.com>" );
MODULE_DESCRIPTION("National Semiconductor SC1x00 CPU watchdog driver" );
MODULE_ALIAS_MISCDEV(WATCHDOG_MINOR);

Message édité par M300A le 15-09-2006 à 04:03:19

M300A

Vu qu'il est pas énorme j'ai posté wd1100.c

M300A

:hot:

M300A

:hello:

Je suis sur que quelqu'un dev un peu et pourrait m'expliquer

dchost99

ctrl+F =
wd_table_header = register_sysctl_table(wd_root_table, 1);
if (wd_table_header == NULL)
printk(KERN_ERR "wd1100.c: Can't register sysctl.\n" );
}

c'est l'err wd_table_header

Ok ok je sort... :whistle:

Message édité par dchost99 le 15-09-2006 à 23:50:42

M300A

Merci j'avais même déduit que soit la fonction n'a pas marché "register_sysctl_table" soit que la structure "wd_root_table" qui a été passée à la fonction n'etait pas valide.

J'ai pa su aller plus loin par contre

Ok ok je sors aussi... :whistle:

ellimac

moi j'ai regarder un peu, je mettrais bien un 0 à la place du 1 dans l'appel de register_sysctl_table. Sinon c'est pas facile de trouver de la doc sur sysctl donc ...

je sort

edit: tout bien réfléchis ça devrais pas changer grand chose. Sinon le problème c'est qu'il n'arrive pas à créer les fichiers de controle dans /proc/sys/dev ...

Message cité 1 fois
Message édité par ellimac le 16-09-2006 à 01:14:10

M300A

ellimac a écrit :

moi j'ai regarder un peu, je mettrais bien un 0 à la place du 1 dans l'appel de register_sysctl_table. Sinon c'est pas facile de trouver de la doc sur sysctl donc ...

je sort

Tu as une raison pour mettre un 0 au lieu de 1 ?

ellimac

Code :

static ctl_table wd_table[] = {
{
.ctl_name = DEV_WD_TIMEOUT,
.procname = "timeout",
.data = &sysctl_wd_timeout,
.maxlen = sizeof(int),
.mode = 0644,
.proc_handler = &proc_wd_timeout
},
{
.ctl_name = DEV_WD_GRACEFUL,
.procname = "graceful",
.data = &sysctl_wd_graceful,
.maxlen = sizeof(int),
.mode = 0644,
.proc_handler = &proc_wd_graceful
},
{ .ctl_name = 0}
};
static ctl_table wd_dir_table[] = {
{
.ctl_name = DEV_WATCHDOG,
.procname = "wd",
.maxlen = 0,
.mode = 0555,
.child = wd_table
},
{ .ctl_name = 0}
};
static ctl_table wd_root_table[] = {
{
.ctl_name = CTL_DEV,
.procname = "dev",
.maxlen = 0,
.mode = 0555,
.child =wd_dir_table
},
{.ctl_name = 0}
};

Sinon essaye de remplacer les déclaration de la ligne 110 à 132 par ce code là ( modulo mes éventuels erreur de frappe )

Message édité par ellimac le 16-09-2006 à 01:47:29

Publicité

M300A

Je teste ca immédiatement

PS : C'est normalisé cette indentation, c'est très étrange ?

ellimac

j'ai trouvé ça dans d'autres fichiers source du kernel et j'ai adapter à ton cas en croisant les doigts. L'indentation j'ai pas trop fait gaffe mais grosso modo c'étais présenter comme ça, et les .clt_name et autre ça permet de nommer les champs directement pour être sur que tout est au bon endroit.

M300A

Tu développe pour le kernel ou dans un module ? :??:

ellimac

ni l'un ni l'autre, enfin juste un module qui fait rien pour l'école d'ingé ...

M300A

CC [M] drivers/char/watchdog/wd1100.o
drivers/char/watchdog/wd1100.c:118: error: unknown field ‘date’ specified in initializer
drivers/char/watchdog/wd1100.c:129: error: unknown field ‘date’ specified in initializer
drivers/char/watchdog/wd1100.c:164: error: request for member ‘mode’ in something not a structure or union
drivers/char/watchdog/wd1100.c:165: error: request for member ‘child’ in something not a structure or union
make[4]: *** [drivers/char/watchdog/wd1100.o] Erreur 1
make[3]: *** [drivers/char/watchdog] Erreur 2
make[2]: *** [drivers/char] Erreur 2
make[1]: *** [drivers] Erreur 2
make[1]: quittant le répertoire « /usr/src/wrap-kernel/2.6.15/linux-2.6.15.7 »
make: *** [debian/stamp-build-kernel] Erreur 2

Ca passe pas

ellimac

date = data , désolé pour la faute de frappe, et il manque quelque virgule, j'ai corrigé

Message édité par ellimac le 16-09-2006 à 01:47:51

M300A

J'ai corrigé aussi, mci

Le module est passé je termine le build du kernel et je le test sur la machine !

M300A

Toujours le même message lors de l'insertion

M300A

Tu as raison il est temps d'aller dormir

Merci pour ton aide en tout cas, si tu as d'autre idées n'hésite pas.

Etant donné que cette watchdog n'est présente que sur les Geode SC1100 d'AMD je ne trouve aucune info part google...

ellimac

t'as bien CONFIG_SYSCTL=y dans ton .config ? on sait jamais

M300A

Oops

/me se frappe la tête contre le clavier

Je rebuild avec le support sysctrl, ca risque de mieux marcher...

Merci beaucoup :jap:

M300A

Le moduel s'insere maintenant sans message cependant il ne fait rien

J'ai créé /dev/watchdog et j'ai inseré le module de cette facon :

modprobe wd1100 sysctl_wd_graceful=0 sysctl_wd_timeout=30

Evidemment au bout de 30sec, aucun reboot matériel (je n'ai pas le demon watchdog installé).

Une idée ?

ellimac

je pense que tu dois avoir besoin du demon, car d'apres les commentaires dans le code, le reglage du watchdog se fait à l'ouverture du péripherique.

M300A

Okay :jap:

M300A

Tu pourrais m'expliquer ou ca se fait dans le code, j'essaye de comprendre si possible

Je ne vois aucun références à /dev/watchdog ??

ellimac

En fait pour l'écriture d'un module du noyau, il y a 4 méthode obligatoire si mes souvenirs sont bons, register, unregister, open, release. register s'éxécute au chargement du module, unregister au déchargement du module, open quand un processus ouvre le peripherique, release quand il le ferme ( bizarre il me semblais que c'étais close) donc ici on a wd_register, wd_unregister, wd_open, wd_release. dans le code wd_open, on a une indication comme quoi c'est reset_wd qui démarre le watchdog, et il n'y a pas d'appel à reset_wd dans wd_register, j'en déduis donc quand tant que le peripherique n'a pas été ouvert, le watchdog fonctionne pas malgré que le module soit chargé.

Message édité par ellimac le 16-09-2006 à 13:26:39

M300A

Merci beaucoup pour ton aide :jap:

Le module est maintenant fonctionnel. Si je fais un echo vers le dev, 30 sec apres ca reboot hard et au rebot le module m'indique que le dernier reboot à été provoqué par la watchdog

Plus qu'a mettre le démon en place heheh !

PS : Tu crois que c'est possible de compiler ce module de facon indépendante ? Pour l'instant il est distribué sous forme de patch sur les sources du kernel.

J'ai essayé d'écrire un Makefile mais c'est pas concluant...

M300A

Code :

KVERS = $(shell uname -r)
KSRCS = /lib/modules/$(KVERS)/build
TARGET = wd1100.o
SOURCE = wd1100.c
MODNAME = wd1100
CFLAGS+ = -c -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fomit-frame-pointer -fno-strict-aliasing -fno-common -pipe
INCLUDE = -I$(KSRCS)/include
all: $(TARGET)
help:
@echo Possible targets:
@echo -e all\\t- default target, builds kernel module
@echo -e install\\t- copies module binary to /lib/modules/$(KVERS)/extra/
@echo -e clean\\t- removes all binaries and temporary files
wd1100.o: $(SOURCE)
$(CC) $(INCLUDE) $(CFLAGS) -DMODVERSIONS -DMODULE -D__KERNEL__ -o $(TARGET) $(SOURCE)
# $(MAKE) -C $(KSRCS) SUBDIRS=$(PWD) modules
clean:
rm -f *.ko
load: $(TARGET)
insmod $(TARGET)
unload:
rmmod $(MODNAME)
install: $(TARGET)
mkdir -p /lib/modules/$(KVERS)/extra
cp -v $(TARGET) /lib/modules/$(KVERS)/extra/
depmod -a

Message édité par M300A le 16-09-2006 à 16:30:56

ellimac

Code :

obj-m := wd1100.o
KDIR := /lib/modules/$(shell uname -r)/build
PWD := $(shell pwd)
all:
$(MAKE) -C $(KDIR) SUBDIRS=$(PWD) modules

c'est un peu minimal mais c'est ce que j'utilisais, tu peut peut-être t'en inspirer pour corriger le tien.

cela dis il y a peut-être des corrections à faire dans le source, j'ai un unresolve symbol au chargement sur verify_area.

Message édité par ellimac le 16-09-2006 à 21:46:39

M300A

Il y'a sans doute des modifs à faire, tout ce que je peux dire c'est qu'il roule sur le 2.6.15 vanilla.

Merci pour ton aide encore une fois :jap:

M300A

KVERS = $(shell uname -r)
KSRCS = /lib/modules/$(KVERS)/build
TARGET = wd1100.o
SOURCE = wd1100.c
MODNAME = wd1100

obj-m := wd1100.o

all: $(TARGET)

help:
@echo Possible targets:
@echo -e all\\t- default target, builds kernel module
@echo -e install\\t- copies module binary to /lib/modules/$(KVERS)/extra/
@echo -e clean\\t- removes all binaries and temporary files

wd1100.o: $(SOURCE)
$(MAKE) -C $(KSRCS) SUBDIRS=$(PWD) modules

clean:
rm -f *.ko *.o *.mod.c

load: $(TARGET)
insmod $(TARGET)

unload:
rmmod $(MODNAME)

install: $(TARGET)
mkdir -p /lib/modules/$(KVERS)/extra
cp -v $(TARGET) /lib/modules/$(KVERS)/extra/
depmod -a

Voila un Makefile qui roule
:jap:

Quand j'essaye de le build contre un 2.6.17 je me prend ces erreurs :

gandalf@hellscream:~/devel/wd1100$ make
make -C /lib/modules/2.6.17-2-686/build SUBDIRS=/home/gandalf/devel/wd1100 modules
make[1]: entrant dans le répertoire « /usr/src/linux-headers-2.6.17-2-686 »
CC [M] /home/gandalf/devel/wd1100/wd1100.o
/home/gandalf/devel/wd1100/wd1100.c:87: error: expected ‘)’ before string constant
/home/gandalf/devel/wd1100/wd1100.c:90: error: expected ‘)’ before string constant
/home/gandalf/devel/wd1100/wd1100.c:93: error: expected ‘)’ before string constant
make[2]: *** [/home/gandalf/devel/wd1100/wd1100.o] Erreur 1
make[1]: *** [_module_/home/gandalf/devel/wd1100] Erreur 2
make[1]: quittant le répertoire « /usr/src/linux-headers-2.6.17-2-686 »
make: *** [wd1100.o] Erreur 2

Tu peux me donner une piste ? :jap:

ellimac

il faut remplacer les lignes

Code :

MODULE_PARM (gcb, "i" );
MODULE_PARM (sysctl_wd_graceful, "i" );
MODULE_PARM (sysctl_wd_timeout, "i" );

par

Code :

module_param(gcb,int ,0);
module_param(sysctl_wd_graceful, uint, 0);
module_param(sysctl_wd_timeout, uint, 0);

bon évidemment je ne peut pas tester, j'ai pas le matériel en question, mais bon ça compile sur mon 2.6.17

M300A

gandalf@hellscream:~/devel/wd1100$ make
make -C /lib/modules/2.6.17-2-686/build SUBDIRS=/home/gandalf/devel/wd1100 modules
make[1]: entrant dans le répertoire « /usr/src/linux-headers-2.6.17-2-686 »
CC [M] /home/gandalf/devel/wd1100/wd1100.o
/home/gandalf/devel/wd1100/wd1100.c:87: error: expected ‘)’ before ‘int’
/home/gandalf/devel/wd1100/wd1100.c:90: error: expected ‘)’ before ‘uint’
/home/gandalf/devel/wd1100/wd1100.c:93: error: expected ‘)’ before ‘uint’
make[2]: *** [/home/gandalf/devel/wd1100/wd1100.o] Erreur 1
make[1]: *** [_module_/home/gandalf/devel/wd1100] Erreur 2
make[1]: quittant le répertoire « /usr/src/linux-headers-2.6.17-2-686 »
make: *** [wd1100.o] Erreur 2

Pas mieux

ellimac

euh t'a bien changé toute la ligne, la case compte en C, il suffit pas de rajouter int et uint dans la macro. Sinon c'est que j'ai fait une autre modification pour que ça compile mais là je vois pas.

M300A

Oui j'ai bien tout remplacer

Tu peux me filer ton source ?

Si tu sais pas ou le mettre : gandalf_at_le-vert_dot_net

ellimac

ok, je te l'ai envoyé, tu me dira quand même ce que j'ai changé pasque là je vois pas du tout.

M300A

Erreur de typo, s/parm/param/

Sinon il y'a ca comme différence encore

- i = access_ok(VERIFY_WRITE, (void *)arg, sizeof(struct watchdog_info));
+ i = verify_area(VERIFY_WRITE, (void *)arg, sizeof(struct watchdog_info));

Tu peux me transmettre ton nom et ton mail pour que je t'ajoute au copyright et au changelog ?

ellimac

pour cette différence laisse access_ok, j'ai pas du partir du même patch que toi. je t'envoie le reste par mail.

M300A

printk(KERN_ERR "wd1100.c: Can't register sysctl.\nwd1100.c: Is this kernel built with CONFIG_SYSCTL=y ?\n" );

Elle ne te pose pas de problèmes cette ligne ?

ellimac

ça ne devrais pas poser de problème, juste s/nwd1100.c/wd1100.c . On pourrais aussi modifier le source pour ne pas être obligé d'utiliser sysctl.

Message cité 1 fois

Publicité

Page : 1 2

Page Suivante

Page Précédente

Haut de page

FORUM HardWare.fr

Linux et OS Alternatifs

Divers

[GURU KERNEL] wd1100.c: Can't register sysctl

Sujets relatifs
nvidia mandriva 2006 compil kernel	[PHP] Register globals différent selon différents virtualhost ?
[Mandriva 2006 x86_64] Erreurs + kernel panic	kernel panic au demarage debian sarge 3.1 (megaraid inside)
Boot PXE / Initrd / Kernel	Comment avoir les sources de mon kernel
Reconnaitre ma version de kernel	[Kernel] Compiler libata en dur
Besoin d'aide sur un kernel panic	[debian] Sources du kernel 2.6.17-1-k7
Plus de sujets relatifs à : [GURU KERNEL] wd1100.c: Can't register sysctl

Page générée en 0.275 secondes