Fix a typo in md.c

Started by Masahiko Sawadaalmost 3 years ago3 messageshackers
Jump to latest
#1Masahiko Sawada
sawada.mshk@gmail.com

Hi,

I've attached the patch for $subject. In the following comment,

/*
* If available and useful, use posix_fallocate() (via FileAllocate())
* to extend the relation. That's often more efficient than using
* write(), as it commonly won't cause the kernel to allocate page
* cache space for the extended pages.
*
* However, we don't use FileAllocate() for small extensions, as it
* defeats delayed allocation on some filesystems. Not clear where
* that decision should be made though? For now just use a cutoff of
* 8, anything between 4 and 8 worked OK in some local testing.

s/FileAllocate()/FileFallocate()/

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com

Attachments:

fix_typo_md.patchapplication/octet-stream; name=fix_typo_md.patchDownload+2-2
#2Michael Paquier
michael@paquier.xyz
In reply to: Masahiko Sawada (#1)
Re: Fix a typo in md.c

On Wed, Jun 14, 2023 at 06:33:18AM +0900, Masahiko Sawada wrote:

I've attached the patch for $subject. In the following comment,

LGTM.
--
Michael

#3Masahiko Sawada
sawada.mshk@gmail.com
In reply to: Michael Paquier (#2)
Re: Fix a typo in md.c

On Wed, Jun 14, 2023 at 9:19 AM Michael Paquier <michael@paquier.xyz> wrote:

On Wed, Jun 14, 2023 at 06:33:18AM +0900, Masahiko Sawada wrote:

I've attached the patch for $subject. In the following comment,

LGTM.

Thanks, pushed.

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com