Re: [PATCH] Documentation/features: refresh powerpc arch support files

On 2020-08-10 at 17:09:51 +0200, Christophe Leroy [off-list ref] wrote:

Le 10/08/2020 à 12:09, Tobias Klauser a écrit :

quoted

Support for these was added by commit aa65ff6b18e0 ("powerpc/64s:
Implement queued spinlocks and rwlocks").

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
---
  Documentation/features/locking/queued-rwlocks/arch-support.txt  | 2 +-
  .../features/locking/queued-spinlocks/arch-support.txt          | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/features/locking/queued-rwlocks/arch-support.txt b/Documentation/features/locking/queued-rwlocks/arch-support.txt
index 5c6bcfcf8e1f..4dd5e554873f 100644
--- a/Documentation/features/locking/queued-rwlocks/arch-support.txt
+++ b/Documentation/features/locking/queued-rwlocks/arch-support.txt

@@ -22,7 +22,7 @@
      |       nios2: | TODO |
      |    openrisc: |  ok  |
      |      parisc: | TODO |
-    |     powerpc: | TODO |
+    |     powerpc: |  ok  |

In your commit log you are refering to a commit titled "powerpc/64s:"

Are you sure it is now OK for all powerpc, not only for book3s/64 as
suggested by yout text ?

The change was generated by running
Documentation/features/scripts/features-refresh.sh
Sorry, I should have mentioned this in the commit message. I noticed the
updated features for powerpc after updating the RISC-V supported
features [1].

[1] https://lore.kernel.org/linux-riscv/20200810095000.32092-1-tklauser@distanz.ch/T/#u (local)

AFAIK, the features-refresh.sh script has no way of distinguishing
between different types of an architecture. It just checks for the
respective Kconfig symbols listed in the
Documentation/features/**/arch-support.txt files in all arch/**/Kconfig
files and updates the feature to "ok" if it finds the Kconfig symbol.