Re: [Ocfs2-devel] [PATCH V3] ocfs2: xattr: fix inlined xattr reflink

2013-07-01 Thread Junxiao Bi
Hi Joel, On 07/02/2013 09:50 AM, Joel Becker wrote: > On Tue, Jul 02, 2013 at 09:08:36AM +0800, Junxiao Bi wrote: >> Hi Joel, >> >> On 07/02/2013 06:57 AM, Joel Becker wrote: >>> On Fri, Jun 28, 2013 at 10:56:23AM +0800, Junxiao Bi wrote: Inlined xattr shared free space of inode block with in

Re: [Ocfs2-devel] why oracle give up dlm by disk on ocfs2? because performance?

2013-07-01 Thread Jensen
On 2013/7/2 9:35, Sunil Mushran wrote: > A general purpose file system requires one to manage over a million locks > concurrently. So performance is the main reason. > Thanks for your comments. Has Oracle compared the performance between ocfs2 and ocfs1? > > On Mon, Jul 1, 2013 at 6:22 PM, J

Re: [Ocfs2-devel] [PATCH V3] ocfs2: xattr: fix inlined xattr reflink

2013-07-01 Thread Joel Becker
On Tue, Jul 02, 2013 at 09:08:36AM +0800, Junxiao Bi wrote: > Hi Joel, > > On 07/02/2013 06:57 AM, Joel Becker wrote: > > On Fri, Jun 28, 2013 at 10:56:23AM +0800, Junxiao Bi wrote: > >> Inlined xattr shared free space of inode block with inlined data > >> or data extent record, so the size of the

Re: [Ocfs2-devel] why oracle give up dlm by disk on ocfs2? because performance?

2013-07-01 Thread Sunil Mushran
A general purpose file system requires one to manage over a million locks concurrently. So performance is the main reason. On Mon, Jul 1, 2013 at 6:22 PM, Jensen wrote: > Hi Mark, sunil, jeff an Joel, >Do you know why ? Thanks. > > Jensen. > 2013-7-2 > > On 2013/6/29 11:27, Jensen wrote: >

Re: [Ocfs2-devel] why oracle give up dlm by disk on ocfs2? because performance?

2013-07-01 Thread Jensen
Hi Mark, sunil, jeff an Joel, Do you know why ? Thanks. Jensen. 2013-7-2 On 2013/6/29 11:27, Jensen wrote: > Hi all, >I found ocfs1.0 use the Distributed lock manager by disk. on ocfs2 why > give up? because performance or other reason? > >Btw: > on the ocfs1.0 why not use scsi

Re: [Ocfs2-devel] [PATCH V3] ocfs2: xattr: fix inlined xattr reflink

2013-07-01 Thread Junxiao Bi
Hi Joel, On 07/02/2013 06:57 AM, Joel Becker wrote: > On Fri, Jun 28, 2013 at 10:56:23AM +0800, Junxiao Bi wrote: >> Inlined xattr shared free space of inode block with inlined data >> or data extent record, so the size of the later two should be >> adjusted when inlined xattr is enabled. See ocfs

Re: [Ocfs2-devel] [RESENT PATCH 4/4] ocfs2: adjust switch_case syntax at o2net_state_change()

2013-07-01 Thread Joel Becker
On Mon, Jul 01, 2013 at 04:22:28PM -0700, Andrew Morton wrote: > On Mon, 1 Jul 2013 16:04:21 -0700 Joel Becker wrote: > > > On Sat, Jun 29, 2013 at 12:01:19PM +0800, Jeff Liu wrote: > > > From: Jie Liu > > > > > > Adjust switch..case syntax at o2net_state_change to meet > > > the kernel coding

Re: [Ocfs2-devel] [RESENT PATCH 4/4] ocfs2: adjust switch_case syntax at o2net_state_change()

2013-07-01 Thread Andrew Morton
On Mon, 1 Jul 2013 16:04:21 -0700 Joel Becker wrote: > On Sat, Jun 29, 2013 at 12:01:19PM +0800, Jeff Liu wrote: > > From: Jie Liu > > > > Adjust switch..case syntax at o2net_state_change to meet > > the kernel coding standard. > > > > s/printk/pr_info/. > > > > Signed-off-by: Jie Liu > >

Re: [Ocfs2-devel] [RESENT PATCH 3/4] ocfs2: fix a comments typo at o2quo_hb_still_up()

2013-07-01 Thread Joel Becker
On Sat, Jun 29, 2013 at 12:01:14PM +0800, Jeff Liu wrote: > From: Jie Liu > > Fix a comments typo at o2quo_hb_still_up() > > Signed-off-by: Jie Liu Acked-by: Joel Becker > --- > fs/ocfs2/cluster/quorum.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/ocfs2/c

Re: [Ocfs2-devel] [RESENT PATCH 4/4] ocfs2: adjust switch_case syntax at o2net_state_change()

2013-07-01 Thread Joel Becker
On Sat, Jun 29, 2013 at 12:01:19PM +0800, Jeff Liu wrote: > From: Jie Liu > > Adjust switch..case syntax at o2net_state_change to meet > the kernel coding standard. > > s/printk/pr_info/. > > Signed-off-by: Jie Liu Switching to pr_info() hasn't been discussed as part of the project.

Re: [Ocfs2-devel] [PATCH V3] ocfs2: xattr: fix inlined xattr reflink

2013-07-01 Thread Joel Becker
On Fri, Jun 28, 2013 at 10:56:23AM +0800, Junxiao Bi wrote: > Inlined xattr shared free space of inode block with inlined data > or data extent record, so the size of the later two should be > adjusted when inlined xattr is enabled. See ocfs2_xattr_ibody_init(). > But this isn't done well when refl

Re: [Ocfs2-devel] [PATCH 2/2] ocfs2: xattr: remove useless free space checking

2013-07-01 Thread Joel Becker
On Thu, Jun 20, 2013 at 05:01:25PM +0800, Jeff Liu wrote: > On 06/20/2013 02:17 PM, Junxiao Bi wrote: > > > free space checking will be done in ocfs2_xattr_ibody_init(). > > So remove here. > > > > Signed-off-by: Junxiao Bi > > Reviewed-by: Jie Liu Acked-by: Joel Becker > > > --- > > fs/o

Re: [Ocfs2-devel] [PATCH] unlink performance: wait for open lock in case of dirs

2013-07-01 Thread Goldwyn Rodrigues
On Sun, Jun 30, 2013 at 1:44 AM, Jeff Liu wrote: > On 06/29/2013 07:12 AM, Andrew Morton wrote: > >> On Fri, 14 Jun 2013 21:05:10 -0500 Goldwyn Rodrigues >> wrote: >> >>> This patch is to improve the unlink performance. Here is the scenario: >>> >>> On node A, create multiple directories say d1-

[Ocfs2-devel] BUG: soft lockup - CPU#0 stuck for 22s! [kworker/u:2:28128]

2013-07-01 Thread Eugene Istomin
Hello, i have CPU lockup after scheduling a reflink script: [452296.152056] BUG: soft lockup - CPU#2 stuck for 22s! [kworker/u:2:28128] [452296.155270] Modules linked in: dm_mod xt_tcpudp xt_pkttype xt_LOG xt_limit ocfs2_dlmfs ocfs2_stack_o2cb ocfs2_dlm af_packet nbd blktap blktap2 pciback usbb