All of lore.kernel.org
 help / color / mirror / Atom feed
From: Catalin Marinas <catalin.marinas@arm.com>
To: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
Cc: Jamie Cunliffe <Jamie.Cunliffe@arm.com>,
	linux-arm-kernel@lists.infradead.org,
	rust-for-linux@vger.kernel.org, Miguel Ojeda <ojeda@kernel.org>,
	Will Deacon <will@kernel.org>,
	steve.capper@arm.com, Asahi Lina <lina@asahilina.net>,
	boqun.feng@gmail.com, andrew@lunn.ch
Subject: Re: [PATCH v4 2/2] arm64: rust: Enable Rust support for AArch64
Date: Sun, 22 Oct 2023 11:48:56 +0100	[thread overview]
Message-ID: <ZTT+GMucKZ192yuv@arm.com> (raw)
In-Reply-To: <CANiq72=hpO8dP_B0=gOocCeaC-GDNQJx1MeA2HDZcEKDt4ZepA@mail.gmail.com>

On Sat, Oct 21, 2023 at 03:40:21PM +0200, Miguel Ojeda wrote:
> On Fri, Oct 20, 2023 at 6:46 PM Catalin Marinas <catalin.marinas@arm.com> wrote:
> > Miguel, Will acked this already so I think it's good to go. Do you plan
> > to pick both patches via the Rust tree?
> 
> Thanks for taking a look!
> 
> As you prefer -- ideally the changes for each architecture would go
> through their own trees (we are trying to get maintainers of the
> different parts of the kernel involved with Rust for their own bits
> and pieces), but if you think there is a reason to avoid that, I can
> also take them as fallback for the time being.

The main reason I asked was that the first patch touches x86 and none of
the x86 maintainers have been cc'ed or acked the patch. I thought
there's some deal that Rust-related changes go via you tree.

> > Acked-by: Catalin Marinas <catalin.marinas@arm.com>
> 
> If you decide to take them:
> 
> Acked-by: Miguel Ojeda <ojeda@kernel.org>
> 
> I have boot tested it in aarch64 QEMU and saw the KUnit Rust tests
> passing and checked that the x86 side still works fine.

Thanks. I'll reply to the first patch and cc some x86 maintainers. I
assume your ack applies to the first patch as well.

-- 
Catalin

WARNING: multiple messages have this Message-ID (diff)
From: Catalin Marinas <catalin.marinas@arm.com>
To: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
Cc: Jamie Cunliffe <Jamie.Cunliffe@arm.com>,
	linux-arm-kernel@lists.infradead.org,
	rust-for-linux@vger.kernel.org, Miguel Ojeda <ojeda@kernel.org>,
	Will Deacon <will@kernel.org>,
	steve.capper@arm.com, Asahi Lina <lina@asahilina.net>,
	boqun.feng@gmail.com, andrew@lunn.ch
Subject: Re: [PATCH v4 2/2] arm64: rust: Enable Rust support for AArch64
Date: Sun, 22 Oct 2023 11:48:56 +0100	[thread overview]
Message-ID: <ZTT+GMucKZ192yuv@arm.com> (raw)
In-Reply-To: <CANiq72=hpO8dP_B0=gOocCeaC-GDNQJx1MeA2HDZcEKDt4ZepA@mail.gmail.com>

On Sat, Oct 21, 2023 at 03:40:21PM +0200, Miguel Ojeda wrote:
> On Fri, Oct 20, 2023 at 6:46 PM Catalin Marinas <catalin.marinas@arm.com> wrote:
> > Miguel, Will acked this already so I think it's good to go. Do you plan
> > to pick both patches via the Rust tree?
> 
> Thanks for taking a look!
> 
> As you prefer -- ideally the changes for each architecture would go
> through their own trees (we are trying to get maintainers of the
> different parts of the kernel involved with Rust for their own bits
> and pieces), but if you think there is a reason to avoid that, I can
> also take them as fallback for the time being.

The main reason I asked was that the first patch touches x86 and none of
the x86 maintainers have been cc'ed or acked the patch. I thought
there's some deal that Rust-related changes go via you tree.

> > Acked-by: Catalin Marinas <catalin.marinas@arm.com>
> 
> If you decide to take them:
> 
> Acked-by: Miguel Ojeda <ojeda@kernel.org>
> 
> I have boot tested it in aarch64 QEMU and saw the KUnit Rust tests
> passing and checked that the x86 side still works fine.

Thanks. I'll reply to the first patch and cc some x86 maintainers. I
assume your ack applies to the first patch as well.

-- 
Catalin

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2023-10-22 10:49 UTC|newest]

