[ofw] Patch: Fix using lists on ipoib without locks
Ishai Rabinovitz
ishai at mellanox.co.il
Mon Nov 3 14:09:26 PST 2008
I'm not sure Tzachi left it by mistake.
I'm not sure it is a bad idea to change the name so it will be easy to
find it in the future.
Maybe using p_endpt1 is not such a good name, but using some other name
for the field (e.g., include the name of the containing structure as a
prefix: p_send_desc_endpt or p_sd_endpt), does not look like a bad idea.
Ishai
> -----Original Message-----
> From: ofw-bounces at lists.openfabrics.org
> [mailto:ofw-bounces at lists.openfabrics.org] On Behalf Of Fab Tillier
> Sent: Monday, November 03, 2008 11:53 PM
> To: Tzachi Dar; Alex Estrin; ofw at lists.openfabrics.org
> Subject: RE: [ofw] Patch: Fix using lists on ipoib without locks
>
> > While looking for the race on the lists, I wanted to make
> sure that I
> > know where all the places that this structure is being
> used. Since the
> > string p_endpt is used many times in the code I wanted to have a
> > unique identifier that my editor will be able to find easily.
>
> Changes to work around a lacking editor should not find their
> way into the final check in. I believe this happened before
> and you received the same feedback - change the variable back
> before checking in so as to minimize the changes in the check in.
>
> -Fab
> _______________________________________________
> ofw mailing list
> ofw at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw
>
More information about the ofw
mailing list