4.4.86-stable review
Greg Kroah-Hartman
maintainercover 4.4.86-stable review
This is the start of the stable review cycle for the 4.4.86 release. There are 16 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Sep 2 15:42:07 UTC 2017. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.4.86-rc1.gz
or in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.4.y
and the diffstat can be found below.
thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:
Greg Kroah-Hartman <[email protected]>
Linux 4.4.86-rc1
Chao Yu <[email protected]>
f2fs: do more integrity verification for superblock
Greg Kroah-Hartman <[email protected]>
drm/i915: fix compiler warning in drivers/gpu/drm/i915/intel_uncore.c
Hannes Reinecke <[email protected]>
scsi: sg: reset 'res_in_use' after unlinking reserved array
Hannes Reinecke <[email protected]>
scsi: sg: protect accesses to 'reserved' page array
Dave Martin <[email protected]>
arm64: fpsimd: Prevent registers leaking across exec
Arnd Bergmann <[email protected]>
x86/io: Add "memory" clobber to insb/insw/insl/outsb/outsw/outsl
Mark Rutland <[email protected]>
arm64: mm: abort uaccess retries upon fatal signal
James Smart <[email protected]>
lpfc: Fix Device discovery failures during switch reboot test.
Jiri Slaby <[email protected]>
p54: memset(0) whole array
Javier González <[email protected]>
lightnvm: initialize ppa_addr in dev_to_generic_addr()
Martin Liska <[email protected]>
gcov: support GCC 7.1
Florian Meier <[email protected]>
gcov: add support for gcc version >= 6
Wolfram Sang <[email protected]>
i2c: jz4780: drop superfluous init
Colin Ian King <[email protected]>
btrfs: remove duplicate const specifier
Takashi Iwai <[email protected]>
ALSA: au88x0: Fix zero clear of stream->resources
Arnd Bergmann <[email protected]>
scsi: isci: avoid array subscript warning
-------------
Diffstat:
Makefile | 4 +- arch/arm64/kernel/fpsimd.c | 2 + arch/arm64/mm/fault.c | 5 +- arch/x86/include/asm/io.h | 4 +- drivers/gpu/drm/i915/intel_uncore.c | 3 +- drivers/i2c/busses/i2c-jz4780.c | 4 -- drivers/net/wireless/p54/fwio.c | 2 +- drivers/scsi/isci/remote_node_context.c | 3 + drivers/scsi/lpfc/lpfc_els.c | 5 +- drivers/scsi/sg.c | 49 ++++++++++------- fs/btrfs/volumes.c | 2 +- fs/f2fs/super.c | 98 +++++++++++++++++++++++++++++++++ include/linux/lightnvm.h | 1 + kernel/gcov/base.c | 6 ++ kernel/gcov/gcc_4_7.c | 4 +- sound/pci/au88x0/au88x0_core.c | 14 ++--- 16 files changed, 162 insertions(+), 44 deletions(-)
Greg Kroah-Hartman
maintainerPATCH 1/16 scsi: isci: avoid array subscript warning
4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Arnd Bergmann <[email protected]>
commit 5cfa2a3c7342bd0b50716c8bb32ee491af43c785 upstream.
I'm getting a new warning with gcc-7:
isci/remote_node_context.c: In function 'sci_remote_node_context_destruct':isci/remote_node_context.c:69:16: error: array subscript is above array bounds [-Werror=array-bounds]
This is odd, since we clearly cover all values for enum scis_sds_remote_node_context_states here. Anyway, checking for an array overflow can't harm and it makes the warning go away.
--- drivers/scsi/isci/remote_node_context.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/scsi/isci/remote_node_context.c +3 -0
@@ -66,6 +66,9 @@ const char *rnc_state_name(enum scis_sds {static const char *const strings[] =RNC_STATES ; +if (state >=ARRAY_SIZE (strings)) +return "UNKNOWN" ; +return strings[state]; }#undef C
Greg Kroah-Hartman
maintainerPATCH 2/16 ALSA: au88x0: Fix zero clear of stream->resources
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Takashi Iwai <[email protected]>
commit 639db596165746ca87bbcb56559b094fd9042890 upstream.
There are a few calls of memset() to stream->resources, but they all are called in a wrong size, sizeof(unsigned char) * VORTEX_RESOURCE_LAST, while this field is a u32 array. This may leave the memories not zero-cleared.
Fix it by replacing them with a simpler sizeof(stream->resources) instead.
--- sound/pci/au88x0/au88x0_core.c | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-)
sound/pci/au88x0/au88x0_core.c +5 -9
@@ -2150,8 +2150,7 @@ vortex_adb_allocroute(vortex_t *vortex, stream->resources, en, VORTEX_RESOURCE_SRC)) < 0) {memset (stream->resources,0 , -sizeof (unsigned char ) * -VORTEX_RESOURCE_LAST ); +sizeof (stream->resources));return -EBUSY ; }if (stream->type !=VORTEX_PCM_A3D ) {
@@ -2161,7 +2160,7 @@ vortex_adb_allocroute(vortex_t *vortex, VORTEX_RESOURCE_MIXIN)) < 0) {memset (stream->resources,0 , -sizeof (unsigned char ) *VORTEX_RESOURCE_LAST ); +sizeof (stream->resources));return -EBUSY ; } }
@@ -2174,8 +2173,7 @@ vortex_adb_allocroute(vortex_t *vortex, stream->resources, en, VORTEX_RESOURCE_A3D)) < 0) {memset (stream->resources,0 , -sizeof (unsigned char ) * -VORTEX_RESOURCE_LAST ); +sizeof (stream->resources));dev_err (vortex->card->dev,"out of A3D sources. Sorry\n" );return -EBUSY ;
@@ -2289,8 +2287,7 @@ vortex_adb_allocroute(vortex_t *vortex, VORTEX_RESOURCE_MIXOUT)) < 0) {memset (stream->resources,0 , -sizeof (unsigned char ) * -VORTEX_RESOURCE_LAST ); +sizeof (stream->resources));return -EBUSY ; } if ((src[i] =
@@ -2298,8 +2295,7 @@ vortex_adb_allocroute(vortex_t *vortex, stream->resources, en, VORTEX_RESOURCE_SRC)) < 0) {memset (stream->resources,0 , -sizeof (unsigned char ) * -VORTEX_RESOURCE_LAST ); +sizeof (stream->resources));return -EBUSY ; } }
Greg Kroah-Hartman
maintainerPATCH 3/16 btrfs: remove duplicate const specifier
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Colin Ian King <[email protected]>
commit fb75d857a31d600cc0c37b8c7d914014f7fa3f9a upstream.
duplicate const is redundant so remove it
--- fs/btrfs/volumes.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
fs/btrfs/volumes.c +1 -1
@@ -108,7 +108,7 @@ const struct btrfs_raid_attr btrfs_raid_ }, }; -const u64 const btrfs_raid_group[BTRFS_NR_RAID_TYPES ] = { +const u64 btrfs_raid_group[BTRFS_NR_RAID_TYPES ] = { [BTRFS_RAID_RAID10 ] =BTRFS_BLOCK_GROUP_RAID10 , [BTRFS_RAID_RAID1 ] =BTRFS_BLOCK_GROUP_RAID1 , [BTRFS_RAID_DUP ] =BTRFS_BLOCK_GROUP_DUP ,
Greg Kroah-Hartman
maintainerPATCH 4/16 i2c: jz4780: drop superfluous init
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Wolfram Sang <[email protected]>
commit 27bfeb5a0619554d9734fb39e14f0e80fa7c342c upstream.
David reported that the length for memset was incorrect (element sizes were not taken into account). Then I saw that we are clearing kzalloced memory, so we can simply drop this code.
--- drivers/i2c/busses/i2c-jz4780.c | 4 ---- 1 file changed, 4 deletions(-)
drivers/i2c/busses/i2c-jz4780.c +0 -4
@@ -786,10 +786,6 @@ static int jz4780_i2c_probe(struct platfjz4780_i2c_writew (i2c,JZ4780_I2C_INTM ,0x0 ); - i2c->cmd =0 ; -memset (i2c->cmd_buf,0 ,BUFSIZE ); -memset (i2c->data_buf,0 ,BUFSIZE ); - i2c->irq =platform_get_irq (pdev,0 ); ret =devm_request_irq (&pdev->dev, i2c->irq, jz4780_i2c_irq,0 ,dev_name (&pdev->dev), i2c);
Greg Kroah-Hartman
maintainerPATCH 5/16 gcov: add support for gcc version >= 6
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Florian Meier <[email protected]>
commit d02038f972538b93011d78c068f44514fbde0a8c upstream.
--- kernel/gcov/gcc_4_7.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
kernel/gcov/gcc_4_7.c +1 -1
@@ -18,7 +18,7 @@#include <linux/vmalloc.h> #include "gcov.h" -#if __GNUC__ ==5 && __GNUC_MINOR__ >=1 +#if (__GNUC__ >5 ) || (__GNUC__ ==5 && __GNUC_MINOR__ >=1 )#define GCOV_COUNTERS 10#elif __GNUC__ ==4 && __GNUC_MINOR__ >=9 #define GCOV_COUNTERS 9
Greg Kroah-Hartman
maintainerPATCH 6/16 gcov: support GCC 7.1
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Martin Liska <[email protected]>
commit 05384213436ab690c46d9dfec706b80ef8d671ab upstream.
Starting from GCC 7.1, __gcov_exit is a new symbol expected to be implemented in a profiling runtime.
--- kernel/gcov/base.c | 6 ++++++ kernel/gcov/gcc_4_7.c | 4 +++- 2 files changed, 9 insertions(+), 1 deletion(-)
kernel/gcov/base.c +6 -0
@@ -98,6 +98,12 @@ void __gcov_merge_icall_topn(gcov_type * }EXPORT_SYMBOL (__gcov_merge_icall_topn); +void __gcov_exit (void ) +{ +/* Unused. */ +} +EXPORT_SYMBOL (__gcov_exit); +/** * gcov_enable_events - enable event reporting through gcov_event() *
kernel/gcov/gcc_4_7.c +3 -1
@@ -18,7 +18,9 @@#include <linux/vmalloc.h> #include "gcov.h" -#if (__GNUC__ >5 ) || (__GNUC__ ==5 && __GNUC_MINOR__ >=1 ) +#if (__GNUC__ >=7 ) +#define GCOV_COUNTERS 9 +#elif (__GNUC__ >5 ) || (__GNUC__ ==5 && __GNUC_MINOR__ >=1 )#define GCOV_COUNTERS 10#elif __GNUC__ ==4 && __GNUC_MINOR__ >=9 #define GCOV_COUNTERS 9
Greg Kroah-Hartman
maintainerPATCH 8/16 p54: memset(0) whole array
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jiri Slaby <[email protected]>
commit 6f17581788206444cbbcdbc107498f85e9765e3d upstream.
gcc 7 complains:drivers/net/wireless/intersil/p54/fwio.c: In function 'p54_scan':drivers/net/wireless/intersil/p54/fwio.c:491:4: warning: 'memset' used with length equal to number of elements without multiplication by element size [-Wmemset-elt-size]
Fix that by passing the correct size to memset.
--- drivers/net/wireless/p54/fwio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/wireless/p54/fwio.c +1 -1
@@ -488,7 +488,7 @@ int p54_scan(struct p54_common *priv, u1 entry +=sizeof (__le16); chan->pa_points_per_curve =8 ; -memset (chan->curve_data,0 ,sizeof (*chan->curve_data)); +memset (chan->curve_data,0 ,sizeof (chan->curve_data));memcpy (chan->curve_data, entry,sizeof (struct p54_pa_curve_data_sample ) *min ((u8 )8 , curve_data->points_per_channel));
Greg Kroah-Hartman
maintainerPATCH 9/16 lpfc: Fix Device discovery failures during switch reboot test.
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: James Smart <[email protected]>
commit 342b59caa66240b670285d519fdfe2c44289b516 upstream.
When the switch is rebooted, the lpfc driver fails to log into the fabric, and Unexpected timeout message is seen.
Fix: Do not issue RegVFI if the FLOGI was internally aborted.
--- drivers/scsi/lpfc/lpfc_els.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
drivers/scsi/lpfc/lpfc_els.c +4 -1
@@ -1054,7 +1054,10 @@ stop_rr_fcf_flogi:lpfc_sli4_unreg_all_rpis (vport); } } -lpfc_issue_reg_vfi (vport); + +/* Do not register VFI if the driver aborted FLOGI */ +if (!lpfc_error_lost_link (irsp)) +lpfc_issue_reg_vfi (vport);lpfc_nlp_put (ndlp); goto out; }
Greg Kroah-Hartman
maintainerPATCH 10/16 arm64: mm: abort uaccess retries upon fatal signal
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Mark Rutland <[email protected]>
commit 289d07a2dc6c6b6f3e4b8a62669320d99dbe6c3d upstream.
When there's a fatal signal pending, arm64's do_page_fault() implementation returns 0. The intent is that we'll return to the faulting userspace instruction, delivering the signal on the way.
However, if we take a fatal signal during fixing up a uaccess, this results in a return to the faulting kernel instruction, which will be instantly retried, resulting in the same fault being taken forever. As the task never reaches userspace, the signal is not delivered, and the task is left unkillable. While the task is stuck in this state, it can inhibit the forward progress of the system.
To avoid this, we must ensure that when a fatal signal is pending, we apply any necessary fixup for a faulting kernel instruction. Thus we will return to an error path, and it is up to that code to make forward progress towards delivering the fatal signal.
--- arch/arm64/mm/fault.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
arch/arm64/mm/fault.c +4 -1
@@ -313,8 +313,11 @@ retry:* signal first. We do not need to release the mmap_sem because it * would already be released in __lock_page_or_retry in mm/filemap.c. */ -if ((fault &VM_FAULT_RETRY ) &&fatal_signal_pending (current)) +if ((fault &VM_FAULT_RETRY ) &&fatal_signal_pending (current)) { +if (!user_mode (regs)) + goto no_context;return 0 ; + }/* * Major/minor page fault accounting is only done on the initial
Greg Kroah-Hartman
maintainerPATCH 12/16 arm64: fpsimd: Prevent registers leaking across exec
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Dave Martin <[email protected]>
commit 096622104e14d8a1db4860bd557717067a0515d2 upstream.
There are some tricky dependencies between the different stages of flushing the FPSIMD register state during exec, and these can race with context switch in ways that can cause the old task's regs to leak across. In particular, a context switch during the memset() can cause some of the task's old FPSIMD registers to reappear.
Disabling preemption for this small window would be no big deal for
performance: preemption is already disabled for similar scenarios like updating the FPSIMD registers in sigreturn.
So, instead of rearranging things in ways that might swap existing subtle bugs for new ones, this patch just disables preemption around the FPSIMD state flushing so that races of this type can't occur here. This brings fpsimd_flush_thread() into line with other code paths.
---
For stable only.
3.17.x-4.0.x don't appear active, and this patch isn't sufficient to fix them (they would need 674c242c9323 also).
arch/arm64/kernel/fpsimd.c | 2 ++ 1 file changed, 2 insertions(+)
arch/arm64/kernel/fpsimd.c +2 -0
@@ -157,9 +157,11 @@ void fpsimd_thread_switch(struct task_stvoid fpsimd_flush_thread (void ) { +preempt_disable ();memset (¤t->thread.fpsimd_state,0 ,sizeof (struct fpsimd_state ));fpsimd_flush_task_state (current);set_thread_flag (TIF_FOREIGN_FPSTATE ); +preempt_enable (); }/*
Greg Kroah-Hartman
maintainerPATCH 13/16 scsi: sg: protect accesses to reserved page array
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hannes Reinecke <[email protected]>
commit 1bc0eb0446158cc76562176b80623aa119afee5b upstream.
The 'reserved' page array is used as a short-cut for mapping data, saving us to allocate pages per request. However, the 'reserved' array is only capable of holding one request, so this patch introduces a mutex for protect 'sg_fd' against concurrent accesses.
[[email protected]: backport to 3.18-4.9, fixup for bad ioctl SG_SET_FORCE_LOW_DMA code removed in later versions and not modified by the original patch.]
--- drivers/scsi/sg.c | 47 ++++++++++++++++++++++++++--------------------- 1 file changed, 26 insertions(+), 21 deletions(-)
drivers/scsi/sg.c +26 -21
@@ -153,6 +153,7 @@ typedef struct sg_fd { /* holds the stastruct sg_device *parentdp;/* owning device */ wait_queue_head_t read_wait;/* queue read until command done */ rwlock_t rq_list_lock;/* protect access to list in req_arr */ +struct mutex f_mutex;/* protect against changes in this fd */ int timeout;/* defaults to SG_DEFAULT_TIMEOUT */ int timeout_user;/* defaults to SG_DEFAULT_TIMEOUT_USER */ Sg_scatter_hold reserve;/* buffer held for this file descriptor */
@@ -166,6 +167,7 @@ typedef struct sg_fd { /* holds the staunsigned char next_cmd_len;/* 0: automatic, >0: use on next write() */ char keep_orphan;/* 0 -> drop orphan (def), 1 -> keep for read() */ char mmap_called;/* 0 -> mmap() never called on this fd */ +char res_in_use;/* 1 -> 'reserve' array in use */ struct kref f_ref;struct execute_work ew; } Sg_fd;
@@ -209,7 +211,6 @@ static void sg_remove_sfp(struct kref *)static Sg_request *sg_get_rq_mark (Sg_fd * sfp,int pack_id);static Sg_request *sg_add_request (Sg_fd * sfp);static int sg_remove_request (Sg_fd * sfp,Sg_request * srp); -static int sg_res_in_use (Sg_fd * sfp);static Sg_device *sg_get_dev (int dev);static void sg_device_destroy (struct kref *kref);
@@ -625,6 +626,7 @@ sg_write(struct file *filp, const char _ } buf +=SZ_SG_HEADER ;__get_user (opcode, buf); +mutex_lock (&sfp->f_mutex);if (sfp->next_cmd_len >0 ) { cmd_size = sfp->next_cmd_len; sfp->next_cmd_len =0 ;/* reset so only this write() effected */
@@ -633,6 +635,7 @@ sg_write(struct file *filp, const char _if ((opcode >=0xc0 ) && old_hdr.twelve_byte) cmd_size =12 ; } +mutex_unlock (&sfp->f_mutex);SCSI_LOG_TIMEOUT (4 ,sg_printk (KERN_INFO , sdp,"sg_write: scsi opcode=0x%02x, cmd_size=%d\n" , (int ) opcode, cmd_size));/* Determine buffer size. */
@@ -732,7 +735,7 @@ sg_new_write(Sg_fd *sfp, struct file *fisg_remove_request (sfp, srp);return -EINVAL ;/* either MMAP_IO or DIRECT_IO (not both) */ } -if (sg_res_in_use (sfp)) { +if (sfp->res_in_use) {sg_remove_request (sfp, srp);return -EBUSY ;/* reserve buffer already being used */ }
@@ -902,7 +905,7 @@ sg_ioctl(struct file *filp, unsigned intreturn result;if (val) { sfp->low_dma =1 ; -if ((0 == sfp->low_dma) && (0 ==sg_res_in_use (sfp))) { +if ((0 == sfp->low_dma) && !sfp->res_in_use) { val = (int ) sfp->reserve.bufflen;sg_remove_scat (sfp, &sfp->reserve);sg_build_reserve (sfp, val);
@@ -977,12 +980,18 @@ sg_ioctl(struct file *filp, unsigned intreturn -EINVAL ; val =min_t (int, val,max_sectors_bytes (sdp->device->request_queue)); +mutex_lock (&sfp->f_mutex);if (val != sfp->reserve.bufflen) { -if (sg_res_in_use (sfp) || sfp->mmap_called) +if (sfp->mmap_called || + sfp->res_in_use) { +mutex_unlock (&sfp->f_mutex);return -EBUSY ; + } +sg_remove_scat (sfp, &sfp->reserve);sg_build_reserve (sfp, val); } +mutex_unlock (&sfp->f_mutex);return 0 ;case SG_GET_RESERVED_SIZE : val = min_t(int, sfp->reserve.bufflen,
@@ -1737,13 +1746,22 @@ sg_start_req(Sg_request *srp, unsigned c md = &map_data;if (md) { -if (!sg_res_in_use (sfp) && dxfer_len <= rsv_schp->bufflen) +mutex_lock (&sfp->f_mutex); +if (dxfer_len <= rsv_schp->bufflen && + !sfp->res_in_use) { + sfp->res_in_use =1 ;sg_link_reserve (sfp, srp, dxfer_len); -else { + }else if ((hp->flags &SG_FLAG_MMAP_IO ) && sfp->res_in_use) { +mutex_unlock (&sfp->f_mutex); +return -EBUSY ; + }else { res =sg_build_indirect (req_schp, sfp, dxfer_len); -if (res) +if (res) { +mutex_unlock (&sfp->f_mutex);return res; + } } +mutex_unlock (&sfp->f_mutex); md->pages = req_schp->pages; md->page_order = req_schp->page_order;
@@ -2145,6 +2163,7 @@ sg_add_sfp(Sg_device * sdp)rwlock_init (&sfp->rq_list_lock);kref_init (&sfp->f_ref); +mutex_init (&sfp->f_mutex); sfp->timeout =SG_DEFAULT_TIMEOUT ; sfp->timeout_user =SG_DEFAULT_TIMEOUT_USER ; sfp->force_packid =SG_DEF_FORCE_PACK_ID ;
@@ -2220,20 +2239,6 @@ sg_remove_sfp(struct kref *kref)schedule_work (&sfp->ew.work); } -static int -sg_res_in_use (Sg_fd * sfp) -{ -const Sg_request *srp; -unsigned long iflags; - -read_lock_irqsave (&sfp->rq_list_lock, iflags); -for (srp = sfp->headrp; srp; srp = srp->nextrp) -if (srp->res_used) -break ; -read_unlock_irqrestore (&sfp->rq_list_lock, iflags); -return srp ?1 :0 ; -} -#ifdef CONFIG_SCSI_PROC_FS static int sg_idr_max_id (int id,void *p,void *data)
Greg Kroah-Hartman
maintainerPATCH 14/16 scsi: sg: reset res_in_use after unlinking reserved array
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hannes Reinecke <[email protected]>
commit e791ce27c3f6a1d3c746fd6a8f8e36c9540ec6f9 upstream.
Once the reserved page array is unused we can reset the 'res_in_use' state; here we can do a lazy update without holding the mutex as we only need to check against concurrent access, not concurrent release.
[mkp: checkpatch]
--- drivers/scsi/sg.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/scsi/sg.c +2 -0
@@ -2052,6 +2052,8 @@ sg_unlink_reserve(Sg_fd * sfp, Sg_reques req_schp->sglist_len =0 ; sfp->save_scat_len =0 ; srp->res_used =0 ; +/* Called without mutex lock to avoid deadlock */ + sfp->res_in_use =0 ; } static Sg_request *
Greg Kroah-Hartman
maintainerPATCH 16/16 f2fs: do more integrity verification for superblock
In reply to Greg Kroah-Hartman4.4-stable review patch. If anyone has any objections, please let me know.
------------------
From: Chao Yu <[email protected]>
commit 9a59b62fd88196844cee5fff851bee2cfd7afb6e upstream.
Do more sanity check for superblock during ->mount.
--- fs/f2fs/super.c | 98 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 98 insertions(+)
fs/f2fs/super.c +98 -0
@@ -991,6 +991,79 @@ static inline bool sanity_check_area_boureturn false; } +static inline bool sanity_check_area_boundary (struct super_block *sb, +struct f2fs_super_block *raw_super) +{ +u32 segment0_blkaddr =le32_to_cpu (raw_super->segment0_blkaddr); +u32 cp_blkaddr =le32_to_cpu (raw_super->cp_blkaddr); +u32 sit_blkaddr =le32_to_cpu (raw_super->sit_blkaddr); +u32 nat_blkaddr =le32_to_cpu (raw_super->nat_blkaddr); +u32 ssa_blkaddr =le32_to_cpu (raw_super->ssa_blkaddr); +u32 main_blkaddr =le32_to_cpu (raw_super->main_blkaddr); +u32 segment_count_ckpt =le32_to_cpu (raw_super->segment_count_ckpt); +u32 segment_count_sit =le32_to_cpu (raw_super->segment_count_sit); +u32 segment_count_nat =le32_to_cpu (raw_super->segment_count_nat); +u32 segment_count_ssa =le32_to_cpu (raw_super->segment_count_ssa); +u32 segment_count_main =le32_to_cpu (raw_super->segment_count_main); +u32 segment_count =le32_to_cpu (raw_super->segment_count); +u32 log_blocks_per_seg =le32_to_cpu (raw_super->log_blocks_per_seg); + +if (segment0_blkaddr != cp_blkaddr) { +f2fs_msg (sb,KERN_INFO , +"Mismatch start address, segment0(%u) cp_blkaddr(%u)" , + segment0_blkaddr, cp_blkaddr); +return true; + } + +if (cp_blkaddr + (segment_count_ckpt << log_blocks_per_seg) != + sit_blkaddr) { +f2fs_msg (sb,KERN_INFO , +"Wrong CP boundary, start(%u) end(%u) blocks(%u)" , + cp_blkaddr, sit_blkaddr, + segment_count_ckpt << log_blocks_per_seg); +return true; + } + +if (sit_blkaddr + (segment_count_sit << log_blocks_per_seg) != + nat_blkaddr) { +f2fs_msg (sb,KERN_INFO , +"Wrong SIT boundary, start(%u) end(%u) blocks(%u)" , + sit_blkaddr, nat_blkaddr, + segment_count_sit << log_blocks_per_seg); +return true; + } + +if (nat_blkaddr + (segment_count_nat << log_blocks_per_seg) != + ssa_blkaddr) { +f2fs_msg (sb,KERN_INFO , +"Wrong NAT boundary, start(%u) end(%u) blocks(%u)" , + nat_blkaddr, ssa_blkaddr, + segment_count_nat << log_blocks_per_seg); +return true; + } + +if (ssa_blkaddr + (segment_count_ssa << log_blocks_per_seg) != + main_blkaddr) { +f2fs_msg (sb,KERN_INFO , +"Wrong SSA boundary, start(%u) end(%u) blocks(%u)" , + ssa_blkaddr, main_blkaddr, + segment_count_ssa << log_blocks_per_seg); +return true; + } + +if (main_blkaddr + (segment_count_main << log_blocks_per_seg) != + segment0_blkaddr + (segment_count << log_blocks_per_seg)) { +f2fs_msg (sb,KERN_INFO , +"Wrong MAIN_AREA boundary, start(%u) end(%u) blocks(%u)" , + main_blkaddr, + segment0_blkaddr + (segment_count << log_blocks_per_seg), + segment_count_main << log_blocks_per_seg); +return true; + } + +return false; +} +static int sanity_check_raw_super (struct super_block *sb,struct f2fs_super_block *raw_super) {
@@ -1028,6 +1101,14 @@ static int sanity_check_raw_super(structreturn 1 ; } +/* check log blocks per segment */ +if (le32_to_cpu (raw_super->log_blocks_per_seg) !=9 ) { +f2fs_msg (sb,KERN_INFO , +"Invalid log blocks per segment (%u)\n" , +le32_to_cpu (raw_super->log_blocks_per_seg)); +return 1 ; + } +/* Currently, support 512/1024/2048/4096 bytes sector size */ if (le32_to_cpu(raw_super->log_sectorsize) > F2FS_MAX_LOG_SECTOR_SIZE ||
@@ -1109,6 +1190,23 @@ static int sanity_check_ckpt(struct f2fsf2fs_msg (sbi->sb,KERN_ERR ,"A bug case: need to run fsck" );return 1 ; } + +/* check reserved ino info */ +if (le32_to_cpu (raw_super->node_ino) !=1 || +le32_to_cpu (raw_super->meta_ino) !=2 || +le32_to_cpu (raw_super->root_ino) !=3 ) { +f2fs_msg (sb,KERN_INFO , +"Invalid Fs Meta Ino: node(%u) meta(%u) root(%u)" , +le32_to_cpu (raw_super->node_ino), +le32_to_cpu (raw_super->meta_ino), +le32_to_cpu (raw_super->root_ino)); +return 1 ; + } + +/* check CP/SIT/NAT/SSA/MAIN_AREA area boundary */ +if (sanity_check_area_boundary (sb, raw_super)) +return 1 ; +return 0 ; }
Greg Kroah-Hartman
maintainerRe: [PATCH 4.4 16/16] f2fs: do more integrity verification for superblock
Quoted 1 line · Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know.
I object, this doesn't build, I shouldn't have included it, sorry for the noise.
thanks,
greg k-h
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.4.86 release. There are 16 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Sep 2 15:42:07 UTC 2017. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 9 lines
kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.4.86-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-4.4.y
and the diffstat can be found below.thanks,
greg k-h
Compiled and booted on my test system. No dmesg regressions.
thanks,
-- Shuah
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.4.86 release. There are 16 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sat Sep 2 15:42:07 UTC 2017. Anything received after that time might be too late.
Test results are for v4.4.85-16-ga0e79fa.
Build results:
total: 145 pass: 145 fail: 0 Qemu test results: total: 115 pass: 115 fail: 0
Details are available at http://kerneltests.org/builders.
Guenter