mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Damien Le Moal <dlemoal@kernel.org>
To: Xichao Zhao <zhao.xichao@vivo.com>,
	Naohiro Aota <naohiro.aota@wdc.com>,
	Johannes Thumshirn <jth@kernel.org>,
	"open list:ZONEFS FILESYSTEM" <linux-fsdevel@vger.kernel.org>,
	open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 6/6] zonefs: fix "writen"->"written"
Date: Fri, 8 Aug 2025 16:50:07 +0900	[thread overview]
Message-ID: <c08243fc-06fd-4d7e-84e3-f231e97fe451@kernel.org> (raw)
In-Reply-To: <20250808071459.174087-7-zhao.xichao@vivo.com>

On 8/8/25 4:14 PM, Xichao Zhao wrote:
> Trivial fix to spelling mistake in comment text.
> 
> Signed-off-by: Xichao Zhao <zhao.xichao@vivo.com>

Please squash this with patch 5/6. There is no good reason to have 2 patches
for typo fixes.

Also split this into separate patches for each FS. The entire patch series is
not going to be applied in a single tree. The different FS maintainers wil (or
not) take the patches for their FS in their tree.

> ---
>  fs/zonefs/super.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/zonefs/super.c b/fs/zonefs/super.c
> index 4dc7f967c861..70be0b3dda49 100644
> --- a/fs/zonefs/super.c
> +++ b/fs/zonefs/super.c
> @@ -268,7 +268,7 @@ static void zonefs_handle_io_error(struct inode *inode, struct blk_zone *zone,
>  	 * Check the zone condition: if the zone is not "bad" (offline or
>  	 * read-only), read errors are simply signaled to the IO issuer as long
>  	 * as there is no inconsistency between the inode size and the amount of
> -	 * data writen in the zone (data_size).
> +	 * data written in the zone (data_size).
>  	 */
>  	data_size = zonefs_check_zone_condition(sb, z, zone);
>  	isize = i_size_read(inode);
> @@ -282,7 +282,7 @@ static void zonefs_handle_io_error(struct inode *inode, struct blk_zone *zone,
>  	 * For the latter case, the cause may be a write IO error or an external
>  	 * action on the device. Two error patterns exist:
>  	 * 1) The inode size is lower than the amount of data in the zone:
> -	 *    a write operation partially failed and data was writen at the end
> +	 *    a write operation partially failed and data was written at the end
>  	 *    of the file. This can happen in the case of a large direct IO
>  	 *    needing several BIOs and/or write requests to be processed.
>  	 * 2) The inode size is larger than the amount of data in the zone:


-- 
Damien Le Moal
Western Digital Research

      reply	other threads:[~2025-08-08  7:52 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-08  7:14 [PATCH 0/6] fs: correct some spelling mistakes Xichao Zhao
2025-08-08  7:14 ` [PATCH 1/6] fs: fix "writen"->"written" Xichao Zhao
2025-08-08  7:14 ` [PATCH 2/6] NFSv4: fix prefered -> preferred Xichao Zhao
2025-08-08  7:14 ` [PATCH 3/6] fsnotify: fix "rewriten"->"rewritten" Xichao Zhao
2025-08-08  7:14 ` [PATCH 4/6] xfs: fix "acheive"->"achieve" Xichao Zhao
2025-08-08  7:14 ` [PATCH 5/6] zonefs: fix "unwriten"->"unwritten" Xichao Zhao
2025-08-08  7:14 ` [PATCH 6/6] zonefs: fix "writen"->"written" Xichao Zhao
2025-08-08  7:50   ` Damien Le Moal [this message]

Reply instructions:

You may reply publicly 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=c08243fc-06fd-4d7e-84e3-f231e97fe451@kernel.org \
    --to=dlemoal@kernel.org \
    --cc=jth@kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=naohiro.aota@wdc.com \
    --cc=zhao.xichao@vivo.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®