Skip to content
Commits on Aug 17, 2010
  1. [GSoC Encrypted Storage] Macro handling of libuafs

    Edit code where libuafs ignores printk's and also
    the division function by using the UKERNEL
    macro definition.
    
    Conflicts:
    
    	src/afs/VNOPS/afs_vnop_create.c
    committed Aug 17, 2010
  2. [GSoC Encrypted Storage]Free allocated space in read/write routines

    Free space used due to extenting, in the
    afs_UFSRead and afs_UFSWrite routines.
    
    Minor warnings removed
    committed Aug 16, 2010
  3. [GSoC Encrypted Storage]Move file along with metadata file

    Add code to vnop_rename, to move file along with
    metadata file, if one such exists
    
    Conflicts:
    
    	src/afs/VNOPS/afs_vnop_rename.c
    committed Aug 17, 2010
  4. [GSoC Encrypted Storage]Removal of metadata file on rm filename

    Remove the files associated with a encrypted file,
    checks for an encrypted file using afs_lookup
    
    Conflicts:
    
    	src/afs/VNOPS/afs_vnop_create.c
    committed Aug 17, 2010
  5. [GSoC Encrypted Storage]Metadata file creation/read/write

    Implementation of metadata file. File created on a
    new create in afs_vnop_create. Also each file vcache
    updated with the encryption information by modifying
    the vcache structure.
    
    Reading completely supported now
    
    Conflicts:
    
    	src/afs/VNOPS/afs_vnop_write.c
    committed Aug 17, 2010
  6. [GSoC Encrypted Storage]Read encryption functionality with extents an…

    …d chunking LINUX
    
    Rough edges: iov_count considered to be zero, but
    should be fixable
    committed Jul 7, 2010
  7. [GSoC Encrypted Storage]Encrypt functionality

    1. Adds a file afs_enc.c which presently implements the
    encryption functions.
    2. Adds the headers of afs_enc.c to afs_protoypes.h
    3. Modifies the make file for compilation
    4. Inserts the relevant code in afs_vnop_read/write.c
    5. Moddifies LINUX/osi_vnodeops.c to disable fastpath caching
    in linux.
    6. ROT-13 encryption prototype
    7. Extetns and chunking API
    
    Conflicts:
    
    	src/afs/afs_chunkops.h
    committed Aug 17, 2010
  8. @jaltman @jaltman

    Windows: Fix remove 'register' patch

    The register removal patch broke the Windows build.
    Fix it.
    
    Change-Id: I0121da17a98e1eb35932d18c851989737d9325b5
    Reviewed-on: http://gerrit.openafs.org/2571
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    jaltman committed with jaltman Aug 17, 2010
  9. death to trailing whitespace

    if we're gonna clean up...
    
    Change-Id: I5ab03f29468577b62dacab41a67eadfd8c43f812
    Reviewed-on: http://gerrit.openafs.org/2463
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Marcus Watts committed with Derrick Brashear Aug 16, 2010
  10. death to register

    in soviet compilers, compiler optimizes you.
    stop providing dated (and annoying for debugging) register keywords.
    
    Change-Id: Ibcac0aa3f353fe531b9be0beedca919fb947bfab
    Reviewed-on: http://gerrit.openafs.org/2436
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Marcus Watts committed with Derrick Brashear Jul 14, 2010
Commits on Aug 16, 2010
  1. openafs 1.5.76

    make openafs 1.5.76
    
    Change-Id: I56ef6618ae6ff8d0ee2e06a145aed7d61f98d682
    Reviewed-on: http://gerrit.openafs.org/2537
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Derrick Brashear committed Aug 12, 2010
  2. @jaltman @jaltman

    Windows Change Log for 1.5.76

    Change-Id: Ia5004d37fceb506ed7ca894f31a903d2b8cb66cb
    Reviewed-on: http://gerrit.openafs.org/2540
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    jaltman committed with jaltman Aug 12, 2010
  3. @jaltman @jaltman

    Windows: Return error for SetToken PIOCTL_LOGON if not LocalSystem

    If the PIOCTL_LOGON flag is set and the user is not LocalSystem,
    return Access Denied.
    
    LICENSE MIT
    
    Change-Id: I25f042353fb8b70aba4fb01ba1a5b204470416f5
    Reviewed-on: http://gerrit.openafs.org/2568
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    jaltman committed with jaltman Aug 16, 2010
  4. @jaltman @jaltman

    Windows: prevent buffer overrun in cklog

    The Windows version of klog.exe will overwrite the realm and
    password buffers if the command line input is too long.  Generate
    an error and terminate the program instead.
    
    Change-Id: I80671adcf92e9140f14a943b2677a352d2223eee
    Reviewed-on: http://gerrit.openafs.org/2558
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    jaltman committed with jaltman Aug 16, 2010
  5. @jaltman @jaltman

    Windows: Only Local System Account may set tokens for alternate ids

    Only permit tokens to be set for alternate ids from the system
    account.
    
    LICENSE MIT
    
    Change-Id: Ibeaf16950b0317db99dfebccb9534697a258db78
    Reviewed-on: http://gerrit.openafs.org/2541
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    jaltman committed with jaltman Aug 12, 2010
  6. @jaltman @jaltman

    Windows: Parse Freelance entries and enforce trailing dot but no trai…

    …ling ws
    
    Make sure that there is a trailing dot and no trailing whitespace
    for both Mount Point and Symlink entries read from the Freelance
    registry key.
    
    LICENSE MIT
    
    Change-Id: I339d1bd2a8fc3e5f44362c65e872396adf64fdf6
    Reviewed-on: http://gerrit.openafs.org/2555
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    jaltman committed with jaltman Aug 13, 2010
Commits on Aug 12, 2010
  1. @chazchandler

    autoconf detection of label support

    acinclude.m4 now has struct label support detection and sets
    HAVE_STRUCT_LABEL_SUPPORT in afsconfig.h.  Obviates complicated
    ifdefs in various src/afs/ and src/rx/ files.  Must run regen.sh
    to generate new configure and afsconfig.h.in, then configure will
    detect label support and set afsconfig.h appropriately.  Not sure
    what to do for Windows, but should be a step in the right
    direction.
    
    Fixes issue with compilation of libuafs on IRIX and sunpro by accurately
    detecting label support.
    
    Change-Id: I092eda47c4cfcc2517dc39f5e48b5038c413cb01
    Change-Id: I2ff8449b28db645d9cfd27796b5f89e5c328a2b2
    Reviewed-on: http://gerrit.openafs.org/2516
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    chazchandler committed with Derrick Brashear Aug 4, 2010
  2. @mdionne

    Linux: avoid oops in memcpy_toiovec

    With ADAPT_PMTU enabled, kernel_recvmsg can be called to receive
    errors.  The call is made with no iovec supplied (we pass a NULL
    pointer), but with a size of 256.  In the kernel, the memcpy_toiovec
    function tests for size > 0, and proceeds to check iovec->iov_len,
    which causes an oops.  Fix by setting the size to 0.
    
    Change-Id: I3c672c2e67d0aa5af2bd54d0c14aaeae3bb3c530
    Reviewed-on: http://gerrit.openafs.org/2546
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    mdionne committed with Derrick Brashear Aug 12, 2010
  3. @mdionne

    Linux: Rework statfs super block operations configure test

    The configure test to detect if the statfs super block operation
    needs a dentry argument is based on vfs_statfs, and assumes that
    its signature matches the one of the operation.  In 2.6.36 this is
    no longer true and the test fails.
    
    Rework the test to actually test the operation we're interested in.
    
    This change is required for 2.6.36.
    
    Change-Id: I5a712fb47e48c1bcc7ea2cfbf6c53c2b27627723
    Reviewed-on: http://gerrit.openafs.org/2545
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    mdionne committed with Derrick Brashear Aug 11, 2010
  4. @mdionne

    Linux: switch to evict_inode

    In 2.6.36, the delete_inode and clear_inode inode operations
    are replaced by evict_inode.
    Rename our current clear_inode to evict_inode, and add a few
    things that were previously handled by the generic delete_inode.
    
    This is required for 2.6.36.
    
    Change-Id: I68d75830b43df710ccce173ab36f3bb39074c59e
    Reviewed-on: http://gerrit.openafs.org/2544
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    mdionne committed with Derrick Brashear Aug 11, 2010
  5. @mdionne

    Linux: adapt to truncate sequence changes

    As part of changes to the truncate sequence, inode_setattr() no
    longer exists, and all filesystems have to define the setattr op
    so we can assume that it is not NULL.
    
    Introduce a compat inline function afs_inode_setattr that hides
    the tests and the different versions from the main code.
    
    Note that the existing test for the inode_setattr() return type
    will fail, but the value is no longer used in that case.
    
    This is required for 2.6.36
    
    Change-Id: I2f5e8a0b660b48453d0152b6c4db64e57539f91a
    Reviewed-on: http://gerrit.openafs.org/2543
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    mdionne committed with Derrick Brashear Aug 11, 2010
  6. @mdionne

    Linux: use %pI4 if NIPQUAD is not available

    The definition of NIPQUAD has been removed from the kernel headers
    in the mainline kernel.  Replace it by using the %pI4 format
    specifier which takes the IP address as a single argument.
    
    There should be no change in the output.
    
    This fix is required for 2.6.36.
    
    Change-Id: I269e5bd754b7aef49b005081eb6b3af1b2679293
    Reviewed-on: http://gerrit.openafs.org/2542
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    mdionne committed with Derrick Brashear Aug 11, 2010
  7. disable pmtu error gathering

    don't enable kernel pmtu, don't collect errors.
    
    a report from Marc Dionne suggests there are still issues here
    
    Change-Id: Ib54fa522ba53efe50e8f08b27422ebfc8af7f9e5
    Reviewed-on: http://gerrit.openafs.org/2538
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Derrick Brashear committed Aug 12, 2010
  8. @jaltman @jaltman

    Windows: comparisons of nul strings should match

    When comparing nul strings in cm_strnicmp_utf8 and cm_strnicmp_utf16
    the functions should return 0.
    
    LICENSE MIT
    
    Change-Id: I30c1321f42135a5e0160407467549b13cc92393f
    Reviewed-on: http://gerrit.openafs.org/2539
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    jaltman committed with jaltman Aug 12, 2010
  9. @adeason

    viced: VNOVOL on deleted volumes

    When the volserver deletes a volume, the fileserver should respond to
    future access to that volume with VNOVOL and not VOFFLINE, since the
    volume doesn't exist anymore. Do this in DAFS via a new state,
    VOL_STATE_DELETED, and in non-DAFS by just setting specialStatus to
    VNOVOL.
    
    In the future we should also make sure the vp's for deleted volumes
    get freed after a couple of hours, but not yet.
    
    Change-Id: I6dec3e0a5e9e54f6ad09fad9f2355b513cce3bf6
    Reviewed-on: http://gerrit.openafs.org/2533
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    adeason committed with Derrick Brashear Aug 10, 2010
  10. @adeason

    volser: Ignore duplicate file tags when restoring

    Currently, if the volserver encounters multiple file tags for a vnode
    when restoring a volume dump, it will create extra vnodes which are
    not recorded anywhere, and are leaked when the volume is removed.
    Instead of doing that, ignore any duplicate file tags that are
    encountered (which is what happens with other tags), and log a
    warning.
    
    Change-Id: I1ac76b0217096d013db355b899f1a53174d20eb5
    Reviewed-on: http://gerrit.openafs.org/2531
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    adeason committed with Derrick Brashear Aug 6, 2010
Commits on Aug 11, 2010
  1. @SimonWilkinson @jaltman

    LINUX: More flock fixes

    Turns out the we have two almost identical lock routines on Linux,
    one for POSIX locks, and one for flocks. Fix them both. Also,
    while we're at it, make the same fixes in the LINUX24 tree.
    
    Thanks to Andrew Deason for pointing out the oversight in the
    original patch (5abe706)
    
    Change-Id: I59f0f1e12af36a97e490df36537ac17e8ed87303
    Reviewed-on: http://gerrit.openafs.org/2535
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    SimonWilkinson committed with jaltman Aug 11, 2010
  2. fs discon support alternate uid

    this is a short-term hack. macos lacks pags, and anywhere without
    pags won't have tokens when fs discon online is run as root unless
    the tokens are explicitly set for root... which can be bad mojo.
    
    instead, allow a uid to be passed in.
    
    Change-Id: I44ebf9468976fa43d449f72d392fbb9e830e44d6
    Reviewed-on: http://gerrit.openafs.org/2515
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Derrick Brashear committed Aug 4, 2010
  3. macos disconnected vnode holding fix

    tweak how we hold vcaches to avoid issues with the underlying
    vnode having an iocount underrun due to races.
    
    Change-Id: I96526ed52c11aac1124a8cf66458ba3e25e7efb2
    Reviewed-on: http://gerrit.openafs.org/2501
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Derrick Brashear committed Aug 2, 2010
  4. @SimonWilkinson

    LINUX: Use correct type of error in flock code

    The flock code gets a kernel error code from afs_posix_lock_file()
    but then passes this error code through afs_convert_code (which sees
    that it is already negative, and returns EIO). Instead, we should just
    return afs_posix_lock_file()'s code direct to our caller.
    
    Change-Id: I28bbc2dc8535d28a4e9e9f564392fd4cbfa76569
    Reviewed-on: http://gerrit.openafs.org/2534
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    SimonWilkinson committed with Derrick Brashear Aug 11, 2010
  5. @jaltman @jaltman

    Windows: cm_Analyze should treat VOFFLINE like VMOVED or VNOVOL

    Volume package bugs in the file server can result in VOFFLINE
    being returned to the client instead of VNOVOL or VMOVED.  As
    a result the Unix CM treats VOFFLINE the same as VMOVED and VNOVOL.
    The Windows client has not.  As a result, bugs in the file server
    can cause the Windows client to lose if the volume has in fact
    been moved to another server.
    
    As part of this change, the volume location list is updated prior
    to the volume status being applied to the server from which the
    error was received.
    
    LICENSE MIT
    
    Change-Id: I01036aa9d1999d0ba6822beb1b73500d365bf0b3
    Reviewed-on: http://gerrit.openafs.org/2532
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    jaltman committed with jaltman Aug 11, 2010
  6. @adeason

    salvage: update inodes array after CopyAndSalvage

    CopyAndSalvage makes a copy of the directory to salvage it. So, the
    new directory will have a new inode. Update the array of directory
    inodes to reflect this. Without this, we can fail to alter the
    copied/salvaged directory later on, since we will try to read/write to
    the wrong inode.
    
    Change-Id: I2466981e8ee54c5aecf1a166a37d8146b126de24
    Reviewed-on: http://gerrit.openafs.org/2530
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    adeason committed with Derrick Brashear Jul 27, 2010
  7. @adeason

    salvage: Remove redundant 'code' declaration

    'code' is declared at the top of SalvageVolume; no need to declare it
    again in a lower block.
    
    Change-Id: I9b48830c17e5043e6abcda4dd4dd3cc040031d9a
    Reviewed-on: http://gerrit.openafs.org/2529
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    adeason committed with Derrick Brashear Jul 27, 2010
  8. @adeason

    salvage: Do not look at not-found root dir

    If we have not found any root directory, don't even look at
    'oldrootdir' or 'rootdir', since they will be uninitialized at this
    point.
    
    Change-Id: Ia1a7037e06a223a7e1fbc48c0bfadd3b7aaa0e4b
    Reviewed-on: http://gerrit.openafs.org/2528
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    adeason committed with Derrick Brashear Jul 28, 2010
Something went wrong with that request. Please try again.