mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: Viacheslav Dubeyko <slava@dubeyko.com>
Cc: Christoph Hellwig <hch@lst.de>,
	glaubitz@physik.fu-berlin.de, frank.li@vivo.com,
	linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
	vdubeyko@coreweave.com, willy@infradead.org, brauner@kernel.org,
	djwong@kernel.org
Subject: Re: [PATCH v4 6/7] hfsplus: introduce iomap-based file_operations
Date: Fri, 25 Sep 2026 09:15:03 +0200	[thread overview]
Message-ID: <20260925071503.GA4737@lst.de> (raw)
In-Reply-To: <1056cf9f4393443de00137bc8937922d6ab40202.camel@dubeyko.com>

On Thu, Sep 24, 2026 at 05:07:11PM -0700, Viacheslav Dubeyko wrote:
> On Fri, 2026-09-18 at 15:39 +0200, Christoph Hellwig wrote:
> > On Mon, Sep 14, 2026 at 04:39:40PM -0700, Viacheslav Dubeyko wrote:
> > > This patch
> > 
> > Patch descriptions are not supposed to start with "this patch".
> > By the time this fets into git, it isn't a patch any more :)
> 
> It depends on how we can defines a patch. From my point of view, patch
> is a piece of code that can be applied on some state of git tree. But
> even if this piece of code was adopted and applied into the git tree,
> then it could be extracted as patch again. So, it exists as patch
> forever in the git tree because it is incremental change of the tree.

Either way the kernel style asks this not to be done.  See
Documentation/process/submitting-patches.rst.


  reply	other threads:[~2026-09-25  7:15 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-14 23:39 [PATCH v4 0/7] hfsplus: convert regular file I/O to iomap-based operations Viacheslav Dubeyko
2026-09-14 23:39 ` [PATCH v4 1/7] hfs/hfsplus: exchange hardcoded number of extents on named constants Viacheslav Dubeyko
2026-09-18 13:28   ` Christoph Hellwig
2026-09-14 23:39 ` [PATCH v4 2/7] hfsplus: rework hfsplus_get_block() logic Viacheslav Dubeyko
2026-09-15  2:34   ` Darrick J. Wong
2026-09-15 18:34     ` Viacheslav Dubeyko
2026-09-18 13:29   ` Christoph Hellwig
2026-09-14 23:39 ` [PATCH v4 3/7] hfsplus: take the bitmap page lock for allocate/free Viacheslav Dubeyko
2026-09-18 13:30   ` Christoph Hellwig
2026-09-14 23:39 ` [PATCH v4 4/7] hfsplus: add iomap operations for regular file data Viacheslav Dubeyko
2026-09-18 13:36   ` Christoph Hellwig
2026-09-23 23:19     ` Viacheslav Dubeyko
2026-09-14 23:39 ` [PATCH v4 5/7] hfsplus: move file related operations to file.c Viacheslav Dubeyko
2026-09-15  2:22   ` Darrick J. Wong
2026-09-18 13:31   ` Christoph Hellwig
2026-09-14 23:39 ` [PATCH v4 6/7] hfsplus: introduce iomap-based file_operations Viacheslav Dubeyko
2026-09-18 13:39   ` Christoph Hellwig
2026-09-25  0:07     ` Viacheslav Dubeyko
2026-09-25  7:15       ` Christoph Hellwig [this message]
2026-09-14 23:39 ` [PATCH v4 7/7] hfsplus: switch address_space_operations on iomap-based support Viacheslav Dubeyko
2026-09-15  2:21   ` Darrick J. Wong
2026-09-15 18:37     ` Viacheslav Dubeyko
2026-09-18 13:57   ` Christoph Hellwig

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=20260925071503.GA4737@lst.de \
    --to=hch@lst.de \
    --cc=brauner@kernel.org \
    --cc=djwong@kernel.org \
    --cc=frank.li@vivo.com \
    --cc=glaubitz@physik.fu-berlin.de \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=slava@dubeyko.com \
    --cc=vdubeyko@coreweave.com \
    --cc=willy@infradead.org \
    /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®