Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <d7c250e0-a3a8-51d0-a761-7a81305b39b2@infradead.org>
Date:   Tue, 4 Apr 2023 17:01:46 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.9.0
Subject: Re: Build regressions/improvements in v6.3-rc4 (parport_pc)
Content-Language: en-US
From:   Randy Dunlap <rdunlap@infradead.org>
To:     Sudip Mukherjee <sudipm.mukherjee@gmail.com>,
        Geert Uytterhoeven <geert@linux-m68k.org>
Cc:     linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org,
        sparclinux@vger.kernel.org, "Maciej W. Rozycki" <macro@orcam.me.uk>
References: <CAHk-=whcaHLNpb7Mu_QX7ABwPgyRyfW-V8=v4Mv0S22fpjY4JQ@mail.gmail.com>
 <20230327072641.3591802-1-geert@linux-m68k.org>
 <eb55ca34-ca71-ed19-dae2-6e5e87c170@linux-m68k.org>
 <CADVatmNHYar6tR3r9q42vZR5fsqZeeDb8LEmmBcrrOwhA7HpmQ@mail.gmail.com>
 <c3718ac3-3dd1-2273-68ef-3d2923778ab7@infradead.org>
In-Reply-To: <c3718ac3-3dd1-2273-68ef-3d2923778ab7@infradead.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 4/4/23 15:40, Randy Dunlap wrote:
> 
> 
> On 4/4/23 14:16, Sudip Mukherjee wrote:
>> On Mon, 27 Mar 2023 at 08:29, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>>>
>>> On Mon, 27 Mar 2023, Geert Uytterhoeven wrote:
>>>> JFYI, when comparing v6.3-rc4[1] to v6.3-rc3[3], the summaries are:
>>>>  - build errors: +9/-1
>>>
>>>    + /kisskb/src/drivers/net/wireless/cisco/airo.c: error: 'status_rid.currentXmitRate' is used uninitialized [-Werror=uninitialized]:  => 6163:45
>>>
>>> sh4-gcc11/sh-allmodconfig
>>> seen before
>>>
>>>    + error: modpost: "ebus_dma_enable" [drivers/parport/parport_pc.ko] undefined!:  => N/A
>>>    + error: modpost: "ebus_dma_irq_enable" [drivers/parport/parport_pc.ko] undefined!:  => N/A
>>>    + error: modpost: "ebus_dma_prepare" [drivers/parport/parport_pc.ko] undefined!:  => N/A
>>>    + error: modpost: "ebus_dma_register" [drivers/parport/parport_pc.ko] undefined!:  => N/A
>>>    + error: modpost: "ebus_dma_request" [drivers/parport/parport_pc.ko] undefined!:  => N/A
>>>    + error: modpost: "ebus_dma_residue" [drivers/parport/parport_pc.ko] undefined!:  => N/A
>>>    + error: modpost: "ebus_dma_unregister" [drivers/parport/parport_pc.ko] undefined!:  => N/A
>>>    + error: modpost: "ns87303_lock" [drivers/parport/parport_pc.ko] undefined!:  => N/A
>>>
>>> sparc64-gcc11/sparc-allmodconfig
>>> seen before
>>
>> Tried sparc64 allmodconfig with gcc-11 and did not see the parport
>> errors with 6ab608fe852b ("Merge tag 'for-6.3-rc4-tag' of
>> git://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux").
>> Is it still being seen?
>>
> 
> This is reported at  http://kisskb.ellerman.id.au/kisskb/buildresult/14905002/
> 
> I still see it on sparc32 using gcc-12.2.0.
> 
> The compiler info vs. the sparc-allmodconfig is confusing IMO.
> I am compiling with sparc-linux, not sparc64-linux.
> 
> The .config file says:
> # CONFIG_64BIT is not set
> CONFIG_SPARC=y
> CONFIG_SPARC32=y

It looks to me like we have 2 choices (though there may be others
that are more complicated):

(1) restrict PARPORT_PC to !SPARC32
This fixes the build errors.

(2) We could build ebus.o and some (new) binary that includes
ns87303_lock for SPARC32. They are currently only built for SPARC64.

I prefer option (1) unless we need to support PARPORT_PC on
Sparc32 (which I have no idea about).  Patch for it is below for inspection.

---
From: Randy Dunlap <rdunlap@infradead.org>
Subject: [PATCH] parport_pc: don't allow driver for SPARC32

arch/sparc/kernel/ebus.o is only built for SPARC64.
ns87303_lock is only built for SPARC64.
Also, arch/sparc/include/asm/parport.h says that it is for sparc64.

Therefore don't allow PARPORT_PC to be built for SPARC32.

Fixes these build errors on SPARC32:

ERROR: modpost: "ebus_dma_irq_enable" [drivers/parport/parport_pc.ko] undefined!
ERROR: modpost: "ebus_dma_unregister" [drivers/parport/parport_pc.ko] undefined!
ERROR: modpost: "ebus_dma_register" [drivers/parport/parport_pc.ko] undefined!
ERROR: modpost: "ns87303_lock" [drivers/parport/parport_pc.ko] undefined!
ERROR: modpost: "ebus_dma_enable" [drivers/parport/parport_pc.ko] undefined!
ERROR: modpost: "ebus_dma_prepare" [drivers/parport/parport_pc.ko] undefined!
ERROR: modpost: "ebus_dma_request" [drivers/parport/parport_pc.ko] undefined!
ERROR: modpost: "ebus_dma_residue" [drivers/parport/parport_pc.ko] undefined!

Fixes: 66bcd06099bb ("parport_pc: Also enable driver for PCI systems")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Cc: Maciej W. Rozycki <macro@orcam.me.uk>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: sparclinux@vger.kernel.org
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: linux-parport@lists.infradead.org
---
 drivers/parport/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff -- a/drivers/parport/Kconfig b/drivers/parport/Kconfig
--- a/drivers/parport/Kconfig
+++ b/drivers/parport/Kconfig
@@ -42,7 +42,7 @@ if PARPORT
 
 config PARPORT_PC
 	tristate "PC-style hardware"
-	depends on ARCH_MIGHT_HAVE_PC_PARPORT || (PCI && !S390)
+	depends on ARCH_MIGHT_HAVE_PC_PARPORT || (PCI && !S390 && !SPARC32)
 	help
 	  You should say Y here if you have a PC-style parallel port. All
 	  IBM PC compatible computers and some Alphas have PC-style