Re: [PATCH 5.15 000/135] 5.15.51-rc1 review

Re: [PATCH 5.15 000/135] 5.15.51-rc1 review

[Bagas Sanjaya]

On Mon, Jun 27, 2022 at 01:20:07PM +0200, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 5.15.51 release.
> There are 135 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.
> 

Successfully cross-compiled for arm (multi_v7_defconfig, GCC 12.1.0,
armv7 with neon FPU) and arm64 (bcm2711_defconfig, GCC 12.1.0).

But I see a warning on arm64 build:

  CC [M]  drivers/staging/r8188eu/core/rtw_br_ext.o
  CC [M]  net/batman-adv/tvlv.o
In function '__nat25_add_pppoe_tag',
    inlined from 'nat25_db_handle' at drivers/staging/r8188eu/core/rtw_br_ext.c:520:11:
drivers/staging/r8188eu/core/rtw_br_ext.c:83:9: warning: 'memcpy' reading between 2052 and 9220 bytes from a region of size 40 [-Wstringop-overread]
   83 |         memcpy((unsigned char *)ph->tag, tag, data_len);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/staging/r8188eu/core/rtw_br_ext.c: In function 'nat25_db_handle':
drivers/staging/r8188eu/core/rtw_br_ext.c:489:63: note: source object 'tag_buf' of size 40
  489 |                                                 unsigned char tag_buf[40];
      |                                                               ^~~~~~~

Introduced by commit 15865124feed88 ("staging: r8188eu: introduce new core dir
for RTL8188eu driver").

Anyway,
Tested-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara

Re: [PATCH 5.15 000/135] 5.15.51-rc1 review

[Greg Kroah-Hartman]

On Tue, Jun 28, 2022 at 10:38:19AM +0700, Bagas Sanjaya wrote:
> On Mon, Jun 27, 2022 at 01:20:07PM +0200, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 5.15.51 release.
> > There are 135 patches in this series, all will be posted as a response
> > to this one.  If anyone has any issues with these being applied, please
> > let me know.
> > 
> 
> Successfully cross-compiled for arm (multi_v7_defconfig, GCC 12.1.0,
> armv7 with neon FPU) and arm64 (bcm2711_defconfig, GCC 12.1.0).
> 
> But I see a warning on arm64 build:
> 
>   CC [M]  drivers/staging/r8188eu/core/rtw_br_ext.o
>   CC [M]  net/batman-adv/tvlv.o
> In function '__nat25_add_pppoe_tag',
>     inlined from 'nat25_db_handle' at drivers/staging/r8188eu/core/rtw_br_ext.c:520:11:
> drivers/staging/r8188eu/core/rtw_br_ext.c:83:9: warning: 'memcpy' reading between 2052 and 9220 bytes from a region of size 40 [-Wstringop-overread]
>    83 |         memcpy((unsigned char *)ph->tag, tag, data_len);
>       |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/staging/r8188eu/core/rtw_br_ext.c: In function 'nat25_db_handle':
> drivers/staging/r8188eu/core/rtw_br_ext.c:489:63: note: source object 'tag_buf' of size 40
>   489 |                                                 unsigned char tag_buf[40];
>       |                                                               ^~~~~~~
> 
> Introduced by commit 15865124feed88 ("staging: r8188eu: introduce new core dir
> for RTL8188eu driver").

That is due to gcc-12 stuff.

thanks for testing.

thanks,

greg k-h