RE: [EXT] Re: [PATCH BlueZ v2 1/1] gobex/gobex.c : Increase default obex timeout from 10 sec to 500 sec

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Luiz and Paul,

> -----Original Message-----
> From: Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx>
> Sent: Friday, October 20, 2023 11:19 PM
> To: Paul Menzel <pmenzel@xxxxxxxxxxxxx>
> Cc: Purendra Singh <purendra.singh@xxxxxxx>; linux-
> bluetooth@xxxxxxxxxxxxxxx; Devyani Godbole <devyani.godbole@xxxxxxx>;
> Nitin Jadhav <nitin.jadhav@xxxxxxx>
> Subject: [EXT] Re: [PATCH BlueZ v2 1/1] gobex/gobex.c : Increase default
> obex timeout from 10 sec to 500 sec
> 
> Caution: This is an external email. Please take care when clicking links or
> opening attachments. When in doubt, report the message using the 'Report
> this email' button
> 
> 
> Hi,
> 
> On Fri, Oct 20, 2023 at 2:44 AM Paul Menzel <pmenzel@xxxxxxxxxxxxx>
> 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’s really seconds. Isn’t greater than 8 minute timeout *not*
> > very user friendly?
> 
> Yeah, something like that would probably cause the user to  assume the
> operation didn't work.

This is the maximum timeout we have set based on experimenting
different reference devices mentioned in earlier comment.
This will only come into picture if the reference devices slow in 
responding the file received status.
> 
> > > ---
> > >   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
> 
> 
> 
> --
> Luiz Augusto von Dentz

Thanks 
Purendra




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux