[GIT PULL] orangefs: an adjustment and a fix...

[GIT PULL] orangefs: an adjustment and a fix...

[Mike Marshall]

The following changes since commit 13311e74253fe64329390df80bed3f07314ddd61:

  Linux 5.13-rc7 (2021-06-20 15:03:15 -0700)

are available in the Git repository at:

  https://git.kernel.org/pub/scm/linux/kernel/git/hubcap/linux.git
tags/for-linus-5.14-ofs1

for you to fetch changes up to 0fdec1b3c9fbb5e856a40db5993c9eaf91c74a83:

  orangefs: fix orangefs df output. (2021-06-28 08:40:08 -0400)

----------------------------------------------------------------
Orangefs: and adjustment and a fix

The readahead adjustment was suggested by Matthew Wilcox and looks like
how I should have written it in the first place... the "df fix" was
suggested by Walt Ligon, some Orangefs users have been complaining
about whacky df output...

----------------------------------------------------------------
Mike Marshall (2):
      orangefs: readahead adjustment
      orangefs: fix orangefs df output.

 fs/orangefs/inode.c | 7 +++----
 fs/orangefs/super.c | 2 +-
 2 files changed, 4 insertions(+), 5 deletions(-)

Re: [GIT PULL] orangefs: an adjustment and a fix...

[pr-tracker-bot]

The pull request you sent on Tue, 6 Jul 2021 09:36:40 -0400:

> https://git.kernel.org/pub/scm/linux/kernel/git/hubcap/linux.git tags/for-linus-5.14-ofs1

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/729437e334a9d9e079e2be9a42629316bee8a17e

Thank you!

-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html

Re: [GIT PULL] orangefs: an adjustment and a fix...

[Mike Marshall]

Fair enough... I promise it is my generally my habit to do
this and pay attention to the output before I send stuff
to you:

 1011  rm fs/orangefs/*.o
 1012  make

>>so clearly this has gotten absolutely zero testing.

I wouldn't do you that way :-) ... I ran this through xfstests
and left it on linux-next for a cycle.

-Mike

On Sat, Jun 26, 2021 at 1:02 PM Linus Torvalds
<torvalds@linux-foundation.org> wrote:
>
> On Sat, Jun 26, 2021 at 7:14 AM Mike Marshall <hubcap@omnibond.com> wrote:
> >
> > If it is not too late for these... the readahead adjustment
> > was suggested by Matthew Wilcox and looks like how I should
> > have written it in the first place...
>
> It wasn't too late - I pulled these.
>
> And then I un-pulled them. You removed all the uses of "file", but
> left the variable, so it all warns about
>
>   fs/orangefs/inode.c: In function ‘orangefs_readahead’:
>   fs/orangefs/inode.c:252:22: warning: unused variable ‘file’
> [-Wunused-variable]
>     252 |         struct file *file = rac->file;
>         |                      ^~~~
>
> so clearly this has gotten absolutely zero testing.
>
>            Linus

Re: [GIT PULL] orangefs: an adjustment and a fix...

[Linus Torvalds]

On Sat, Jun 26, 2021 at 7:14 AM Mike Marshall <hubcap@omnibond.com> wrote:
>
> If it is not too late for these... the readahead adjustment
> was suggested by Matthew Wilcox and looks like how I should
> have written it in the first place...

It wasn't too late - I pulled these.

And then I un-pulled them. You removed all the uses of "file", but
left the variable, so it all warns about

  fs/orangefs/inode.c: In function ‘orangefs_readahead’:
  fs/orangefs/inode.c:252:22: warning: unused variable ‘file’
[-Wunused-variable]
    252 |         struct file *file = rac->file;
        |                      ^~~~

so clearly this has gotten absolutely zero testing.

           Linus

[GIT PULL] orangefs: an adjustment and a fix...

[Mike Marshall]

The following changes since commit 13311e74253fe64329390df80bed3f07314ddd61:

  Linux 5.13-rc7 (2021-06-20 15:03:15 -0700)

are available in the Git repository at:

  https://git.kernel.org/pub/scm/linux/kernel/git/hubcap/linux.git
tags/for-linus-5.13-ofs2

for you to fetch changes up to 1815bba0a5c935d60f0fa180873d9152feb29d09:

  orangefs: fix orangefs df output. (2021-06-25 12:00:04 -0400)

----------------------------------------------------------------
Orangefs: and adjustment and a fix

If it is not too late for these... the readahead adjustment
was suggested by Matthew Wilcox and looks like how I should
have written it in the first place... the "df fix" was
suggested by Walt Ligon, some Orangefs users have been complaining
about whacky df output...

----------------------------------------------------------------
Mike Marshall (2):
      orangefs: readahead adjustment
      orangefs: fix orangefs df output.

 fs/orangefs/inode.c | 6 +++---
 fs/orangefs/super.c | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)