diff mbox series

Use CSR numbers instead of names

Message ID mvmr25z4toj.fsf@suse.de (mailing list archive)
State New, archived
Headers show
Series Use CSR numbers instead of names | expand

Commit Message

Andreas Schwab Aug. 5, 2019, 1:40 p.m. UTC
Fixes: f6635f873a60 ("riscv: move switch_mm to its own file")
Fixes: 671f9a3e2e24 ("RISC-V: Setup initial page tables in two stages")
Signed-off-by: Andreas Schwab <schwab@suse.de>
---
 arch/riscv/mm/context.c | 7 +------
 arch/riscv/mm/init.c    | 2 +-
 2 files changed, 2 insertions(+), 7 deletions(-)

Comments

Anup Patel Aug. 5, 2019, 3:05 p.m. UTC | #1
> -----Original Message-----
> From: linux-riscv <linux-riscv-bounces@lists.infradead.org> On Behalf Of
> Andreas Schwab
> Sent: Monday, August 5, 2019 7:10 PM
> To: linux-riscv@lists.infradead.org
> Cc: linux-kernel@lists.kernel.org
> Subject: [PATCH] Use CSR numbers instead of names
> 
> Fixes: f6635f873a60 ("riscv: move switch_mm to its own file")
> Fixes: 671f9a3e2e24 ("RISC-V: Setup initial page tables in two stages")
> Signed-off-by: Andreas Schwab <schwab@suse.de>
> ---
>  arch/riscv/mm/context.c | 7 +------
>  arch/riscv/mm/init.c    | 2 +-
>  2 files changed, 2 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/riscv/mm/context.c b/arch/riscv/mm/context.c index
> 89ceb3cbe218..beeb5d7f92ea 100644
> --- a/arch/riscv/mm/context.c
> +++ b/arch/riscv/mm/context.c
> @@ -57,12 +57,7 @@ void switch_mm(struct mm_struct *prev, struct
> mm_struct *next,
>  	cpumask_clear_cpu(cpu, mm_cpumask(prev));
>  	cpumask_set_cpu(cpu, mm_cpumask(next));
> 
> -	/*
> -	 * Use the old spbtr name instead of using the current satp
> -	 * name to support binutils 2.29 which doesn't know about the
> -	 * privileged ISA 1.10 yet.
> -	 */
> -	csr_write(sptbr, virt_to_pfn(next->pgd) | SATP_MODE);
> +	csr_write(CSR_SATP, virt_to_pfn(next->pgd) | SATP_MODE);
>  	local_flush_tlb_all();
> 
>  	flush_icache_deferred(next);
> diff --git a/arch/riscv/mm/init.c b/arch/riscv/mm/init.c index
> 42bf939693d3..238fc4157f8d 100644
> --- a/arch/riscv/mm/init.c
> +++ b/arch/riscv/mm/init.c
> @@ -435,7 +435,7 @@ static void __init setup_vm_final(void)
>  	clear_fixmap(FIX_PMD);
> 
>  	/* Move to swapper page table */
> -	csr_write(sptbr, PFN_DOWN(__pa(swapper_pg_dir)) |
> SATP_MODE);
> +	csr_write(CSR_SATP, PFN_DOWN(__pa(swapper_pg_dir)) |
> SATP_MODE);
>  	local_flush_tlb_all();
>  }
> 
> --
> 2.22.0
> 
> 
> --
> Andreas Schwab, SUSE Labs, schwab@suse.de
> GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE  1748 E4D4 88E3 0EEA B9D7
> "And now for something completely different."
> 
> _______________________________________________
> linux-riscv mailing list
> linux-riscv@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv

Looks good to me.

Reviewed-by: Anup Patel <anup.patel@wdc.com>

Regards,
Anup
Christoph Hellwig Aug. 6, 2019, 8:49 p.m. UTC | #2
Looks good.  I've been trying to get the same patch in for about two
month now, maybe this one has more luck..
Paul Walmsley Aug. 7, 2019, 12:04 a.m. UTC | #3
On Tue, 6 Aug 2019, Christoph Hellwig wrote:

> Looks good.  I've been trying to get the same patch in for about two
> month now, maybe this one has more luck..

Thanks for your patience.  If you have patches from one of your earlier 
series that you think should go in as fixes, it's helpful for us if you 
split them out from patches that are intended for the regular merge 
window.


- Paul
Bin Meng Aug. 7, 2019, 1:31 a.m. UTC | #4
On Wed, Aug 7, 2019 at 8:04 AM Paul Walmsley <paul.walmsley@sifive.com> wrote:
>
> On Tue, 6 Aug 2019, Christoph Hellwig wrote:
>
> > Looks good.  I've been trying to get the same patch in for about two
> > month now, maybe this one has more luck..
>
> Thanks for your patience.  If you have patches from one of your earlier
> series that you think should go in as fixes, it's helpful for us if you
> split them out from patches that are intended for the regular merge
> window.

