ACK: [SRU][B][PATCH 0/1] zfs: fix potential build error

Colin Ian King colin.king at canonical.com
Thu May 23 13:54:06 UTC 2019


On 15/05/2019 11:58, Andrea Righi wrote:
> Buglink: https://bugs.launchpad.net/bugs/1828763
> 
> [Impact]
> 
> * In b49151d684f44 tx_waited has been renamed to tx_dirty_delayed, but only in
>   the tracepoint definition (in trace_dmu.h) and not in the rest of the code,
>   causing build errors if zfs tracepoints are enabled.
> 
> * Fix by reverting tx_dirty_delayed back to the original name tx_waited
> 
> NOTE: this bug doesn't show up in regular kernel builds, because zfs
> tracepoints are not enabled by default, it is possible to see this problem only
> by enabling them and recompiling zfs outside our regular build process, but
> it's a good cleanup anyway (in case we need to enable zfs tracepoints in the
> future).
> 
> [Test Case]
> 
>  * enable zfs tracepoints in config and build zfs
> 
> [Fix]
> 
>  * Restore the old struct name to fix the build bug
> 
> [Regression Potential]
> 
>  * It is a very small fix (just a rename of a struct member), so regression
>    potential is minimal
> 
>  trace_dmu.h |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> 

Acked-by: Colin Ian King <colin.king at canonical.com>



More information about the kernel-team mailing list