Re: [PATCH 1/2] NFS: Clear key construction data if the idmap upcall fails

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

 



On Thu, 2012-08-16 at 18:09 -0700, Shai Lazmi wrote:
> I've looked at the repo (http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=blob;f=fs/nfs/idmap.c;h=b701358c39c351d0613d1db29ebcf382ca8cb0a6;hb=ddf343f635fe4440cad528e12f96f28bd50aa099), seems just like what I have
> Can it be already changed on the tree?

No. It hasn't been merged upstream yet.

However, I've now pushed the patches upstream to the NFS bugfixes
branch. Instead of applying the patches, perhaps you could just pull the
full set of NFS client bugfixes by doing

git pull git://git.linux-nfs.org/projects/trondmy/linux-nfs.git bugfixes

Cheers
  Trond

> -----Original Message-----
> From: Myklebust, Trond [mailto:Trond.Myklebust@xxxxxxxxxx]
> Sent: Thursday, August 16, 2012 5:22 PM
> To: Shai Lazmi
> Cc: William Dauchy; Schumaker, Bryan; linux-nfs@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/2] NFS: Clear key construction data if the idmap upcall fails
> 
> On Thu, 2012-08-16 at 17:15 -0700, Shai Lazmi wrote:
> > git apply --check 0001-NFS-Clear-key-construction-data-if-the-idmap-upcall-.patch
> > error: patch failed: fs/nfs/idmap.c:61
> > error: fs/nfs/idmap.c: patch does not apply
> >
> >
> > idmap.c
> > define NFS_UINT_MAXLEN 11
> >
> > static const struct cred *id_resolver_cache;
> > static struct key_type key_type_id_resolver_legacy;
> >
> > struct idmap {
> >         struct rpc_pipe         *idmap_pipe;
> >         struct key_construction *idmap_key_cons;
> >         struct mutex            idmap_mutex;
> > };
> 
> That's how it should more or less look after applying the patch. I'm
> guessing that one of your previous attempts has partially worked, and
> has left your checked out git tree in a "dirty" state.
> 
> Please try the following:
> 
> $ git reset --hard HEAD
> $ git apply --check 0001-NFS-Clear-key-construction-data-if-the-idmap-upcall-.patch
> $ git apply --check 0002-NFS-return-ENOKEY-when-the-upcall-fails-to-map-the-n.patch
> 
> 
> >
> > -----Original Message-----
> > From: Myklebust, Trond [mailto:Trond.Myklebust@xxxxxxxxxx]
> > Sent: Thursday, August 16, 2012 5:14 PM
> > To: Shai Lazmi
> > Cc: William Dauchy; Schumaker, Bryan; linux-nfs@xxxxxxxxxxxxxxx
> > Subject: Re: [PATCH 1/2] NFS: Clear key construction data if the idmap upcall fails
> >
> > On Thu, 2012-08-16 at 17:07 -0700, Shai Lazmi wrote:
> > > Sorry - company's policy ...
> > > What idmap version was this patch used against? The one I have (b701358c) seems to be a bit different (I used latest pull commit ddf343f635fe4440cad528e12f96f28bd50aa099), hence the patch fails again
> >
> > It works fine for me against commit ddf343f635fe4440cad528e12f96f28bd50aa099:
> >
> > [trondmy@lade linux-nfs]$ git branch test-apply ddf343f635fe4440cad528e12f96f28bd50aa099
> > [trondmy@lade linux-nfs]$ git checkout test-apply
> > Switched to branch 'test-apply'
> > [trondmy@lade linux-nfs]$ git am ~/Desktop/for_shai/0001-NFS-Clear-key-construction-data-if-the-idmap-upcall-.patch
> > Applying: NFS: Clear key construction data if the idmap upcall fails
> > [trondmy@lade linux-nfs]$ git am ~/Desktop/for_shai/0002-NFS-return-ENOKEY-when-the-upcall-fails-to-map-the-n.patch
> > Applying: NFS: return -ENOKEY when the upcall fails to map the name
> >
> > What error messages are you seeing?
> >
> > Cheers
> >   Trond
> >
> > > -----Original Message-----
> > > From: Myklebust, Trond [mailto:Trond.Myklebust@xxxxxxxxxx]
> > > Sent: Thursday, August 16, 2012 4:57 PM
> > > To: Shai Lazmi
> > > Cc: William Dauchy; Schumaker, Bryan; linux-nfs@xxxxxxxxxxxxxxx
> > > Subject: Re: [PATCH 1/2] NFS: Clear key construction data if the idmap upcall fails
> > >
> > > On Thu, 2012-08-16 at 16:50 -0700, Shai Lazmi wrote:
> > > > MS outlook
> > >
> > > Don't ever use LookOut for patches. Its "I just reformatted your email"
> > > wonder feature breaks all the whitespace.
> > >
> > > > Best thing is to attached the patch
> > >
> > > We do not normally use attachments on the mailing list because it makes it impossible to comment the patches inline. Ah well, here goes...
> > >
> > > --
> > > Trond Myklebust
> > > Linux NFS client maintainer
> > >
> > > NetApp
> > > Trond.Myklebust@xxxxxxxxxx
> > > www.netapp.com
> > >
> > >
> > > This message and any attached documents contain information from QLogic Corporation or its wholly-owned subsidiaries that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message.
> >
> > --
> > Trond Myklebust
> > Linux NFS client maintainer
> >
> > NetApp
> > Trond.Myklebust@xxxxxxxxxx
> > www.netapp.com
> >
> >
> > This message and any attached documents contain information from QLogic Corporation or its wholly-owned subsidiaries that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message.
> 
> --
> Trond Myklebust
> Linux NFS client maintainer
> 
> NetApp
> Trond.Myklebust@xxxxxxxxxx
> www.netapp.com
> 
> 
> This message and any attached documents contain information from QLogic Corporation or its wholly-owned subsidiaries that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message.

-- 
Trond Myklebust
Linux NFS client maintainer

NetApp
Trond.Myklebust@xxxxxxxxxx
www.netapp.com

��.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