Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1210040 > unrolled thread
| Started by | Laurent Dufour <ldufour@linux.vnet.ibm.com> |
|---|---|
| First post | 2015-08-20 00:00 +0200 |
| Last post | 2015-08-20 06:30 +0200 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH] powerpc/hvsi: Fix endianness issues in the HVSI driver Laurent Dufour <ldufour@linux.vnet.ibm.com> - 2015-08-20 00:00 +0200
Re: [PATCH] powerpc/hvsi: Fix endianness issues in the HVSI driver Michael Ellerman <mpe@ellerman.id.au> - 2015-08-20 03:50 +0200
Re: [PATCH] powerpc/hvsi: Fix endianness issues in the HVSI driver Michael Ellerman <mpe@ellerman.id.au> - 2015-08-20 06:30 +0200
| From | Laurent Dufour <ldufour@linux.vnet.ibm.com> |
|---|---|
| Date | 2015-08-20 00:00 +0200 |
| Subject | Re: [PATCH] powerpc/hvsi: Fix endianness issues in the HVSI driver |
| Message-ID | <pZjyN-1GO-3@gated-at.bofh.it> |
On 04/08/2015 02:51, Michael Ellerman wrote: > On Fri, 2015-07-31 at 11:29 +0200, Laurent Dufour wrote: >> This patch fixes several endianness issues detected when running the HVSI >> driver in little endian mode. >> >> These issues are raised in little endian mode because the data exchanged in >> memory between the kernel and the hypervisor has to be in big endian >> format. > > Can you include the sparse output before and after? Hi Michael, Here is the output message displayed on the console when the bug occurred: [ 0.000517] irq: (null) didn't like hwirq-0x1000a00 to VIRQ16 mapping (rc=-22) [ 0.000578] hvsi_console_init: couldn't create irq mapping for 0x1000a00 With the patch is applied, the hvsi driver is initializing correctly and no message is displayed, except the one saying the number of device the hvsi driver has configured. For instance: [ 1.535783] HVSI: registered 1 devices Cheers, Laurent. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Michael Ellerman <mpe@ellerman.id.au> |
|---|---|
| Date | 2015-08-20 03:50 +0200 |
| Message-ID | <pZn9o-6St-7@gated-at.bofh.it> |
| In reply to | #1210040 |
On Wed, 2015-08-19 at 23:53 +0200, Laurent Dufour wrote: > On 04/08/2015 02:51, Michael Ellerman wrote: > > On Fri, 2015-07-31 at 11:29 +0200, Laurent Dufour wrote: > >> This patch fixes several endianness issues detected when running the HVSI > >> driver in little endian mode. > >> > >> These issues are raised in little endian mode because the data exchanged in > >> memory between the kernel and the hypervisor has to be in big endian > >> format. > > > > Can you include the sparse output before and after? > > Hi Michael, > > Here is the output message displayed on the console when the bug occurred: > > [ 0.000517] irq: (null) didn't like hwirq-0x1000a00 to VIRQ16 mapping (rc=-22) > [ 0.000578] hvsi_console_init: couldn't create irq mapping for 0x1000a00 > > With the patch is applied, the hvsi driver is initializing correctly and > no message is displayed, except the one saying the number of device the > hvsi driver has configured. For instance: > [ 1.535783] HVSI: registered 1 devices OK that's also good. I was talking about the output from sparse endian checking, as the driver seems to already be endian annotated. ie, before: $ make C=2 CF=-D__CHECK_ENDIAN__ ... drivers/tty/hvc/hvsi.c:245:36: warning: restricted __be32 degrades to integer drivers/tty/hvc/hvsi.c:243:23: warning: restricted __be16 degrades to integer drivers/tty/hvc/hvsi.c:243:23: warning: restricted __be16 degrades to integer drivers/tty/hvc/hvsi.c:277:36: warning: restricted __be32 degrades to integer drivers/tty/hvc/hvsi.c:279:36: warning: restricted __be32 degrades to integer drivers/tty/hvc/hvsi.c:298:26: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:298:26: expected restricted __be16 [assigned] [usertype] seqno drivers/tty/hvc/hvsi.c:298:26: got int drivers/tty/hvc/hvsi.c:299:21: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:299:21: expected restricted __be16 [assigned] [usertype] verb drivers/tty/hvc/hvsi.c:299:21: got int drivers/tty/hvc/hvsi.c:301:28: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:301:28: expected restricted __be16 [assigned] [usertype] query_seqno drivers/tty/hvc/hvsi.c:301:28: got int drivers/tty/hvc/hvsi.c:322:60: warning: incorrect type in argument 2 (different base types) drivers/tty/hvc/hvsi.c:322:60: expected unsigned short [unsigned] [usertype] query_seqno drivers/tty/hvc/hvsi.c:322:60: got restricted __be16 [usertype] seqno drivers/tty/hvc/hvsi.c:558:26: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:558:26: expected restricted __be16 [assigned] [usertype] seqno drivers/tty/hvc/hvsi.c:558:26: got int drivers/tty/hvc/hvsi.c:559:21: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:559:21: expected restricted __be16 [assigned] [usertype] verb drivers/tty/hvc/hvsi.c:559:21: got unsigned short [unsigned] [usertype] verb drivers/tty/hvc/hvsi.c:600:26: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:600:26: expected restricted __be16 [assigned] [usertype] seqno drivers/tty/hvc/hvsi.c:600:26: got int drivers/tty/hvc/hvsi.c:602:21: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:602:21: expected restricted __be16 [assigned] [usertype] verb drivers/tty/hvc/hvsi.c:602:21: got int drivers/tty/hvc/hvsi.c:603:21: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:603:21: expected restricted __be32 [assigned] [usertype] mask drivers/tty/hvc/hvsi.c:603:21: got int drivers/tty/hvc/hvsi.c:606:29: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:606:29: expected restricted __be32 [assigned] [usertype] word drivers/tty/hvc/hvsi.c:606:29: got int drivers/tty/hvc/hvsi.c:683:26: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:683:26: expected restricted __be16 [assigned] [usertype] seqno drivers/tty/hvc/hvsi.c:683:26: got int drivers/tty/hvc/hvsi.c:700:26: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:700:26: expected restricted __be16 [assigned] [usertype] seqno drivers/tty/hvc/hvsi.c:700:26: got int drivers/tty/hvc/hvsi.c:702:21: warning: incorrect type in assignment (different base types) drivers/tty/hvc/hvsi.c:702:21: expected restricted __be16 [assigned] [usertype] verb drivers/tty/hvc/hvsi.c:702:21: got int ... And with your patch applied there are no warnings from hvsi.c! So it seems you fixed all the issues, or at least all the issues we can detect with sparse. So I'll merge this as-is with an updated changelog. cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Michael Ellerman <mpe@ellerman.id.au> |
|---|---|
| Date | 2015-08-20 06:30 +0200 |
| Message-ID | <pZpEd-2d1-3@gated-at.bofh.it> |
| In reply to | #1210112 |
On Thu, 2015-08-20 at 11:40 +1000, Michael Ellerman wrote: > On Wed, 2015-08-19 at 23:53 +0200, Laurent Dufour wrote: > > On 04/08/2015 02:51, Michael Ellerman wrote: > > > On Fri, 2015-07-31 at 11:29 +0200, Laurent Dufour wrote: > > >> This patch fixes several endianness issues detected when running the HVSI > > >> driver in little endian mode. > > >> > > >> These issues are raised in little endian mode because the data exchanged in > > >> memory between the kernel and the hypervisor has to be in big endian > > >> format. ... > > So it seems you fixed all the issues, or at least all the issues we can detect > with sparse. So I'll merge this as-is with an updated changelog. Unless Greg wants to merge it? But I'll assume he's not bothered unless he says otherwise as it's a powerpc only driver. cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web