mbox series

[SRU,J,0/2] UBUNTU: SAUCE: shiftfs: -EOVERFLOW fixes

Message ID 20230131173833.557692-1-aleksandr.mikhalitsyn@canonical.com
Headers show
Series UBUNTU: SAUCE: shiftfs: -EOVERFLOW fixes | expand

Message

Aleksandr Mikhalitsyn Jan. 31, 2023, 5:38 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1990849

Alexander Mikhalitsyn (1):
  UBUNTU: [SAUCE] shiftfs: fix -EOVERFLOW inside the container

Andrea Righi (1):
  UBUNTU: SAUCE: shiftfs: always rely on init_user_ns

 fs/shiftfs.c | 66 +++++++++++++++++++++++++++++-----------------------
 1 file changed, 37 insertions(+), 29 deletions(-)

Comments

Andrea Righi Jan. 31, 2023, 5:41 p.m. UTC | #1
On Tue, Jan 31, 2023 at 06:38:31PM +0100, Alexander Mikhalitsyn wrote:
> BugLink: https://bugs.launchpad.net/bugs/1990849
> 
> Alexander Mikhalitsyn (1):
>   UBUNTU: [SAUCE] shiftfs: fix -EOVERFLOW inside the container
> 
> Andrea Righi (1):
>   UBUNTU: SAUCE: shiftfs: always rely on init_user_ns
> 
>  fs/shiftfs.c | 66 +++++++++++++++++++++++++++++-----------------------
>  1 file changed, 37 insertions(+), 29 deletions(-)

This looks good to me, we need both patches to make shiftfs in jammy
consistent with the other releases, therefore:

Acked-by: Andrea Righi <andrea.righi@canonical.com>
Tim Gardner Jan. 31, 2023, 6:04 p.m. UTC | #2
On 1/31/23 10:38 AM, Alexander Mikhalitsyn wrote:
> BugLink: https://bugs.launchpad.net/bugs/1990849
> 
> Alexander Mikhalitsyn (1):
>    UBUNTU: [SAUCE] shiftfs: fix -EOVERFLOW inside the container
> 
> Andrea Righi (1):
>    UBUNTU: SAUCE: shiftfs: always rely on init_user_ns
> 
>   fs/shiftfs.c | 66 +++++++++++++++++++++++++++++-----------------------
>   1 file changed, 37 insertions(+), 29 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>

I wouldn't mind some test results though.
Aleksandr Mikhalitsyn Feb. 7, 2023, 10:22 a.m. UTC | #3
Hi Tim!

Do you have a plan to apply these patches to Jammy? As far as I know
these patches were applied for Kinetic/Lunar.

Kind regards,
Alex

On Tue, Jan 31, 2023 at 7:04 PM Tim Gardner <tim.gardner@canonical.com> wrote:
>
> On 1/31/23 10:38 AM, Alexander Mikhalitsyn wrote:
> > BugLink: https://bugs.launchpad.net/bugs/1990849
> >
> > Alexander Mikhalitsyn (1):
> >    UBUNTU: [SAUCE] shiftfs: fix -EOVERFLOW inside the container
> >
> > Andrea Righi (1):
> >    UBUNTU: SAUCE: shiftfs: always rely on init_user_ns
> >
> >   fs/shiftfs.c | 66 +++++++++++++++++++++++++++++-----------------------
> >   1 file changed, 37 insertions(+), 29 deletions(-)
> >
> Acked-by: Tim Gardner <tim.gardner@canonical.com>
>
> I wouldn't mind some test results though.
> --
> -----------
> Tim Gardner
> Canonical, Inc
>
Tim Gardner Feb. 8, 2023, 3:59 p.m. UTC | #4
Hi Aleksandr,

I'm staying in my lane. The stable team will apply this patch to Jammy 
at the appropriate time, probably in the 2023.02.20 SRU cycle. See 
kernel.ubuntu.com for dates.

rtg

On 2/7/23 3:22 AM, Aleksandr Mikhalitsyn wrote:
> Hi Tim!
> 
> Do you have a plan to apply these patches to Jammy? As far as I know
> these patches were applied for Kinetic/Lunar.
> 
> Kind regards,
> Alex
> 
> On Tue, Jan 31, 2023 at 7:04 PM Tim Gardner <tim.gardner@canonical.com> wrote:
>>
>> On 1/31/23 10:38 AM, Alexander Mikhalitsyn wrote:
>>> BugLink: https://bugs.launchpad.net/bugs/1990849
>>>
>>> Alexander Mikhalitsyn (1):
>>>     UBUNTU: [SAUCE] shiftfs: fix -EOVERFLOW inside the container
>>>
>>> Andrea Righi (1):
>>>     UBUNTU: SAUCE: shiftfs: always rely on init_user_ns
>>>
>>>    fs/shiftfs.c | 66 +++++++++++++++++++++++++++++-----------------------
>>>    1 file changed, 37 insertions(+), 29 deletions(-)
>>>
>> Acked-by: Tim Gardner <tim.gardner@canonical.com>
>>
>> I wouldn't mind some test results though.
>> --
>> -----------
>> Tim Gardner
>> Canonical, Inc
>>
Stefan Bader Feb. 10, 2023, 3:07 p.m. UTC | #5
On 31.01.23 18:38, Alexander Mikhalitsyn wrote:
> BugLink: https://bugs.launchpad.net/bugs/1990849
> 
> Alexander Mikhalitsyn (1):
>    UBUNTU: [SAUCE] shiftfs: fix -EOVERFLOW inside the container
> 
> Andrea Righi (1):
>    UBUNTU: SAUCE: shiftfs: always rely on init_user_ns
> 
>   fs/shiftfs.c | 66 +++++++++++++++++++++++++++++-----------------------
>   1 file changed, 37 insertions(+), 29 deletions(-)
> 

Applied to jammy:linux/master-next. Thanks.

-Stefan