Re: [PATCH 05/16] IB uverbs: core implementation

[Greg KH]

On Tue, Jun 28, 2005 at 04:03:43PM -0700, Roland Dreier wrote:
> +++ linux/drivers/infiniband/core/uverbs_main.c	2005-06-28 15:20:04.363963991 -0700
> @@ -0,0 +1,708 @@
> +/*
> + * Copyright (c) 2005 Topspin Communications.  All rights reserved.
> + * Copyright (c) 2005 Cisco Systems.  All rights reserved.
> + *
> + * This software is available to you under a choice of one of two
> + * licenses.  You may choose to be licensed under the terms of the GNU
> + * General Public License (GPL) Version 2, available from the file
> + * COPYING in the main directory of this source tree, or the
> + * OpenIB.org BSD license below:

Ok, I've complained about this before, but due to the fact that you are
calling EXPORT_SYMBOL_GPL() only functions in this code, the ability for
it for someone to use the BSD license on it in the future, is pretty
much impossible, right?

Wasn't the openib group going to drop this horrible license, or are they
still insisting on porting this to other operating systems?

> +static ssize_t show_dev(struct class_device *class_dev, char *buf)
> +{
> +	struct ib_uverbs_device *dev =
> +		container_of(class_dev, struct ib_uverbs_device, class_dev);
> +
> +	return print_dev_t(buf, dev->dev.dev);
> +}
> +static CLASS_DEVICE_ATTR(dev, S_IRUGO, show_dev, NULL);

This is no longer needed with the class device interface in the kernel
today.  Please use the new api (basically just set dev_t in the
class_device, and you get this for free.)

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/