mbox series

[SRU,Xenial,0/1] s390/qeth: use vzalloc for QUERY OAT buffer

Message ID cover.1537259280.git.joseph.salisbury@canonical.com
Headers show
Series s390/qeth: use vzalloc for QUERY OAT buffer | expand

Message

Joseph Salisbury Sept. 20, 2018, 9:42 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1793086

== SRU Justification ==
IBM reports the qethqoat tool fails.  qeth_query_oat_command() currently
allocates the kernel buffer for the SIOC_QETH_QUERY_OAT ioctl with kzalloc.

This commit will cause the use of vzalloc() instead of kzalloc(), backing
the allocation with non-contiguous memory.

Bionic and Cosmic also need this commit, but they are clean cherry picks
so have been SRU'd separatly.

== Fix ==
aec45e857c55 ("s390/qeth: use vzalloc for QUERY OAT buffer")

== Regression Potential ==
Low, limited to s390.

== Test Case ==
IBM verified testing of this commit up-front.

Wenjia Zhang (1):
  s390/qeth: use vzalloc for QUERY OAT buffer

 drivers/s390/net/qeth_core_main.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Kleber Sacilotto de Souza Oct. 2, 2018, 10:18 a.m. UTC | #1
On 09/20/18 11:42, Joseph Salisbury wrote:
> BugLink: https://bugs.launchpad.net/bugs/1793086
> 
> == SRU Justification ==
> IBM reports the qethqoat tool fails.  qeth_query_oat_command() currently
> allocates the kernel buffer for the SIOC_QETH_QUERY_OAT ioctl with kzalloc.
> 
> This commit will cause the use of vzalloc() instead of kzalloc(), backing
> the allocation with non-contiguous memory.
> 
> Bionic and Cosmic also need this commit, but they are clean cherry picks
> so have been SRU'd separatly.
> 
> == Fix ==
> aec45e857c55 ("s390/qeth: use vzalloc for QUERY OAT buffer")
> 
> == Regression Potential ==
> Low, limited to s390.
> 
> == Test Case ==
> IBM verified testing of this commit up-front.
> 
> Wenjia Zhang (1):
>   s390/qeth: use vzalloc for QUERY OAT buffer
> 
>  drivers/s390/net/qeth_core_main.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 

Applied to xenial/master-next branch.

Thanks,
Kleber