On Saturday 08 September 2007 19:43, Goswin von Brederlow wrote:
> Nick Piggin <[EMAIL PROTECTED]> writes:
> > Lustre should probably have to be ported over to write_begin/write_end in
> > order to use it too. With the patches in -mm, if a filesystem is still
> > using prepare_write/commit_write,
Nick Piggin <[EMAIL PROTECTED]> writes:
> Lustre should probably have to be ported over to write_begin/write_end in
> order to use it too. With the patches in -mm, if a filesystem is still using
> prepare_write/commit_write, the vm reverts to a safe path which avoids
> the deadlock (and allows
Nick Piggin [EMAIL PROTECTED] writes:
Lustre should probably have to be ported over to write_begin/write_end in
order to use it too. With the patches in -mm, if a filesystem is still using
prepare_write/commit_write, the vm reverts to a safe path which avoids
the deadlock (and allows
On Saturday 08 September 2007 19:43, Goswin von Brederlow wrote:
Nick Piggin [EMAIL PROTECTED] writes:
Lustre should probably have to be ported over to write_begin/write_end in
order to use it too. With the patches in -mm, if a filesystem is still
using prepare_write/commit_write, the vm
On Saturday 08 September 2007 17:25, Nick Piggin wrote:
> On Saturday 08 September 2007 07:12, Goswin von Brederlow wrote:
> > Nick Piggin <[EMAIL PROTECTED]> writes:
> > > On Saturday 08 September 2007 06:01, Goswin von Brederlow wrote:
> > >> b) a segment boundary
> > >
> > > This is done, as
On Saturday 08 September 2007 07:12, Goswin von Brederlow wrote:
> Nick Piggin <[EMAIL PROTECTED]> writes:
> > On Saturday 08 September 2007 06:01, Goswin von Brederlow wrote:
> >> b) a segment boundary
> >
> > This is done, as I said, because of the deadlock issue. While the issue
> > is more
On Saturday 08 September 2007 07:00, Goswin von Brederlow wrote:
> Nick Piggin <[EMAIL PROTECTED]> writes:
> > Anyway, there are fixes for this deadlock in Andrew's -mm tree, but
> > also a workaround for the NFSD problem in git commit 29dbb3fc. Did
> > you try a later kernel to see if it is fixed
Nick Piggin <[EMAIL PROTECTED]> writes:
> On Saturday 08 September 2007 06:01, Goswin von Brederlow wrote:
>> Nick Piggin <[EMAIL PROTECTED]> writes:
>> > So I believe the problem is that for a multi-segment iovec, we currently
>> > prepare_write/commit_write once for each segment, right? We do
Nick Piggin <[EMAIL PROTECTED]> writes:
> Anyway, there are fixes for this deadlock in Andrew's -mm tree, but
> also a workaround for the NFSD problem in git commit 29dbb3fc. Did
> you try a later kernel to see if it is fixed there?
I had a chance to look up that commit (git clone took a while
Nick Piggin <[EMAIL PROTECTED]> writes:
> On Thursday 06 September 2007 03:41, Bernd Schubert wrote:
> Minor nit: when resubmitting a patch, you should include everything
> (ie. the full changelog of problem statement and fix description) in a
> single mail. It's just a bit easier...
Will do
On Saturday 08 September 2007 06:01, Goswin von Brederlow wrote:
> Nick Piggin <[EMAIL PROTECTED]> writes:
> > On Thursday 06 September 2007 03:41, Bernd Schubert wrote:
> > Minor nit: when resubmitting a patch, you should include everything
> > (ie. the full changelog of problem statement and fix
On Thursday 06 September 2007 03:41, Bernd Schubert wrote:
> > This comment block should be:
> >
> > /**
> > * generic_file_buffered_write - handle an iov
> > * @iocb: file operations
> > * @iov:vector of data to write
> > * @nr_segs:number of iov segments
> > * @pos:
On Thursday 06 September 2007 03:41, Bernd Schubert wrote:
This comment block should be:
/**
* generic_file_buffered_write - handle an iov
* @iocb: file operations
* @iov:vector of data to write
* @nr_segs:number of iov segments
* @pos:position in the file
On Saturday 08 September 2007 06:01, Goswin von Brederlow wrote:
Nick Piggin [EMAIL PROTECTED] writes:
On Thursday 06 September 2007 03:41, Bernd Schubert wrote:
Minor nit: when resubmitting a patch, you should include everything
(ie. the full changelog of problem statement and fix
Nick Piggin [EMAIL PROTECTED] writes:
On Thursday 06 September 2007 03:41, Bernd Schubert wrote:
Minor nit: when resubmitting a patch, you should include everything
(ie. the full changelog of problem statement and fix description) in a
single mail. It's just a bit easier...
Will do next
Nick Piggin [EMAIL PROTECTED] writes:
Anyway, there are fixes for this deadlock in Andrew's -mm tree, but
also a workaround for the NFSD problem in git commit 29dbb3fc. Did
you try a later kernel to see if it is fixed there?
I had a chance to look up that commit (git clone took a while so
Nick Piggin [EMAIL PROTECTED] writes:
On Saturday 08 September 2007 06:01, Goswin von Brederlow wrote:
Nick Piggin [EMAIL PROTECTED] writes:
So I believe the problem is that for a multi-segment iovec, we currently
prepare_write/commit_write once for each segment, right? We do this
It is
On Saturday 08 September 2007 07:00, Goswin von Brederlow wrote:
Nick Piggin [EMAIL PROTECTED] writes:
Anyway, there are fixes for this deadlock in Andrew's -mm tree, but
also a workaround for the NFSD problem in git commit 29dbb3fc. Did
you try a later kernel to see if it is fixed there?
On Saturday 08 September 2007 07:12, Goswin von Brederlow wrote:
Nick Piggin [EMAIL PROTECTED] writes:
On Saturday 08 September 2007 06:01, Goswin von Brederlow wrote:
b) a segment boundary
This is done, as I said, because of the deadlock issue. While the issue
is more completely
On Saturday 08 September 2007 17:25, Nick Piggin wrote:
On Saturday 08 September 2007 07:12, Goswin von Brederlow wrote:
Nick Piggin [EMAIL PROTECTED] writes:
On Saturday 08 September 2007 06:01, Goswin von Brederlow wrote:
b) a segment boundary
This is done, as I said, because of
Hello Randy,
thanks for your review.
On Wednesday 05 September 2007 17:35:29 Randy Dunlap wrote:
> On Wed, 5 Sep 2007 15:45:36 +0200 Bernd Schubert wrote:
> > Hi,
>
> meta-comments:
> > filemap.c | 144
> > +-
> > 1 file changed, 96
Hello Randy,
thanks for your review.
On Wednesday 05 September 2007 17:35:29 Randy Dunlap wrote:
On Wed, 5 Sep 2007 15:45:36 +0200 Bernd Schubert wrote:
Hi,
meta-comments:
filemap.c | 144
+-
1 file changed, 96
22 matches
Mail list logo