From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="lyFJ3EZx" Received: from mail-pl1-x62d.google.com (mail-pl1-x62d.google.com [IPv6:2607:f8b0:4864:20::62d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 55B2E10EF; Mon, 20 Nov 2023 21:37:08 -0800 (PST) Received: by mail-pl1-x62d.google.com with SMTP id d9443c01a7336-1cf57430104so16108315ad.1; Mon, 20 Nov 2023 21:37:08 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1700545027; x=1701149827; darn=vger.kernel.org; h=in-reply-to:subject:cc:to:from:message-id:date:from:to:cc:subject :date:message-id:reply-to; bh=QrByEh4HtrdrZh+TurV7QStnfhXuPXFCWiwNep0ga/M=; b=lyFJ3EZxbmhn1A8WbKJwyxn3IuTylpoxO1EzU4dfa8HCr6jFC07lwWfiotVyRyPSiz u7Ca6akSBxOUVjxlAk9+hcvKpqonKwxOeKlRCJBC+QtcGDYIZPSnJGCqcjWUozmqITv4 GuLjOxIj7moloEHWESPjWRQ8esU2GUJb/hvUvI1Gx+zqaA5d6Rd3a8go39idOB4fUYu+ CXMnDrJbnEV6bVkR1HTZ5PoybwZTmsttA1ctYodChR9j1/uyTyx4dAnp6dS3PJJ2JQPV x1LH22SkXaEQTgc2w9ZpdiNM34SI9SlcMDTOiw21I/Abb1cl5h/QUTKv+awCVwhfumgr 8EVA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1700545027; x=1701149827; h=in-reply-to:subject:cc:to:from:message-id:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=QrByEh4HtrdrZh+TurV7QStnfhXuPXFCWiwNep0ga/M=; b=op/pCVDj8IQT0OtMQQl3awXMJnf3JyHr/d63mEnsCK+CkR8QnvGwC939JRcmQF+RBy VBE42HaCpkOMTpL2cJsuS0hPi2MzB5cHFjfHVJTEEXpKtBDvuEq/IXGZtM+PLpbcsCnq e/JpGH94Yk8rRrqmPQjvHUfS8Tz3vjXGgjnLgFjRK2vrb+rsUx5prfw6gZIOnkotFoWj 6cVwz6PjRGOrtdZR49vBFMFcO6bfUuR25Y8gAnD7JK/9QJnmw3fRUGsZfXdTLpbNuyWb CuocTliujZPYTE5itehEewFkJMea4en3+3PyHSLyNCvtmYCoHX0Q6aD5Bbiezb3sxRJY ivZQ== X-Gm-Message-State: AOJu0Yzr0kH+YrUb1qVPQ3mYi/ef1n5nPkEg0ZFMMUmhdfVUQ/NZUV34 wJU9KRKEmABAMii4bPQhClGjLtc7n4E= X-Google-Smtp-Source: AGHT+IHrjYDFU/5fcDTmlV3i0L+bDqr5nK4xRZjP93bGVB4QcT5/TFL+feYNarEcP+XkAEG6Xjy1qQ== X-Received: by 2002:a17:902:c94c:b0:1cf:5362:7e88 with SMTP id i12-20020a170902c94c00b001cf53627e88mr7961546pla.32.1700545026678; Mon, 20 Nov 2023 21:37:06 -0800 (PST) Received: from dw-tp ([129.41.58.19]) by smtp.gmail.com with ESMTPSA id c4-20020a170902c2c400b001c726147a46sm6939204pla.234.2023.11.20.21.37.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 21:37:06 -0800 (PST) Date: Tue, 21 Nov 2023 11:06:53 +0530 Message-Id: <877cmby8ei.fsf@doe.com> From: Ritesh Harjani (IBM) To: Christoph Hellwig Cc: linux-ext4@vger.kernel.org, Jan Kara , linux-fsdevel@vger.kernel.org Subject: Re: [RFC 1/3] ext2: Fix ki_pos update for DIO buffered-io fallback case In-Reply-To: Precedence: bulk X-Mailing-List: linux-ext4@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Christoph Hellwig writes: > On Tue, Nov 21, 2023 at 12:35:19AM +0530, Ritesh Harjani (IBM) wrote: >> Commit "filemap: update ki_pos in generic_perform_write", made updating >> of ki_pos into common code in generic_perform_write() function. >> This also causes generic/091 to fail. >> >> Fixes: 182c25e9c157 ("filemap: update ki_pos in generic_perform_write") > > Looks like this really was an in-flight collision with: > fb5de4358e1a ("ext2: Move direct-io to use iomap"). How did we manage > to not notice the failure for months, though? Yes, it was due to in-flight collision. I found it during this conversion and also noticed that generic/091 fails on upstream kernel. > > Otherwise looks good: > > Reviewed-by: Christoph Hellwig Thanks! -ritesh