| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 38/49] xfs: remove __KERNEL__ check from xfs_dir2_leaf.c |
| From: | Mark Tinguely <tinguely@xxxxxxx> |
| Date: | Tue, 06 Aug 2013 11:12:18 -0500 |
| Cc: | xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1374216324-8781-39-git-send-email-david@xxxxxxxxxxxxx> |
| References: | <1374216324-8781-1-git-send-email-david@xxxxxxxxxxxxx> <1374216324-8781-39-git-send-email-david@xxxxxxxxxxxxx> |
| User-agent: | Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120122 Thunderbird/9.0 |
On 07/19/13 01:45, Dave Chinner wrote: From: Dave Chinner<dchinner@xxxxxxxxxx> It's actually an ifndef section, which means it is only included in userspace. however, it's deep within the libxfs code, so it's unlikely that the condition checked in userspace can actually occur (search an empty leaf) through the libxfs interfaces. i.e. if it can happen in usrspace, it can happen in the kernel, so remove it from userspace too.... Signed-off-by: Dave Chinner<dchinner@xxxxxxxxxx> --- user equivalent of the for Linux 3.12 kernel patch 31/49. compares okay to the kernel patch. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 37/49] xfs: remove __KERNEL__ from debug code, Mark Tinguely |
|---|---|
| Next by Date: | Re: [PATCH 39/49] xfs: move kernel specific type definitions to xfs.h, Mark Tinguely |
| Previous by Thread: | Re: [PATCH 37/49] xfs: remove __KERNEL__ from debug code, Mark Tinguely |
| Next by Thread: | Re: [PATCH 39/49] xfs: move kernel specific type definitions to xfs.h, Mark Tinguely |
| Indexes: | [Date] [Thread] [Top] [All Lists] |