RE: [PATCH 2/3] pnfs: introduce pnfs private workqueue

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

 



Hi, Boaz,
> -----Original Message-----
> From: linux-nfs-owner@xxxxxxxxxxxxxxx [mailto:linux-nfs-owner@xxxxxxxxxxxxxxx]
> On Behalf Of Boaz Harrosh
> Sent: Wednesday, September 21, 2011 6:38 PM
> To: Peng, Tao
> Cc: bhalevy@xxxxxxxxxx; Trond.Myklebust@xxxxxxxxxx; linux-nfs@xxxxxxxxxxxxxxx;
> honey@xxxxxxxxxxxxxx; rees@xxxxxxxxx
> Subject: Re: [PATCH 2/3] pnfs: introduce pnfs private workqueue
> 
> On 09/21/2011 01:23 PM, tao.peng@xxxxxxx wrote:
> >
> > Unless we do following:
> > 1. preallocate memory for extent state convertion
> > 2. use nfsiod/rpciod to handle bl_write_cleanup
> > 3. for pnfs error case, create a kthread to recollapse and resend to MDS
> >
> > not sure if it worth the complexity though...
> >
> > Cheers,
> > Tao
> >
> 
> Please forgive my ignorance but what is the big difference between
> a kthread and a workqueue? I thought a workqueue is just a kthread
> and some memory structures (list).
Sorry for being not clear...
By kthread, I mean a thread that runs only once and does what we need and then exits, if I read code correctly, just like the state recovery thread that Trond mentioned above. And as Trond explained, since the error case should be rare, it is not necessary to keep a thread up and waiting for it all the time.
And by workqueue, I think you know exactly what it is :)

Cheers,
Tao
> 
> I agree with "keep things simple"
> 
> But what do I know ;-)
> 
> Thanks
> Boaz
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

��.n��������+%������w��{.n�����{��w���jg��������ݢj����G�������j:+v���w�m������w�������h�����٥



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux