2018-08-02 08:51:25

by Alexey Brodkin

[permalink] [raw]
Subject: [PATCH] ARC: [plat-axs10x] Enable SWAP

SWAP support on ARC was fixed earlier by
commit 6e3761145a9b ("ARC: Fix CONFIG_SWAP")
so now we may safely enable it on platforms that
have external media like USB and SD-card.

Signed-off-by: Alexey Brodkin <[email protected]>
Cc: [email protected] # 6e3761145a9b: ARC: Fix CONFIG_SWAP
---
arch/arc/configs/axs101_defconfig | 1 -
arch/arc/configs/axs103_defconfig | 1 -
arch/arc/configs/axs103_smp_defconfig | 1 -
3 files changed, 3 deletions(-)

diff --git a/arch/arc/configs/axs101_defconfig b/arch/arc/configs/axs101_defconfig
index 3dbb2b3f60af..96b7258a76be 100644
--- a/arch/arc/configs/axs101_defconfig
+++ b/arch/arc/configs/axs101_defconfig
@@ -1,4 +1,3 @@
-# CONFIG_SWAP is not set
CONFIG_SYSVIPC=y
CONFIG_POSIX_MQUEUE=y
# CONFIG_CROSS_MEMORY_ATTACH is not set
diff --git a/arch/arc/configs/axs103_defconfig b/arch/arc/configs/axs103_defconfig
index 4d7b85d4b2dc..72fc21ef1334 100644
--- a/arch/arc/configs/axs103_defconfig
+++ b/arch/arc/configs/axs103_defconfig
@@ -1,4 +1,3 @@
-# CONFIG_SWAP is not set
CONFIG_SYSVIPC=y
CONFIG_POSIX_MQUEUE=y
# CONFIG_CROSS_MEMORY_ATTACH is not set
diff --git a/arch/arc/configs/axs103_smp_defconfig b/arch/arc/configs/axs103_smp_defconfig
index 92417e2e39f3..2ac89bd00e28 100644
--- a/arch/arc/configs/axs103_smp_defconfig
+++ b/arch/arc/configs/axs103_smp_defconfig
@@ -1,4 +1,3 @@
-# CONFIG_SWAP is not set
CONFIG_SYSVIPC=y
CONFIG_POSIX_MQUEUE=y
# CONFIG_CROSS_MEMORY_ATTACH is not set
--
2.17.1



2018-08-06 18:29:45

by Vineet Gupta

[permalink] [raw]
Subject: Re: [PATCH] ARC: [plat-axs10x] Enable SWAP

On 08/02/2018 01:50 AM, Alexey Brodkin wrote:
> SWAP support on ARC was fixed earlier by
> commit 6e3761145a9b ("ARC: Fix CONFIG_SWAP")
> so now we may safely enable it on platforms that
> have external media like USB and SD-card.
>
> Signed-off-by: Alexey Brodkin <[email protected]>
> Cc: [email protected] # 6e3761145a9b: ARC: Fix CONFIG_SWAP

Does this really qualify as a fix for backport ?

> ---
> arch/arc/configs/axs101_defconfig | 1 -
> arch/arc/configs/axs103_defconfig | 1 -
> arch/arc/configs/axs103_smp_defconfig | 1 -

Also hsdk ?


2018-08-08 13:36:10

by Alexey Brodkin

[permalink] [raw]
Subject: Re: [PATCH] ARC: [plat-axs10x] Enable SWAP

Hi Vineet,

On Mon, 2018-08-06 at 10:56 -0700, Vineet Gupta wrote:
> On 08/02/2018 01:50 AM, Alexey Brodkin wrote:
> > SWAP support on ARC was fixed earlier by
> > commit 6e3761145a9b ("ARC: Fix CONFIG_SWAP")
> > so now we may safely enable it on platforms that
> > have external media like USB and SD-card.
> >
> > Signed-off-by: Alexey Brodkin <[email protected]>
> > Cc: [email protected] # 6e3761145a9b: ARC: Fix CONFIG_SWAP
>
> Does this really qualify as a fix for backport ?
>
> > ---
> > arch/arc/configs/axs101_defconfig | 1 -
> > arch/arc/configs/axs103_defconfig | 1 -
> > arch/arc/configs/axs103_smp_defconfig | 1 -
>
> Also hsdk ?

Exactly - in HSDK we happened to have SWAP from the very beginning
so my intention was to align other boards so we get exactly the same
behavior on all boards once SWAP was fixed by
commit 6e3761145a9b ("ARC: Fix CONFIG_SWAP").

Otherwise since this is just a matter of toggling a config option
we may leave it up to user to decide if he/she wants to have
SWAP enabled.

That said it should be fine in either way.

-Alexey

2018-08-08 16:30:37

by Vineet Gupta

[permalink] [raw]
Subject: Re: [PATCH] ARC: [plat-axs10x] Enable SWAP

On 08/08/2018 06:35 AM, Alexey Brodkin wrote:
> Exactly - in HSDK we happened to have SWAP from the very beginning
> so my intention was to align other boards so we get exactly the same
> behavior on all boards once SWAP was fixed by
> commit 6e3761145a9b ("ARC: Fix CONFIG_SWAP").

I figured this - the patch was pushed to my for-curr yesterday for 4.19 !