Re: [PATCH 0/1] hw: aspeed_gpio: Fix GPIO array indexing

2021-09-29 Thread Peter Delevoryas
> On Sep 27, 2021, at 8:43 PM, p...@fb.com wrote: > > From: Peter Delevoryas > > Hey everyone, > > I think there might be a bug in aspeed_gpio_update, where it's selecting > a GPIO IRQ to update. The indexing that maps from GPIO pin to IRQ leads > to an out-of-bounds array access and a segfau

[PATCH 0/1] hw: aspeed_gpio: Fix GPIO array indexing

2021-09-27 Thread pdel
From: Peter Delevoryas Hey everyone, I think there might be a bug in aspeed_gpio_update, where it's selecting a GPIO IRQ to update. The indexing that maps from GPIO pin to IRQ leads to an out-of-bounds array access and a segfault after that. tl;dr There's 8 rows of 32 pins (8 * 32 == 256 total

[PATCH 0/1] hw: aspeed_gpio: Fix GPIO array indexing

2021-09-23 Thread pdel
From: Peter Delevoryas Hey everyone, I think there might be a bug aspeed_gpio_update, when it's selecting a GPIO IRQ to update. I was testing booting Facebook's OpenBMC platform "YosemiteV2" (fby2), and I was hitting a segfault in QEMU: qemu-system-arm -machine ast2500-evb \ -drive file=fby