mbox series

[v2,0/2] PCI: hv: Hardening changes

Message ID 20220511223207.3386-1-parri.andrea@gmail.com (mailing list archive)
Headers show
Series PCI: hv: Hardening changes | expand

Message

Andrea Parri May 11, 2022, 10:32 p.m. UTC
Changes since v1[1]:
  - Add validation in q_resource_requirements()

Patch #2 depends on changes in hyperv-next.  (Acknowledging that hyperv
is entering EOM, for review.)

Thanks,
  Andrea

[1] https://lkml.kernel.org/r/20220504125039.2598-1-parri.andrea@gmail.com

Andrea Parri (Microsoft) (2):
  PCI: hv: Add validation for untrusted Hyper-V values
  PCI: hv: Fix synchronization between channel callback and
    hv_pci_bus_exit()

 drivers/pci/controller/pci-hyperv.c | 59 +++++++++++++++++++++--------
 1 file changed, 43 insertions(+), 16 deletions(-)

Comments

Lorenzo Pieralisi May 13, 2022, 10:11 a.m. UTC | #1
On Thu, May 12, 2022 at 12:32:05AM +0200, Andrea Parri (Microsoft) wrote:
> Changes since v1[1]:
>   - Add validation in q_resource_requirements()
> 
> Patch #2 depends on changes in hyperv-next.  (Acknowledging that hyperv
> is entering EOM, for review.)

I suppose then it is best for me to ACK it and ask Hyper-V maintainers
to pick it up.

For the series:

Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>

> 
> Thanks,
>   Andrea
> 
> [1] https://lkml.kernel.org/r/20220504125039.2598-1-parri.andrea@gmail.com
> 
> Andrea Parri (Microsoft) (2):
>   PCI: hv: Add validation for untrusted Hyper-V values
>   PCI: hv: Fix synchronization between channel callback and
>     hv_pci_bus_exit()
> 
>  drivers/pci/controller/pci-hyperv.c | 59 +++++++++++++++++++++--------
>  1 file changed, 43 insertions(+), 16 deletions(-)
> 
> -- 
> 2.25.1
>
Wei Liu May 13, 2022, 4:51 p.m. UTC | #2
On Fri, May 13, 2022 at 11:11:32AM +0100, Lorenzo Pieralisi wrote:
> On Thu, May 12, 2022 at 12:32:05AM +0200, Andrea Parri (Microsoft) wrote:
> > Changes since v1[1]:
> >   - Add validation in q_resource_requirements()
> > 
> > Patch #2 depends on changes in hyperv-next.  (Acknowledging that hyperv
> > is entering EOM, for review.)
> 
> I suppose then it is best for me to ACK it and ask Hyper-V maintainers
> to pick it up.
> 

Yes.

> For the series:
> 
>Pending Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> 

Thanks!
Wei Liu May 13, 2022, 4:58 p.m. UTC | #3
On Thu, May 12, 2022 at 12:32:05AM +0200, Andrea Parri (Microsoft) wrote:
> Changes since v1[1]:
>   - Add validation in q_resource_requirements()
> 
> Patch #2 depends on changes in hyperv-next.  (Acknowledging that hyperv
> is entering EOM, for review.)
> 
> Thanks,
>   Andrea
> 
> [1] https://lkml.kernel.org/r/20220504125039.2598-1-parri.andrea@gmail.com
> 
> Andrea Parri (Microsoft) (2):
>   PCI: hv: Add validation for untrusted Hyper-V values
>   PCI: hv: Fix synchronization between channel callback and
>     hv_pci_bus_exit()

Applied to hyperv-next. Thanks.