mbox series

[0/3] ntb_hw_switchtec enable 256 LUTs

Message ID 20250213225319.1965-1-Maciej.Grochowski@sony.com (mailing list archive)
Headers show
Series ntb_hw_switchtec enable 256 LUTs | expand

Message

Maciej Grochowski Feb. 13, 2025, 10:53 p.m. UTC
Microchip NTB devices support up to 512 LUTs shared across all NT
partitions. This short patch series increases MAX_MWS to 256 and also
address issues when the number of mw is equal to 0 or MAX_MWS

Maciej Grochowski (3):
  ntb: ntb_hw_switchtec: Fix shift-out-of-bounds for 0 mw lut
  ntb: ntb_hw_switchtec: Fix array-index-out-of-bounds access
  ntb: ntb_hw_switchtec: Increase MAX_MWS limit to 256

 drivers/ntb/hw/mscc/ntb_hw_switchtec.c | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

Comments

Dave Jiang Feb. 13, 2025, 10:58 p.m. UTC | #1
On 2/13/25 3:53 PM, Maciej Grochowski wrote:
> Microchip NTB devices support up to 512 LUTs shared across all NT
> partitions. This short patch series increases MAX_MWS to 256 and also
> address issues when the number of mw is equal to 0 or MAX_MWS
> 
> Maciej Grochowski (3):
>   ntb: ntb_hw_switchtec: Fix shift-out-of-bounds for 0 mw lut
>   ntb: ntb_hw_switchtec: Fix array-index-out-of-bounds access
>   ntb: ntb_hw_switchtec: Increase MAX_MWS limit to 256
> 
>  drivers/ntb/hw/mscc/ntb_hw_switchtec.c | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)
> 
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
for the series
Logan Gunthorpe Feb. 14, 2025, 12:49 a.m. UTC | #2
On 2025-02-13 15:53, Maciej Grochowski wrote:
> Microchip NTB devices support up to 512 LUTs shared across all NT
> partitions. This short patch series increases MAX_MWS to 256 and also
> address issues when the number of mw is equal to 0 or MAX_MWS
> 
> Maciej Grochowski (3):
>   ntb: ntb_hw_switchtec: Fix shift-out-of-bounds for 0 mw lut
>   ntb: ntb_hw_switchtec: Fix array-index-out-of-bounds access
>   ntb: ntb_hw_switchtec: Increase MAX_MWS limit to 256
> 
>  drivers/ntb/hw/mscc/ntb_hw_switchtec.c | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)
> 

All these patches look good to me. Thanks!

Reviewed-by: Logan Gunthorpe <logang@deltatee.com>