All of lore.kernel.org
 help / color / mirror / Atom feed
* [mmotm:master 187/302] mm/page_io.c:405:18: error: 'REQ_HIPRI' undeclared; did you mean 'RWF_HIPRI'?
@ 2018-12-14 23:04 kbuild test robot
  2018-12-15  4:29 ` Jens Axboe
  0 siblings, 1 reply; 2+ messages in thread
From: kbuild test robot @ 2018-12-14 23:04 UTC (permalink / raw)
  To: Jens Axboe
  Cc: kbuild-all, Johannes Weiner, Andrew Morton, Linux Memory Management List

[-- Attachment #1: Type: text/plain, Size: 3283 bytes --]

tree:   git://git.cmpxchg.org/linux-mmotm.git master
head:   6d5b029d523e959579667282e713106a29c193d2
commit: 80509a64e354a3de67cfe4dee1cf51c8a2512e44 [187/302] mm/page_io.c: fix polled swap page in
config: x86_64-rhel-7.2-clear (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        git checkout 80509a64e354a3de67cfe4dee1cf51c8a2512e44
        # save the attached .config to linux build tree
        make ARCH=x86_64 

Note: the mmotm/master HEAD 6d5b029d523e959579667282e713106a29c193d2 builds fine.
      It only hurts bisectibility.

All errors (new ones prefixed by >>):

   mm/page_io.c: In function 'swap_readpage':
>> mm/page_io.c:405:18: error: 'REQ_HIPRI' undeclared (first use in this function); did you mean 'RWF_HIPRI'?
      bio->bi_opf |= REQ_HIPRI;
                     ^~~~~~~~~
                     RWF_HIPRI
   mm/page_io.c:405:18: note: each undeclared identifier is reported only once for each function it appears in

vim +405 mm/page_io.c

   350	
   351	int swap_readpage(struct page *page, bool synchronous)
   352	{
   353		struct bio *bio;
   354		int ret = 0;
   355		struct swap_info_struct *sis = page_swap_info(page);
   356		blk_qc_t qc;
   357		struct gendisk *disk;
   358	
   359		VM_BUG_ON_PAGE(!PageSwapCache(page) && !synchronous, page);
   360		VM_BUG_ON_PAGE(!PageLocked(page), page);
   361		VM_BUG_ON_PAGE(PageUptodate(page), page);
   362		if (frontswap_load(page) == 0) {
   363			SetPageUptodate(page);
   364			unlock_page(page);
   365			goto out;
   366		}
   367	
   368		if (sis->flags & SWP_FS) {
   369			struct file *swap_file = sis->swap_file;
   370			struct address_space *mapping = swap_file->f_mapping;
   371	
   372			ret = mapping->a_ops->readpage(swap_file, page);
   373			if (!ret)
   374				count_vm_event(PSWPIN);
   375			return ret;
   376		}
   377	
   378		ret = bdev_read_page(sis->bdev, swap_page_sector(page), page);
   379		if (!ret) {
   380			if (trylock_page(page)) {
   381				swap_slot_free_notify(page);
   382				unlock_page(page);
   383			}
   384	
   385			count_vm_event(PSWPIN);
   386			return 0;
   387		}
   388	
   389		ret = 0;
   390		bio = get_swap_bio(GFP_KERNEL, page, end_swap_bio_read);
   391		if (bio == NULL) {
   392			unlock_page(page);
   393			ret = -ENOMEM;
   394			goto out;
   395		}
   396		disk = bio->bi_disk;
   397		/*
   398		 * Keep this task valid during swap readpage because the oom killer may
   399		 * attempt to access it in the page fault retry time check.
   400		 */
   401		get_task_struct(current);
   402		bio->bi_private = current;
   403		bio_set_op_attrs(bio, REQ_OP_READ, 0);
   404		if (synchronous)
 > 405			bio->bi_opf |= REQ_HIPRI;
   406		count_vm_event(PSWPIN);
   407		bio_get(bio);
   408		qc = submit_bio(bio);
   409		while (synchronous) {
   410			set_current_state(TASK_UNINTERRUPTIBLE);
   411			if (!READ_ONCE(bio->bi_private))
   412				break;
   413	
   414			if (!blk_poll(disk->queue, qc))
   415				io_schedule();
   416		}
   417		__set_current_state(TASK_RUNNING);
   418		bio_put(bio);
   419	
   420	out:
   421		return ret;
   422	}
   423	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 41457 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [mmotm:master 187/302] mm/page_io.c:405:18: error: 'REQ_HIPRI' undeclared; did you mean 'RWF_HIPRI'?
  2018-12-14 23:04 [mmotm:master 187/302] mm/page_io.c:405:18: error: 'REQ_HIPRI' undeclared; did you mean 'RWF_HIPRI'? kbuild test robot
@ 2018-12-15  4:29 ` Jens Axboe
  0 siblings, 0 replies; 2+ messages in thread
From: Jens Axboe @ 2018-12-15  4:29 UTC (permalink / raw)
  To: kbuild test robot
  Cc: kbuild-all, Johannes Weiner, Andrew Morton, Linux Memory Management List

On 12/14/18 4:04 PM, kbuild test robot wrote:
> tree:   git://git.cmpxchg.org/linux-mmotm.git master
> head:   6d5b029d523e959579667282e713106a29c193d2
> commit: 80509a64e354a3de67cfe4dee1cf51c8a2512e44 [187/302] mm/page_io.c: fix polled swap page in
> config: x86_64-rhel-7.2-clear (attached as .config)
> compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
> reproduce:
>         git checkout 80509a64e354a3de67cfe4dee1cf51c8a2512e44
>         # save the attached .config to linux build tree
>         make ARCH=x86_64 
> 
> Note: the mmotm/master HEAD 6d5b029d523e959579667282e713106a29c193d2 builds fine.
>       It only hurts bisectibility.
> 
> All errors (new ones prefixed by >>):
> 
>    mm/page_io.c: In function 'swap_readpage':
>>> mm/page_io.c:405:18: error: 'REQ_HIPRI' undeclared (first use in this function); did you mean 'RWF_HIPRI'?
>       bio->bi_opf |= REQ_HIPRI;
>                      ^~~~~~~~~
>                      RWF_HIPRI
>    mm/page_io.c:405:18: note: each undeclared identifier is reported only once for each function it appears in

The patch just needs to be ordered after the block tree in -mm, then
there are no concerns.

-- 
Jens Axboe

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-12-15  4:30 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-14 23:04 [mmotm:master 187/302] mm/page_io.c:405:18: error: 'REQ_HIPRI' undeclared; did you mean 'RWF_HIPRI'? kbuild test robot
2018-12-15  4:29 ` Jens Axboe

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.