On Wed, 2011-12-07 at 18:37 +0530, Archit Taneja wrote:
> On Tuesday 22 November 2011 02:51 PM, Tomi Valkeinen wrote:

> > +/* wait until no extra_info updates are pending */
> > +static void wait_pending_extra_info_updates(void)
> > +{
> > +   bool updating;
> > +   unsigned long flags;
> > +   unsigned long t;
> > +
> > +   spin_lock_irqsave(&data_lock, flags);
> > +
> > +   updating = extra_info_update_ongoing();
> > +
> > +   if (!updating) {
> > +           spin_unlock_irqrestore(&data_lock, flags);
> > +           return;
> > +   }
> > +
> > +   init_completion(&extra_updated_completion);
> > +
> > +   spin_unlock_irqrestore(&data_lock, flags);
> > +
> > +   t = msecs_to_jiffies(500);
> > +   wait_for_completion_timeout(&extra_updated_completion, t);
> > +
> > +   updating = extra_info_update_ongoing();
> > +
> > +   WARN_ON(updating);
> > +}
> 
> This function isn't used, are you planning to use it later?

Yes, it's used with the fifo-merge stuff. It could be added later, but
as it's related to the extra_info stuff, I think it fits better here
than with the fifo merge series.

 Tomi

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to