Linux-Fsdevel Archive on lore.kernel.org
 help / Atom feed
From: Jan Harkes <jaharkes@cs.cmu.edu>
To: Sasha Levin <sashal@kernel.org>
Cc: linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH 01/22] coda: pass the host file in vma->vm_file on mmap
Date: Sat, 18 May 2019 09:18:57 -0400
Message-ID: <A67240F2-2712-460E-A17F-EB8D24AD76DE@cs.cmu.edu> (raw)
In-Reply-To: <20190518122241.D867120B7C@mail.kernel.org>

That is actually great, 4 out of 7 patched without any issues, I love this bot.

The remaining 3 probably are all failing because of the missing call_mmap helper.

f74ac01520c9 ("mm: use helper for calling f_op->mmap()")

I can make a separate patch that doesn't use the helper. I will also check if this change is necessary for the older kernels, although right now I assume it probably is.

Jan

On May 18, 2019 8:22:41 AM EDT, Sasha Levin <sashal@kernel.org> wrote:
>Hi,
>
>[This is an automated email]
>
>This commit has been processed because it contains a -stable tag.
>The stable tag indicates that it's relevant for the following trees:
>all
>
>The bot has tested the following trees: v5.1.3, v5.0.17, v4.19.44,
>v4.14.120, v4.9.177, v4.4.180, v3.18.140.
>
>v5.1.3: Build OK!
>v5.0.17: Build OK!
>v4.19.44: Build OK!
>v4.14.120: Build OK!
>v4.9.177: Failed to apply! Possible dependencies:
>0f78d06ac1e9 ("vfs: pass type instead of fn to
>do_{loop,iter}_readv_writev()")
>7687a7a4435f ("vfs: extract common parts of
>{compat_,}do_readv_writev()")
>bb7462b6fd64 ("vfs: use helpers for calling f_op->{read,write}_iter()")
>    f74ac01520c9 ("mm: use helper for calling f_op->mmap()")
>
>v4.4.180: Failed to apply! Possible dependencies:
>0b944d3a4bba ("aio: hold an extra file reference over AIO read/write
>operations")
>    1bd816f12071 ("drm/vgem: Use lockless gem BO free callback")
>    2710fd7e00b4 ("f2fs: introduce dirty list node in inode info")
>2dbf0d90971a ("drm/i915: Use CPU mapping for userspace dma-buf mmap()")
>    4cf185379b75 ("f2fs: add a tracepoint for sync_dirty_inodes")
>    6ad7609a183a ("f2fs: introduce __remove_dirty_inode")
>    6d5a1495eebd ("f2fs: let user being aware of IO error")
>    70fe2f48152e ("aio: fix freeze protection of aio writes")
>    89319d31d2d0 ("fs: remove aio_run_iocb")
>b439b103a6c9 ("f2fs: move dio preallocation into f2fs_file_write_iter")
>bb7462b6fd64 ("vfs: use helpers for calling f_op->{read,write}_iter()")
>    c227f912732f ("f2fs: record dirty status of regular/symlink inode")
>    d323d005ac4a ("f2fs: support file defragment")
>    dde0c2e79848 ("fs: add IOCB_SYNC and IOCB_DSYNC")
>    e6f15b763ab2 ("drm/vgem: Enable dmabuf interface for export")
>   eb7e813cc791 ("f2fs: fix to remove directory inode from dirty list")
>    f74ac01520c9 ("mm: use helper for calling f_op->mmap()")
>
>v3.18.140: Failed to apply! Possible dependencies:
>    1bd816f12071 ("drm/vgem: Use lockless gem BO free callback")
>2dbf0d90971a ("drm/i915: Use CPU mapping for userspace dma-buf mmap()")
>    301120134628 ("block: loop: say goodby to bio")
>    502e95c66785 ("drm/vgem: implement virtual GEM")
>    990ed2720717 ("drm/vgem: drop DRIVER_PRIME (v2)")
>    af65aa8ea78b ("block: loop: don't handle REQ_FUA explicitly")
>    b5dd2f6047ca ("block: loop: improve performance via blk-mq")
>bb7462b6fd64 ("vfs: use helpers for calling f_op->{read,write}_iter()")
>    bc07c10a3603 ("block: loop: support DIO & AIO")
>cf655d953422 ("block: loop: introduce lo_discard() and lo_req_flush()")
>    e6f15b763ab2 ("drm/vgem: Enable dmabuf interface for export")
>    f74ac01520c9 ("mm: use helper for calling f_op->mmap()")
>
>
>How should we proceed with this patch?
>
>--
>Thanks,
>Sasha

  parent reply index

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-17 18:36 [PATCH 00/22] Coda updates Jan Harkes
2019-05-17 18:36 ` [PATCH 01/22] coda: pass the host file in vma->vm_file on mmap Jan Harkes
     [not found]   ` <20190518122241.D867120B7C@mail.kernel.org>
2019-05-18 13:18     ` Jan Harkes [this message]
2019-05-17 18:36 ` [PATCH 02/22] uapi linux/coda.h: use __kernel_pid_t for userspace Jan Harkes
2019-05-17 18:36 ` [PATCH 03/22] uapi linux/coda_psdev.h: move upc_req definition from uapi to kernel side headers Jan Harkes
2019-05-17 18:36 ` [PATCH 04/22] coda: add error handling for fget Jan Harkes
2019-05-17 18:36 ` [PATCH 05/22] coda: potential buffer overflow in coda_psdev_write() Jan Harkes
2019-05-17 18:36 ` [PATCH 06/22] coda: Fix build using bare-metal toolchain Jan Harkes
2019-05-17 18:36 ` [PATCH 07/22] coda: don't try to print names that were considered too long Jan Harkes
2019-05-17 18:36 ` [PATCH 08/22] uapi linux/coda_psdev.h: Move CODA_REQ_ from uapi to kernel side headers Jan Harkes
2019-05-17 18:36 ` [PATCH 09/22] coda: clean up indentation, replace spaces with tab Jan Harkes
2019-05-17 18:36 ` [PATCH 10/22] coda: stop using 'struct timespec' in user API Jan Harkes
2019-05-17 18:36 ` [PATCH 11/22] coda: change Coda's user api to use 64-bit time_t in timespec Jan Harkes
2019-05-17 18:36 ` [PATCH 12/22] coda: get rid of CODA_ALLOC() Jan Harkes
2019-05-17 18:36 ` [PATCH 13/22] coda: get rid of CODA_FREE() Jan Harkes
2019-05-17 18:36 ` [PATCH 14/22] coda: bump module version Jan Harkes
2019-05-17 18:36 ` [PATCH 15/22] coda: Move internal defs out of include/linux/ [ver #2] Jan Harkes
2019-05-17 18:36 ` [PATCH 16/22] coda: remove uapi/linux/coda_psdev.h Jan Harkes
2019-05-17 23:29   ` Andrew Morton
2019-05-17 23:49     ` Andrew Morton
2019-05-17 23:51     ` Jan Harkes
2019-05-17 18:36 ` [PATCH 17/22] coda: destroy mutex in put_super() Jan Harkes
2019-05-17 18:36 ` [PATCH 18/22] coda: use SIZE() for stat Jan Harkes
2019-05-17 18:36 ` [PATCH 19/22] coda: add __init to init_coda_psdev() Jan Harkes
2019-05-17 18:36 ` [PATCH 20/22] coda: remove sysctl object from module when unused Jan Harkes
2019-05-17 18:36 ` [PATCH 21/22] coda: remove sb test in coda_fid_to_inode() Jan Harkes
2019-05-17 18:37 ` [PATCH 22/22] coda: ftoc validity check integration Jan Harkes

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=A67240F2-2712-460E-A17F-EB8D24AD76DE@cs.cmu.edu \
    --to=jaharkes@cs.cmu.edu \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=sashal@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-Fsdevel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-fsdevel/0 linux-fsdevel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-fsdevel linux-fsdevel/ https://lore.kernel.org/linux-fsdevel \
		linux-fsdevel@vger.kernel.org linux-fsdevel@archiver.kernel.org
	public-inbox-index linux-fsdevel


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-fsdevel


AGPL code for this site: git clone https://public-inbox.org/ public-inbox