/[LeafOK_CVS]/lbbs/src/section_list.c
ViewVC logotype

Diff of /lbbs/src/section_list.c

Parent Directory Parent Directory | Revision Log Revision Log | View Patch Patch

Revision 1.13 by sysadm, Fri May 23 14:04:05 2025 UTC Revision 1.20 by sysadm, Sun May 25 08:15:03 2025 UTC
# Line 14  Line 14 
14   *                                                                         *   *                                                                         *
15   ***************************************************************************/   ***************************************************************************/
16    
17    #define _GNU_SOURCE
18    
19  #include "section_list.h"  #include "section_list.h"
20  #include "log.h"  #include "log.h"
21  #include "trie_dict.h"  #include "trie_dict.h"
# Line 24  Line 26 
26  #include <stdlib.h>  #include <stdlib.h>
27  #include <errno.h>  #include <errno.h>
28  #include <sys/param.h>  #include <sys/param.h>
29    #include <sys/sem.h>
30  #include <sys/shm.h>  #include <sys/shm.h>
31  #include <sys/ipc.h>  #include <sys/ipc.h>
32    
33    #ifdef _SEM_SEMUN_UNDEFINED
34    union semun
35    {
36            int val;                           /* Value for SETVAL */
37            struct semid_ds *buf;  /* Buffer for IPC_STAT, IPC_SET */
38            unsigned short *array; /* Array for GETALL, SETALL */
39            struct seminfo *__buf; /* Buffer for IPC_INFO
40                                                              (Linux-specific) */
41    };
42    #endif // #ifdef _SEM_SEMUN_UNDEFINED
43    
44  #define ARTICLE_BLOCK_PER_SHM 400                 // sizeof(ARTICLE_BLOCK) * ARTICLE_BLOCK_PER_SHM is the size of each shm segment to allocate  #define ARTICLE_BLOCK_PER_SHM 400                 // sizeof(ARTICLE_BLOCK) * ARTICLE_BLOCK_PER_SHM is the size of each shm segment to allocate
45  #define ARTICLE_BLOCK_SHM_COUNT_LIMIT 256 // limited by length (8-bit) of proj_id in ftok(path, proj_id)  #define ARTICLE_BLOCK_SHM_COUNT_LIMIT 200 // limited by length (8-bit) of proj_id in ftok(path, proj_id)
46  #define ARTICLE_BLOCK_PER_POOL (ARTICLE_BLOCK_PER_SHM * ARTICLE_BLOCK_SHM_COUNT_LIMIT)  #define ARTICLE_BLOCK_PER_POOL (ARTICLE_BLOCK_PER_SHM * ARTICLE_BLOCK_SHM_COUNT_LIMIT)
47    
48  #define CALCULATE_PAGE_THRESHOLD 100 // Adjust to tune performance of move topic  #define CALCULATE_PAGE_THRESHOLD 100 // Adjust to tune performance of move topic
49    
50    #define SID_STR_LEN 5 // 32-bit + NULL
51    
52  struct article_block_t  struct article_block_t
53  {  {
54          ARTICLE articles[ARTICLE_PER_BLOCK];          ARTICLE articles[ARTICLE_PER_BLOCK];
# Line 58  struct article_block_pool_t Line 74  struct article_block_pool_t
74  };  };
75  typedef struct article_block_pool_t ARTICLE_BLOCK_POOL;  typedef struct article_block_pool_t ARTICLE_BLOCK_POOL;
76    
77    static int article_block_pool_shmid;
78  static ARTICLE_BLOCK_POOL *p_article_block_pool = NULL;  static ARTICLE_BLOCK_POOL *p_article_block_pool = NULL;
79    
80    static int section_list_pool_semid;
81  static int section_list_pool_shmid;  static int section_list_pool_shmid;
82  static SECTION_LIST *p_section_list_pool = NULL;  static SECTION_LIST *p_section_list_pool = NULL;
83    
84  static int section_list_count = 0;  static int section_list_count = 0;
85  static TRIE_NODE *p_trie_dict_section_list = NULL;  static TRIE_NODE *p_trie_dict_section_by_name = NULL;
86    static TRIE_NODE *p_trie_dict_section_by_sid = NULL;
87    
88  int article_block_init(const char *filename, int block_count)  int article_block_init(const char *filename, int block_count)
89  {  {
# Line 89  int article_block_init(const char *filen Line 109  int article_block_init(const char *filen
109                  return -2;                  return -2;
110          }          }
111    
112          p_article_block_pool = calloc(1, sizeof(ARTICLE_BLOCK_POOL));          // Allocate shared memory
113          if (p_article_block_pool == NULL)          proj_id = ARTICLE_BLOCK_SHM_COUNT_LIMIT; // keep different from proj_id used to create block shm
114            key = ftok(filename, proj_id);
115            if (key == -1)
116          {          {
117                  log_error("calloc(ARTICLE_BLOCK_POOL) OOM\n");                  log_error("ftok(%s, %d) error (%d)\n", filename, proj_id, errno);
118                  return -2;                  return -3;
119          }          }
120    
121          // Allocate shared memory          size = sizeof(ARTICLE_BLOCK_POOL);
122            shmid = shmget(key, size, IPC_CREAT | IPC_EXCL | 0600);
123            if (shmid == -1)
124            {
125                    log_error("shmget(article_block_pool_shm, size = %d) error (%d)\n", size, errno);
126                    return -3;
127            }
128            p_shm = shmat(shmid, NULL, 0);
129            if (p_shm == (void *)-1)
130            {
131                    log_error("shmat(shmid = %d) error (%d)\n", shmid, errno);
132                    return -3;
133            }
134    
135            article_block_pool_shmid = shmid;
136            p_article_block_pool = p_shm;
137    
138          p_article_block_pool->shm_count = 0;          p_article_block_pool->shm_count = 0;
139          pp_block_next = &(p_article_block_pool->p_block_free_list);          pp_block_next = &(p_article_block_pool->p_block_free_list);
140    
# Line 105  int article_block_init(const char *filen Line 143  int article_block_init(const char *filen
143                  block_count_in_shm = MIN(block_count, ARTICLE_BLOCK_PER_SHM);                  block_count_in_shm = MIN(block_count, ARTICLE_BLOCK_PER_SHM);
144                  block_count -= block_count_in_shm;                  block_count -= block_count_in_shm;
145    
146                  proj_id = getpid() + p_article_block_pool->shm_count;                  proj_id = p_article_block_pool->shm_count;
147                  key = ftok(filename, proj_id);                  key = ftok(filename, proj_id);
148                  if (key == -1)                  if (key == -1)
149                  {                  {
# Line 114  int article_block_init(const char *filen Line 152  int article_block_init(const char *filen
152                          return -3;                          return -3;
153                  }                  }
154    
155                  size = sizeof(shmid) + sizeof(ARTICLE_BLOCK) * (size_t)block_count_in_shm;                  size = sizeof(ARTICLE_BLOCK) * (size_t)block_count_in_shm;
156                  shmid = shmget(key, size, IPC_CREAT | IPC_EXCL | 0600);                  shmid = shmget(key, size, IPC_CREAT | IPC_EXCL | 0600);
157                  if (shmid == -1)                  if (shmid == -1)
158                  {                  {
# Line 158  int article_block_init(const char *filen Line 196  int article_block_init(const char *filen
196    
197  void article_block_cleanup(void)  void article_block_cleanup(void)
198  {  {
199          if (p_article_block_pool != NULL)          if (p_article_block_pool == NULL)
200          {          {
201                  for (int i = 0; i < p_article_block_pool->shm_count; i++)                  return;
202            }
203    
204            for (int i = 0; i < p_article_block_pool->shm_count; i++)
205            {
206                    if (shmdt((p_article_block_pool->shm_pool + i)->p_shm) == -1)
207                  {                  {
208                          if (shmdt((p_article_block_pool->shm_pool + i)->p_shm) == -1)                          log_error("shmdt(shmid = %d) error (%d)\n", (p_article_block_pool->shm_pool + i)->shmid, errno);
209                          {                  }
                                 log_error("shmdt(shmid = %d) error (%d)\n", (p_article_block_pool->shm_pool + i)->shmid, errno);  
                         }  
210    
211                          if (shmctl((p_article_block_pool->shm_pool + i)->shmid, IPC_RMID, NULL) == -1)                  if (shmctl((p_article_block_pool->shm_pool + i)->shmid, IPC_RMID, NULL) == -1)
212                          {                  {
213                                  log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", (p_article_block_pool->shm_pool + i)->shmid, errno);                          log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", (p_article_block_pool->shm_pool + i)->shmid, errno);
                         }  
214                  }                  }
215            }
216    
217                  free(p_article_block_pool);          if (shmdt(p_article_block_pool) == -1)
218                  p_article_block_pool = NULL;          {
219                    log_error("shmdt(shmid = %d) error (%d)\n", article_block_pool_shmid, errno);
220          }          }
221    
222            if (shmctl(article_block_pool_shmid, IPC_RMID, NULL) == -1)
223            {
224                    log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", article_block_pool_shmid, errno);
225            }
226    
227            p_article_block_pool = NULL;
228  }  }
229    
230  inline static ARTICLE_BLOCK *pop_free_article_block(void)  inline static ARTICLE_BLOCK *pop_free_article_block(void)
# Line 297  ARTICLE *article_block_find_by_index(int Line 346  ARTICLE *article_block_find_by_index(int
346    
347          if (index < 0 || index / ARTICLE_PER_BLOCK >= p_article_block_pool->block_count)          if (index < 0 || index / ARTICLE_PER_BLOCK >= p_article_block_pool->block_count)
348          {          {
349                  log_error("section_data_find_article_by_index(%d) is out of boundary of block [0, %d)\n", index, p_article_block_pool->block_count);                  log_error("article_block_find_by_index(%d) is out of boundary of block [0, %d)\n", index, p_article_block_pool->block_count);
350                  return NULL;                  return NULL;
351          }          }
352    
# Line 305  ARTICLE *article_block_find_by_index(int Line 354  ARTICLE *article_block_find_by_index(int
354    
355          if (index % ARTICLE_PER_BLOCK >= p_block->article_count)          if (index % ARTICLE_PER_BLOCK >= p_block->article_count)
356          {          {
357                  log_error("section_data_find_article_by_index(%d) is out of boundary of article [0, %d)\n", index, p_block->article_count);                  log_error("article_block_find_by_index(%d) is out of boundary of article [0, %d)\n", index, p_block->article_count);
358                  return NULL;                  return NULL;
359          }          }
360    
361          return (p_block->articles + (index % ARTICLE_PER_BLOCK));          return (p_block->articles + (index % ARTICLE_PER_BLOCK));
362  }  }
363    
364  extern int section_list_pool_init(const char *filename)  extern int section_list_init(const char *filename)
365  {  {
366            int semid;
367          int shmid;          int shmid;
368          int proj_id;          int proj_id;
369          key_t key;          key_t key;
370          size_t size;          size_t size;
371          void *p_shm;          void *p_shm;
372            union semun arg;
373            int i;
374    
375          if (p_section_list_pool == NULL || p_trie_dict_section_list == NULL)          if (p_section_list_pool == NULL || p_trie_dict_section_by_name == NULL || p_trie_dict_section_by_sid == NULL)
376          {          {
377                  section_list_pool_cleanup();                  section_list_cleanup();
         }  
   
         p_section_list_pool = calloc(BBS_max_section, sizeof(SECTION_LIST));  
         if (p_section_list_pool == NULL)  
         {  
                 log_error("calloc(%d SECTION_LIST) OOM\n", BBS_max_section);  
                 return -1;  
378          }          }
379    
380          proj_id = (int)(time(NULL) % getpid());          proj_id = (int)(time(NULL) % getpid());
# Line 340  extern int section_list_pool_init(const Line 385  extern int section_list_pool_init(const
385                  return -3;                  return -3;
386          }          }
387    
388          size = sizeof(shmid) + sizeof(SECTION_LIST) * BBS_max_section;          size = 2 * (BBS_max_section + 1); // r_sem and w_sem per section, the last pair for all sections
389            semid = semget(key, (int)size, IPC_CREAT | IPC_EXCL | 0600);
390            if (semid == -1)
391            {
392                    log_error("semget(section_list_pool_sem, size = %d) error (%d)\n", size, errno);
393                    return -3;
394            }
395    
396            // Initialize sem value to 0
397            arg.val = 0;
398            for (i = 0; i < size; i++)
399            {
400                    if (semctl(semid, i, SETVAL, arg) == -1)
401                    {
402                            log_error("semctl(section_list_pool_sem, SETVAL) error (%d)\n", errno);
403                            return -3;
404                    }
405            }
406    
407            section_list_pool_semid = semid;
408    
409            size = sizeof(SECTION_LIST) * BBS_max_section;
410          shmid = shmget(key, size, IPC_CREAT | IPC_EXCL | 0600);          shmid = shmget(key, size, IPC_CREAT | IPC_EXCL | 0600);
411          if (shmid == -1)          if (shmid == -1)
412          {          {
413                  log_error("shmget(section_list_pool, size = %d) error (%d)\n", size, errno);                  log_error("shmget(section_list_pool_shm, size = %d) error (%d)\n", size, errno);
414                  return -3;                  return -3;
415          }          }
416          p_shm = shmat(shmid, NULL, 0);          p_shm = shmat(shmid, NULL, 0);
# Line 358  extern int section_list_pool_init(const Line 424  extern int section_list_pool_init(const
424          p_section_list_pool = p_shm;          p_section_list_pool = p_shm;
425          section_list_count = 0;          section_list_count = 0;
426    
427          p_trie_dict_section_list = trie_dict_create();          p_trie_dict_section_by_name = trie_dict_create();
428          if (p_trie_dict_section_list == NULL)          if (p_trie_dict_section_by_name == NULL)
429            {
430                    log_error("trie_dict_create() OOM\n", BBS_max_section);
431                    return -2;
432            }
433    
434            p_trie_dict_section_by_sid = trie_dict_create();
435            if (p_trie_dict_section_by_sid == NULL)
436          {          {
437                  log_error("trie_dict_create() OOM\n", BBS_max_section);                  log_error("trie_dict_create() OOM\n", BBS_max_section);
438                  return -2;                  return -2;
# Line 368  extern int section_list_pool_init(const Line 441  extern int section_list_pool_init(const
441          return 0;          return 0;
442  }  }
443    
444    inline static void sid_to_str(int32_t sid, char *p_sid_str)
445    {
446            uint32_t u_sid;
447            int i;
448    
449            u_sid = (uint32_t)sid;
450            for (i = 0; i < SID_STR_LEN - 1; i++)
451            {
452                    p_sid_str[i] = (char)(u_sid % 255 + 1);
453                    u_sid /= 255;
454            }
455            p_sid_str[i] = '\0';
456    }
457    
458  SECTION_LIST *section_list_create(int32_t sid, const char *sname, const char *stitle, const char *master_name)  SECTION_LIST *section_list_create(int32_t sid, const char *sname, const char *stitle, const char *master_name)
459  {  {
460          SECTION_LIST *p_section;          SECTION_LIST *p_section;
461            char sid_str[SID_STR_LEN];
462    
463          if (p_section_list_pool == NULL || p_trie_dict_section_list == NULL)          if (p_section_list_pool == NULL || p_trie_dict_section_by_name == NULL || p_trie_dict_section_by_sid == NULL)
464          {          {
465                  log_error("session_list_pool not initialized\n");                  log_error("session_list_pool not initialized\n");
466                  return NULL;                  return NULL;
# Line 384  SECTION_LIST *section_list_create(int32_ Line 472  SECTION_LIST *section_list_create(int32_
472                  return NULL;                  return NULL;
473          }          }
474    
475            sid_to_str(sid, sid_str);
476    
477          p_section = p_section_list_pool + section_list_count;          p_section = p_section_list_pool + section_list_count;
478    
479          p_section->sid = sid;          p_section->sid = sid;
# Line 397  SECTION_LIST *section_list_create(int32_ Line 487  SECTION_LIST *section_list_create(int32_
487          strncpy(p_section->master_name, master_name, sizeof(p_section->master_name - 1));          strncpy(p_section->master_name, master_name, sizeof(p_section->master_name - 1));
488          p_section->master_name[sizeof(p_section->master_name - 1)] = '\0';          p_section->master_name[sizeof(p_section->master_name - 1)] = '\0';
489    
490          if (trie_dict_set(p_trie_dict_section_list, sname, section_list_count) != 1)          if (trie_dict_set(p_trie_dict_section_by_name, sname, section_list_count) != 1)
491          {          {
492                  log_error("trie_dict_set(section_data, %s, %d) error\n", sname, section_list_count);                  log_error("trie_dict_set(section, %s, %d) error\n", sname, section_list_count);
493                    return NULL;
494            }
495    
496            if (trie_dict_set(p_trie_dict_section_by_sid, sid_str, section_list_count) != 1)
497            {
498                    log_error("trie_dict_set(section, %d, %d) error\n", sid, section_list_count);
499                  return NULL;                  return NULL;
500          }          }
501    
# Line 423  void section_list_reset_articles(SECTION Line 519  void section_list_reset_articles(SECTION
519          p_section->last_page_visible_article_count = 0;          p_section->last_page_visible_article_count = 0;
520  }  }
521    
522  void section_list_pool_cleanup(void)  void section_list_cleanup(void)
523  {  {
524          if (p_trie_dict_section_list != NULL)          if (p_trie_dict_section_by_name != NULL)
525            {
526                    trie_dict_destroy(p_trie_dict_section_by_name);
527                    p_trie_dict_section_by_name = NULL;
528            }
529    
530            if (p_trie_dict_section_by_sid != NULL)
531          {          {
532                  trie_dict_destroy(p_trie_dict_section_list);                  trie_dict_destroy(p_trie_dict_section_by_sid);
533                  p_trie_dict_section_list = NULL;                  p_trie_dict_section_by_sid = NULL;
534          }          }
535    
536          if (p_section_list_pool != NULL)          if (p_section_list_pool != NULL)
# Line 443  void section_list_pool_cleanup(void) Line 545  void section_list_pool_cleanup(void)
545                  {                  {
546                          log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", section_list_pool_shmid, errno);                          log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", section_list_pool_shmid, errno);
547                  }                  }
548    
549                    if (semctl(section_list_pool_semid, 0, IPC_RMID) == -1)
550                    {
551                            log_error("semctl(semid = %d, IPC_RMID) error (%d)\n", section_list_pool_semid, errno);
552                    }
553          }          }
554    
555          section_list_count = 0;          section_list_count = 0;
# Line 452  SECTION_LIST *section_list_find_by_name( Line 559  SECTION_LIST *section_list_find_by_name(
559  {  {
560          int64_t index;          int64_t index;
561    
562          if (p_section_list_pool == NULL || p_trie_dict_section_list == NULL)          if (p_section_list_pool == NULL || p_trie_dict_section_by_name == NULL)
563            {
564                    log_error("section_list not initialized\n");
565                    return NULL;
566            }
567    
568            if (trie_dict_get(p_trie_dict_section_by_name, sname, &index) != 1)
569            {
570                    log_error("trie_dict_get(section, %s) error\n", sname);
571                    return NULL;
572            }
573    
574            return (p_section_list_pool + index);
575    }
576    
577    SECTION_LIST *section_list_find_by_sid(int32_t sid)
578    {
579            int64_t index;
580            char sid_str[SID_STR_LEN];
581    
582            if (p_section_list_pool == NULL || p_trie_dict_section_by_sid == NULL)
583          {          {
584                  log_error("section_list not initialized\n");                  log_error("section_list not initialized\n");
585                  return NULL;                  return NULL;
586          }          }
587    
588          if (trie_dict_get(p_trie_dict_section_list, sname, &index) != 1)          sid_to_str(sid, sid_str);
589    
590            if (trie_dict_get(p_trie_dict_section_by_sid, sid_str, &index) != 1)
591          {          {
592                  log_error("trie_dict_get(section_data, %s) error\n", sname);                  log_error("trie_dict_get(section, %d) error\n", sid);
593                  return NULL;                  return NULL;
594          }          }
595    
# Line 487  int section_list_append_article(SECTION_ Line 616  int section_list_append_article(SECTION_
616                  return -1;                  return -1;
617          }          }
618    
619            if (p_section->sid != p_article_src->sid)
620            {
621                    log_error("section_list_append_article() error: section sid %d != article sid %d\n", p_section->sid, p_article_src->sid);
622                    return -2;
623            }
624    
625          if (p_section->article_count >= BBS_article_limit_per_section)          if (p_section->article_count >= BBS_article_limit_per_section)
626          {          {
627                  log_error("section_list_append_article() error: article_count reach limit in section %d\n", p_section->sid);                  log_error("section_list_append_article() error: article_count reach limit in section %d\n", p_section->sid);
# Line 519  int section_list_append_article(SECTION_ Line 654  int section_list_append_article(SECTION_
654          // AID of articles should be strictly ascending          // AID of articles should be strictly ascending
655          if (p_article_src->aid <= last_aid)          if (p_article_src->aid <= last_aid)
656          {          {
657                  log_error("section_data_append_article(aid=%d) error: last_aid=%d\n", p_article_src->aid, last_aid);                  log_error("section_list_append_article(aid=%d) error: last_aid=%d\n", p_article_src->aid, last_aid);
658                  return -3;                  return -3;
659          }          }
660    
# Line 617  int section_list_set_article_visible(SEC Line 752  int section_list_set_article_visible(SEC
752                  return -1; // Not found                  return -1; // Not found
753          }          }
754    
755            if (p_section->sid != p_article->sid)
756            {
757                    log_error("section_list_set_article_visible() error: section sid %d != article sid %d\n", p_section->sid, p_article->sid);
758                    return -2;
759            }
760    
761          if (p_article->visible == visible)          if (p_article->visible == visible)
762          {          {
763                  return 0; // Already set                  return 0; // Already set
# Line 783  int section_list_calculate_page(SECTION_ Line 924  int section_list_calculate_page(SECTION_
924    
925          if (start_aid > 0)          if (start_aid > 0)
926          {          {
927                    p_article = article_block_find_by_aid(start_aid);
928                    if (p_article == NULL)
929                    {
930                            return -1; // Not found
931                    }
932    
933                    if (p_section->sid != p_article->sid)
934                    {
935                            log_error("section_list_calculate_page() error: section sid %d != start article sid %d\n", p_section->sid, p_article->sid);
936                            return -2;
937                    }
938    
939                  p_article = section_list_find_article_with_offset(p_section, start_aid, &page, &offset, &p_next);                  p_article = section_list_find_article_with_offset(p_section, start_aid, &page, &offset, &p_next);
940                  if (p_article == NULL)                  if (p_article == NULL)
941                  {                  {
# Line 851  int section_list_calculate_page(SECTION_ Line 1004  int section_list_calculate_page(SECTION_
1004          return 0;          return 0;
1005  }  }
1006    
1007  int section_list_count_of_topic_articles(int32_t aid)  int article_count_of_topic(int32_t aid)
1008  {  {
1009          ARTICLE *p_article;          ARTICLE *p_article;
1010          int article_count;          int article_count;
# Line 866  int section_list_count_of_topic_articles Line 1019  int section_list_count_of_topic_articles
1019    
1020          do          do
1021          {          {
1022                    if (p_article->tid != 0 && p_article->tid != aid)
1023                    {
1024                            log_error("article_count_of_topic(%d) error: article %d not linked to the topic\n", aid, p_article->aid);
1025                            break;
1026                    }
1027    
1028                  article_count++;                  article_count++;
1029                  p_article = p_article->p_topic_next;                  p_article = p_article->p_topic_next;
1030          } while (p_article->aid != aid);          } while (p_article->aid != aid);
# Line 891  int section_list_move_topic(SECTION_LIST Line 1050  int section_list_move_topic(SECTION_LIST
1050                  return -1;                  return -1;
1051          }          }
1052    
1053          if ((p_article = section_list_find_article_with_offset(p_section_src, aid, &page, &offset, &p_next)) == NULL)          if ((p_article = article_block_find_by_aid(aid)) == NULL)
1054          {          {
1055                  log_error("section_list_move_topic() error: article %d not found in section %d\n", aid, p_section_src->sid);                  log_error("section_list_move_topic() error: article %d not found in block\n", aid);
1056                    return -2;
1057            }
1058    
1059            if (p_section_src->sid != p_article->sid)
1060            {
1061                    log_error("section_list_move_topic() error: src section sid %d != article %d sid %d\n",
1062                                      p_section_src->sid, p_article->aid, p_article->sid);
1063                  return -2;                  return -2;
1064          }          }
1065    
# Line 905  int section_list_move_topic(SECTION_LIST Line 1071  int section_list_move_topic(SECTION_LIST
1071    
1072          last_unaffected_aid_src = (p_article == p_section_src->p_article_head ? 0 : p_article->p_prior->aid);          last_unaffected_aid_src = (p_article == p_section_src->p_article_head ? 0 : p_article->p_prior->aid);
1073    
1074          move_article_count = section_list_count_of_topic_articles(aid);          move_article_count = article_count_of_topic(aid);
1075          if (move_article_count <= 0)          if (move_article_count <= 0)
1076          {          {
1077                  log_error("section_list_count_of_topic_articles(aid = %d) <= 0\n", aid);                  log_error("section_list_count_of_topic_articles(aid = %d) <= 0\n", aid);
# Line 925  int section_list_move_topic(SECTION_LIST Line 1091  int section_list_move_topic(SECTION_LIST
1091    
1092          do          do
1093          {          {
1094                  if (section_list_find_article_with_offset(p_section_dest, p_article->aid, &page, &offset, &p_next) != NULL)                  if (p_section_src->sid != p_article->sid)
1095                  {                  {
1096                          log_error("section_list_move_topic() error: article %d already in section %d\n", p_article->aid, p_section_dest->sid);                          log_error("section_list_move_topic() error: src section sid %d != article %d sid %d\n",
1097                          return -4;                                            p_section_src->sid, p_article->aid, p_article->sid);
1098                            return -2;
1099                  }                  }
1100    
1101                  // Remove from bi-directional article list of src section                  // Remove from bi-directional article list of src section
# Line 949  int section_list_move_topic(SECTION_LIST Line 1116  int section_list_move_topic(SECTION_LIST
1116                  p_article->p_prior->p_next = p_article->p_next;                  p_article->p_prior->p_next = p_article->p_next;
1117                  p_article->p_next->p_prior = p_article->p_prior;                  p_article->p_next->p_prior = p_article->p_prior;
1118    
1119                    // Update sid of article
1120                    p_article->sid = p_section_dest->sid;
1121    
1122                    if (section_list_find_article_with_offset(p_section_dest, p_article->aid, &page, &offset, &p_next) != NULL)
1123                    {
1124                            log_error("section_list_move_topic() error: article %d already in section %d\n", p_article->aid, p_section_dest->sid);
1125                            return -4;
1126                    }
1127    
1128                  // Insert into bi-directional article list of dest section                  // Insert into bi-directional article list of dest section
1129                  if (p_next == NULL) // empty section                  if (p_next == NULL) // empty section
1130                  {                  {
# Line 1014  int section_list_move_topic(SECTION_LIST Line 1190  int section_list_move_topic(SECTION_LIST
1190                          // Re-calculate pages of desc section                          // Re-calculate pages of desc section
1191                          if (section_list_calculate_page(p_section_dest, first_inserted_aid_dest) < 0)                          if (section_list_calculate_page(p_section_dest, first_inserted_aid_dest) < 0)
1192                          {                          {
1193                                  log_error("section_list_calculate_page(section = %d, aid = %d) error\n",                                  log_error("section_list_calculate_page(dest section = %d, aid = %d) error\n",
1194                                                    p_section_dest->sid, first_inserted_aid_dest);                                                    p_section_dest->sid, first_inserted_aid_dest);
1195                          }                          }
1196    
# Line 1031  int section_list_move_topic(SECTION_LIST Line 1207  int section_list_move_topic(SECTION_LIST
1207          // Re-calculate pages of src section          // Re-calculate pages of src section
1208          if (section_list_calculate_page(p_section_src, last_unaffected_aid_src) < 0)          if (section_list_calculate_page(p_section_src, last_unaffected_aid_src) < 0)
1209          {          {
1210                  log_error("section_list_calculate_page(section = %d, aid = %d) error at aid = %d\n",                  log_error("section_list_calculate_page(src section = %d, aid = %d) error at aid = %d\n",
1211                                    p_section_src->sid, last_unaffected_aid_src, aid);                                    p_section_src->sid, last_unaffected_aid_src, aid);
1212          }          }
1213    
# Line 1040  int section_list_move_topic(SECTION_LIST Line 1216  int section_list_move_topic(SECTION_LIST
1216                  // Re-calculate pages of desc section                  // Re-calculate pages of desc section
1217                  if (section_list_calculate_page(p_section_dest, first_inserted_aid_dest) < 0)                  if (section_list_calculate_page(p_section_dest, first_inserted_aid_dest) < 0)
1218                  {                  {
1219                          log_error("section_list_calculate_page(section = %d, aid = %d) error\n",                          log_error("section_list_calculate_page(dest section = %d, aid = %d) error\n",
1220                                            p_section_dest->sid, first_inserted_aid_dest);                                            p_section_dest->sid, first_inserted_aid_dest);
1221                  }                  }
1222          }          }
1223    
1224          return move_article_count;          return move_article_count;
1225  }  }
1226    
1227    int get_section_index(SECTION_LIST *p_section)
1228    {
1229            int index;
1230    
1231            if (p_section_list_pool == NULL)
1232            {
1233                    log_error("get_section_index() error: uninitialized\n");
1234                    return -1;
1235            }
1236    
1237            if (p_section == NULL)
1238            {
1239                    index = BBS_max_section;
1240            }
1241            else
1242            {
1243                    index = (int)(p_section - p_section_list_pool);
1244                    if (index < 0 || index >= BBS_max_section)
1245                    {
1246                            log_error("get_section_index(%d) error: index out of range\n", index);
1247                            return -2;
1248                    }
1249            }
1250    
1251            return index;
1252    }
1253    
1254    int section_list_try_rd_lock(SECTION_LIST *p_section, int wait_sec)
1255    {
1256            int index;
1257            struct sembuf sops[4];
1258            struct timespec timeout;
1259            int ret;
1260    
1261            index = get_section_index(p_section);
1262            if (index < 0)
1263            {
1264                    return -2;
1265            }
1266    
1267            sops[0].sem_num = (unsigned short)(index * 2 + 1); // w_sem of section index
1268            sops[0].sem_op = 0;                                                                // wait until unlocked
1269            sops[0].sem_flg = 0;
1270    
1271            sops[1].sem_num = (unsigned short)(index * 2); // r_sem of section index
1272            sops[1].sem_op = 1;                                                        // lock
1273            sops[1].sem_flg = SEM_UNDO;                                        // undo on terminate
1274    
1275            // Read lock on any specific section will also acquire single read lock on "all section"
1276            // so that write lock on all section only need to acquire single write on on "all section"
1277            // rather than to acquire multiple write locks on all the available sections.
1278            if (index == BBS_max_section)
1279            {
1280                    sops[2].sem_num = BBS_max_section * 2 + 1; // w_sem of all section
1281                    sops[2].sem_op = 0;                                                // wait until unlocked
1282                    sops[2].sem_flg = 0;
1283    
1284                    sops[3].sem_num = BBS_max_section * 2; // r_sem of all section
1285                    sops[3].sem_op = 1;                                        // lock
1286                    sops[3].sem_flg = SEM_UNDO;                        // undo on terminate
1287            }
1288    
1289            timeout.tv_sec = wait_sec;
1290            timeout.tv_nsec = 0;
1291    
1292            ret = semtimedop(section_list_pool_semid, sops, (index == BBS_max_section ? 4 : 2), &timeout);
1293            if (ret == -1 && errno != EAGAIN && errno != EINTR)
1294            {
1295                    log_error("semtimedop(index = %d, lock read) error %d\n", index, errno);
1296            }
1297    
1298            return ret;
1299    }
1300    
1301    int section_list_try_rw_lock(SECTION_LIST *p_section, int wait_sec)
1302    {
1303            int index;
1304            struct sembuf sops[3];
1305            struct timespec timeout;
1306            int ret;
1307    
1308            index = get_section_index(p_section);
1309            if (index < 0)
1310            {
1311                    return -2;
1312            }
1313    
1314            sops[0].sem_num = (unsigned short)(index * 2 + 1); // w_sem of section index
1315            sops[0].sem_op = 0;                                                                // wait until unlocked
1316            sops[0].sem_flg = 0;
1317    
1318            sops[1].sem_num = (unsigned short)(index * 2 + 1); // w_sem of section index
1319            sops[1].sem_op = 1;                                                                // lock
1320            sops[1].sem_flg = SEM_UNDO;                                                // undo on terminate
1321    
1322            sops[2].sem_num = (unsigned short)(index * 2); // r_sem of section index
1323            sops[2].sem_op = 0;                                                        // wait until unlocked
1324            sops[2].sem_flg = 0;
1325    
1326            timeout.tv_sec = wait_sec;
1327            timeout.tv_nsec = 0;
1328    
1329            ret = semtimedop(section_list_pool_semid, sops, 3, &timeout);
1330            if (ret == -1 && errno != EAGAIN && errno != EINTR)
1331            {
1332                    log_error("semtimedop(index = %d, lock write) error %d\n", index, errno);
1333            }
1334    
1335            return ret;
1336    }
1337    
1338    int section_list_rd_unlock(SECTION_LIST *p_section)
1339    {
1340            int index;
1341            struct sembuf sops[2];
1342            int ret;
1343    
1344            index = get_section_index(p_section);
1345            if (index < 0)
1346            {
1347                    return -2;
1348            }
1349    
1350            sops[0].sem_num = (unsigned short)(index * 2); // r_sem of section index
1351            sops[0].sem_op = -1;                                               // unlock
1352            sops[0].sem_flg = IPC_NOWAIT | SEM_UNDO;           // no wait
1353    
1354            // The same reason as section_list_try_rd_lock()
1355            if (index == BBS_max_section)
1356            {
1357                    sops[1].sem_num = BBS_max_section * 2;   // r_sem of all section
1358                    sops[1].sem_op = -1;                                     // unlock
1359                    sops[1].sem_flg = IPC_NOWAIT | SEM_UNDO; // no wait
1360            }
1361    
1362            ret = semop(section_list_pool_semid, sops, (index == BBS_max_section ? 2 : 1));
1363            if (ret == -1 && errno != EAGAIN && errno != EINTR)
1364            {
1365                    log_error("semop(index = %d, unlock read) error %d\n", index, errno);
1366            }
1367    
1368            return ret;
1369    }
1370    
1371    int section_list_rw_unlock(SECTION_LIST *p_section)
1372    {
1373            int index;
1374            struct sembuf sops[1];
1375            int ret;
1376    
1377            index = get_section_index(p_section);
1378            if (index < 0)
1379            {
1380                    return -2;
1381            }
1382    
1383            sops[0].sem_num = (unsigned short)(index * 2 + 1); // w_sem of section index
1384            sops[0].sem_op = -1;                                                       // unlock
1385            sops[0].sem_flg = IPC_NOWAIT | SEM_UNDO;                   // no wait
1386    
1387            ret = semop(section_list_pool_semid, sops, 1);
1388            if (ret == -1 && errno != EAGAIN && errno != EINTR)
1389            {
1390                    log_error("semop(index = %d, unlock write) error %d\n", index, errno);
1391            }
1392    
1393            return ret;
1394    }

Legend:
Removed lines/characters  
Changed lines/characters
  Added lines/characters
webmaster@leafok.com
 ViewVC Help
Powered by ViewVC 1.3.0-beta1