Hi Gary,
Can you please rebase this patch (I pushed #517) and float it.
Thanks
-Nagu
> -----Original Message-----
> From: Nagendra Kumar
> Sent: 19 May 2016 16:42
> To: Anders Widell; Gary Lee
> Cc: [email protected]; [email protected]
> Subject: Re: [devel] [PATCH 1 of 1] amfnd: add preliminary support for BAD
> OM handles [#1833]
>
> Hi Anders,
> I forgot to mention that we already pushed fix for #1819, though the
> issue is solved by floated patch of #517. Now, this patch should be on top of
> #517. So, we need to have some where #517 and starts toping on it.
>
> That's my point and it is good for all of us.
>
> Sorry for the inconvenience if it is causing.
>
> Thanks
> -Nagu
>
> > -----Original Message-----
> > From: Nagendra Kumar
> > Sent: 19 May 2016 16:27
> > To: Anders Widell; Gary Lee
> > Cc: [email protected]; [email protected]
> > Subject: Re: [devel] [PATCH 1 of 1] amfnd: add preliminary support for
> > BAD OM handles [#1833]
> >
> > Hi Anders,
> >
> > Yes, but it need to go as per
> > sequence. If we have #517 (a separate thread already created), you can
> > simply change the code for BAD_HANDLE.
> >
> > When I have to push #517, then I need to again refactor it (for making
> > changes in a separate thread).
> >
> >
> >
> > Neither #517 nor #1833 fixes all problems mentioned in #1607/9, we are
> > just progressing towards it.
> >
> >
> >
> > Thanks
> >
> > -Nagu
> >
> >
> >
> > From: Anders Widell [mailto:[email protected]]
> > Sent: 19 May 2016 16:13
> > To: Gary Lee; Nagendra Kumar
> > Cc: [email protected]; [email protected]; opensaf-
> > [email protected]
> > Subject: Re: [PATCH 1 of 1] amfnd: add preliminary support for BAD OM
> > handles [#1833]
> >
> >
> >
> > I agree with Gary that the full solution (outlined in tickets [#1607]
> > and
> > [#1609]) is too big to introduce in a maintenance release (OpenSAF
> > 5.0.1). It will have to wait until the next major release.
> >
> > regards,
> > Anders Widell
> >
> > On 05/19/2016 12:08 PM, Gary Lee wrote:
> >
> > That is why I tried to restrict the number of retries. Handling bad
> > handle is different to me between OI and OM.
> >
> > I think what you did in #517 is great. But it only covers a subset of
> > the places where IMM reads occur.
> >
> > I don't see a feasible way of fixing this fully with threads with the
> > restricted time and amount of code changes we can make for 5.0?
> >
> > On 19 May 2016 7:57 PM, Nagendra Kumar HYPERLINK
> > "mailto:[email protected]";<[email protected]> wrote:
> >
> > Hi Gary,
> >
> > Yes, but this will make Amfnd to hang and will get
> > created the same problem as mentioned in #517. Please check Amfd
> > handling the BAD_HANDLE in a separate thread and the tickets Ander W
> has mentioned.
> >
> >
> >
> > Thanks
> >
> > -Nagu
> >
> >
> >
> > From: Gary Lee [mailto:[email protected]]
> > Sent: 19 May 2016 15:21
> > To: Nagendra Kumar
> > Cc: HYPERLINK
> > "mailto:[email protected]"[email protected];
> > HYPERLINK "mailto:[email protected]"opensaf-
> > [email protected]; HYPERLINK
> > "mailto:[email protected]"[email protected];
> > HYPERLINK
> > "mailto:[email protected]"[email protected]
> > Subject: RE: [PATCH 1 of 1] amfnd: add preliminary support for BAD OM
> > handles [#1833]
> >
> >
> >
> > Hi Nagu
> >
> > I think we definitely should for the next release, but what about 5.0?
> >
> > Gary
> >
> >
> > ----------------------------------------------------------------------
> > -------- Mobile security can be enabling, not merely restricting.
> > Employees who bring their own devices (BYOD) to work are irked by the
> > imposition of MDM restrictions. Mobile Device Manager Plus allows you
> > to control only the apps on BYO-devices by containerizing them,
> > leaving personal data untouched!
> > https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
> > _______________________________________________
> > Opensaf-devel mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/opensaf-devel
>
> ------------------------------------------------------------------------------
> Mobile security can be enabling, not merely restricting. Employees who bring
> their own devices (BYOD) to work are irked by the imposition of MDM
> restrictions. Mobile Device Manager Plus allows you to control only the apps
> on BYO-devices by containerizing them, leaving personal data untouched!
> https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
> _______________________________________________
> Opensaf-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/opensaf-devel
------------------------------------------------------------------------------
Attend Shape: An AT&T Tech Expo July 15-16. Meet us at AT&T Park in San
Francisco, CA to explore cutting-edge tech and listen to tech luminaries
present their vision of the future. This family event has something for
everyone, including kids. Get more information and register today.
http://sdm.link/attshape
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel
