[SRU][B][PATCH v2] UBUNTU: SAUCE: (noup) Update zfs to 0.7.5-1ubuntu16.5

Colin Ian King colin.king at canonical.com
Thu May 23 08:07:18 UTC 2019


On 20/05/2019 10:46, Andrea Righi wrote:
> BugLink: https://bugs.launchpad.net/bugs/1828763
> 
> 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 builds, because zfs
> tracepoints are not enabled by default.
> 
> Signed-off-by: Andrea Righi <andrea.righi at canonical.com>
> ---
>  zfs/META                    | 2 +-
>  zfs/include/sys/trace_dmu.h | 6 +++---
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/zfs/META b/zfs/META
> index 9864f8f64432..7e598b52dca6 100644
> --- a/zfs/META
> +++ b/zfs/META
> @@ -2,7 +2,7 @@ Meta:         1
>  Name:         zfs
>  Branch:       1.0
>  Version:      0.7.5
> -Release:      1ubuntu16.4
> +Release:      1ubuntu16.5
>  Release-Tags: relext
>  License:      CDDL
>  Author:       OpenZFS on Linux
> diff --git a/zfs/include/sys/trace_dmu.h b/zfs/include/sys/trace_dmu.h
> index 24e57f5146a4..51304ddc925c 100644
> --- a/zfs/include/sys/trace_dmu.h
> +++ b/zfs/include/sys/trace_dmu.h
> @@ -50,7 +50,7 @@ DECLARE_EVENT_CLASS(zfs_delay_mintime_class,
>  	    __field(uint64_t,			tx_lastsnap_txg)
>  	    __field(uint64_t,			tx_lasttried_txg)
>  	    __field(boolean_t,			tx_anyobj)
> -	    __field(boolean_t,			tx_dirty_delayed)
> +	    __field(boolean_t,			tx_waited)
>  	    __field(hrtime_t,			tx_start)
>  	    __field(boolean_t,			tx_wait_dirty)
>  	    __field(int,			tx_err)
> @@ -62,7 +62,7 @@ DECLARE_EVENT_CLASS(zfs_delay_mintime_class,
>  	    __entry->tx_lastsnap_txg		= tx->tx_lastsnap_txg;
>  	    __entry->tx_lasttried_txg		= tx->tx_lasttried_txg;
>  	    __entry->tx_anyobj			= tx->tx_anyobj;
> -	    __entry->tx_dirty_delayed		= tx->tx_dirty_delayed;
> +	    __entry->tx_waited			= tx->tx_waited;
>  	    __entry->tx_start			= tx->tx_start;
>  	    __entry->tx_wait_dirty		= tx->tx_wait_dirty;
>  	    __entry->tx_err			= tx->tx_err;
> @@ -74,7 +74,7 @@ DECLARE_EVENT_CLASS(zfs_delay_mintime_class,
>  	    "} dirty %llu min_tx_time %llu",
>  	    __entry->tx_txg, __entry->tx_lastsnap_txg,
>  	    __entry->tx_lasttried_txg, __entry->tx_anyobj,
> -	    __entry->tx_dirty_delayed, __entry->tx_start,
> +	    __entry->tx_waited, __entry->tx_start,
>  	    __entry->tx_wait_dirty, __entry->tx_err,
>  	    __entry->dirty, __entry->min_tx_time)
>  );
> 

Has this been tested against the ZFS tests?

Colin



More information about the kernel-team mailing list