mbox series

[GIT,PULL] fscrypt update for 4.21

Message ID 20190106134912.GA27028@mit.edu (mailing list archive)
State Accepted
Headers show
Series [GIT,PULL] fscrypt update for 4.21 | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/tytso/fscrypt.git tags/fscrypt_for_linus

Message

Theodore Ts'o Jan. 6, 2019, 1:49 p.m. UTC
The following changes since commit 7beb01f74415c56f5992922b5b902b45d365e694:

  f2fs: clean up f2fs_sb_has_##feature_name (2018-11-26 15:53:55 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/tytso/fscrypt.git tags/fscrypt_for_linus

for you to fetch changes up to 8094c3ceb21ad93896fd4d238e8ba41911932eaf:

  fscrypt: add Adiantum support (2019-01-06 08:36:21 -0500)

----------------------------------------------------------------
Add Adiantum support for fscrypt

----------------------------------------------------------------
Eric Biggers (1):
      fscrypt: add Adiantum support

 Documentation/filesystems/fscrypt.rst | 179 ++++++++++++---------
 fs/crypto/crypto.c                    |  28 ++--
 fs/crypto/fname.c                     |  22 +--
 fs/crypto/fscrypt_private.h           |  67 +++++++-
 fs/crypto/keyinfo.c                   | 351 ++++++++++++++++++++++++++++++++----------
 fs/crypto/policy.c                    |   5 +-
 include/uapi/linux/fs.h               |   4 +-
 7 files changed, 468 insertions(+), 188 deletions(-)

Comments

Eric Biggers Jan. 6, 2019, 5:20 p.m. UTC | #1
On Sun, Jan 06, 2019 at 08:49:12AM -0500, Theodore Y. Ts'o wrote:
> The following changes since commit 7beb01f74415c56f5992922b5b902b45d365e694:
> 
>   f2fs: clean up f2fs_sb_has_##feature_name (2018-11-26 15:53:55 -0800)
> 
> are available in the Git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/tytso/fscrypt.git tags/fscrypt_for_linus
> 
> for you to fetch changes up to 8094c3ceb21ad93896fd4d238e8ba41911932eaf:
> 
>   fscrypt: add Adiantum support (2019-01-06 08:36:21 -0500)
> 
> ----------------------------------------------------------------
> Add Adiantum support for fscrypt
> 
> ----------------------------------------------------------------
> Eric Biggers (1):
>       fscrypt: add Adiantum support
> 
>  Documentation/filesystems/fscrypt.rst | 179 ++++++++++++---------
>  fs/crypto/crypto.c                    |  28 ++--
>  fs/crypto/fname.c                     |  22 +--
>  fs/crypto/fscrypt_private.h           |  67 +++++++-
>  fs/crypto/keyinfo.c                   | 351 ++++++++++++++++++++++++++++++++----------
>  fs/crypto/policy.c                    |   5 +-
>  include/uapi/linux/fs.h               |   4 +-
>  7 files changed, 468 insertions(+), 188 deletions(-)

Hi Ted, that tag is still on an old commit.  Probably you forgot to push.

Also, FYI Linus - the reason this is so late in the merge window is that this
patch was in the fscrypt tree which also had the fsverity stuff, which had
concerns raised.  But this patch is independent and has been in linux-next since
Dec 4 (albeit under a different commit ID since it was applied after the
fsverity stuff), and also tested on some Android devices.

- Eric
Eric Biggers Jan. 6, 2019, 5:52 p.m. UTC | #2
On Sun, Jan 06, 2019 at 09:20:07AM -0800, Eric Biggers wrote:
> On Sun, Jan 06, 2019 at 08:49:12AM -0500, Theodore Y. Ts'o wrote:
> > The following changes since commit 7beb01f74415c56f5992922b5b902b45d365e694:
> > 
> >   f2fs: clean up f2fs_sb_has_##feature_name (2018-11-26 15:53:55 -0800)
> > 
> > are available in the Git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/tytso/fscrypt.git tags/fscrypt_for_linus
> > 
> > for you to fetch changes up to 8094c3ceb21ad93896fd4d238e8ba41911932eaf:
> > 
> >   fscrypt: add Adiantum support (2019-01-06 08:36:21 -0500)
> > 
> > ----------------------------------------------------------------
> > Add Adiantum support for fscrypt
> > 
> > ----------------------------------------------------------------
> > Eric Biggers (1):
> >       fscrypt: add Adiantum support
> > 
> >  Documentation/filesystems/fscrypt.rst | 179 ++++++++++++---------
> >  fs/crypto/crypto.c                    |  28 ++--
> >  fs/crypto/fname.c                     |  22 +--
> >  fs/crypto/fscrypt_private.h           |  67 +++++++-
> >  fs/crypto/keyinfo.c                   | 351 ++++++++++++++++++++++++++++++++----------
> >  fs/crypto/policy.c                    |   5 +-
> >  include/uapi/linux/fs.h               |   4 +-
> >  7 files changed, 468 insertions(+), 188 deletions(-)
> 
> Hi Ted, that tag is still on an old commit.  Probably you forgot to push.
> 

Fixed now, so either Ted pushed or I screwed something up.

> Also, FYI Linus - the reason this is so late in the merge window is that this
> patch was in the fscrypt tree which also had the fsverity stuff, which had
> concerns raised.  But this patch is independent and has been in linux-next since
> Dec 4 (albeit under a different commit ID since it was applied after the
> fsverity stuff), and also tested on some Android devices.
> 

Also I forgot: last month I switched my personal desktop (which I'm sending this
email from) over to use an Adiantum encrypted home directory :-)

- Eric
Theodore Ts'o Jan. 6, 2019, 7:56 p.m. UTC | #3
On Sun, Jan 06, 2019 at 09:20:07AM -0800, Eric Biggers wrote:

> Hi Ted, that tag is still on an old commit.  Probably you forgot to push.

My scripts check to see that it was right (at least from my view of
git.kernel.org) before it generates the push.  I just checked and it's
correct now, and I haven't done any intervening pushes.  I think it
must have been a stale cache entry that you saw?

	    	       	     	- Ted
pr-tracker-bot@kernel.org Jan. 6, 2019, 9:35 p.m. UTC | #4
The pull request you sent on Sun, 6 Jan 2019 08:49:12 -0500:

> git://git.kernel.org/pub/scm/linux/kernel/git/tytso/fscrypt.git tags/fscrypt_for_linus

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/baa6707381285e68cc472efba58e7e736057aacc

Thank you!