如何确保insmod在出错时失败?

时间:2014-02-27 10:29:14

标签: linux-kernel linux-device-driver insmod

我为Linux开发了一个外设驱动程序。 .probe函数执行常见的错误检查,如内存分配失败,并尝试与硬件和任何类型的错误进行通信,取消分配任何内存并返回错误代码,如-ENOMEM或-EIO。

问题是,虽然模块探测功能在硬件无法访问时返回-EIO,但我仍然看到模块列在lsmod输出中。当初始化期间出现问题时,是否可以确保insmod完全失败?

这是我目前的探测功能。所有特定于设备的功能在发生故障时都会返回相应的错误代码,通常是-EIO。

static int mlx90399_probe(struct i2c_client *client,
        const struct i2c_device_id *id)
{
    int err;
    struct mlx90399_data *data;

    data = kzalloc(sizeof(*data), GFP_KERNEL);
    if (!data) {
        dev_err(&client->dev, "Memory allocation fails\n");
        err = -ENOMEM;
        goto exit;
    }
    i2c_set_clientdata(client, data);
    data->client = client;
    mutex_init(&data->lock);
    data->mode = MLX90399_MODE_OFF;

    err = mlx90399_reset(client);
    if (err < 0)
        goto exit;

    msleep(1); /* nominal 0.6ms from reset */
    err = mlx90399_write_register_defaults(client);
    if (err < 0)
        goto exit;

    err = mlx90399_update_scale(client);
    if (err < 0)
        goto exit;

    data->indio_dev = iio_allocate_device(0);
    if (data->indio_dev == NULL) {
        err = -ENOMEM;
        goto exit;
    }

    data->indio_dev->dev.parent = &client->dev;
    data->indio_dev->info = &mlx90399_info;
    data->indio_dev->dev_data = (void *)(data);
    data->indio_dev->modes = INDIO_DIRECT_MODE;

    mlx90399_setup_irq(client);

    err = iio_device_register(data->indio_dev);
    if(err < 0)
        goto exit;

    return 0;

exit:
    kfree(data);
    return err;
}

2 个答案:

答案 0 :(得分:2)

请参阅__driver_attach()中的评论:

/*
 * Lock device and try to bind to it. We drop the error
 * here and always return 0, because we need to keep trying
 * to bind to devices and some drivers will return an error
 * simply if it didn't support the device.
 *
 * driver_probe_device() will spit a warning if there
 * is an error.
 */

要使模块初始化失败,请取消注册驱动程序并从init函数返回错误代码。

答案 1 :(得分:2)

请注意,模块和设备之间不一定是1:1的关系。一个模块可以用于多个设备。例如,通过使用设备树,设备树可以声明几个板载UART,所有这些都使用一个串行设备内核模块。模块的probe功能将被调用多次,每个设备一次。仅仅因为一个probe调用失败,这并不一定意味着应该卸载该模块。