All of lore.kernel.org
 help / color / mirror / Atom feed
From: Roman Gushchin <roman.gushchin@linux.dev>
To: Lorenzo Stoakes <ljs@kernel.org>
Cc: Matthew Brost <matthew.brost@intel.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	David Hildenbrand <david@kernel.org>,
	"Liam R. Howlett" <liam@infradead.org>,
	Vlastimil Babka <vbabka@kernel.org>,
	Mike Rapoport <rppt@kernel.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: fixing sashiko failure to apply (was Re: [PATCH v5 00/16] mm/rmap: index MAP_PRIVATE file-backed folios by anonymous pgoff)
Date: Fri, 14 Aug 2026 15:53:36 +0200	[thread overview]
Message-ID: <F700B4F3-2068-4DFD-975A-6324DAAB384F@linux.dev> (raw)
In-Reply-To: <an7ewWbMwN20xklN@lucifer>



> On Aug 14, 2026, at 11:30 AM, Lorenzo Stoakes (ARM) <ljs@kernel.org> wrote:
> 
> [ trim cc list ]
> 
> +cc Roman
> 
>> On Fri, Aug 14, 2026 at 02:13:51AM -0700, Matthew Brost wrote:
>>> On Fri, Aug 14, 2026 at 10:01:19AM +0100, Lorenzo Stoakes (ARM) wrote:
>>> On Thu, Aug 13, 2026 at 11:53:46AM -0700, Andrew Morton wrote:
>>>> You'll be mortified to hear that Sashiko wasn't able to find anything
>>>> to which to apply this.
>>> 
>>> :))
>>> 
>>> Well, when it's right it's useful, when it's wrong or suggesting unrelated
>>> what-nots it's less useful :>)
>>> 
>> 
>> Questioning your assumptions is useful, even when they turn out to be wrong.
>> Show more lines
>> 
>>> I do locally put things through claude + Chris Mason's prompts a lot, I
>>> don't always invoke local sashiko as it's very slow and token-heavy or has
>>> been so far, but am planning to do that more also in future.
>>> 
>> 
>> Yes, it's kind of odd that Sashiko burns more tokens than a full day of
>> breakfast, lunch, and dinner service. Running Sashiko is a bottleneck in
>> my workflow, so I'll defer to others on this list.
> 
> Yup, not sure if there are recommended configs for something saner :)
> 
> Maybe Roman has some advice on that?

Sorry, no magic way to save tokens without hurting the quality. But I am curious what are your numbers?
Can be model-dependent too. In prod on average it burns 3-4M tokens per patch with Gemini 3.1 Pro, 
but maybe mm patches are more complex than average, Idk.

One option is to run only some discovery stages (—stages), but this unlikely will save you that much.
I’d say use a cheaper and faster model for the development, but it has it’s downsides too.

If you have an example of a patch(set) which is particularly token-hungry, I can take a look.
> 
>> 
>>>> 
>>>> Sashiko can be guided with a base-commit: tag but I'm not sure how to
>>>> tell it what tree/branch to try, or even if that's necessary.  Perhaps
>>>> someone can figure this out sometime.
>>> 
>>> b4 gives a base commit, but I think because the trees are rebased it ends
>>> up being the incorrect one.
>>> 
>>> Not sure what the solution is!
>>> 
>> 
>> We have seen this on the Xe list (our list is based on drm-tip),
>> typically with cross-subsystem patches. Some cross-subsystem patches
>> apply and run correctly, while others do not but public CI flows run
>> based on drm-tip. I do not have a bisect or a clear understanding of
>> what works and what doesn't, but I think it would be very useful if the
>> community could better understand the root cause.
> 
> As Mike said, mm-unstable/mm-new is heavily rebased and also carries the old
> version of the series before the new one is applied, so it's super unclear what
> the base commit should be there.
> 
> But in general, I wonder if it's possible that we could tell sashiko
> after-the-fact what base commit to look at once the series is in, or re-trigger
> it somehow once it's in-tree?
> 
> Roman - any suggestions on what we could do to help sashiko find things?
> 
> (Once mm-next is in place everything with change again, but can address that
> then :)

I can implement any reasonable logic here, the problem is that my understanding is
the current mm process is a bit vague here. Which likely will be also an issue for the mm ci.
I’ll merge a support for b4-like dependencies specification soon.

Re re-starting with manual selection it’s on my todo list, but maybe a bit lfurther away, as it requires
an authorization, etc.

Thanks

  reply	other threads:[~2026-08-14 13:53 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-13 17:32 [PATCH v5 00/16] mm/rmap: index MAP_PRIVATE file-backed folios by anonymous pgoff Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 01/16] mm/vma: introduce VMA anon page offset field and add helpers Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 02/16] mm: provide vma_[flags_]is_cow_mapping() and remove is_cow_mapping() Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 03/16] mm: introduce linear_anon_page_index() Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 04/16] mm: abstract vma_address() and introduce vma_anon_address() Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 05/16] mm: update print_bad_page_map() to show anon index if appropriate Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 06/16] mm: introduce and use vma_filebacked_address() Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 07/16] mm/vma: fix self-merge check in copy_vma() Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 08/16] tools/testing/vma: add tests for copy_vma() self-merge Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 09/16] mm: propagate VMA anonymous page offset on map, remap, split + merge Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 10/16] mm/rmap: track whether the page VMA mapped pgoff is anonymous Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 11/16] mm: clean up vma_address_end() Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 12/16] mm/huge_memory: update remove_migration_pmd() to accept a folio Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 13/16] mm/migrate: calculate large folio page index using PFN Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 14/16] mm/rmap: use anon pgoff to track MAP_PRIVATE file-backed anon folios Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 15/16] tools/testing/vma: expand VMA merge tests to assert anon pgoff Lorenzo Stoakes (ARM)
2026-08-13 17:32 ` [PATCH v5 16/16] tools/testing/selftests/mm: test anonymous page offset merge behaviour Lorenzo Stoakes (ARM)
2026-08-13 18:53 ` [PATCH v5 00/16] mm/rmap: index MAP_PRIVATE file-backed folios by anonymous pgoff Andrew Morton
2026-08-14  9:01   ` Lorenzo Stoakes (ARM)
2026-08-14  9:13     ` Matthew Brost
2026-08-14  9:29       ` fixing sashiko failure to apply (was Re: [PATCH v5 00/16] mm/rmap: index MAP_PRIVATE file-backed folios by anonymous pgoff) Lorenzo Stoakes (ARM)
2026-08-14 13:53         ` Roman Gushchin [this message]
2026-08-14  9:18     ` [PATCH v5 00/16] mm/rmap: index MAP_PRIVATE file-backed folios by anonymous pgoff Mike Rapoport
2026-08-14  9:23       ` Lorenzo Stoakes (ARM)
2026-08-14  9:45         ` Mike Rapoport

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=F700B4F3-2068-4DFD-975A-6324DAAB384F@linux.dev \
    --to=roman.gushchin@linux.dev \
    --cc=akpm@linux-foundation.org \
    --cc=david@kernel.org \
    --cc=liam@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=ljs@kernel.org \
    --cc=matthew.brost@intel.com \
    --cc=rppt@kernel.org \
    --cc=vbabka@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.