All of lore.kernel.org
 help / color / mirror / Atom feed
* revalidate inode before generic_file_aio_read
@ 2011-03-10  8:32 Pavel Shilovsky
       [not found] ` <AANLkTinfzBXvfNmerWHoMuZ8NkeZLJxNE=MJYcFhA8Nu-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Pavel Shilovsky @ 2011-03-10  8:32 UTC (permalink / raw)
  To: Steve French, Jeff Layton, Suresh Jayaraman, Shirish Pargaonkar
  Cc: linux-cifs-u79uwXL29TY76Z2rM5mHXA

Hi all!

As soon as we have tunable actimeo, I think we should revalidate inode
all the time we want to read from it, like NFS does it.

What do you think about it? If it's ok, I am ready to provide a patch for this.

-- 
Best regards,
Pavel Shilovsky.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: revalidate inode before generic_file_aio_read
       [not found] ` <AANLkTinfzBXvfNmerWHoMuZ8NkeZLJxNE=MJYcFhA8Nu-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2011-03-10 10:14   ` Suresh Jayaraman
       [not found]     ` <4D78A476.7050800-l3A5Bk7waGM@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Suresh Jayaraman @ 2011-03-10 10:14 UTC (permalink / raw)
  To: Pavel Shilovsky
  Cc: Steve French, Jeff Layton, Shirish Pargaonkar,
	linux-cifs-u79uwXL29TY76Z2rM5mHXA

On 03/10/2011 02:02 PM, Pavel Shilovsky wrote:
> Hi all!
> 
> As soon as we have tunable actimeo, I think we should revalidate inode
> all the time we want to read from it, like NFS does it.

Did you mean while using strict cache along with actimeo? Conceptually
one should not be needing to use them together, right?

> What do you think about it? If it's ok, I am ready to provide a patch for this.
> 


-- 
Suresh Jayaraman

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: revalidate inode before generic_file_aio_read
       [not found]     ` <4D78A476.7050800-l3A5Bk7waGM@public.gmane.org>
@ 2011-03-10 10:33       ` Pavel Shilovsky
       [not found]         ` <AANLkTinVkxs9vO9QP_BdmFfRcK5duVWviFAbgg6NoSHw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Pavel Shilovsky @ 2011-03-10 10:33 UTC (permalink / raw)
  To: Suresh Jayaraman
  Cc: Steve French, Jeff Layton, Shirish Pargaonkar,
	linux-cifs-u79uwXL29TY76Z2rM5mHXA

2011/3/10 Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org>:
> Did you mean while using strict cache along with actimeo? Conceptually
> one should not be needing to use them together, right?

I don't mean strict cache mode and suggested to do it for
non-strictcache mounts. Sorry for didn't make it clear in the first
email.

-- 
Best regards,
Pavel Shilovsky.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: revalidate inode before generic_file_aio_read
       [not found]         ` <AANLkTinVkxs9vO9QP_BdmFfRcK5duVWviFAbgg6NoSHw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2011-03-10 20:49           ` Jeff Layton
       [not found]             ` <20110310154933.1fa017a0-xSBYVWDuneFaJnirhKH9O4GKTjYczspe@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Jeff Layton @ 2011-03-10 20:49 UTC (permalink / raw)
  To: Pavel Shilovsky
  Cc: Suresh Jayaraman, Steve French, Shirish Pargaonkar,
	linux-cifs-u79uwXL29TY76Z2rM5mHXA

On Thu, 10 Mar 2011 13:33:32 +0300
Pavel Shilovsky <piastryyy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:

> 2011/3/10 Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org>:
> > Did you mean while using strict cache along with actimeo? Conceptually
> > one should not be needing to use them together, right?
> 
> I don't mean strict cache mode and suggested to do it for
> non-strictcache mounts. Sorry for didn't make it clear in the first
> email.
> 

I think it's a reasonable thing to do. I proposed something similar
about a year ago, but Steve NAK'ed it. It was part of this patchset,
some of which made it in:

http://lists.samba.org/archive/linux-cifs-client/2010-February/005548.html

-- 
Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: revalidate inode before generic_file_aio_read
       [not found]             ` <20110310154933.1fa017a0-xSBYVWDuneFaJnirhKH9O4GKTjYczspe@public.gmane.org>
@ 2011-03-11  9:01               ` Pavel Shilovsky
  0 siblings, 0 replies; 5+ messages in thread
From: Pavel Shilovsky @ 2011-03-11  9:01 UTC (permalink / raw)
  To: Jeff Layton, Steve French
  Cc: Suresh Jayaraman, Shirish Pargaonkar,
	linux-cifs-u79uwXL29TY76Z2rM5mHXA

2011/3/10 Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>:
> On Thu, 10 Mar 2011 13:33:32 +0300
> Pavel Shilovsky <piastryyy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>
>> 2011/3/10 Suresh Jayaraman <sjayaraman-l3A5Bk7waGM@public.gmane.org>:
>> > Did you mean while using strict cache along with actimeo? Conceptually
>> > one should not be needing to use them together, right?
>>
>> I don't mean strict cache mode and suggested to do it for
>> non-strictcache mounts. Sorry for didn't make it clear in the first
>> email.
>>
>
> I think it's a reasonable thing to do. I proposed something similar
> about a year ago, but Steve NAK'ed it. It was part of this patchset,
> some of which made it in:
>
> http://lists.samba.org/archive/linux-cifs-client/2010-February/005548.html
>
> --
> Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
>

Steve, can you comment, please?

-- 
Best regards,
Pavel Shilovsky.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-03-11  9:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-10  8:32 revalidate inode before generic_file_aio_read Pavel Shilovsky
     [not found] ` <AANLkTinfzBXvfNmerWHoMuZ8NkeZLJxNE=MJYcFhA8Nu-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-03-10 10:14   ` Suresh Jayaraman
     [not found]     ` <4D78A476.7050800-l3A5Bk7waGM@public.gmane.org>
2011-03-10 10:33       ` Pavel Shilovsky
     [not found]         ` <AANLkTinVkxs9vO9QP_BdmFfRcK5duVWviFAbgg6NoSHw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-03-10 20:49           ` Jeff Layton
     [not found]             ` <20110310154933.1fa017a0-xSBYVWDuneFaJnirhKH9O4GKTjYczspe@public.gmane.org>
2011-03-11  9:01               ` Pavel Shilovsky

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.