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

Re: Pre-approval request for dpkg sync() changes for squeeze



Hi,

On Fri, Oct 22, 2010 at 11:35:54AM +0200, Guillem Jover wrote:
>   1) Switch back from sync() to fsync() before rename() (while keeping
>      the sync() code around for the benefit of other distributions
>      that might not want to switch just yet). So to avoid unrelated
>      I/O when there's background work being done for example. This
>      hack also only works on Linux where sync() is synchronous, so
>      it would unify that code path for all dpkg supported platforms.
> 
>      Bug: #588339
> 
>   <http://git.hadrons.org/?p=debian/dpkg/dpkg.git;a=commitdiff;h=87740373>

I'm still not sure we have all needed information.  Would you mind doing
a matrix what this change would cause?  I.e. if we get performance
regressions on ext4, btrfs or even ext3 with the squeeze kernel (we're
really only interested in that one, not some random other revision)
or if we even get data safety regressions.

Thanks
Philipp Kern

Attachment: signature.asc
Description: Digital signature


Reply to: