mbox series

[SRU,Artful,0/2] Fixes for LP:1757565

Message ID cover.1521824345.git.joseph.salisbury@canonical.com
Headers show
Series Fixes for LP:1757565 | expand

Message

Joseph Salisbury March 23, 2018, 8:14 p.m. UTC
BugLink: http://bugs.launchpad.net/bugs/1757565

== SRU Justification ==
This bug causes btrfs and tar sparse to truncate archives.  This bug has been 
fixed in v4.15-rc3 by commit e3b8a4858566, which has a prereq commit of f48bf66b662e.

These commits were cc'd to upstream stable and are already in Bionic.  However, upstream
4.13 is EOL, so Artful never recieved the fixes, which is the reason for
the SRU request.

== Fixes ==
f48bf66b662e ("Btrfs: move definition of the function btrfs_find_new_delalloc_bytes")
e3b8a4858566 ("Btrfs: fix reported number of inode blocks after buffered append writes")

== Regression Potential ==
Low.  This fix was also sent and accepted to stable, so it has had additional upstream review.

== Test Case ==
A test kernel was built with these patches and tested by the original bug reporter.
The bug reporter states the test kernel resolved the bug.

Filipe Manana (2):
  Btrfs: move definition of the function btrfs_find_new_delalloc_bytes
  Btrfs: fix reported number of inode blocks after buffered append
    writes

 fs/btrfs/ctree.h                 |   1 +
 fs/btrfs/extent_io.h             |   5 +-
 fs/btrfs/file.c                  | 125 +++++++++++++++++++++------------------
 fs/btrfs/inode.c                 |   9 +--
 fs/btrfs/relocation.c            |   3 +-
 fs/btrfs/tests/extent-io-tests.c |   6 +-
 fs/btrfs/tests/inode-tests.c     |  12 ++--
 7 files changed, 87 insertions(+), 74 deletions(-)

Comments

Stefan Bader March 28, 2018, 10:13 a.m. UTC | #1
On 23.03.2018 21:14, Joseph Salisbury wrote:
> BugLink: http://bugs.launchpad.net/bugs/1757565
> 
> == SRU Justification ==
> This bug causes btrfs and tar sparse to truncate archives.  This bug has been 
> fixed in v4.15-rc3 by commit e3b8a4858566, which has a prereq commit of f48bf66b662e.
> 
> These commits were cc'd to upstream stable and are already in Bionic.  However, upstream
> 4.13 is EOL, so Artful never recieved the fixes, which is the reason for
> the SRU request.
> 
> == Fixes ==
> f48bf66b662e ("Btrfs: move definition of the function btrfs_find_new_delalloc_bytes")
> e3b8a4858566 ("Btrfs: fix reported number of inode blocks after buffered append writes")
> 
> == Regression Potential ==
> Low.  This fix was also sent and accepted to stable, so it has had additional upstream review.
> 
> == Test Case ==
> A test kernel was built with these patches and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
> 
> Filipe Manana (2):
>   Btrfs: move definition of the function btrfs_find_new_delalloc_bytes
>   Btrfs: fix reported number of inode blocks after buffered append
>     writes
> 
>  fs/btrfs/ctree.h                 |   1 +
>  fs/btrfs/extent_io.h             |   5 +-
>  fs/btrfs/file.c                  | 125 +++++++++++++++++++++------------------
>  fs/btrfs/inode.c                 |   9 +--
>  fs/btrfs/relocation.c            |   3 +-
>  fs/btrfs/tests/extent-io-tests.c |   6 +-
>  fs/btrfs/tests/inode-tests.c     |  12 ++--
>  7 files changed, 87 insertions(+), 74 deletions(-)
> 
Acked-by: Stefan Bader <stefan.bader@canonical.com>

Existing test case for verification.
Kleber Sacilotto de Souza April 3, 2018, 1:50 p.m. UTC | #2
On 03/23/18 21:14, Joseph Salisbury wrote:
> BugLink: http://bugs.launchpad.net/bugs/1757565
> 
> == SRU Justification ==
> This bug causes btrfs and tar sparse to truncate archives.  This bug has been 
> fixed in v4.15-rc3 by commit e3b8a4858566, which has a prereq commit of f48bf66b662e.
> 
> These commits were cc'd to upstream stable and are already in Bionic.  However, upstream
> 4.13 is EOL, so Artful never recieved the fixes, which is the reason for
> the SRU request.
> 
> == Fixes ==
> f48bf66b662e ("Btrfs: move definition of the function btrfs_find_new_delalloc_bytes")
> e3b8a4858566 ("Btrfs: fix reported number of inode blocks after buffered append writes")
> 
> == Regression Potential ==
> Low.  This fix was also sent and accepted to stable, so it has had additional upstream review.
> 
> == Test Case ==
> A test kernel was built with these patches and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
> 
> Filipe Manana (2):
>   Btrfs: move definition of the function btrfs_find_new_delalloc_bytes
>   Btrfs: fix reported number of inode blocks after buffered append
>     writes
> 
>  fs/btrfs/ctree.h                 |   1 +
>  fs/btrfs/extent_io.h             |   5 +-
>  fs/btrfs/file.c                  | 125 +++++++++++++++++++++------------------
>  fs/btrfs/inode.c                 |   9 +--
>  fs/btrfs/relocation.c            |   3 +-
>  fs/btrfs/tests/extent-io-tests.c |   6 +-
>  fs/btrfs/tests/inode-tests.c     |  12 ++--
>  7 files changed, 87 insertions(+), 74 deletions(-)
> 

Acked-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
Kleber Sacilotto de Souza April 3, 2018, 3:48 p.m. UTC | #3
On 03/23/18 21:14, Joseph Salisbury wrote:
> BugLink: http://bugs.launchpad.net/bugs/1757565
> 
> == SRU Justification ==
> This bug causes btrfs and tar sparse to truncate archives.  This bug has been 
> fixed in v4.15-rc3 by commit e3b8a4858566, which has a prereq commit of f48bf66b662e.
> 
> These commits were cc'd to upstream stable and are already in Bionic.  However, upstream
> 4.13 is EOL, so Artful never recieved the fixes, which is the reason for
> the SRU request.
> 
> == Fixes ==
> f48bf66b662e ("Btrfs: move definition of the function btrfs_find_new_delalloc_bytes")
> e3b8a4858566 ("Btrfs: fix reported number of inode blocks after buffered append writes")
> 
> == Regression Potential ==
> Low.  This fix was also sent and accepted to stable, so it has had additional upstream review.
> 
> == Test Case ==
> A test kernel was built with these patches and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
> 
> Filipe Manana (2):
>   Btrfs: move definition of the function btrfs_find_new_delalloc_bytes
>   Btrfs: fix reported number of inode blocks after buffered append
>     writes
> 
>  fs/btrfs/ctree.h                 |   1 +
>  fs/btrfs/extent_io.h             |   5 +-
>  fs/btrfs/file.c                  | 125 +++++++++++++++++++++------------------
>  fs/btrfs/inode.c                 |   9 +--
>  fs/btrfs/relocation.c            |   3 +-
>  fs/btrfs/tests/extent-io-tests.c |   6 +-
>  fs/btrfs/tests/inode-tests.c     |  12 ++--
>  7 files changed, 87 insertions(+), 74 deletions(-)
> 

Applied to artful/master-next branch.

Thanks,
Kleber