lkml.org 
[lkml]   [2005]   [May]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [ANNOUNCE] mini_fo-0.6.0 overlay file system
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 <ezk@cs.sunysb.edu>");
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 majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-05-10 19:04    [W:0.115 / U:0.328 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site