I have already sent a similar patch (with more places fixed) on Jul 11.
https://patchwork.kernel.org/patch/11039421/

Regards,
Bin
Paul Walmsley Aug. 7, 2019, 1:39 a.m. UTC | #5
On Wed, 7 Aug 2019, Bin Meng wrote:

> On Wed, Aug 7, 2019 at 8:04 AM Paul Walmsley <paul.walmsley@sifive.com> wrote:
> >
> > On Tue, 6 Aug 2019, Christoph Hellwig wrote:
> >
> > > Looks good.  I've been trying to get the same patch in for about two
> > > month now, maybe this one has more luck..
> >
> > Thanks for your patience.  If you have patches from one of your earlier
> > series that you think should go in as fixes, it's helpful for us if you
> > split them out from patches that are intended for the regular merge
> > window.
> 
> I have already sent a similar patch (with more places fixed) on Jul 11.
> https://patchwork.kernel.org/patch/11039421/

Christoph's patch predates yours.

If all looks well with your patch, I'll update it to remove the overlap 
and queue it after his.


- Paul
Christoph Hellwig Aug. 7, 2019, 5:40 a.m. UTC | #6
On Tue, Aug 06, 2019 at 06:39:59PM -0700, Paul Walmsley wrote:
> > 
> > I have already sent a similar patch (with more places fixed) on Jul 11.
> > https://patchwork.kernel.org/patch/11039421/
> 
> Christoph's patch predates yours.
> 
> If all looks well with your patch, I'll update it to remove the overlap 
> and queue it after his.

The one from Bin looks most comprehensive, so maybe go with that, plus
the hunk from Andreas for the instance that was only recently introduced.
Bin Meng Aug. 7, 2019, 1:34 p.m. UTC | #7
On Wed, Aug 7, 2019 at 1:40 PM Christoph Hellwig <hch@infradead.org> wrote:
>
> On Tue, Aug 06, 2019 at 06:39:59PM -0700, Paul Walmsley wrote:
> > >
> > > I have already sent a similar patch (with more places fixed) on Jul 11.
> > > https://patchwork.kernel.org/patch/11039421/
> >
> > Christoph's patch predates yours.
> >
> > If all looks well with your patch, I'll update it to remove the overlap
> > and queue it after his.
>
> The one from Bin looks most comprehensive, so maybe go with that, plus
> the hunk from Andreas for the instance that was only recently introduced.

Thanks Christoph.

I've included your SoB and Andreas's into my v2 patch which added the
changes of arch/riscv/mm/init.c that was recently changed.
https://patchwork.kernel.org/patch/11082147/

Regards,
Bin
Christoph Hellwig Aug. 7, 2019, 3:13 p.m. UTC | #8
On Wed, Aug 07, 2019 at 09:34:03PM +0800, Bin Meng wrote:
> I've included your SoB and Andreas's into my v2 patch which added the
> changes of arch/riscv/mm/init.c that was recently changed.
> https://patchwork.kernel.org/patch/11082147/

I don;t think you need mine at all, as you did the same change
independently.  And maybe well just keep the hunk from Andreas separate
to keep it simple.
Paul Walmsley Aug. 7, 2019, 3:47 p.m. UTC | #9
On Wed, 7 Aug 2019, Bin Meng wrote:

> On Wed, Aug 7, 2019 at 1:40 PM Christoph Hellwig <hch@infradead.org> wrote:
> >
> > On Tue, Aug 06, 2019 at 06:39:59PM -0700, Paul Walmsley wrote:
> > > >
> > > > I have already sent a similar patch (with more places fixed) on Jul 11.
> > > > https://patchwork.kernel.org/patch/11039421/
> > >
> > > Christoph's patch predates yours.
> > >
> > > If all looks well with your patch, I'll update it to remove the overlap
> > > and queue it after his.
> >
> > The one from Bin looks most comprehensive, so maybe go with that, plus
> > the hunk from Andreas for the instance that was only recently introduced.
> 
> Thanks Christoph.
> 
> I've included your SoB and Andreas's into my v2 patch which added the
> changes of arch/riscv/mm/init.c that was recently changed.
> https://patchwork.kernel.org/patch/11082147/

Best not to add others' Signed-off-by:s unless they explicitly offer them. 
Adding a Signed-off-by: for the kernel denotes that that person has 
accepted the Developer's Statement of Origin here:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n431

Could you please either convert those into Co-developed-by: lines or ask 
Christoph and Andreas for explicit Signed-off-by:s, Acked-by:s, or 
Reviewed-by:s ?


- Paul
Paul Walmsley Aug. 7, 2019, 3:49 p.m. UTC | #10
On Wed, 7 Aug 2019, Paul Walmsley wrote:

