Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [RFC]  [Question] memset.S old code stuff
@ 2026-07-22 23:17 Bradley Morgan
  2026-08-04 15:57 ` Catalin Marinas
  0 siblings, 1 reply; 4+ messages in thread
From: Bradley Morgan @ 2026-07-22 23:17 UTC (permalink / raw)
  To: will, catalin.marinas, linux-arm-kernel, linux-kernel

Hey folks, sorry for the crazy asm emails, I see that in memset.S in
arch/arm64/lib/memset.S, the code is incredibly old code, like 2013 old
(heh).


I was wondering if I could say, adapt it from optimized-routines (same
treatment as memcpy.S essentially.
Thanks!


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

* Re: [RFC]  [Question] memset.S old code stuff
  2026-07-22 23:17 [RFC] [Question] memset.S old code stuff Bradley Morgan
@ 2026-08-04 15:57 ` Catalin Marinas
  2026-08-04 17:20   ` Mark Rutland
  0 siblings, 1 reply; 4+ messages in thread
From: Catalin Marinas @ 2026-08-04 15:57 UTC (permalink / raw)
  To: Bradley Morgan; +Cc: will, linux-arm-kernel, linux-kernel

On Thu, Jul 23, 2026 at 12:17:15AM +0100, Bradley Morgan wrote:
> Hey folks, sorry for the crazy asm emails, I see that in memset.S in
> arch/arm64/lib/memset.S, the code is incredibly old code, like 2013 old
> (heh).
> 
> I was wondering if I could say, adapt it from optimized-routines (same
> treatment as memcpy.S essentially.

I think Robin attempted until he realised the use of the Q registers in
the Cortex Strings library, mentioned here:

https://lore.kernel.org/all/cover.1571073960.git.robin.murphy@arm.com/

So that's where Cortex Strings diverged from what we can use in the
kernel.

-- 
Catalin


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

* Re: [RFC]  [Question] memset.S old code stuff
  2026-08-04 15:57 ` Catalin Marinas
@ 2026-08-04 17:20   ` Mark Rutland
  2026-08-04 17:29     ` Bradley Morgan
  0 siblings, 1 reply; 4+ messages in thread
From: Mark Rutland @ 2026-08-04 17:20 UTC (permalink / raw)
  To: Catalin Marinas; +Cc: Bradley Morgan, will, linux-arm-kernel, linux-kernel

On Tue, Aug 04, 2026 at 04:57:54PM +0100, Catalin Marinas wrote:
> On Thu, Jul 23, 2026 at 12:17:15AM +0100, Bradley Morgan wrote:
> > Hey folks, sorry for the crazy asm emails, I see that in memset.S in
> > arch/arm64/lib/memset.S, the code is incredibly old code, like 2013 old
> > (heh).
> > 
> > I was wondering if I could say, adapt it from optimized-routines (same
> > treatment as memcpy.S essentially.
> 
> I think Robin attempted until he realised the use of the Q registers in
> the Cortex Strings library, mentioned here:
> 
> https://lore.kernel.org/all/cover.1571073960.git.robin.murphy@arm.com/

We also subsequently looked at moving away from Cortex-strings entirely,
e.g.

  https://lore.kernel.org/linux-arm-kernel/20230314115030.347976-1-mark.rutland@arm.com/

... but there were open questions on the semantics, and no clear path
forwards at the time.

Writing the code isn't the hard part. We need to get wider agreement on
the semantics, test on a variety of platforms, etc.

For now, I think it'd be best to leave as-is, since it's a time sink for
reviewers.

Mark.


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

* Re: [RFC]  [Question] memset.S old code stuff
  2026-08-04 17:20   ` Mark Rutland
@ 2026-08-04 17:29     ` Bradley Morgan
  0 siblings, 0 replies; 4+ messages in thread
From: Bradley Morgan @ 2026-08-04 17:29 UTC (permalink / raw)
  To: Mark Rutland, Catalin Marinas; +Cc: will, linux-arm-kernel, linux-kernel

On 4 August 2026 18:20:43 BST, Mark Rutland <mark.rutland@arm.com> wrote:
>On Tue, Aug 04, 2026 at 04:57:54PM +0100, Catalin Marinas wrote:
>> On Thu, Jul 23, 2026 at 12:17:15AM +0100, Bradley Morgan wrote:
>> > Hey folks, sorry for the crazy asm emails, I see that in memset.S in
>> > arch/arm64/lib/memset.S, the code is incredibly old code, like 2013
>old
>> > (heh).
>> > 
>> > I was wondering if I could say, adapt it from optimized-routines (same
>> > treatment as memcpy.S essentially.
>> 
>> I think Robin attempted until he realised the use of the Q registers in
>> the Cortex Strings library, mentioned here:
>>

I would say scalar port??

>> https://lore.kernel.org/all/cover.1571073960.git.robin.murphy@arm.com/
>
>We also subsequently looked at moving away from Cortex-strings entirely,
>e.g.
>
>  https://lore.kernel.org/linux-arm-kernel/20230314115030.347976-1-mark.rutland@arm.com/
>
>... but there were open questions on the semantics, and no clear path
>forwards at the time.
>
>Writing the code isn't the hard part. We need to get wider agreement on
>the semantics, test on a variety of platforms, etc.
>
>For now, I think it'd be best to leave as-is, since it's a time sink for
>reviewers.

Ack. Again, a scalar port is iffy, not 100%. 

That would require time.


>Mark.
>

Thanks!


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

end of thread, other threads:[~2026-08-04 17:30 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-22 23:17 [RFC] [Question] memset.S old code stuff Bradley Morgan
2026-08-04 15:57 ` Catalin Marinas
2026-08-04 17:20   ` Mark Rutland
2026-08-04 17:29     ` Bradley Morgan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox