Re: [patch] zfs livelock and thread priorities

From: Adam McDougall <mcdouga9_at_egr.msu.edu>
Date: Mon, 27 Apr 2009 02:22:00 -0400
On Sat, Apr 25, 2009 at 01:49:33PM -0400, Ben Kelly wrote:
  
  On Apr 24, 2009, at 11:59 PM, Artem Belevich wrote:
  
  > Ah! Indeed.
  >
  >> va_list va;
  >> va_start(ap, fmt);
  >> va_copy(va, ap);
  >> vsnprintf(p2->p_comm, sizeof(p2->p_comm), fmt, va);
  >> vsnprintf(td->td_name, sizeof(td->td_name), fmt, ap);
  >
  > We also need va_end(va) to clear up after va_copy();
  
  I've updated the patch to use va_copy().  Unfortunately I don't have  
  an AMD64 machine to test on at the moment.  However, I did look at  
  other places in the kernel where va_copy() is used so I am hopeful I  
  got it right.
  
  Thanks again for the help!
  
  - Ben
  
I have the patch running on a amd64 desktop at home,
so far so good and I'll run some portupgrades overnight.
Assuming no huge problems with that, I have one or more
systems at work I can try it with as well, as I have time.
Received on Mon Apr 27 2009 - 04:22:01 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:46 UTC