> On Wed, 7 Aug 2019, Bin Meng wrote:
> 
> > On Wed, Aug 7, 2019 at 1:40 PM Christoph Hellwig <hch@infradead.org> wrote:
> > >
> > > On Tue, Aug 06, 2019 at 06:39:59PM -0700, Paul Walmsley wrote:
> > > > >
> > > > > I have already sent a similar patch (with more places fixed) on Jul 11.
> > > > > https://patchwork.kernel.org/patch/11039421/
> > > >
> > > > Christoph's patch predates yours.
> > > >
> > > > If all looks well with your patch, I'll update it to remove the overlap
> > > > and queue it after his.
> > >
> > > The one from Bin looks most comprehensive, so maybe go with that, plus
> > > the hunk from Andreas for the instance that was only recently introduced.
> > 
> > Thanks Christoph.
> > 
> > I've included your SoB and Andreas's into my v2 patch which added the
> > changes of arch/riscv/mm/init.c that was recently changed.
> > https://patchwork.kernel.org/patch/11082147/
> 
> Best not to add others' Signed-off-by:s unless they explicitly offer them. 
> Adding a Signed-off-by: for the kernel denotes that that person has 
> accepted the Developer's Statement of Origin here:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n431
> 
> Could you please either convert those into Co-developed-by: lines or ask 

Actually even Co-developed-by: lines require Signed-off-by:s as well:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n548

- Paul
Bin Meng Aug. 7, 2019, 4:09 p.m. UTC | #11
On Wed, Aug 7, 2019 at 11:47 PM Paul Walmsley <paul.walmsley@sifive.com> wrote:
>
> On Wed, 7 Aug 2019, Bin Meng wrote:
>
> > On Wed, Aug 7, 2019 at 1:40 PM Christoph Hellwig <hch@infradead.org> wrote:
> > >
> > > On Tue, Aug 06, 2019 at 06:39:59PM -0700, Paul Walmsley wrote:
> > > > >
> > > > > I have already sent a similar patch (with more places fixed) on Jul 11.
> > > > > https://patchwork.kernel.org/patch/11039421/
> > > >
> > > > Christoph's patch predates yours.
> > > >
> > > > If all looks well with your patch, I'll update it to remove the overlap
> > > > and queue it after his.
> > >
> > > The one from Bin looks most comprehensive, so maybe go with that, plus
> > > the hunk from Andreas for the instance that was only recently introduced.
> >
> > Thanks Christoph.
> >
> > I've included your SoB and Andreas's into my v2 patch which added the
> > changes of arch/riscv/mm/init.c that was recently changed.
> > https://patchwork.kernel.org/patch/11082147/
>
> Best not to add others' Signed-off-by:s unless they explicitly offer them.
> Adding a Signed-off-by: for the kernel denotes that that person has
> accepted the Developer's Statement of Origin here:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n431
>
> Could you please either convert those into Co-developed-by: lines or ask
> Christoph and Andreas for explicit Signed-off-by:s, Acked-by:s, or
> Reviewed-by:s ?

OK, I will remove these SoB tags.

Regards,
Bin
diff mbox series

Patch

diff --git a/arch/riscv/mm/context.c b/arch/riscv/mm/context.c
index 89ceb3cbe218..beeb5d7f92ea 100644
--- a/arch/riscv/mm/context.c
+++ b/arch/riscv/mm/context.c
@@ -57,12 +57,7 @@  void switch_mm(struct mm_struct *prev, struct mm_struct *next,
 	cpumask_clear_cpu(cpu, mm_cpumask(prev));
 	cpumask_set_cpu(cpu, mm_cpumask(next));
 
-	/*
-	 * Use the old spbtr name instead of using the current satp
-	 * name to support binutils 2.29 which doesn't know about the
-	 * privileged ISA 1.10 yet.
-	 */
-	csr_write(sptbr, virt_to_pfn(next->pgd) | SATP_MODE);
+	csr_write(CSR_SATP, virt_to_pfn(next->pgd) | SATP_MODE);
 	local_flush_tlb_all();
 
 	flush_icache_deferred(next);
diff --git a/arch/riscv/mm/init.c b/arch/riscv/mm/init.c
index 42bf939693d3..238fc4157f8d 100644
--- a/arch/riscv/mm/init.c
+++ b/arch/riscv/mm/init.c
@@ -435,7 +435,7 @@  static void __init setup_vm_final(void)
 	clear_fixmap(FIX_PMD);
 
 	/* Move to swapper page table */
-	csr_write(sptbr, PFN_DOWN(__pa(swapper_pg_dir)) | SATP_MODE);
+	csr_write(CSR_SATP, PFN_DOWN(__pa(swapper_pg_dir)) | SATP_MODE);
 	local_flush_tlb_all();
 }