Re: [PATCH v2 1/2] fuse: Fix possible deadlock when writing back dirty pages

2021-04-15 Thread Peng Tao
On Wed, Apr 14, 2021 at 9:20 PM Miklos Szeredi wrote: > > On Wed, Apr 14, 2021 at 2:22 PM Peng Tao wrote: > > > > > > --- a/fs/fuse/file.c > > > +++ b/fs/fuse/file.c > > > @@ -1117,17 +1117,12 @@ static ssize_t fuse_send_write_pages(str > > >

Re: [PATCH v2 1/2] fuse: Fix possible deadlock when writing back dirty pages

2021-04-14 Thread Peng Tao
+0x8c/0xc8 [fuse] > > > [1450578.907128] fuse_perform_write+0x240/0x4e0 [fuse] > > > [1450578.912082] fuse_file_write_iter+0x1dc/0x290 [fuse] > > > [1450578.917207] do_iter_readv_writev+0x110/0x188 > > > [1450578.921724] do_iter_write+0x90/0x1c8 > > > [1450578.9255

Re: [PATCH RESEND V12 3/8] fuse: Definitions and ioctl for passthrough

2021-02-18 Thread Peng Tao
On Wed, Feb 17, 2021 at 9:41 PM Miklos Szeredi wrote: > > On Mon, Jan 25, 2021 at 4:31 PM Alessio Balsini wrote: > > > > Expose the FUSE_PASSTHROUGH interface to user space and declare all the > > basic data structures and functions as the skeleton on top of which the > > FUSE passthrough functio

Re: [PATCH RESEND V12 2/8] fuse: 32-bit user space ioctl compat for fuse device

2021-02-05 Thread Peng Tao
On Thu, Jan 28, 2021 at 10:15 PM Alessio Balsini wrote: > > Hi all, > > I'm more than happy to change the interface into something that is > objectively better and accepted by everyone. > I would really love to reach the point at which we have a "stable-ish" > UAPI as soon as possible. > > I've be

Re: [PATCH RESEND V12 7/8] fuse: Use daemon creds in passthrough mode

2021-02-05 Thread Peng Tao
On Mon, Jan 25, 2021 at 11:31 PM Alessio Balsini wrote: > > When using FUSE passthrough, read/write operations are directly > forwarded to the lower file system file through VFS, but there is no > guarantee that the process that is triggering the request has the right > permissions to access the l

Re: [PATCH V10 0/5] fuse: Add support for passthrough read/write

2020-11-27 Thread Peng Tao
On Tue, Oct 27, 2020 at 1:00 AM Alessio Balsini wrote: > > This is the 10th version of the series. Please find the changelog at the > bottom of this cover letter. > > Add support for file system passthrough read/write of files when enabled in > userspace through the option FUSE_PASSTHROUGH. > > Th

Re: [PATCH V10 2/5] fuse: Passthrough initialization and release

2020-11-27 Thread Peng Tao
On Fri, Nov 27, 2020 at 9:41 PM Alessio Balsini wrote: > > Hi Peng, > > Thanks for the heads up! > > On Thu, Nov 26, 2020 at 09:33:34PM +0800, Peng Tao wrote: > > On Tue, Oct 27, 2020 at 12:19 AM Alessio Balsini > > wrote: > > > [...] > > >

Re: [PATCH V10 2/5] fuse: Passthrough initialization and release

2020-11-26 Thread Peng Tao
On Tue, Oct 27, 2020 at 12:19 AM Alessio Balsini wrote: > > Implement the FUSE passthrough ioctl() that associates the lower > (passthrough) file system file with the fuse_file. > > The file descriptor passed to the ioctl() by the FUSE daemon is used to > access the relative file pointer, that wil

[PATCH RFC] scsi: wait sd probing in manual scan

2017-04-28 Thread Peng Tao
;James E.J. Bottomley" CC: "Martin K. Petersen" Signed-off-by: Peng Tao --- drivers/scsi/scsi_scan.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index 6f7128f..0208f40 100644 --- a/drivers/scsi/scsi_

Re: [PATCH] staging/lustre: use rcu_dereference to access rcu protected current->real_parent field

2014-08-09 Thread Peng Tao
On Fri, Aug 8, 2014 at 1:32 PM, Greg Kroah-Hartman wrote: > On Fri, Aug 08, 2014 at 01:06:15AM -0400, Oleg Drokin wrote: >> >> On Aug 8, 2014, at 12:42 AM, Greg Kroah-Hartman wrote: >> >> > On Fri, Aug 08, 2014 at 12:03:20AM -0400, Oleg Drokin wrote: >> >> Hello! >> >> >> >> On Aug 7, 2014, at 11:

Re: [PATCH RFC] sched: introduce add_wait_queue_exclusive_head

2014-03-18 Thread Peng Tao
On Wed, Mar 19, 2014 at 12:23 AM, Oleg Nesterov wrote: > On 03/18, Peng Tao wrote: >> >> On Tue, Mar 18, 2014 at 10:05 PM, Peter Zijlstra >> wrote: >> > >> > Unless you cannot use ___wait() and really need to open-code the >> > wait_event() stuf

Re: [PATCH RFC] sched: introduce add_wait_queue_exclusive_head

2014-03-18 Thread Peng Tao
On Tue, Mar 18, 2014 at 11:47 PM, Oleg Nesterov wrote: > On 03/18, Peter Zijlstra wrote: >> >> I think we can avoid the entire function if we add >> WQ_FLAG_LIFO and make prepare_to_wait_event() DTRT. > > Agreed, this looks very natural. > > prepare_to_wait_event(WQ_FLAG_LIFO) should probably skip

Re: [PATCH RFC] sched: introduce add_wait_queue_exclusive_head

2014-03-18 Thread Peng Tao
On Tue, Mar 18, 2014 at 10:05 PM, Peter Zijlstra wrote: > On Tue, Mar 18, 2014 at 09:51:04PM +0800, Peng Tao wrote: >> > Firstly I think the _head postfix for LIFO is a bad name, >> Do you have any preference on the name? add_wait_queue_exclusive_lifo()? > > I think

Re: [PATCH RFC] sched: introduce add_wait_queue_exclusive_head

2014-03-18 Thread Peng Tao
On Tue, Mar 18, 2014 at 9:33 PM, Peter Zijlstra wrote: > On Tue, Mar 18, 2014 at 09:10:08PM +0800, Peng Tao wrote: >> Normally wait_queue_t is a FIFO list for exclusive waiting tasks. >> As a side effect, if there are many threads waiting on the same >> condition (which is co

[PATCH RFC] sched: introduce add_wait_queue_exclusive_head

2014-03-18 Thread Peng Tao
. Of course there is an alternative approach to just open code it but we'd like to ask first to see if there is objection to making it generic. Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/include/linux/l

[PATCH 4/5] staging/lustre/libcfs: remove schedule_timeout_and_set_state

2014-03-18 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_prim.h |1 - drivers/staging/lustre/lnet/lnet/router.c |4 ++-- drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c|4 ++-- drivers/staging/lustre/lustre/libcfs

[PATCH 3/5] staging/lustre/libcfs: remove waitq_timedwait

2014-03-18 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_prim.h |1 - .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c |2 +- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c |5 ++--- drivers/staging/lustre/lnet/lnet/lib-eq.c

[PATCH 2/5] staging/lustre/libcfs: remove waitq_wait

2014-03-18 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_prim.h |1 - .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c |2 +- drivers/staging/lustre/lnet/lnet/lib-eq.c |2 +- drivers/staging/lustre/lustre/fid

[PATCH 0/5] staging/lustre/libcfs: clean up wait queue related wrappers

2014-03-18 Thread Peng Tao
Hi Greg, Following patches remove wait queue related wrappers from lustre. The only thing left is add_wait_queue_exclusive_head() that we hope to make into generic helpers in a separate patch. Thanks, Tao Cc: Andreas Dilger Cc: Oleg Drokin Peng Tao (5): staging/lustre/libcfs

[PATCH 5/5] staging/lustre/libcfs: remove cfs_pause

2014-03-18 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_prim.h |5 - .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c|6 -- .../staging/lustre/lnet/klnds/socklnd/socklnd.c|6 -- .../staging/lustre/lnet/klnds

[PATCH 1/5] staging/lustre/libcfs: remove init_waitqueue_entry_current

2014-03-18 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_prim.h |1 - .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c |6 +++--- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c |4 ++-- drivers/staging/lustre/lnet/lnet/lib

[PATCH 6/6] staging/lustre/libcfs: remove NIDSTR_LOCK/UNLOCK

2014-03-05 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- drivers/staging/lustre/lustre/libcfs/nidstrings.c |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/staging/lustre/lustre/libcfs/nidstrings.c b/drivers/staging/lustre/lustre/libcfs/nidstrings.c

[PATCH 5/6] staging/lustre/libcfs: remove obsolete function declaration

2014-03-05 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_string.h| 12 1 files changed, 0 insertions(+), 12 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_string.h b/drivers/staging/lustre/include

[PATCH 4/6] staging/lustre/libcfs: remove cfs_iswhite

2014-03-05 Thread Peng Tao
Kernel provides isspace(). Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_string.h| 15 --- drivers/staging/lustre/lnet/lnet/config.c | 12 ++-- .../staging/lustre/lustre/libcfs/libcfs_string.c

[PATCH 2/6] staging/lustre/libcfs: remove cfs_strncasecmp

2014-03-05 Thread Peng Tao
The only difference against generic strncasecmp() is that cfs_strncasecmp() check NULL pointers and zero length parameter, which all callers have already taken care of. Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- drivers/staging/lustre/lnet/lnet/router_proc.c |4

[PATCH 3/6] staging/lustre/libcfs_string.h: don't export internal functions

2014-03-05 Thread Peng Tao
cfs_expr_list_print is removed due to no callers. Other internal functions are held unexported. Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_string.h|3 --- .../staging/lustre/lustre/libcfs/libcfs_string.c | 20

[PATCH 1/6] staging/lustre/lnet: remove kiblnd_debug_conn/rx/tx

2014-03-05 Thread Peng Tao
/klnds/o2iblnd/o2iblnd.c:597: warning: ‘kiblnd_debug_tx’ defined but not used Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c| 68 1 files changed, 0 insertions(+), 68 deletions(-) diff --git a/drivers

[PATCH 3/3] staging/lustre: fix coccinelle warnings

2014-02-13 Thread Peng Tao
From: Fengguang Wu drivers/staging/lustre/lustre/llite/dir.c:1416:2-8: Replace memcpy with struct assignment Generated by: coccinelle/misc/memcpy-assign.cocci CC: Peng Tao CC: Andreas Dilger CC: Oleg Drokin CC: Greg Kroah-Hartman Signed-off-by: Fengguang Wu --- drivers/staging/lustre

[PATCH 2/3] staging/lustre/libcfs: remove cfs_hash_long

2014-02-13 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/include/linux/libcfs/libcfs_hash.h |2 -- .../staging/lustre/include/linux/lnet/lib-lnet.h |2 +- drivers/staging/lustre/lnet/lnet/api-ni.c |2 +- drivers/staging/lustre/lnet/lnet/lib-ptl.c

[PATCH 1/3] staging/lustre/libcfs: remove user space code from kernel_user_comm.c

2014-02-13 Thread Peng Tao
Cc: Andreas Dilger Cc: Oleg Drokin Signed-off-by: Peng Tao --- .../lustre/lustre/libcfs/kernel_user_comm.c| 102 1 files changed, 0 insertions(+), 102 deletions(-) diff --git a/drivers/staging/lustre/lustre/libcfs/kernel_user_comm.c b/drivers/staging/lustre

Re: [PATCH 6/6] staging/lustre/libcfs: remove cfs_capable

2014-02-11 Thread Peng Tao
On Sat, Jan 25, 2014 at 2:50 AM, Dilger, Andreas wrote: > On 2014/01/22, 6:47 AM, "Peng Tao" wrote: >>diff --git a/drivers/staging/lustre/lustre/libcfs/linux/linux-curproc.c >>b/drivers/staging/lustre/lustre/libcfs/linux/linux-curproc.c >>index 8b3af7f..6d0bd22

Re: [PATCH v2 RESEND 6/8] staging/lustre/obdclass: remove extra break in class_process_config

2014-02-07 Thread Peng Tao
On Sat, Feb 8, 2014 at 1:43 AM, Greg Kroah-Hartman wrote: > On Wed, Jan 22, 2014 at 09:36:17PM +0800, Peng Tao wrote: >> From: Artem Blagodarenko >> >> This is only part of the original Lustre commit, splitted to do the >> cleanup work. >> >> Intel-bug-id

[PATCH 6/6] staging/lustre/libcfs: remove cfs_capable

2014-01-22 Thread Peng Tao
Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../staging/lustre/include/linux/libcfs/curproc.h |1 - .../lustre/lustre/libcfs/linux/linux-curproc.c |6 -- .../lustre/lustre/libcfs/linux/linux-module.c |4 ++-- drivers/staging/lustre/lustre/llite/dir.c |8

[PATCH 4/6] staging/lustre/libcfs: remove cfs_cap_{un}pack

2014-01-22 Thread Peng Tao
Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../lustre/lustre/libcfs/linux/linux-curproc.c | 24 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/drivers/staging/lustre/lustre/libcfs/linux/linux-curproc.c b/drivers/staging/lustre/lustre/libcfs/linux

[PATCH 5/6] staging/lustre/libcfs: remove CAPABILITY_VERSION tests

2014-01-22 Thread Peng Tao
_LINUX_CAPABILITY_VERSION is only for backward compatibility in user space. Kernel code doesn't care about it. Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../lustre/lustre/libcfs/linux/linux-curproc.c | 16 1 file changed, 16 deletions(-) diff --git a/dr

[PATCH 3/6] staging/lustre/libcfs: remove cfs_curproc_cap_unpack

2014-01-22 Thread Peng Tao
no user. Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../staging/lustre/include/linux/libcfs/curproc.h |1 - .../lustre/lustre/libcfs/linux/linux-curproc.c | 10 -- 2 files changed, 11 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/curproc.h b

[PATCH 2/6] staging/lustre/libcfs: remove cfs_curproc_groups_nr

2014-01-22 Thread Peng Tao
no user. Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../staging/lustre/include/linux/libcfs/curproc.h |7 --- .../lustre/lustre/libcfs/linux/linux-curproc.c | 11 --- 2 files changed, 18 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/curproc.h

[PATCH 1/6] staging/lustre/o2iblnd: fix is_vmalloc_addr build warning

2014-01-22 Thread Peng Tao
] Cc: Laura Abbott Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c b/drivers/staging/lustre/lnet/klnds/o2iblnd

Re: [PATCH 1/6] staging/lustre/o2iblnd: fix is_vmalloc_addr build warning

2014-01-22 Thread Peng Tao
Hi Greg, I got gmail failure after git send-email sent the first two patches. Please drop the two and I'll resend the whole series. Thanks, Tao On Wed, Jan 22, 2014 at 9:40 PM, Peng Tao wrote: > I got this building Lustre: > drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c:

[PATCH 2/6] staging/lustre/libcfs: remove cfs_curproc_groups_nr

2014-01-22 Thread Peng Tao
no user. Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../staging/lustre/include/linux/libcfs/curproc.h |7 --- .../lustre/lustre/libcfs/linux/linux-curproc.c | 11 --- 2 files changed, 18 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/curproc.h

[PATCH 1/6] staging/lustre/o2iblnd: fix is_vmalloc_addr build warning

2014-01-22 Thread Peng Tao
] Cc: Laura Abbott Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c b/drivers/staging/lustre/lnet/klnds/o2iblnd

[PATCH v2 RESEND 7/8] staging/lustre/lustre_user.h: remove obsolete comments

2014-01-22 Thread Peng Tao
: JC Lafoucriere Signed-off-by: Henri Doreau Reviewed-by: John L. Hammond Reviewed-by: Andreas Dilger Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/lustre/lustre_user.h |6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/lustre

[PATCH v2 RESEND 4/8] staging/lustre/idl: remove LASSERT/CLASSERT from lustre_idl.h

2014-01-22 Thread Peng Tao
ug-id: https://jira.hpdd.intel.com/browse/LU-1606 Signed-off-by: Andreas Dilger Signed-off-by: John L. Hammond Reviewed-by: Andreas Dilger Reviewed-by: Christopher J. Morrone Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/lustre/lustre_idl.h |

[PATCH v2 RESEND 8/8] staging/lustre/ptlrpc: flock deadlock detection does not work

2014-01-22 Thread Peng Tao
https://jira.hpdd.intel.com/browse/LU-1715 Lustre-change: http://review.whamcloud.com/3553 Signed-off-by: Andriy Skulysh Reviewed-by: Vitaly Fertman Reviewed-by: Bruce Korb Reviewed-by: Keith Mannthey Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/l

[PATCH v2 RESEND 6/8] staging/lustre/obdclass: remove extra break in class_process_config

2014-01-22 Thread Peng Tao
Liu Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../staging/lustre/lustre/obdclass/obd_config.c|4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/obdclass/obd_config.c b/drivers/staging/lustre/lustre/obdclass/obd_config.c index 27f56c0

[PATCH v2 RESEND 5/8] staging/lustre/mgs: set_param -P option that sets value permanently

2014-01-22 Thread Peng Tao
eviewed-by: Andreas Dilger Reviewed-by: Emoly Liu Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/include/lustre_cfg.h |2 + .../staging/lustre/lustre/include/lustre_disk.h|2 + drivers/staging/lustre/lustre/include/obd_class.h |9 +- drive

[PATCH v2 RESEND 3/8] staging/lustre/llite: pass correct pointer to obd_iocontrol()

2014-01-22 Thread Peng Tao
leg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/dir.c | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c b/drivers/staging/lustre/lustre/llite/d

[PATCH v2 RESEND 1/8] staging/lustre/llite: remove ll_d_root_ops

2014-01-22 Thread Peng Tao
rowse/LU-3486 Signed-off-by: Lai Siyao Reviewed-by: James Simmons Reviewed-by: Peng Tao Reviewed-by: Bobi Jam Reviewed-by: Fan Yong Reviewed-by: Alexey Shvetsov Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/dcache.c

[PATCH v2 RESEND 2/8] staging/lustre/llite: don't d_add for create only files

2014-01-22 Thread Peng Tao
-off-by: Lai Siyao Reviewed-by: James Simmons Reviewed-by: Peng Tao Reviewed-by: Bobi Jam Reviewed-by: Fan Yong Reviewed-by: Alexey Shvetsov Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/namei.c |5 + 1 file

[PATCH v2 RESEND 0/8] staging/lustre: sync with external tree

2014-01-22 Thread Peng Tao
Hi Greg, Here are the v2 of previous series. Each patch is checkpatch'ed and build-tested independently. Please see if it is OK to merge them. Thanks, Tao Cc: Andreas Dilger Andreas Dilger (1): staging/lustre/idl: remove LASSERT/CLASSERT from lustre_idl.h Andriy Skulysh (1): staging/lustr

Re: [PATCH 2/2] NFSv4.1: Fix a race in nfs4_write_inode

2014-01-17 Thread Peng Tao
On Fri, Jan 17, 2014 at 1:11 AM, Trond Myklebust wrote: > > On Jan 16, 2014, at 10:49, Peng Tao wrote: >> On Tue, Jan 14, 2014 at 2:45 AM, Trond Myklebust >> wrote: >>> void pnfs_set_lo_fail(struct pnfs_layout_segment *lseg) >>> @@ -1881,43 +1887,37 @@

Re: [PATCH 2/2] NFSv4.1: Fix a race in nfs4_write_inode

2014-01-16 Thread Peng Tao
On Tue, Jan 14, 2014 at 2:45 AM, Trond Myklebust wrote: > nfs4_write_inode() must not be allowed to exit until the layoutcommit > is done. That means that both NFS_INO_LAYOUTCOMMIT and > NFS_INO_LAYOUTCOMMITTING have to be cleared. > > Signed-off-by: Trond Myklebust > --- > fs/nfs/nfs4super.c |

[PATCH v2 1/8] staging/lustre/llite: remove ll_d_root_ops

2014-01-03 Thread Peng Tao
rowse/LU-3486 Signed-off-by: Lai Siyao Reviewed-by: James Simmons Reviewed-by: Peng Tao Reviewed-by: Bobi Jam Reviewed-by: Fan Yong Reviewed-by: Alexey Shvetsov Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/dcache.c

[PATCH v2 2/8] staging/lustre/llite: don't d_add for create only files

2014-01-03 Thread Peng Tao
-off-by: Lai Siyao Reviewed-by: James Simmons Reviewed-by: Peng Tao Reviewed-by: Bobi Jam Reviewed-by: Fan Yong Reviewed-by: Alexey Shvetsov Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/namei.c |5 + 1 files

[PATCH v2 6/8] staging/lustre/obdclass: remove extra break in class_process_config

2014-01-03 Thread Peng Tao
Liu Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../staging/lustre/lustre/obdclass/obd_config.c|4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/obdclass/obd_config.c b/drivers/staging/lustre/lustre/obdclass/obd_config.c

[PATCH v2 5/8] staging/lustre/mgs: set_param -P option that sets value permanently

2014-01-03 Thread Peng Tao
eviewed-by: Andreas Dilger Reviewed-by: Emoly Liu Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/include/lustre_cfg.h |2 + .../staging/lustre/lustre/include/lustre_disk.h|2 + drivers/staging/lustre/lustre/include/obd_class.h |9 ++- drive

[PATCH v2 3/8] staging/lustre/llite: pass correct pointer to obd_iocontrol()

2014-01-03 Thread Peng Tao
leg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/dir.c | 23 ++- 1 files changed, 14 insertions(+), 9 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c b/drivers/staging/lustre/lustre/llite/d

[PATCH v2 7/8] staging/lustre/lustre_user.h: remove obsolete comments

2014-01-03 Thread Peng Tao
: JC Lafoucriere Signed-off-by: Henri Doreau Reviewed-by: John L. Hammond Reviewed-by: Andreas Dilger Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/lustre/lustre_user.h |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/drivers

[PATCH v2 8/8] staging/lustre/ptlrpc: flock deadlock detection does not work

2014-01-03 Thread Peng Tao
https://jira.hpdd.intel.com/browse/LU-1715 Lustre-change: http://review.whamcloud.com/3553 Signed-off-by: Andriy Skulysh Reviewed-by: Vitaly Fertman Reviewed-by: Bruce Korb Reviewed-by: Keith Mannthey Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/l

[PATCH v2 0/8] staging/lustre: sync with external tree

2014-01-03 Thread Peng Tao
Hi Greg, Happy New Year! I hope it is long enough to start sending you new patches. Here are the updated version of previous series. Each patch is checkpatch'ed and build-tested independently. Please see if it is OK to merge them. Thanks, Tao Cc: Andreas Dilger Andreas Dilger (1): staging/l

[PATCH v2 4/8] staging/lustre/idl: remove LASSERT/CLASSERT from lustre_idl.h

2014-01-03 Thread Peng Tao
ug-id: https://jira.hpdd.intel.com/browse/LU-1606 Signed-off-by: Andreas Dilger Signed-off-by: John L. Hammond Reviewed-by: Andreas Dilger Reviewed-by: Christopher J. Morrone Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/lustre/lustre_idl.h |

RE: randconfig build error with 3.13-rc6, in drivers/staging/lustre/lustre/lmv/lproc_lmv.c

2013-12-30 Thread Peng, Tao
>-Original Message- >From: Jim Davis [mailto:jim.ep...@gmail.com] >Sent: Tuesday, December 31, 2013 1:02 AM >To: linux-kernel@vger.kernel.org; Greg Kroah-Hartman; >andreas.dil...@intel.com; Peng, Tao; sachin.ka...@linaro.org; >de...@driverdev.osuosl.org >Subject: r

Re: [PATCH 06/13] staging/lustre/llite: remove ll_d_root_ops

2013-12-10 Thread Peng Tao
On Wed, Dec 11, 2013 at 10:32 AM, Greg Kroah-Hartman wrote: > On Mon, Dec 09, 2013 at 10:56:58PM +0800, Peng Tao wrote: >> From: Lai Siyao >> >> Mnt root dentry will never be revalidated, but its d_op->d_compare >> will be called for its children, to simplify code,

[PATCH 03/13] staging/lustre/api: HSM import uses new released pattern

2013-12-09 Thread Peng Tao
Reviewed-by: Andreas Dilger Reviewed-by: Jinshan Xiong [Fix up kuid_t/guid_t conversion in llite/file.c -- Peng Tao] Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/lustre/lustre_user.h | 17 ++- drivers/staging/lustre/lustre/llite/file.c | 118

[PATCH 01/13] staging/lustre: remove server macros from lustre_net.h

2013-12-09 Thread Peng Tao
These macros are only used on server side. Cc: Andreas Dilger Signed-off-by: Peng Tao --- drivers/staging/lustre/lustre/include/lustre_net.h | 234 1 file changed, 234 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre_net.h b/drivers/staging/lustre

[PATCH 10/13] staging/lustre/mgs: set_param -P option that sets value permanently

2013-12-09 Thread Peng Tao
eviewed-by: Andreas Dilger Reviewed-by: Emoly Liu Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/include/lustre_cfg.h |2 + .../staging/lustre/lustre/include/lustre_disk.h|2 + drivers/staging/lustre/lustre/include/obd_class.h |9 +- drive

[PATCH 06/13] staging/lustre/llite: remove ll_d_root_ops

2013-12-09 Thread Peng Tao
rowse/LU-3486 Signed-off-by: Lai Siyao Reviewed-by: James Simmons Reviewed-by: Peng Tao Reviewed-by: Bobi Jam Reviewed-by: Fan Yong Reviewed-by: Alexey Shvetsov Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/dcache.c

[PATCH 08/13] staging/lustre/llite: pass correct pointer to obd_iocontrol()

2013-12-09 Thread Peng Tao
leg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/dir.c | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c b/drivers/staging/lustre/lustre/llite/d

[PATCH 12/13] staging/lustre/lustre_user.h: remove obsolete comments

2013-12-09 Thread Peng Tao
: JC Lafoucriere Signed-off-by: Henri Doreau Reviewed-by: John L. Hammond Reviewed-by: Andreas Dilger Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/lustre/lustre_user.h |6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/lustre

[PATCH 04/13] staging/lustre/target: move OUT to the unified target code

2013-12-09 Thread Peng Tao
: Andreas Dilger [pick client side of change. target is mostly server side code -- PengTao] Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/mdc/mdc_request.c | 32 +-- 1 file changed, 1 insertion(+), 31 deletions(-) diff --git a/drivers

[PATCH 13/13] staging/lustre/ptlrpc: flock deadlock detection does not work

2013-12-09 Thread Peng Tao
https://jira.hpdd.intel.com/browse/LU-1715 Lustre-change: http://review.whamcloud.com/3553 Signed-off-by: Andriy Skulysh Reviewed-by: Vitaly Fertman Reviewed-by: Bruce Korb Reviewed-by: Keith Mannthey Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/l

[PATCH 05/13] staging/lustre/seq: remove seq_{query,handle}

2013-12-09 Thread Peng Tao
Zhuravlev Reviewed-by: wangdi Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/include/lustre_fid.h |6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre_fid.h b/drivers/staging

[PATCH 09/13] staging/lustre/idl: remove LASSERT/CLASSERT from lustre_idl.h

2013-12-09 Thread Peng Tao
ug-id: https://jira.hpdd.intel.com/browse/LU-1606 Signed-off-by: Andreas Dilger Signed-off-by: John L. Hammond Reviewed-by: Andreas Dilger Reviewed-by: Christopher J. Morrone Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/lustre/lustre_idl.h |

[PATCH 02/13] staging/lustre/lnet: Add LNet Router Priority parameter

2013-12-09 Thread Peng Tao
com/5663 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2934 Signed-off-by: Doug Oucharek Reviewed-by: Andreas Dilger Reviewed-by: Amir Shehata Reviewed-by: Isaac Huang Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/include/linux/lib

[PATCH 07/13] staging/lustre/llite: don't d_add for create only files

2013-12-09 Thread Peng Tao
-off-by: Lai Siyao Reviewed-by: James Simmons Reviewed-by: Peng Tao Reviewed-by: Bobi Jam Reviewed-by: Fan Yong Reviewed-by: Alexey Shvetsov Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/namei.c |6 +- 1 file

[PATCH 11/13] staging/lustre/obdclass: remove extra break in class_process_config

2013-12-09 Thread Peng Tao
Liu Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../staging/lustre/lustre/obdclass/obd_config.c|4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/obdclass/obd_config.c b/drivers/staging/lustre/lustre/obdclass/obd_config.c index 27f56c0

[PATCH 00/13] staging/lustre: sync with external tree, set 3

2013-12-09 Thread Peng Tao
only files Mikhail Pershin (2): staging/lustre/target: move OUT to the unified target code staging/lustre/seq: remove seq_{query,handle} Peng Tao (1): staging/lustre: remove server macros from lustre_net.h .../lustre/include/linux/libcfs/libcfs_ioctl.h |1 + .../staging/lustre/in

[PATCH v2 3/4] staging/lustre: fix undefined reference if CONFIG_PROC_FS is off

2013-12-03 Thread Peng Tao
licit declaration of function ‘obd_update_maxusage’ The patch defines following symbols when CONFIG_PROC_FS is off: fld_type_proc_dir, obd_update_maxusage(), obd_max_pages, obd_max_alloc, obd_alloc, obd_pages, lprocfs_llog_test_init_vars(). Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- dr

[PATCH v2 1/4] staging/lustre: fix build error in ll_setxattr_common

2013-12-03 Thread Peng Tao
lared identifier is reported only once for each function it appears in It was introduced by 7fc1f83 (staging/lustre/llite: extended attribute cache) that added new reference but didn't take care of CONFIG_FS_POSIX_ACL. Reported-by: Fengguang Wu Cc: Andreas Dilger Signed-off-by: Peng Tao --

[PATCH v2 4/4] staging/lustre: don't compile procfs code when CONFIG_PROC_FS is off

2013-12-03 Thread Peng Tao
needs to be fixed in a separate patch because it is not related to CONFIG_PROC_FS breakage here. Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/fid/Makefile |3 +- drivers/staging/lustre/lustre/fid/lproc_fid.c |2 - drivers/staging/lustre

[PATCH v2 2/4] staging/lustre/llite: remove ll_file_get_iov_count

2013-12-03 Thread Peng Tao
It exactly copies generic_segment_checks() except wrongly calling access_ok(VERIFY_READ) for aio read range. Drop it and just call generic_segment_checks() instead. Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/file.c | 36

[PATCH v2 0/4] staging/lustre: fix build and remove ll_file_get_iov_count

2013-12-03 Thread Peng Tao
Hi Greg, Patch 1 is added because there is a newly introduced (7fc1f83) error when CONFIG_FS_POSIX_ACL is off. Patch 2 is updated to fix build errors in itself. Please see if they can be merged. Thanks and sorry for the noise. Thanks, Tao Cc: Andreas Dilger Peng Tao (4): staging/lustre

Re: [PATCH 7/9] staging/lustre/llite: remove ll_file_get_iov_count

2013-12-03 Thread Peng Tao
On 12/04/2013 01:00 AM, Greg Kroah-Hartman wrote: On Tue, Dec 03, 2013 at 10:42:06PM +0800, Peng Tao wrote: It exactly copies generic_segment_checks() except wrongly calling access_ok(VERIFY_READ) for aio read range. Drop it and just call generic_segment_checks() instead. Signed-off-by: Peng

[PATCH 2/9] staging/lustre/obdclass: lprocfs_{alloc_stats,register} always defined

2013-12-03 Thread Peng Tao
No need to surround them with #ifdef LPROCFS. Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/obdclass/class_obd.c |4 ++-- drivers/staging/lustre/lustre/obdclass/genops.c|3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a

[PATCH 3/9] staging/lustre/ldlm: inline ldlm_proc_setup

2013-12-03 Thread Peng Tao
Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/include/lustre_dlm.h |3 ++- drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c|2 -- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lustre/include

[PATCH 5/9] staging/lustre: fix defined not used warning

2013-12-03 Thread Peng Tao
68:20: warning: ‘lnet_table’ defined but not used [-Wunused-variable] Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lnet/lnet/router_proc.c |4 .../lustre/lustre/libcfs/linux/linux-proc.c|8 2 files changed, 12 deletions(-)

[PATCH 8/9] staging/lustre: fix undefined reference if CONFIG_PROC_FS is off

2013-12-03 Thread Peng Tao
licit declaration of function ‘obd_update_maxusage’ The patch defines following symbols when CONFIG_PROC_FS is off: fld_type_proc_dir, obd_update_maxusage(), obd_max_pages, obd_max_alloc, obd_alloc, obd_pages, lprocfs_llog_test_init_vars(). Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- dr

[PATCH 8/9] staging/lustre: fix undefined reference if CONFIG_PROC_FS is off

2013-12-03 Thread Peng Tao
licit declaration of function ‘obd_update_maxusage’ The patch defines following symbols when CONFIG_PROC_FS is off: fld_type_proc_dir, obd_update_maxusage(), obd_max_pages, obd_max_alloc, obd_alloc, obd_pages, lprocfs_llog_test_init_vars(). Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- dr

[PATCH 4/9] staging/lustre/obdclass: move obd_sysctl_init out of class_procfs_init

2013-12-03 Thread Peng Tao
It is not strictly related to procfs. Besides, we already call obd_sysctl_clean outside of class_procfs_clean. Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/obdclass/class_obd.c |3 +++ .../lustre/lustre/obdclass/linux/linux-module.c|1 - 2

[PATCH 4/9] staging/lustre/obdclass: move obd_sysctl_init out of class_procfs_init

2013-12-03 Thread Peng Tao
It is not strictly related to procfs. Besides, we already call obd_sysctl_clean outside of class_procfs_clean. Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/obdclass/class_obd.c |3 +++ .../lustre/lustre/obdclass/linux/linux-module.c|1 - 2

[PATCH 9/9] staging/lustre: don't compile procfs code when CONFIG_PROC_FS is off

2013-12-03 Thread Peng Tao
needs to be fixed in a separate patch because it is not related to CONFIG_PROC_FS breakage here. Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/fid/Makefile |3 +- drivers/staging/lustre/lustre/fid/lproc_fid.c |2 - drivers/staging/lustre

[PATCH 7/9] staging/lustre/llite: remove ll_file_get_iov_count

2013-12-03 Thread Peng Tao
It exactly copies generic_segment_checks() except wrongly calling access_ok(VERIFY_READ) for aio read range. Drop it and just call generic_segment_checks() instead. Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/llite/file.c | 36

[PATCH 6/9] staging/lustre/llite: fix used uninitialized warning

2013-12-03 Thread Peng Tao
tre/llite/file.c: In function 'll_file_aio_read': drivers/staging/lustre/lustre/llite/file.c:1188:9: warning: 'count' may be used uninitialized in this function [-Wuninitialized] Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/lli

[PATCH 0/9] staging/lustre: minor cleanups and CONFIG_PROC_FS fix up

2013-12-03 Thread Peng Tao
Hi Greg, Patch 1-7 are small cleanup and fixup. Patch 8 and 9 fix build errors when CONFIG_PROC_FS is off. Thanks, Tao Cc: Andreas Dilger Peng Tao (9): staging/lustre/obdclass: fix false used uninitialized warning staging/lustre/obdclass: lprocfs_{alloc_stats,register} always defined

[PATCH 2/9] staging/lustre/obdclass: lprocfs_{alloc_stats,register} always defined

2013-12-03 Thread Peng Tao
No need to surround them with #ifdef LPROCFS. Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/obdclass/class_obd.c |4 ++-- drivers/staging/lustre/lustre/obdclass/genops.c|3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a

[PATCH 1/9] staging/lustre/obdclass: fix false used uninitialized warning

2013-12-03 Thread Peng Tao
: Peng Tao Signed-off-by: Andreas Dilger --- drivers/staging/lustre/lustre/obdclass/llog_test.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/obdclass/llog_test.c b/drivers/staging/lustre/lustre/obdclass/llog_test.c index 178f89e..7427c12

[PATCH-RESEND 2/2] staging/lustre: remove wirehdr.c

2013-12-03 Thread Peng Tao
It is not used. Cc: Andreas Dilger Signed-off-by: Peng Tao --- drivers/staging/lustre/lustre/ptlrpc/wirehdr.c | 47 1 file changed, 47 deletions(-) delete mode 100644 drivers/staging/lustre/lustre/ptlrpc/wirehdr.c diff --git a/drivers/staging/lustre/lustre/ptlrpc

[PATCH-RESEND 1/2] staging/lustre: fix build error when CONFIG_FS_POSIX_ACL is off

2013-12-03 Thread Peng Tao
nction-declaration] acl = posix_acl_dup(lli->lli_posix_acl); Reported-by: Fengguang Wu Cc: Andreas Dilger Signed-off-by: Peng Tao --- .../lustre/lustre/include/linux/lustre_acl.h | 18 +- drivers/staging/lustre/lustre/include/lustre_mdc.h |9 +++--

[PATCH v3 03/15] staging/lustre: constify uuid argument of obd_uuid2str

2013-12-03 Thread Peng Tao
Reviewed-by: Jinshan Xiong Reviewed-by: Andreas Dilger Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: Peng Tao Signed-off-by: Andreas Dilger --- .../lustre/lustre/include/lustre/lustre_user.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

[PATCH v3 00/15] staging/lustre: sync with external tree, set 2

2013-12-03 Thread Peng Tao
ustre: make obd_uuid_equals return bool Jinshan Xiong (1): staging/lustre/hsm: Add hsm_release feature. John L. Hammond (1): staging/lustre/llite: don't check for O_CREAT in it_create_mode Mikhail Pershin (1): staging/lustre/llog: MGC to use OSD API for backup logs Peng Tao (1): Revert &qu

  1   2   3   4   5   >