Received-SPF: pass (google.com: domain of linux-kernel+bounces-17120-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Thu, 4 Jan 2024 13:02:47 -0600
From: Bjorn Helgaas <helgaas@kernel.org>
To: Randy Dunlap <rdunlap@infradead.org>
Cc: Vineet Gupta <vgupta@kernel.org>, linux-kernel@vger.kernel.org,
	Bjorn Helgaas <bhelgaas@google.com>,
	linux-snps-arc@lists.infradead.org
Subject: Re: [PATCH 2/8] ARC: Fix typos
Message-ID: <20240104190247.GA1823635@bhelgaas>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <62c7008e-d460-4314-b2af-2122b93d41dc@infradead.org>

On Wed, Jan 03, 2024 at 04:31:31PM -0800, Randy Dunlap wrote:
> On 1/3/24 15:15, Bjorn Helgaas wrote:
> ...

> > --- a/arch/arc/include/asm/pgtable-bits-arcv2.h
> > +++ b/arch/arc/include/asm/pgtable-bits-arcv2.h
> > @@ -66,7 +66,7 @@
> >   * Other rules which cause the divergence from 1:1 mapping
> >   *
> >   *  1. Although ARC700 can do exclusive execute/write protection (meaning R
> > - *     can be tracked independet of X/W unlike some other CPUs), still to
> > + *     can be tracked independent of X/W unlike some other CPUs), still to
> 
>                          independently

Indeed, embarrassing, thanks.

> >   *     keep things consistent with other archs:
> >   *      -Write implies Read:   W => R
> >   *      -Execute implies Read: X => R
> 
> > diff --git a/arch/arc/kernel/signal.c b/arch/arc/kernel/signal.c
> > index 0b3bb529d246..5414d9f5c40c 100644
> > --- a/arch/arc/kernel/signal.c
> > +++ b/arch/arc/kernel/signal.c
> > @@ -9,7 +9,7 @@
> >   * vineetg: Nov 2009 (Everything needed for TIF_RESTORE_SIGMASK)
> >   *  -do_signal() supports TIF_RESTORE_SIGMASK
> >   *  -do_signal() no loner needs oldset, required by OLD sys_sigsuspend
> > - *  -sys_rt_sigsuspend() now comes from generic code, so discard arch implemen
> > + *  -sys_rt_sigsuspend() now comes from generic code, so discard arch implement
> 
>                                                                          implementation

Thanks again.

Added these and a couple other ones to my local copy in case there's a
v2:

diff --git a/arch/arc/include/asm/pgtable-bits-arcv2.h b/arch/arc/include/asm/pgtable-bits-arcv2.h
index f8f85c04d7a8..8ebec1b21d24 100644
--- a/arch/arc/include/asm/pgtable-bits-arcv2.h
+++ b/arch/arc/include/asm/pgtable-bits-arcv2.h
@@ -66,7 +66,7 @@
  * Other rules which cause the divergence from 1:1 mapping
  *
  *  1. Although ARC700 can do exclusive execute/write protection (meaning R
- *     can be tracked independent of X/W unlike some other CPUs), still to
+ *     can be tracked independently of X/W unlike some other CPUs), still to
  *     keep things consistent with other archs:
  *      -Write implies Read:   W => R
  *      -Execute implies Read: X => R
diff --git a/arch/arc/kernel/signal.c b/arch/arc/kernel/signal.c
index 5414d9f5c40c..3490d005e6d4 100644
--- a/arch/arc/kernel/signal.c
+++ b/arch/arc/kernel/signal.c
@@ -8,15 +8,16 @@
  *
  * vineetg: Nov 2009 (Everything needed for TIF_RESTORE_SIGMASK)
  *  -do_signal() supports TIF_RESTORE_SIGMASK
- *  -do_signal() no loner needs oldset, required by OLD sys_sigsuspend
- *  -sys_rt_sigsuspend() now comes from generic code, so discard arch implement
+ *  -do_signal() no longer needs oldset, required by OLD sys_sigsuspend
+ *  -sys_rt_sigsuspend() now comes from generic code, so discard arch
+ *   implementation
  *  -sys_sigsuspend() no longer needs to fudge ptregs, hence that arg removed
  *  -sys_sigsuspend() no longer loops for do_signal(), sets TIF_xxx and leaves
  *   the job to do_signal()
  *
  * vineetg: July 2009
  *  -Modified Code to support the uClibc provided userland sigreturn stub
- *   to avoid kernel synthesing it on user stack at runtime, costing TLB
+ *   to avoid kernel synthesizing it on user stack at runtime, costing TLB
  *   probes and Cache line flushes.
  *
  * vineetg: July 2009