Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1299861 > unrolled thread
| Started by | Julia Lawall <Julia.Lawall@lip6.fr> |
|---|---|
| First post | 2016-01-01 10:20 +0100 |
| Last post | 2016-01-05 16:40 +0100 |
| Articles | 4 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] OrangeFS: constify export_operations structures Julia Lawall <Julia.Lawall@lip6.fr> - 2016-01-01 10:20 +0100
Re: [PATCH] OrangeFS: constify export_operations structures Mike Marshall <hubcap@omnibond.com> - 2016-01-04 17:20 +0100
Re: [PATCH] OrangeFS: constify export_operations structures Dan Carpenter <dan.carpenter@oracle.com> - 2016-01-04 19:50 +0100
Re: [PATCH] OrangeFS: constify export_operations structures Mike Marshall <hubcap@omnibond.com> - 2016-01-05 16:40 +0100
| From | Julia Lawall <Julia.Lawall@lip6.fr> |
|---|---|
| Date | 2016-01-01 10:20 +0100 |
| Subject | [PATCH] OrangeFS: constify export_operations structures |
| Message-ID | <qM42l-9N-5@gated-at.bofh.it> |
This export_operations structure is never modified, so declare it as const.
Most other structures of this type are already const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
---
Mike, I saw your name on all of the patches for this driver, but you aren't
in the MAINTAINERS file. If you are the maintainer of this file, maybe you
should add yourself there.
fs/orangefs/super.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/orangefs/super.c b/fs/orangefs/super.c
index 52bc522..bee67b3 100644
--- a/fs/orangefs/super.c
+++ b/fs/orangefs/super.c
@@ -342,7 +342,7 @@ out:
return type;
}
-static struct export_operations orangefs_export_ops = {
+static const struct export_operations orangefs_export_ops = {
.encode_fh = orangefs_encode_fh,
.fh_to_dentry = orangefs_fh_to_dentry,
};
--
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/
[toc] | [next] | [standalone]
| From | Mike Marshall <hubcap@omnibond.com> |
|---|---|
| Date | 2016-01-04 17:20 +0100 |
| Message-ID | <qNg1t-5Mg-33@gated-at.bofh.it> |
| In reply to | #1299861 |
Hi Julia... thanks for the patch... I've applied it to my github repo,
so it will be in the next push to kernel.org.
Orangefs is still out-of-tree (only temporarily I hope ;-) so I guess I
don't belong in the MAINTAINERS file yet...
-Mike
On Fri, Jan 1, 2016 at 4:01 AM, Julia Lawall <Julia.Lawall@lip6.fr> wrote:
> This export_operations structure is never modified, so declare it as const.
> Most other structures of this type are already const.
>
> Done with the help of Coccinelle.
>
> Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
>
> ---
>
> Mike, I saw your name on all of the patches for this driver, but you aren't
> in the MAINTAINERS file. If you are the maintainer of this file, maybe you
> should add yourself there.
>
> fs/orangefs/super.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/orangefs/super.c b/fs/orangefs/super.c
> index 52bc522..bee67b3 100644
> --- a/fs/orangefs/super.c
> +++ b/fs/orangefs/super.c
> @@ -342,7 +342,7 @@ out:
> return type;
> }
>
> -static struct export_operations orangefs_export_ops = {
> +static const struct export_operations orangefs_export_ops = {
> .encode_fh = orangefs_encode_fh,
> .fh_to_dentry = orangefs_fh_to_dentry,
> };
>
--
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/
[toc] | [prev] | [next] | [standalone]
| From | Dan Carpenter <dan.carpenter@oracle.com> |
|---|---|
| Date | 2016-01-04 19:50 +0100 |
| Message-ID | <qNimC-7d3-11@gated-at.bofh.it> |
| In reply to | #1300870 |
On Mon, Jan 04, 2016 at 11:18:00AM -0500, Mike Marshall wrote: > Hi Julia... thanks for the patch... I've applied it to my github repo, > so it will be in the next push to kernel.org. > > Orangefs is still out-of-tree (only temporarily I hope ;-) so I guess I > don't belong in the MAINTAINERS file yet... > It's in linux-next though. regards, dan carpenter -- 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/
[toc] | [prev] | [next] | [standalone]
| From | Mike Marshall <hubcap@omnibond.com> |
|---|---|
| Date | 2016-01-05 16:40 +0100 |
| Message-ID | <qNBSi-4Kz-31@gated-at.bofh.it> |
| In reply to | #1300962 |
I checked with Stephen Rothwell too, and he agrees with y'all that there should be a Orangefs stanza in the MAINTAINERS file... it will be there next time I push to kernel.org... Thanks! -Mike On Mon, Jan 4, 2016 at 1:45 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote: > On Mon, Jan 04, 2016 at 11:18:00AM -0500, Mike Marshall wrote: >> Hi Julia... thanks for the patch... I've applied it to my github repo, >> so it will be in the next push to kernel.org. >> >> Orangefs is still out-of-tree (only temporarily I hope ;-) so I guess I >> don't belong in the MAINTAINERS file yet... >> > > It's in linux-next though. > > regards, > dan carpenter > -- 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/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web