* [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED @ 2021-10-06 14:58 Lukas Bulwahn 2021-10-06 15:17 ` Arnd Bergmann 0 siblings, 1 reply; 11+ messages in thread From: Lukas Bulwahn @ 2021-10-06 14:58 UTC (permalink / raw) To: Arnd Bergmann, Luis Chamberlain, Palmer Dabbelt, linux-arch Cc: kernel-janitors, linux-kernel, Lukas Bulwahn Commit 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") introduces the config symbol GENERIC_LIB_DEVMEM_IS_ALLOWED, but then falsely refers to CONFIG_GENERIC_DEVMEM_IS_ALLOWED (note the missing LIB in the reference) in ./include/asm-generic/io.h. Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs: GENERIC_DEVMEM_IS_ALLOWED Referencing files: include/asm-generic/io.h Correct the name of the config to the intended one. Fixes: 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> --- include/asm-generic/io.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/asm-generic/io.h b/include/asm-generic/io.h index cc7338f9e0d1..6364174504d7 100644 --- a/include/asm-generic/io.h +++ b/include/asm-generic/io.h @@ -1125,7 +1125,7 @@ static inline void memcpy_toio(volatile void __iomem *addr, const void *buffer, } #endif -#ifndef CONFIG_GENERIC_DEVMEM_IS_ALLOWED +#ifndef CONFIG_GENERIC_LIB_DEVMEM_IS_ALLOWED extern int devmem_is_allowed(unsigned long pfn); #endif -- 2.26.2 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2021-10-06 14:58 [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED Lukas Bulwahn @ 2021-10-06 15:17 ` Arnd Bergmann 2021-10-06 16:52 ` Palmer Dabbelt 0 siblings, 1 reply; 11+ messages in thread From: Arnd Bergmann @ 2021-10-06 15:17 UTC (permalink / raw) To: Lukas Bulwahn Cc: Arnd Bergmann, Luis Chamberlain, Palmer Dabbelt, linux-arch, kernel-janitors, Linux Kernel Mailing List On Wed, Oct 6, 2021 at 5:00 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > Commit 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > introduces the config symbol GENERIC_LIB_DEVMEM_IS_ALLOWED, but then > falsely refers to CONFIG_GENERIC_DEVMEM_IS_ALLOWED (note the missing LIB > in the reference) in ./include/asm-generic/io.h. > > Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs: > > GENERIC_DEVMEM_IS_ALLOWED > Referencing files: include/asm-generic/io.h > > Correct the name of the config to the intended one. > > Fixes: 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> Acked-by: Arnd Bergmann <arnd@arndb.de> ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2021-10-06 15:17 ` Arnd Bergmann @ 2021-10-06 16:52 ` Palmer Dabbelt 2022-07-07 11:40 ` Lukas Bulwahn 0 siblings, 1 reply; 11+ messages in thread From: Palmer Dabbelt @ 2021-10-06 16:52 UTC (permalink / raw) To: Arnd Bergmann Cc: lukas.bulwahn, Arnd Bergmann, mcgrof, linux-arch, kernel-janitors, linux-kernel On Wed, 06 Oct 2021 08:17:38 PDT (-0700), Arnd Bergmann wrote: > On Wed, Oct 6, 2021 at 5:00 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: >> >> Commit 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") >> introduces the config symbol GENERIC_LIB_DEVMEM_IS_ALLOWED, but then >> falsely refers to CONFIG_GENERIC_DEVMEM_IS_ALLOWED (note the missing LIB >> in the reference) in ./include/asm-generic/io.h. >> >> Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs: >> >> GENERIC_DEVMEM_IS_ALLOWED >> Referencing files: include/asm-generic/io.h >> >> Correct the name of the config to the intended one. >> >> Fixes: 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") >> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> > > Acked-by: Arnd Bergmann <arnd@arndb.de> Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com> Acked-by: Palmer Dabbelt <palmerdabbelt@google.com> Thanks. I'm going to assume this is going in through some other tree, but IIUC I sent the buggy patch up so LMK if you're expecting it to go through mine. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2021-10-06 16:52 ` Palmer Dabbelt @ 2022-07-07 11:40 ` Lukas Bulwahn 2022-07-07 12:13 ` Arnd Bergmann 0 siblings, 1 reply; 11+ messages in thread From: Lukas Bulwahn @ 2022-07-07 11:40 UTC (permalink / raw) To: Palmer Dabbelt Cc: Arnd Bergmann, Luis Chamberlain, linux-arch, kernel-janitors, Linux Kernel Mailing List On Wed, Oct 6, 2021 at 6:52 PM Palmer Dabbelt <palmerdabbelt@google.com> wrote: > > On Wed, 06 Oct 2021 08:17:38 PDT (-0700), Arnd Bergmann wrote: > > On Wed, Oct 6, 2021 at 5:00 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > >> > >> Commit 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > >> introduces the config symbol GENERIC_LIB_DEVMEM_IS_ALLOWED, but then > >> falsely refers to CONFIG_GENERIC_DEVMEM_IS_ALLOWED (note the missing LIB > >> in the reference) in ./include/asm-generic/io.h. > >> > >> Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs: > >> > >> GENERIC_DEVMEM_IS_ALLOWED > >> Referencing files: include/asm-generic/io.h > >> > >> Correct the name of the config to the intended one. > >> > >> Fixes: 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > >> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> > > > > Acked-by: Arnd Bergmann <arnd@arndb.de> > > Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com> > Acked-by: Palmer Dabbelt <palmerdabbelt@google.com> > > Thanks. I'm going to assume this is going in through some other tree, > but IIUC I sent the buggy patch up so LMK if you're expecting it to go > through mine. Palmer, Arnd, the patch in this mail thread got lost and was not picked up yet. MAINTAINERS suggests that Arnd takes patches to include/asm-generic/, since commit 1527aab617af ("asm-generic: list Arnd as asm-generic maintainer") in 2009, but maybe the responsibility for those files has actually moved on to somebody (or nobody) else and we just did not record that yet in MAINTAINERS. Arnd, will you pick this patch and provide it further to Linus Torvalds? Otherwise, Palmer already suggested picking it up himself. Thanks and best regards, Lukas ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2022-07-07 11:40 ` Lukas Bulwahn @ 2022-07-07 12:13 ` Arnd Bergmann 2022-07-07 12:20 ` Conor.Dooley 0 siblings, 1 reply; 11+ messages in thread From: Arnd Bergmann @ 2022-07-07 12:13 UTC (permalink / raw) To: Lukas Bulwahn Cc: Palmer Dabbelt, Arnd Bergmann, Luis Chamberlain, linux-arch, kernel-janitors, Linux Kernel Mailing List On Thu, Jul 7, 2022 at 1:40 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > On Wed, Oct 6, 2021 at 6:52 PM Palmer Dabbelt <palmerdabbelt@google.com> wrote: > > > > On Wed, 06 Oct 2021 08:17:38 PDT (-0700), Arnd Bergmann wrote: > > > On Wed, Oct 6, 2021 at 5:00 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > >> > > >> Commit 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > > >> introduces the config symbol GENERIC_LIB_DEVMEM_IS_ALLOWED, but then > > >> falsely refers to CONFIG_GENERIC_DEVMEM_IS_ALLOWED (note the missing LIB > > >> in the reference) in ./include/asm-generic/io.h. > > >> > > >> Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs: > > >> > > >> GENERIC_DEVMEM_IS_ALLOWED > > >> Referencing files: include/asm-generic/io.h > > >> > > >> Correct the name of the config to the intended one. > > >> > > >> Fixes: 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > > >> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> > > > > > > Acked-by: Arnd Bergmann <arnd@arndb.de> > > > > Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com> > > Acked-by: Palmer Dabbelt <palmerdabbelt@google.com> > > > > Thanks. I'm going to assume this is going in through some other tree, > > but IIUC I sent the buggy patch up so LMK if you're expecting it to go > > through mine. > > Palmer, Arnd, > > the patch in this mail thread got lost and was not picked up yet. > > MAINTAINERS suggests that Arnd takes patches to include/asm-generic/, > since commit 1527aab617af ("asm-generic: list Arnd as asm-generic > maintainer") in 2009, but maybe the responsibility for those files has > actually moved on to somebody (or nobody) else and we just did not > record that yet in MAINTAINERS. > > Arnd, will you pick this patch and provide it further to Linus Torvalds? > > Otherwise, Palmer already suggested picking it up himself. > I've applied it to the asm-generic tree and can send it as a bugfix pull request. I don't have any other fixer for that branch at the moment, so if Palmer has other fixes for the riscv tree already, it would save me making a pull request if he picks it up there. Arnd ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2022-07-07 12:13 ` Arnd Bergmann @ 2022-07-07 12:20 ` Conor.Dooley 2022-07-07 13:07 ` Arnd Bergmann 0 siblings, 1 reply; 11+ messages in thread From: Conor.Dooley @ 2022-07-07 12:20 UTC (permalink / raw) To: arnd, lukas.bulwahn, palmer Cc: palmerdabbelt, mcgrof, linux-arch, kernel-janitors, linux-kernel +CC Palmer's current email (idk if his google one directs, but JIC) On 07/07/2022 13:13, Arnd Bergmann wrote: > On Thu, Jul 7, 2022 at 1:40 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: >> >> On Wed, Oct 6, 2021 at 6:52 PM Palmer Dabbelt <palmerdabbelt@google.com> wrote: >>> >>> On Wed, 06 Oct 2021 08:17:38 PDT (-0700), Arnd Bergmann wrote: >>>> On Wed, Oct 6, 2021 at 5:00 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: >>>>> >>>>> Commit 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") >>>>> introduces the config symbol GENERIC_LIB_DEVMEM_IS_ALLOWED, but then >>>>> falsely refers to CONFIG_GENERIC_DEVMEM_IS_ALLOWED (note the missing LIB >>>>> in the reference) in ./include/asm-generic/io.h. >>>>> >>>>> Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs: >>>>> >>>>> GENERIC_DEVMEM_IS_ALLOWED >>>>> Referencing files: include/asm-generic/io.h >>>>> >>>>> Correct the name of the config to the intended one. >>>>> >>>>> Fixes: 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") >>>>> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> >>>> >>>> Acked-by: Arnd Bergmann <arnd@arndb.de> >>> >>> Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com> >>> Acked-by: Palmer Dabbelt <palmerdabbelt@google.com> >>> >>> Thanks. I'm going to assume this is going in through some other tree, >>> but IIUC I sent the buggy patch up so LMK if you're expecting it to go >>> through mine. >> >> Palmer, Arnd, >> >> the patch in this mail thread got lost and was not picked up yet. >> >> MAINTAINERS suggests that Arnd takes patches to include/asm-generic/, >> since commit 1527aab617af ("asm-generic: list Arnd as asm-generic >> maintainer") in 2009, but maybe the responsibility for those files has >> actually moved on to somebody (or nobody) else and we just did not >> record that yet in MAINTAINERS. >> >> Arnd, will you pick this patch and provide it further to Linus Torvalds? >> >> Otherwise, Palmer already suggested picking it up himself. >> > > I've applied it to the asm-generic tree and can send it as a bugfix > pull request. I don't have any other fixer for that branch at the moment, > so if Palmer has other fixes for the riscv tree already, it would > save me making a pull request if he picks it up there. > > Arnd ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2022-07-07 12:20 ` Conor.Dooley @ 2022-07-07 13:07 ` Arnd Bergmann 2022-07-07 14:41 ` Lukas Bulwahn 0 siblings, 1 reply; 11+ messages in thread From: Arnd Bergmann @ 2022-07-07 13:07 UTC (permalink / raw) To: Conor Dooley Cc: Arnd Bergmann, Lukas Bulwahn, Palmer Dabbelt, Palmer Dabbelt, Luis R. Rodriguez, linux-arch, kernel-janitors, Linux Kernel Mailing List On Thu, Jul 7, 2022 at 2:20 PM <Conor.Dooley@microchip.com> wrote: > On 07/07/2022 13:13, Arnd Bergmann wrote: > > On Thu, Jul 7, 2022 at 1:40 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > >> > >> On Wed, Oct 6, 2021 at 6:52 PM Palmer Dabbelt <palmerdabbelt@google.com> wrote: > >>> > >>> On Wed, 06 Oct 2021 08:17:38 PDT (-0700), Arnd Bergmann wrote: > >>>> On Wed, Oct 6, 2021 at 5:00 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > >>>>> > >>>>> Commit 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > >>>>> introduces the config symbol GENERIC_LIB_DEVMEM_IS_ALLOWED, but then > >>>>> falsely refers to CONFIG_GENERIC_DEVMEM_IS_ALLOWED (note the missing LIB > >>>>> in the reference) in ./include/asm-generic/io.h. > >>>>> > >>>>> Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs: > >>>>> > >>>>> GENERIC_DEVMEM_IS_ALLOWED > >>>>> Referencing files: include/asm-generic/io.h > >>>>> > >>>>> Correct the name of the config to the intended one. > >>>>> > >>>>> Fixes: 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > >>>>> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> > >>>> > >>>> Acked-by: Arnd Bergmann <arnd@arndb.de> > >>> > >>> Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com> > >>> Acked-by: Palmer Dabbelt <palmerdabbelt@google.com> > >>> > >>> Thanks. I'm going to assume this is going in through some other tree, > >>> but IIUC I sent the buggy patch up so LMK if you're expecting it to go > >>> through mine. > >> > >> Palmer, Arnd, > >> > >> the patch in this mail thread got lost and was not picked up yet. > >> > >> MAINTAINERS suggests that Arnd takes patches to include/asm-generic/, > >> since commit 1527aab617af ("asm-generic: list Arnd as asm-generic > >> maintainer") in 2009, but maybe the responsibility for those files has > >> actually moved on to somebody (or nobody) else and we just did not > >> record that yet in MAINTAINERS. > >> > >> Arnd, will you pick this patch and provide it further to Linus Torvalds? > >> > >> Otherwise, Palmer already suggested picking it up himself. > >> > > > > I've applied it to the asm-generic tree and can send it as a bugfix > > pull request. I don't have any other fixer for that branch at the moment, > > so if Palmer has other fixes for the riscv tree already, it would > > save me making a pull request if he picks it up there. lkft just found a build failure: https://gitlab.com/Linaro/lkft/users/arnd.bergmann/asm-generic/-/jobs/2691154818 I have not investigated what went wrong, but it does look like an actual regression, so I'll wait for Lukas to follow up with a new version of the patch. Arnd ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2022-07-07 13:07 ` Arnd Bergmann @ 2022-07-07 14:41 ` Lukas Bulwahn 2022-07-20 10:50 ` Arnd Bergmann 0 siblings, 1 reply; 11+ messages in thread From: Lukas Bulwahn @ 2022-07-07 14:41 UTC (permalink / raw) To: Arnd Bergmann Cc: Conor Dooley, Palmer Dabbelt, Palmer Dabbelt, Luis R. Rodriguez, linux-arch, kernel-janitors, Linux Kernel Mailing List On Thu, Jul 7, 2022 at 3:07 PM Arnd Bergmann <arnd@arndb.de> wrote: > > On Thu, Jul 7, 2022 at 2:20 PM <Conor.Dooley@microchip.com> wrote: > > On 07/07/2022 13:13, Arnd Bergmann wrote: > > > On Thu, Jul 7, 2022 at 1:40 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > >> > > >> On Wed, Oct 6, 2021 at 6:52 PM Palmer Dabbelt <palmerdabbelt@google.com> wrote: > > >>> > > >>> On Wed, 06 Oct 2021 08:17:38 PDT (-0700), Arnd Bergmann wrote: > > >>>> On Wed, Oct 6, 2021 at 5:00 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > >>>>> > > >>>>> Commit 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > > >>>>> introduces the config symbol GENERIC_LIB_DEVMEM_IS_ALLOWED, but then > > >>>>> falsely refers to CONFIG_GENERIC_DEVMEM_IS_ALLOWED (note the missing LIB > > >>>>> in the reference) in ./include/asm-generic/io.h. > > >>>>> > > >>>>> Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs: > > >>>>> > > >>>>> GENERIC_DEVMEM_IS_ALLOWED > > >>>>> Referencing files: include/asm-generic/io.h > > >>>>> > > >>>>> Correct the name of the config to the intended one. > > >>>>> > > >>>>> Fixes: 527701eda5f1 ("lib: Add a generic version of devmem_is_allowed()") > > >>>>> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> > > >>>> > > >>>> Acked-by: Arnd Bergmann <arnd@arndb.de> > > >>> > > >>> Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com> > > >>> Acked-by: Palmer Dabbelt <palmerdabbelt@google.com> > > >>> > > >>> Thanks. I'm going to assume this is going in through some other tree, > > >>> but IIUC I sent the buggy patch up so LMK if you're expecting it to go > > >>> through mine. > > >> > > >> Palmer, Arnd, > > >> > > >> the patch in this mail thread got lost and was not picked up yet. > > >> > > >> MAINTAINERS suggests that Arnd takes patches to include/asm-generic/, > > >> since commit 1527aab617af ("asm-generic: list Arnd as asm-generic > > >> maintainer") in 2009, but maybe the responsibility for those files has > > >> actually moved on to somebody (or nobody) else and we just did not > > >> record that yet in MAINTAINERS. > > >> > > >> Arnd, will you pick this patch and provide it further to Linus Torvalds? > > >> > > >> Otherwise, Palmer already suggested picking it up himself. > > >> > > > > > > I've applied it to the asm-generic tree and can send it as a bugfix > > > pull request. I don't have any other fixer for that branch at the moment, > > > so if Palmer has other fixes for the riscv tree already, it would > > > save me making a pull request if he picks it up there. > > lkft just found a build failure: > > https://gitlab.com/Linaro/lkft/users/arnd.bergmann/asm-generic/-/jobs/2691154818 > > I have not investigated what went wrong, but it does look like an actual > regression, so I'll wait for Lukas to follow up with a new version of the patch. Thanks for your testing. I will look into it. Probably it is due to some more rigor during builds (-Werror and new warning types in the default build) since I proposed the patch in October 2021. That should be easy to fix, but let us see. I will send a PATCH v2 soon. Lukas ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2022-07-07 14:41 ` Lukas Bulwahn @ 2022-07-20 10:50 ` Arnd Bergmann 2022-07-21 6:52 ` Lukas Bulwahn 0 siblings, 1 reply; 11+ messages in thread From: Arnd Bergmann @ 2022-07-20 10:50 UTC (permalink / raw) To: Lukas Bulwahn Cc: Arnd Bergmann, Conor Dooley, Palmer Dabbelt, Palmer Dabbelt, Luis R. Rodriguez, linux-arch, kernel-janitors, Linux Kernel Mailing List On Thu, Jul 7, 2022 at 4:41 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > On Thu, Jul 7, 2022 at 3:07 PM Arnd Bergmann <arnd@arndb.de> wrote: > > On Thu, Jul 7, 2022 at 2:20 PM <Conor.Dooley@microchip.com> wrote: > > lkft just found a build failure: > > > > https://gitlab.com/Linaro/lkft/users/arnd.bergmann/asm-generic/-/jobs/2691154818 > > > > I have not investigated what went wrong, but it does look like an actual > > regression, so I'll wait for Lukas to follow up with a new version of the patch. > > Thanks for your testing. I will look into it. Probably it is due to > some more rigor during builds (-Werror and new warning types in the > default build) since I proposed the patch in October 2021. That should > be easy to fix, but let us see. I will send a PATCH v2 soon. Any update on this? I have another bugfix for asm-generic now and was planning to send a pull request with both. If you don't have the updated patch ready yet, this will have to go into 5.21 instead. Arnd ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2022-07-20 10:50 ` Arnd Bergmann @ 2022-07-21 6:52 ` Lukas Bulwahn 2022-07-21 20:41 ` Lukas Bulwahn 0 siblings, 1 reply; 11+ messages in thread From: Lukas Bulwahn @ 2022-07-21 6:52 UTC (permalink / raw) To: Arnd Bergmann Cc: Conor Dooley, Palmer Dabbelt, Palmer Dabbelt, Luis R. Rodriguez, linux-arch, kernel-janitors, Linux Kernel Mailing List On Wed, Jul 20, 2022 at 12:50 PM Arnd Bergmann <arnd@arndb.de> wrote: > > On Thu, Jul 7, 2022 at 4:41 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > On Thu, Jul 7, 2022 at 3:07 PM Arnd Bergmann <arnd@arndb.de> wrote: > > > On Thu, Jul 7, 2022 at 2:20 PM <Conor.Dooley@microchip.com> wrote: > > > > lkft just found a build failure: > > > > > > https://gitlab.com/Linaro/lkft/users/arnd.bergmann/asm-generic/-/jobs/2691154818 > > > > > > I have not investigated what went wrong, but it does look like an actual > > > regression, so I'll wait for Lukas to follow up with a new version of the patch. > > > > Thanks for your testing. I will look into it. Probably it is due to > > some more rigor during builds (-Werror and new warning types in the > > default build) since I proposed the patch in October 2021. That should > > be easy to fix, but let us see. I will send a PATCH v2 soon. > > Any update on this? I have another bugfix for asm-generic now and was planning > to send a pull request with both. If you don't have the updated patch > ready yet, this > will have to go into 5.21 instead. > It is still on my TODO list for revisiting, but I had other work on patches taking me longer than originally expected. I now moved this patch revisiting to the top of my TODO list; so, I will certainly look into this today. So far, I have not set up an arm64 build on my local machine and have not used tuxbuild (which should simplify all this setup)---the typical challenges for a "part-time kernel contributor/janitor"... Arnd, I will certainly let you know by this evening (European time zone) how far I got. If that is already too late, it is also perfectly fine if this goes in 5.21 instead, but I will try my best to make it 5.20 material. Thanks for the patience so far. Best regards, Lukas > Arnd ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED 2022-07-21 6:52 ` Lukas Bulwahn @ 2022-07-21 20:41 ` Lukas Bulwahn 0 siblings, 0 replies; 11+ messages in thread From: Lukas Bulwahn @ 2022-07-21 20:41 UTC (permalink / raw) To: Arnd Bergmann Cc: Conor Dooley, Palmer Dabbelt, Luis R. Rodriguez, linux-arch, kernel-janitors, Linux Kernel Mailing List On Thu, Jul 21, 2022 at 8:52 AM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > On Wed, Jul 20, 2022 at 12:50 PM Arnd Bergmann <arnd@arndb.de> wrote: > > > > On Thu, Jul 7, 2022 at 4:41 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > > On Thu, Jul 7, 2022 at 3:07 PM Arnd Bergmann <arnd@arndb.de> wrote: > > > > On Thu, Jul 7, 2022 at 2:20 PM <Conor.Dooley@microchip.com> wrote: > > > > > > lkft just found a build failure: > > > > > > > > https://gitlab.com/Linaro/lkft/users/arnd.bergmann/asm-generic/-/jobs/2691154818 > > > > > > > > I have not investigated what went wrong, but it does look like an actual > > > > regression, so I'll wait for Lukas to follow up with a new version of the patch. > > > > > > Thanks for your testing. I will look into it. Probably it is due to > > > some more rigor during builds (-Werror and new warning types in the > > > default build) since I proposed the patch in October 2021. That should > > > be easy to fix, but let us see. I will send a PATCH v2 soon. > > > > Any update on this? I have another bugfix for asm-generic now and was planning > > to send a pull request with both. If you don't have the updated patch > > ready yet, this > > will have to go into 5.21 instead. > > > > It is still on my TODO list for revisiting, but I had other work on > patches taking me longer than originally expected. I now moved this > patch revisiting to the top of my TODO list; so, I will certainly look > into this today. So far, I have not set up an arm64 build on my local > machine and have not used tuxbuild (which should simplify all this > setup)---the typical challenges for a "part-time kernel > contributor/janitor"... > > Arnd, I will certainly let you know by this evening (European time > zone) how far I got. If that is already too late, it is also perfectly > fine if this goes in 5.21 instead, but I will try my best to make it > 5.20 material. > Arnd, I can reproduce the error from your build system. I think I have an idea what the correct patch would actually be: - either make the function declaration completely unconditional, - or put the function declaration under #ifdef and not #ifndef. I guess the first should actually lead to some compiler warning (at W=2 or so) having a function declared multiple times, for some architectures. And hence, the second option is a better one. This is all educated guessing so far, so to confirm, I need to build the kernel with W=2 on the defconfig all architectures before and after patch application and see if I am right. This is going to take some build time on my local (home) machine. So, I hope that I can spend some time tomorrow on just kicking off builds and looking at diffs and then send out a working patch v2. Lukas ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2022-07-21 20:41 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-10-06 14:58 [PATCH] asm-generic: correct reference to GENERIC_LIB_DEVMEM_IS_ALLOWED Lukas Bulwahn 2021-10-06 15:17 ` Arnd Bergmann 2021-10-06 16:52 ` Palmer Dabbelt 2022-07-07 11:40 ` Lukas Bulwahn 2022-07-07 12:13 ` Arnd Bergmann 2022-07-07 12:20 ` Conor.Dooley 2022-07-07 13:07 ` Arnd Bergmann 2022-07-07 14:41 ` Lukas Bulwahn 2022-07-20 10:50 ` Arnd Bergmann 2022-07-21 6:52 ` Lukas Bulwahn 2022-07-21 20:41 ` Lukas Bulwahn
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.