mbox series

[GIT,PULL] 9p fixes for 6.7-rc1

Message ID ZUW82S8bGg2BAeNU@codewreck.org
State New
Headers
Series [GIT,PULL] 9p fixes for 6.7-rc1 |

Pull-request

https://github.com/martinetd/linux tags/9p-for-6.7-rc1

Message

Dominique Martinet Nov. 4, 2023, 3:39 a.m. UTC 
  The following changes since commit 05d3ef8bba77c1b5f98d941d8b2d4aeab8118ef1:

  Linux 6.6-rc7 (2023-10-22 12:11:21 -1000)

are available in the Git repository at:

  https://github.com/martinetd/linux tags/9p-for-6.7-rc1

for you to fetch changes up to ce07087964208eee2ca2f9ee4a98f8b5d9027fe6:

  9p/net: fix possible memory leak in p9_check_errors() (2023-10-27 12:44:13 +0900)

----------------------------------------------------------------
Bunch of small fixes:

- three W=1 warning fixes: the NULL -> "" replacement isn't trivial
but is serialized identically by the protocol layer and has been tested
- one syzbot/KCSAN datarace annotation where we don't care about users
messing with the fd they passed to mount -t 9p
- removing a declaration without implementation
- yet another race fix for trans_fd around connection close:
the 'err' field is also used in potentially racy calls and this
isn't complete, but it's better than what we have.
- and finally a theorical memory leak fix on serialization failure

----------------------------------------------------------------
Dominique Martinet (3):
      9p: v9fs_listxattr: fix %s null argument warning
      9p/net: xen: fix false positive printf format overflow warning
      9p/fs: add MODULE_DESCIPTION
(^ I've just noticed DESCIPTION -> DESCRIPTION typo, but I guess it's
understandable enough and I'd rather not rebase the branch just before
sending the pull request (that last memory leak patch came in late
enough) -- please say if you'd rather I fix it up and I'll resend this)

Hangyu Hua (1):
      9p/net: fix possible memory leak in p9_check_errors()

Marco Elver (1):
      9p/trans_fd: Annotate data-racy writes to file::f_flags

Sishuai Gong (1):
      9p/trans_fd: avoid sending req to a cancelled conn

Yue Haibing (1):
      fs/9p: Remove unused function declaration v9fs_inode2stat()

 fs/9p/v9fs.c       |  1 +
 fs/9p/v9fs_vfs.h   |  1 -
 fs/9p/xattr.c      |  5 +++--
 net/9p/client.c    |  8 +++++---
 net/9p/trans_fd.c  | 21 ++++++++++++++++-----
 net/9p/trans_xen.c | 15 +++++++--------
 6 files changed, 32 insertions(+), 19 deletions(-)

Comments

Linus Torvalds Nov. 4, 2023, 7:23 p.m. UTC | #1 
On Fri, 3 Nov 2023 at 17:39, Dominique Martinet <asmadeus@codewreck.org> wrote:
>
>       9p/fs: add MODULE_DESCIPTION
> (^ I've just noticed DESCIPTION -> DESCRIPTION typo, but I guess it's
> understandable enough and I'd rather not rebase the branch just before
> sending the pull request (that last memory leak patch came in late
> enough) -- please say if you'd rather I fix it up and I'll resend this)

No, no, a typo in the commit message is very much not a reason for
rebasing etc. So sending it to me as-is was th weight thing to do.

I did fix it in the resulting merge message shortlog since you pointed
it out, but hey, if we fixed every typo we had in our commits, we'd be
doing nothing but rebasing.

Kernel people aren't necessarily great at spelling, and that's true
even for those of us where English is our first language.

            Linus
pr-tracker-bot@kernel.org Nov. 4, 2023, 7:39 p.m. UTC | #2 
The pull request you sent on Sat, 4 Nov 2023 12:39:05 +0900:

> https://github.com/martinetd/linux tags/9p-for-6.7-rc1

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

Thank you!