Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
MIME-Version: 1.0
References: <20231020093716.9654-1-purendra.singh@nxp.com> <20231020093716.9654-2-purendra.singh@nxp.com>
 <4cf2cb28-b334-487b-96d6-6318961035e1@molgen.mpg.de>
In-Reply-To: <4cf2cb28-b334-487b-96d6-6318961035e1@molgen.mpg.de>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Fri, 20 Oct 2023 10:49:22 -0700
Message-ID: <CABBYNZKhBp=qw9aB+0CVLu0U1YeYwt2qC80+2fqgwH-sgbWf3A@mail.gmail.com>
Subject: Re: [PATCH BlueZ v2 1/1] gobex/gobex.c : Increase default obex
 timeout from 10 sec to 500 sec
To:     Paul Menzel <pmenzel@molgen.mpg.de>
Cc:     Purendra Singh <purendra.singh@nxp.com>,
        linux-bluetooth@vger.kernel.org, devyani.godbole@nxp.com,
        nitin.jadhav@nxp.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi,

On Fri, Oct 20, 2023 at 2:44=E2=80=AFAM Paul Menzel <pmenzel@molgen.mpg.de>=
 wrote:
>
> Dear Purendra,
>
>
> Thank you for sending version 2. (No space is necessary in the prefix
> before the colon.)
>
>
> Am 20.10.23 um 11:37 schrieb Purendra Singh:
> > This issue is observed when multiple profiles are active from
> > multiple connected devices: A2DP + HID + OBex File transfer
> >
> > File received status of some reference devices is delayed,
> > that causes timeout and file transfer status error (Variation
> > of timeout is between ~100 seconds to ~450 seconds).
> >
> > We tested with Motorola Edge, Samsung M33, OnePlus 6T.
> >
> > Experimentaly we increased the timeout and at 500 seconds no issue was
>
> Experimentally
>
> > observed.
>
> Oh, it=E2=80=99s really seconds. Isn=E2=80=99t greater than 8 minute time=
out *not* very
> user friendly?

Yeah, something like that would probably cause the user to  assume the
operation didn't work.

> > ---
> >   gobex/gobex.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/gobex/gobex.c b/gobex/gobex.c
> > index e9b89cead..f0e1c2c83 100644
> > --- a/gobex/gobex.c
> > +++ b/gobex/gobex.c
> > @@ -22,7 +22,7 @@
> >   #define G_OBEX_MINIMUM_MTU  255
> >   #define G_OBEX_MAXIMUM_MTU  65535
> >
> > -#define G_OBEX_DEFAULT_TIMEOUT       10
> > +#define G_OBEX_DEFAULT_TIMEOUT       500
> >   #define G_OBEX_ABORT_TIMEOUT        5
> >
> >   #define G_OBEX_OP_NONE              0xff
>
>
> Kind regards,
>
> Paul


--=20
Luiz Augusto von Dentz