Thread overview: 81+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-20 15:50 [PATCH v4 0/2] Rust enablement for AArch64 Jamie Cunliffe
2023-10-20 15:50 ` Jamie Cunliffe
2023-10-20 15:50 ` [PATCH v4 1/2] rust: Refactor the build target to allow the use of builtin targets Jamie Cunliffe
2023-10-20 15:50   ` Jamie Cunliffe
2023-10-22 10:55   ` Catalin Marinas
2023-10-22 10:55     ` Catalin Marinas
2023-10-22 11:07     ` Ingo Molnar
2023-10-22 11:07       ` Ingo Molnar
2023-10-31  0:24       ` Trevor Gross
2023-10-31  0:24         ` Trevor Gross
2023-10-31 11:58         ` Catalin Marinas
2023-10-31 11:58           ` Catalin Marinas
2023-10-31 19:01           ` Trevor Gross
2023-10-31 19:01             ` Trevor Gross
2023-12-12 16:47           ` Will Deacon
2023-12-12 16:47             ` Will Deacon
2024-01-22  9:17   ` Trevor Gross
2024-01-22  9:17     ` Trevor Gross
2024-01-25  3:19     ` WANG Rui
2024-01-25  3:19       ` WANG Rui
2024-01-26 19:37       ` Masahiro Yamada
2024-01-26 19:37         ` Masahiro Yamada
2024-01-27  5:08         ` WANG Rui
2024-01-27  5:08           ` WANG Rui
2024-01-27 17:31           ` Miguel Ojeda
2024-01-27 17:31             ` Miguel Ojeda
2024-01-22  9:29   ` Alice Ryhl
2024-01-22  9:29     ` Alice Ryhl
2024-01-24  9:12   ` Masahiro Yamada
2024-01-24  9:12     ` Masahiro Yamada
2023-10-20 15:50 ` [PATCH v4 2/2] arm64: rust: Enable Rust support for AArch64 Jamie Cunliffe
2023-10-20 15:50   ` Jamie Cunliffe
2023-10-20 16:45   ` Catalin Marinas
2023-10-20 16:45     ` Catalin Marinas
2023-10-21 13:40     ` Miguel Ojeda
2023-10-21 13:40       ` Miguel Ojeda
2023-10-22 10:48       ` Catalin Marinas [this message]
2023-10-22 10:48         ` Catalin Marinas
2023-10-22 12:14         ` Miguel Ojeda
2023-10-22 12:14           ` Miguel Ojeda
2023-10-20 17:21   ` Andrew Lunn
2023-10-20 17:21     ` Andrew Lunn
2023-10-20 18:33     ` Boqun Feng
2023-10-20 18:33       ` Boqun Feng
2023-10-20 18:47       ` Andrew Lunn
2023-10-20 18:47         ` Andrew Lunn
2023-10-21 12:50       ` Alice Ryhl
2023-10-21 12:50         ` Alice Ryhl
2023-10-21 13:41       ` Miguel Ojeda
2023-10-21 13:41         ` Miguel Ojeda
2023-10-21 16:03         ` Andrew Lunn
2023-10-21 16:03           ` Andrew Lunn
2023-10-22 12:57           ` Miguel Ojeda
2023-10-22 12:57             ` Miguel Ojeda
2023-10-24  0:57       ` Stephen Boyd
2023-10-24  0:57         ` Stephen Boyd
2023-10-25 23:55         ` Boqun Feng
2023-10-25 23:55           ` Boqun Feng
2023-11-01 15:04       ` Linus Walleij
2023-11-01 15:04         ` Linus Walleij
2023-10-31 18:31   ` Matthew Maurer
2023-10-31 18:31     ` Matthew Maurer
2023-11-28 18:29     ` Boqun Feng
2023-11-28 18:29       ` Boqun Feng
2023-12-13 19:00     ` Miguel Ojeda
2023-12-13 19:00       ` Miguel Ojeda
2024-01-22  2:01   ` Fabien Parent
2024-01-22  2:01     ` Fabien Parent
2024-01-22  5:27   ` Behme Dirk (CM/ESO2)
2024-01-22  5:27     ` Behme Dirk (CM/ESO2)
2024-01-22  9:30   ` Alice Ryhl
2024-01-22  9:30     ` Alice Ryhl
2024-02-09 17:38 ` [PATCH v4 0/2] Rust enablement " Catalin Marinas
2024-02-09 17:38   ` Catalin Marinas
2024-02-09 21:41   ` Miguel Ojeda
2024-02-09 21:41     ` Miguel Ojeda
  -- strict thread matches above, loose matches on Subject: below --
2023-10-24 14:17 [PATCH v4 2/2] arm64: rust: Enable Rust support " Pratham Patel
2023-10-24 15:19 ` Miguel Ojeda
2023-10-24 15:19   ` Miguel Ojeda
2023-10-25  2:01   ` Pratham Patel
2023-10-25  2:01     ` Pratham Patel

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=ZTT+GMucKZ192yuv@arm.com \
    --to=catalin.marinas@arm.com \
    --cc=Jamie.Cunliffe@arm.com \
    --cc=andrew@lunn.ch \
    --cc=boqun.feng@gmail.com \
    --cc=lina@asahilina.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=miguel.ojeda.sandonis@gmail.com \
    --cc=ojeda@kernel.org \
    --cc=rust-for-linux@vger.kernel.org \
    --cc=steve.capper@arm.com \
    --cc=will@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.