squashfs: fix some regressions introduced in the readahead code

[0/3] squashfs: fix some regressions introduced in the readahead code

Posted by Phillip Lougher 3 years, 5 months ago

Hi,

This patch-set fixes 3 regressions introduced by the recent readahead code.
The first regression is causing "snaps" to randomly fail after a couple of
hours or days, which how the regression came to light.

----------------------------------------------------------------
Phillip Lougher (3):
      squashfs: fix read regression introduced in readahead code
      squashfs: fix extending readahead beyond end of file
      squashfs: fix buffer release race condition in readahead code

 fs/squashfs/file.c       | 23 ++++++++++++++---------
 fs/squashfs/page_actor.c |  3 +++
 fs/squashfs/page_actor.h |  6 +++++-
 3 files changed, 22 insertions(+), 10 deletions(-)

Re: [0/3] squashfs: fix some regressions introduced in the readahead code

Posted by Bagas Sanjaya 3 years, 5 months ago

On Thu, Oct 20, 2022 at 11:36:13PM +0100, Phillip Lougher wrote:
> Hi,
> 
> This patch-set fixes 3 regressions introduced by the recent readahead code.
> The first regression is causing "snaps" to randomly fail after a couple of
> hours or days, which how the regression came to light.
> 
> ----------------------------------------------------------------
> Phillip Lougher (3):
>       squashfs: fix read regression introduced in readahead code
>       squashfs: fix extending readahead beyond end of file
>       squashfs: fix buffer release race condition in readahead code
> 
>  fs/squashfs/file.c       | 23 ++++++++++++++---------
>  fs/squashfs/page_actor.c |  3 +++
>  fs/squashfs/page_actor.h |  6 +++++-
>  3 files changed, 22 insertions(+), 10 deletions(-)

The regression goes away with this series, thanks.

For the series,

Tested-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara