Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753464Ab2KCG3H (ORCPT ); Sat, 3 Nov 2012 02:29:07 -0400 Received: from mail-da0-f46.google.com ([209.85.210.46]:54545 "EHLO mail-da0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752939Ab2KCG3D (ORCPT ); Sat, 3 Nov 2012 02:29:03 -0400 From: Namjae Jeon To: hirofumi@mail.parknet.co.jp, akpm@linux-foundation.org Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Namjae Jeon , Amit Sahrawat Subject: [PATCH v2 3/3] fat: warn when discard request fails other than EOPNOTSUPP Date: Sat, 3 Nov 2012 15:28:54 +0900 Message-Id: <1351924134-14320-1-git-send-email-linkinjeon@gmail.com> X-Mailer: git-send-email 1.7.9.5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1359 Lines: 43 FAT also warn user then the discard request fails as ext4(http://patchwork.ozlabs.org/patch/192669/) Signed-off-by: Namjae Jeon Signed-off-by: Amit Sahrawat --- fs/fat/fatent.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/fs/fat/fatent.c b/fs/fat/fatent.c index 260705c..fc1f407 100644 --- a/fs/fat/fatent.c +++ b/fs/fat/fatent.c @@ -583,12 +583,15 @@ int fat_free_clusters(struct inode *inode, int cluster) */ if (cluster != fatent.entry + 1) { int nr_clus = fatent.entry - first_cl + 1; - - sb_issue_discard(sb, - fat_clus_to_blknr(sbi, first_cl), - nr_clus * sbi->sec_per_clus, - GFP_NOFS, 0); - + sector_t sblk = fat_clus_to_blknr(sbi, first_cl); + sector_t nrblk = nr_clus * sbi->sec_per_clus; + + err = sb_issue_discard(sb, + sblk, nrblk, GFP_NOFS, 0); + if (err && err != -EOPNOTSUPP) + fat_msg(sb, KERN_WARNING, "discard request for" + " block:%llu count:%llu failed" + " with %d", sblk, nrblk, err); first_cl = cluster; } } -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/