Re: [ANNOUNCE] mini_fo-0.6.0 overlay file system

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Eric,

Thank you for the feedback.

On Tue, May 10, 2005 at 02:07:37AM -0400, Eric Lammerts wrote:
> Some remarks:
> Some functions return -ENOTSUPP on error, which makes "ls -l" complain 
> loudly when getxattr() fails. This should be -EOPNOTSUPP.

You're right. Fixed in attached patch.

> The module taints the kernel because of MODULE_LICENSE("LGPL").
> Since all your copyright statements say it's GPL software, better change 
> this to "GPL".

It seems to be ok to change this. Patch corrects this too.

Cheers

Markus



diff -Nru mini_fo.ORIG/inode.c mini_fo/inode.c
--- mini_fo.ORIG/inode.c	2005-05-06 23:59:08.000000000 +0200
+++ mini_fo/inode.c	2005-05-10 18:09:47.000000000 +0200
@@ -1259,7 +1259,7 @@
 STATIC int
 mini_fo_getxattr(struct dentry *dentry, const char *name, void *value, size_t size) {
 	struct dentry *hidden_dentry = NULL;
-	int err = -ENOTSUPP;
+	int err = -EOPNOTSUPP;
 	/* Define these anyway so we don't need as much ifdef'ed code. */
 	char *encoded_name = NULL;
 	char *encoded_value = NULL;
@@ -1304,7 +1304,7 @@
 
 {
 	struct dentry *hidden_dentry = NULL;
-	int err = -ENOTSUPP;
+	int err = -EOPNOTSUPP;
 
 	/* Define these anyway, so we don't have as much ifdef'ed code. */
 	char *encoded_value = NULL;
@@ -1340,7 +1340,7 @@
 STATIC int
 mini_fo_removexattr(struct dentry *dentry, const char *name) {
 	struct dentry *hidden_dentry = NULL;
-	int err = -ENOTSUPP;
+	int err = -EOPNOTSUPP;
 	char *encoded_name;
 
 	check_mini_fo_dentry(dentry);
@@ -1372,7 +1372,7 @@
 STATIC int
 mini_fo_listxattr(struct dentry *dentry, char *list, size_t size) {
 	struct dentry *hidden_dentry = NULL;
-	int err = -ENOTSUPP;
+	int err = -EOPNOTSUPP;
 	char *encoded_list = NULL;
 
 	check_mini_fo_dentry(dentry);
diff -Nru mini_fo.ORIG/main.c mini_fo/main.c
--- mini_fo.ORIG/main.c	2005-05-06 23:59:08.000000000 +0200
+++ mini_fo/main.c	2005-05-10 17:54:13.000000000 +0200
@@ -405,7 +405,7 @@
 
 MODULE_AUTHOR("Erez Zadok <[email protected]>");
 MODULE_DESCRIPTION("FiST-generated mini_fo filesystem");
-MODULE_LICENSE("LGPL");
+MODULE_LICENSE("GPL");
 
 /* MODULE_PARM(fist_debug_var, "i"); */
 /* MODULE_PARM_DESC(fist_debug_var, "Debug level"); */


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux