2010-08-26 16:45:24

by David Howells

Subject: [PATCH] Alpha: Fix a missing comma in sys_osf_statfs()

Fix a comma that got accidentally deleted from sys_osf_statfs() leading to the
following warning:

arch/alpha/kernel/osf_sys.c: In function 'SYSC_osf_statfs':
arch/alpha/kernel/osf_sys.c:255: error: syntax error before 'buffer'

Signed-off-by: David Howells <[email protected]>
---

arch/alpha/kernel/osf_sys.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/alpha/kernel/osf_sys.c b/arch/alpha/kernel/osf_sys.c
index fb58150..5d1e6d6 100644
--- a/arch/alpha/kernel/osf_sys.c
+++ b/arch/alpha/kernel/osf_sys.c
@@ -252,7 +252,7 @@ SYSCALL_DEFINE3(osf_statfs, const char __user *, pathname,

retval = user_path(pathname, &path);
if (!retval) {
- retval = do_osf_statfs(&path buffer, bufsiz);
+ retval = do_osf_statfs(&path, buffer, bufsiz);
path_put(&path);
}
return retval;

2010-08-26 18:22:01

by Matt Turner

Subject: Re: [PATCH] Alpha: Fix a missing comma in sys_osf_statfs()

On Thu, Aug 26, 2010 at 12:44 PM, David Howells <[email protected]> wrote:
> Fix a comma that got accidentally deleted from sys_osf_statfs() leading to the
> following warning:
>
> ? ? ? ?arch/alpha/kernel/osf_sys.c: In function 'SYSC_osf_statfs':
> ? ? ? ?arch/alpha/kernel/osf_sys.c:255: error: syntax error before 'buffer'
>
> Signed-off-by: David Howells <[email protected]>
> ---
>
> ?arch/alpha/kernel/osf_sys.c | ? ?2 +-
> ?1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/alpha/kernel/osf_sys.c b/arch/alpha/kernel/osf_sys.c
> index fb58150..5d1e6d6 100644
> --- a/arch/alpha/kernel/osf_sys.c
> +++ b/arch/alpha/kernel/osf_sys.c
> @@ -252,7 +252,7 @@ SYSCALL_DEFINE3(osf_statfs, const char __user *, pathname,
>
> ? ? ? ?retval = user_path(pathname, &path);
> ? ? ? ?if (!retval) {
> - ? ? ? ? ? ? ? retval = do_osf_statfs(&path buffer, bufsiz);
> + ? ? ? ? ? ? ? retval = do_osf_statfs(&path, buffer, bufsiz);
> ? ? ? ? ? ? ? ?path_put(&path);
> ? ? ? ?}
> ? ? ? ?return retval;
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-alpha" in
> the body of a message to [email protected]
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html
>

Thanks, but Michael Cree already sent an identical patch. I've just
been slow to vacuum it up and send it on to Linus. I'll do this ASAP.

Matt