linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Question
@ 2008-08-12 20:06 Morey Roof
  2008-08-12 20:32 ` Question Jeff Schroeder
  0 siblings, 1 reply; 5+ messages in thread
From: Morey Roof @ 2008-08-12 20:06 UTC (permalink / raw)
  To: linux-btrfs

I was wondering if someone could give me a few hints.  I noticed that
all patches have [PATCH] in the subject.  Should this always be done?
Also, any other general rules to follow that could be passed on would
be nice.

Thanks,

Morey

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Question
  2008-08-12 20:06 Question Morey Roof
@ 2008-08-12 20:32 ` Jeff Schroeder
  2008-08-13 13:19   ` Question Chris Mason
  0 siblings, 1 reply; 5+ messages in thread
From: Jeff Schroeder @ 2008-08-12 20:32 UTC (permalink / raw)
  To: Morey Roof; +Cc: linux-btrfs

On Tue, Aug 12, 2008 at 1:06 PM, Morey Roof <moreyroof@gmail.com> wrote:
> I was wondering if someone could give me a few hints.  I noticed that
> all patches have [PATCH] in the subject.  Should this always be done?
> Also, any other general rules to follow that could be passed on would
> be nice.

Chris feel free to correct me on this, but...
Since btrfs is Linux kernel code, the same rules apply for btrfs kernel code as
they do for Linux kernel code. Take a look at Documentation/SubmittingPatches
in a checkout of the kernel source. It is online here:
http://users.sosdg.org/~qiyong/lxr/source/Documentation/SubmittingPatches

You can format your patch submission emails correctly with git-send-email(1).

> Morey

-- 
Jeff Schroeder

Don't drink and derive, alcohol and analysis don't mix.
http://www.digitalprognosis.com

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Question
  2008-08-12 20:32 ` Question Jeff Schroeder
@ 2008-08-13 13:19   ` Chris Mason
  0 siblings, 0 replies; 5+ messages in thread
From: Chris Mason @ 2008-08-13 13:19 UTC (permalink / raw)
  To: jeffschroeder; +Cc: Morey Roof, linux-btrfs

On Tue, 2008-08-12 at 13:32 -0700, Jeff Schroeder wrote:
> On Tue, Aug 12, 2008 at 1:06 PM, Morey Roof <moreyroof@gmail.com> wrote:
> > I was wondering if someone could give me a few hints.  I noticed that
> > all patches have [PATCH] in the subject.  Should this always be done?
> > Also, any other general rules to follow that could be passed on would
> > be nice.
> 
> Chris feel free to correct me on this, but...
> Since btrfs is Linux kernel code, the same rules apply for btrfs kernel code as
> they do for Linux kernel code. Take a look at Documentation/SubmittingPatches
> in a checkout of the kernel source. It is online here:
> http://users.sosdg.org/~qiyong/lxr/source/Documentation/SubmittingPatches
> 
> You can format your patch submission emails correctly with git-send-email(1).

Yes, in general the same rules apply to btrfs as the rest of the kernel.
But, one of my goals it to get new people involved and so I try fixup
imperfect patches as well.

-chris




^ permalink raw reply	[flat|nested] 5+ messages in thread

* question
@ 2013-10-20 15:19 lilofile
  2013-10-20 16:53 ` question Hugo Mills
  0 siblings, 1 reply; 5+ messages in thread
From: lilofile @ 2013-10-20 15:19 UTC (permalink / raw)
  To: linux-btrfs


when i read transaction_kthread function in disk-io.c 

delay = HZ * 30;

why commit transaction is 30 seconds,  Other values 20 seconds,10 seconds?
what considerations about the commit transaction interval?



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: question
  2013-10-20 15:19 question lilofile
@ 2013-10-20 16:53 ` Hugo Mills
  0 siblings, 0 replies; 5+ messages in thread
From: Hugo Mills @ 2013-10-20 16:53 UTC (permalink / raw)
  To: lilofile; +Cc: linux-btrfs

[-- Attachment #1: Type: text/plain, Size: 967 bytes --]

On Sun, Oct 20, 2013 at 11:19:26PM +0800, lilofile wrote:
> 
> when i read transaction_kthread function in disk-io.c 
> 
> delay = HZ * 30;
> 
> why commit transaction is 30 seconds,  Other values 20 seconds,10 seconds?
> what considerations about the commit transaction interval?

   It's a more or less arbitrary number. There's been a patch recently
(last couple of months) to allow modifying the value. If you make it
too large, you end up using lots of RAM for unflushed changes, you
risk losing those changes if the machine crashes. If you make it too
small, you end up writing lots more metadata, because that's always
changing as the FS is modified.

   Hugo.

-- 
=== Hugo Mills: hugo@... carfax.org.uk | darksatanic.net | lug.org.uk ===
  PGP key: 65E74AC0 from wwwkeys.eu.pgp.net or http://www.carfax.org.uk
  --- There are three things you should never see being made: laws, ---  
                        standards,  and sausages.                        

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-10-20 16:53 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-20 15:19 question lilofile
2013-10-20 16:53 ` question Hugo Mills
  -- strict thread matches above, loose matches on Subject: below --
2008-08-12 20:06 Question Morey Roof
2008-08-12 20:32 ` Question Jeff Schroeder
2008-08-13 13:19   ` Question Chris Mason

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).